pen.Dispose() vs Pens.Yellow.Dispose()

RodFromGermany · 27. Juli 2018, 07:19

Ich denke, der Titel sagt schon alles.
Zur Darstellung und Beschriftung von Messkurven habe ich eine Klasse, der ein Pen übergeben wird (analog andere IDisposable Objekte).
Beim finalen Aufräumen müssen diese Objekte zerstört werden.
Nun kommt es vor, dass neben einem New Pen(Color.Red, 2) auch ein Pens.Yellow übergeben wird.
Ersterer kann disposed werden, bei letzterem kommt eine ArgumentException.
Frage:
Gibt es eine einfache Abfrage, ob ein derartiges Objekt dynamisch oder statisch erzeugt wurde?
oder
Ist hier ein Try/Catch im Dispose() die einzige Möglichkeit, das abzufangen?

slice · 27. Juli 2018, 11:18

Grundsatzfrage:
Warum räumst du die denn auf? Die liegen, nach meinem Verständnis, nicht in deiner Hoheit. Der der den Pen erstellt, ist dafür verantwortlich ihn wieder weg zu räumen.

Zu deiner Frage:
So weit ich weiß, in der Situation nicht.

Snake3000 · 27. Juli 2018, 11:52

Guten Morgen,

diese Grundsatzfrage interessiert mich auch gerade sehr.
Was genau liegt denn in der Hoheit des Programmierers? Ich habe mich belesen und gelernt das der Garbage Collektor recht gut auf den verwendeten Speicher aufpasst. Leider geht nirgends genau hervor was man selber aufräumen bzw. freigeben muss; was also "unmanaged Code" ist...

Der Satz "Der der den Pen erstellt, ist dafür verantwortlich ihn wieder weg zu räumen." ist gut, aber es bleibt die Frage: Wann, wo und wie mache ich das am sinnvollsten?

RodFromGermany · 27. Juli 2018, 13:01

slice schrieb:

Grundsatzfrage:

VB.NET-Quellcode

MyObject1 = New OwnObjectClass(New Pen(Color.Red, 2))
MyObject2 = New OwnObjectClass(Pens.Yellow)
' ...
MyObject1.Dispose()
MyObject2.Dispose()

VB.NET-Quellcode

Class OwnObjectClass
Inherits IDisposable
Private myPen As Pen
Public Sub New(p As Pen)
Me.myPen = p
End Sub
Public Sub Dispose()
Me.myPen.Dispose()
End Sub
End Class

slice · 27. Juli 2018, 13:37

Da würde ich den Konstruktor so umbauen, das er als ersten Parameter eine Farbe, als zweiten (Optional) die Breite annimmt und dann in "OwnObjectClass" den Pen erstellen. Damit ist sichergestellt das er "richtig" erstellt wird und nicht von außen während deiner Benutzung Disposed wird.

VB.NET-Quellcode

Class OwnObjectClass
Inherits IDisposable
Private myPen As Pen
Public Sub New(color As Color)
Me.New(color, 1)
End Sub
Public Sub New(color As Color, width As Single)
Me.myPen = New Pen(color, width)
End Sub
Public Sub Dispose()
Me.myPen.Dispose()
End Sub
End Class

@Snake3000 Das mit dem GC etc. ist nochmal ein ganz eigenes Thema, da bin ich auch nicht so tief drin. Ich räume das weg was ich erstelle, alles andere ist nicht meine Sache.
Erstell ich zb. den Pen, räume ich ihn am Ende, wenn mein Code vorbei ist und ich ihn nicht mehr brauche weg. Wird der Pen meinem Code als Parameter übergeben, gehört er nicht mir, also räume ich ihn nicht weg.

Snake3000 · 27. Juli 2018, 15:43

Danke Dir.

Als absoluter Neuling in oops nehme ich mal mit, das ich alles was ich mit "new" constructed habe auch wegräumen muss? hmmmmmm……..

EaranMaleasi · 27. Juli 2018, 16:41

Speicherverwaltung ist in jeder Sprache anders.

In der .NET Welt hast du den GarbageCollector (GC), der sich um 95% aller Allokationen bzw. Aufräumarbeiten kümmert. Die restlichen 5% Implementieren IDisposable bzw. bieten die Methode .Dispose() an. Kurz gesagt bedeutet die Implementierung von IDisposable, dass diese Klasse ein Objekt beinhaltet, dass vom GC nicht ohne weiteres aufgeräumt werden kann/darf. Mit .Dispose() wird dem GC signalisisert, dass dieser nun dieses Objekt aufräumen darf.

Facebamm · 27. Juli 2018, 17:03

Ich muss jetzt doof fragen!, Kannst du Pens.Yellow noch verwenden nach Dispose() oder ist er jetzt g, wie ganz weg ? Oo

*update

Wenn ich mir die Source anschaue wird es komisch

C#-Quellcode

/// <include file='doc\Pens.uex' path='docs/doc[@for="Pens.Yellow"]/*' />
/// <devdoc>
/// <para>[To be supplied.]</para>
/// </devdoc>
public static Pen Yellow {
[ResourceExposure(ResourceScope.Process)]
[ResourceConsumption(ResourceScope.Process | ResourceScope.AppDomain, ResourceScope.Process | ResourceScope.AppDomain)]
get {
Pen yellow = (Pen)SafeNativeMethods.Gdip.ThreadData[YellowKey];
if (yellow == null) {
yellow = new Pen(Color.Yellow, true);
SafeNativeMethods.Gdip.ThreadData[YellowKey] = yellow;
}
return yellow;
}
}

Snake3000 · 27. Juli 2018, 17:17

Ist schon witzig das man sich zu Gigabyte Zeiten wieder um Speicherplatz kümmern muss....

.Dispose() ist klar... sagt dem GC das es weg kann. Aber woher weiss ich das ich es disposen kann?
Ist ne dumme Frage, aber die müssen auch gestellt werden...

RodFromGermany · 27. Juli 2018, 18:02

Snake3000 schrieb:

das man sich zu Gigabyte Zeiten wieder um Speicherplatz kümmern muss....

Wenn Du einen dynamischen Überwachungsprozess an einer Maschine hast, wo im Sekundentakt ca. 100 IDisposable Objekte erstellt werden, läuft der Speicher langsam aber sicher zu.

Niko Ortner · 27. Juli 2018, 18:46

@Snake3000
Das dürfte auch interessant für Dich sein: Was sind verwaltete / nicht verwaltete Ressourcen

@RodFromGermany
Ich glaube beim StreamWriter/Reader gibt es das Konzept, dass im Konstruktor (die Überladung, die einen BaseStream annimmt) angegeben werden kann, ob der BaseStream ebenfalls disposed werden soll, wenn der StreamWriter/Reader disposed wird. Das wäre also eine Option.
In diesem konkreten Fall wäre es aber sinnvoll, alle Pens, die im Konstruktor ankommen, zu klonen. Pen implementiert IClonable. Dieses Objekt darfst Du dann in jedem Fall disposen.
Ansonsten gilt generell, dass man nur das disposed, was man auch selbst erstellt hat. Das heißt, den Pen, den Du in deine eigene Klasse gibst, disposed Du außerhalb dieser Klasse (wenn es ein selbst erstellter Pen ist), oder eben nicht. In der Klasse disposed Du den geklonten Pen.

RodFromGermany · 27. Juli 2018, 19:00

Niko Ortner schrieb:

ob der BaseStream ebenfalls disposed werden soll, wenn der StreamWriter/Reader disposed wird.

Nö.
Ich komme darauf, weil ich den Code meines Vorgängers einer Codeanalyse unterzogen habe.
Da fielen die Pens auf und auch FileStream(Stream).
Für diesen Fall schlägt Microsoft dieses Herangehen vor:

C#-Quellcode

Stream stream = null;
try
{
stream = new FileStream("file.txt", FileMode.OpenOrCreate);
using (StreamWriter writer = new StreamWriter(stream))
{
stream = null;
// Use the writer object...
}
}
finally
{
if(stream != null)
{
stream.Dispose();
}
}

Niko Ortner · 27. Juli 2018, 19:37

Hier hab ich ihn: msdn.microsoft.com/de-de/library/gg712952(v=vs.110).aspx
Ist der leaveOpen-Parameter auf True gesetzt, bleibt der stream offen. So meinte ich das.

RodFromGermany · 27. Juli 2018, 19:54

@Niko Ortner OK, dann war das eine Mistverständnissung.

ErfinderDesRades · 27. Juli 2018, 23:27

RodFromGermany schrieb:

Ist hier ein Try/Catch im Dispose() die einzige Möglichkeit, das abzufangen?

Jo - seh ich auch nix anneres.
Interessanterweise kann man im Haltepunkt sehen, dass Pen ein privates Feld immutable As Boolean hat - also der Pen selbst weiss, ob er ein System-Pen ist oder nicht. Aber von aussen findet man das wohl nur durch Reflection oder Exception heraus.

RodFromGermany · 28. Juli 2018, 09:45

@ErfinderDesRades Try/Catch im Dispose() ist nach meinem Dafürgalten ein NoGo, deswegen dieser Thread.
Das ganze ist offensichtlich ein Fehldesign meines Vorgängers, ich werde es auflösen, wie es @slice vorgeschlagen hat.

ErfinderDesRades · 28. Juli 2018, 10:06

RodFromGermany schrieb:

nach meinem Dafürhalten ein NoGo

nach meim nicht. Der Catch frisst keine Resourcen und das Ignorieren des Fehlers ist eine sichere Behandlung.
Sieht bisserl bescheuert aus - aber ist ja nicht deine Schuld.

Imo ein Fehldesign von MS, die hätten die SystemPens iwie anners designen müssen.

Aber deine Alternative ist ja auch durchaus gangbar.

petaod · 28. Juli 2018, 10:24

slice schrieb:

Public Sub New(color As Color)
Me.New(color, 1)
End Sub
Public Sub New(color As Color, width As Single)
Me.myPen = New Pen(color, width)
End Sub

Ich nehme da ganz gerne Optional anstatt einer Überladung.

VB.NET-Quellcode

Public Sub New(color As Color, Optional width As Single = 1)
Me.myPen = New Pen(color, width)
End Sub

RodFromGermany · 28. Juli 2018, 10:32

@petaod Das hab ich schon so angedacht, es kommen momentan nur die Stärken 1 und 2 vor.

pen.Dispose() vs Pens.Yellow.Dispose()

pen.Dispose() vs Pens.Yellow.Dispose()

slice schrieb:

VB.NET-Quellcode

VB.NET-Quellcode

VB.NET-Quellcode

C#-Quellcode

Snake3000 schrieb:

Niko Ortner schrieb:

C#-Quellcode

RodFromGermany schrieb:

RodFromGermany schrieb:

slice schrieb:

VB.NET-Quellcode

Tags

Ähnliche Themen

8 Benutzer haben hier geschrieben