Ist folgender Code "sicher"?

[BUG]

Es gibt auch noch ein kleines Speicherleck (oder habe ich da ein Dispose übersehen?).

[DeddyH]

Das war mir auch aufgefallen. New -> Dispose, GetMem -> FreeMem, das sollte man schon einhalten.

[Furtbichler]

Lustig, da fragt jemand, ob der Code sicher ist (DIESER Code ist es) und dann wird eine Abhandlung daraus, das Pointer per se unsicher sind. Man kann dann ja auch sagen, das das Leben per se tödlich ist.

Lustig finde ich auch, das man fälschlicherweise ein Speicherleck erwähnt, nur weil in DIESEM Code am Ende ein (an dieser Stelle zudem überflüssiges) FreeMem fehlt. Also ich will mich nicht zu weit aus dem Fenster lehnen, aber ich bin mir ziemlich sicher, das sukzessives Aufrufen dieses kleinen Programms nicht dazu führt, das Windows irgendwann mit einem OutOfMemory anhält.

Natürlich sollte man die immer freigeben, was man anfordert (einfach aus Prinzip), und Pointer (wie auch Endlosschleifen) echt böse und gefährlich sein können, wenn man keine Ahnung hat, aber der Hinweis, das beide Bemerkungen nichts mit der Frage zu tun haben, sondern allgemein gehaltene Tipps sind, wäre imho angebracht. Ansonsten würde ich nämlich annehmen, das die Frage nicht verstanden wurde.

[shmia]

Zitat von Furtbichler:

... aber der Hinweis, das beide Bemerkungen nichts mit der Frage zu tun haben, sondern allgemein gehaltene Tipps sind, wäre imho angebracht. Ansonsten würde ich nämlich annehmen, das die Frage nicht verstanden wurde.

Ich denke JEDER der Beitrag #4 gelesen hat, wird verstehen, dass es sich um allgemeine Tipps handelt; das muss also nicht noch explizit erklärt werden.

Eigentlich wollte ich ja schreiben das der Code aus Beitrag #1 hässlich und altmodisch ist, habe dann aber lieber allgemeine Hinweise gegeben.
Das war zumindest ein konstruktiver Beitrag für alle die dazulernen möchten; im Gegensatz zu deinem Rant.