Du bist nicht angemeldet.

Stilllegung des Forums
Das Forum wurde am 05.06.2023 nach über 20 Jahren stillgelegt (weitere Informationen und ein kleiner Rückblick).
Registrierungen, Anmeldungen und Postings sind nicht mehr möglich. Öffentliche Inhalte sind weiterhin zugänglich.
Das Team von spieleprogrammierer.de bedankt sich bei der Community für die vielen schönen Jahre.
Wenn du eine deutschsprachige Spieleentwickler-Community suchst, schau doch mal im Discord und auf ZFX vorbei!

[C#] CA2000 obwohl Objekt noch gebraucht wird ?

Werbeanzeige

Sylence

Community-Fossil

  • »Sylence« ist der Autor dieses Themas

Beiträge: 1 663

Beruf: Softwareentwickler

  • Private Nachricht senden

1

03.04.2011, 15:20

[C#] CA2000 obwohl Objekt noch gebraucht wird ?

Hallo zusammen :)
Ich hab grad eine Meinungsverschiedenheit mit meinem VS. Es mein das ich Dispose() für 'stream' aufrufen soll, allerdings wird das doch noch benutzt ?

Quellcode

 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15

class Klasse
{
    public Klasse( Stream stream )
    {
        BaseStream = stream;
    }

    Stream BaseStream;
}

Klasse Foo()
{
    FileStream stream = new FileStream( "dateiname", FileMode.Open, FileAccess.Read ); // CA2000
    return new Klasse( stream );
}


'Klasse' sieht in wirklichkeit natürlich etwas anders aus, aber so lässt sich das wohl besser veranschaulichen.
Jetzt ist die Frage wer Recht hat ^^
Da ich den FileStream in der Klasse noch benutze kann ich ihn schlecht bei/nach dem erstellen der Klasse disposen...

Achso ja: den aufruf von 'new Klasse()' in ein try-finally block packen und filestream in falle des falles disposen ändert auch nichts.

BlueCobold

Community-Fossil

Beiträge: 10 738

Beruf: Teamleiter Mobile Applikationen & Senior Software Engineer

  • Private Nachricht senden

2

03.04.2011, 15:58

Ja und nein. Sicher wird der Stream weiter verwendet, das stimmt.
Allerdings (muss wohl durch iPhone-Entwicklung so sein) finde ich es eher unschön, dass sich "Foo" darauf verlässt, dass "Klasse" den Stream selber korrekt wieder freigibt. Denn da hast Du beispielsweise folgendes Problem:

C#-Quelltext

 
1
2
3
4
5
6
7
8

Foo Bar()
{
...
    FileStream stream = new FileStream( "dateiname", FileMode.Open, FileAccess.Read ); // CA2000
    Klasse x = new Klasse( stream );
    Klasse y = new Klasse( stream );
...
}


Wer gibt denn da jetzt den Stream frei? Also "Klasse" und die Instanzen "x" und "y" können das offensichtlich weder wissen, noch vernünftig fehlerfrei durchführen. Aber "Bar" kümmert sich auch nicht drum. Design-Fehler.

Die eiserne Regel (und das hat sich 100%ig aus der iPhone-Entwicklung bei mir eingebrannt) lautet da meiner Meinung nach: Der Ersteller gibt ihn auch wieder frei. Jeder andere darf maximal drauf zugreifen, aber disposen auf keinen Fall.
Teamleiter von Rickety Racquet (ehemals das "Foren-Projekt") und von Marble Theory

Willkommen auf SPPRO, auch dir wird man zu Unity oder zur Unreal-Engine raten, ganz bestimmt.[/Sarkasmus]

captain

Frischling

Beiträge: 7

Wohnort: berlin

  • Private Nachricht senden

3

03.04.2011, 21:49

Unter C# gibt es doch diese IDisposable Schnittstelle, wenn "Klasse" diese implementiert, kann sie den Stream selbst wieder freigeben.

Gab es nicht auch eine Klammer ähnlich try catch, die bei disposable objects automatisch dispose aufruft?
The Terminal Angel Disease - Indie Retro RPG!!!

Sylence

Community-Fossil

  • »Sylence« ist der Autor dieses Themas

Beiträge: 1 663

Beruf: Softwareentwickler

  • Private Nachricht senden

4

03.04.2011, 22:11

Zitat von »captain«

Unter C# gibt es doch diese IDisposable Schnittstelle, wenn "Klasse" diese implementiert, kann sie den Stream selbst wieder freigeben.

Sie implementiert IDisposable schon. Das Interface ist nur dafür da um das objekt selber disposen zu können bzw. zu bestimmen was passiert, wenn der GC drüber fegt (zum beispiel resourcen freigeben, die nicht gemanaget sind)

Zitat von »captain«

Gab es nicht auch eine Klammer ähnlich try catch, die bei disposable objects automatisch dispose aufruft?

Du meinst bestimmt 'using' die funktioniert nur innerhalb des lokalen scopes.

Hab das jetzt aber gelöst, indem ich ein wenig umgebaut hab. Klasse sorgt jetzt selbst dafür, dass sie sich den Stream erstellt und auch wieder freigibt.

Werbeanzeige

Ähnliche Themen