9 Antworten in diesem Thema

[Slixer]

Die Function:

Private Sub AddCurrentKey(ByVal name As String, ByVal path As String)
Dim key As RegistryKey = Registry.LocalMachine.OpenSubKey("Software\Microsoft\Windows\CurrentVersion\Run", True)
key.SetValue(name, path)
key.Close()
End Sub

Der aufruf:

AddCurrentKey("Test", "Path zu exe")

[Blackfox]

Geht auch mit 2 Zeilen direkt.

Dim key As Microsoft.Win32.RegistryKey = My.Computer.Registry.CurrentUser.CreateSubKey("Software\Microsoft\Windows\CurrentVersion\Run")
key.SetValue(My.Application.Info.Title, """" & System.Reflection.Assembly.GetEntryAssembly.Location & """")

[Cube]

Geht auch mit 2 Zeilen direkt.

Dim key As Microsoft.Win32.RegistryKey = My.Computer.Registry.CurrentUser.CreateSubKey("Software\Microsoft\Windows\CurrentVersion\Run")
key.SetValue(My.Application.Info.Title, """" & System.Reflection.Assembly.GetEntryAssembly.Location & """")

Korrgier mich wenn ich falsch lieg.
Aber er hats in 2 Zeilen gelöst das ganze nur in einer Funktion verpackt.
Außerdem müsste man bei dir noch die Ressource "key" freigeben mittels "key.Close()"?

LG

[Blackfox]

Korrgier mich wenn ich falsch lieg.
Aber er hats in 2 Zeilen gelöst das ganze nur in einer Funktion verpackt.
Außerdem müsste man bei dir noch die Ressource "key" freigeben mittels "key.Close()"?

LG

Bei mir funz es auch ohne "key.Close()" da mehrmals getestet.
Aber schadet ja nicht.

[Julius K9]

Natürlich geht das auch. Nur hält er dir key dann dauerhaft offen. Wenn du nachdem Code oben nichtmehr draufzugreifen musst macht das aber keinen Sinn. Wäre so als würdest du dir 10 Brote schmieren aber nur 9 essen eines liegt dann dauerhaft im Speicher.

[SAR]

Kommt drauf an - wenn er es Lokal deklariert und in einen extra Sub packt, dann wird das Objekt automatisch aus dem Speicher entfernt - zumindest wenn der Sub zuende ist.
Aber sauberer Code ist es trotzdem nicht. .png' class='bbc_emoticon' alt='^^' />


Mfg.
SAR

[Blackhook]

Warum wird jeder Müll schön geredet?
Müll bleibt Müll. Das lässt sich eben nicht schön reden.
Das hat nichts mit coden zu tun, höchstens hast du es irgendwo abgeschrieben. Ansonsten hättest du nicht solch gravierende Fehler akzeptiert.
Objekte lässt man nicht offen stehen und versaut den Speicher damit.
Sauber ist was Anderes.

[Blackfox]

Warum wird jeder Müll schön geredet?
Müll bleibt Müll. Das lässt sich eben nicht schön reden.
Das hat nichts mit coden zu tun, höchstens hast du es irgendwo abgeschrieben. Ansonsten hättest du nicht solch gravierende Fehler akzeptiert.
Objekte lässt man nicht offen stehen und versaut den Speicher damit.
Sauber ist was Anderes.

Danke für die Info. Und was hats dir jetzt gemacht? *Schenkelklopf*
Bruder, warum bist du so erbost? Nur weil mal Jemand mal kein "Key.Close ()" in den Post gepackt hat?
Findest du nicht auch, dass es überflüssig ist, jeden Müll wie "Key.Close ()" hier anzuplanzen?
Es ist doch selbst verständlich, dass so etwas doch zum Grundwissen gehört und nicht jedes mal für dich extra immer wieder ausgegraben werden muss. Oder? Schau noch mal drüber, dann klappt es auch mit dem Bereifen ...

Bearbeitet von Blackfox, 14 December 2014 - 21:21 Uhr.

[SAR]

Variablen haben eine Lebensdauer. Das es unsauber ist, darüber brauchen wir nicht diskutieren. Aber es hat mal 0,0 mit dem "Versauem" des Speichers zu tun.

Please Login HERE or Register HERE to see this link!

Aber es gibt noch andere Gründe, warum solche Dinge geschlossen werden sollen. z.B. bei StreamReader/Writer - das Handle der Datei sollte korrekt wieder freigegeben werden. Wenn hier schon begründet wird, dann bitte richtig.


Mfg.
SAR

[Blackhook]

Ich entschuldige mich für mein posting.
Fuck, es war sehr viel Alkohol im Blut.
Sorry noch mal.
Hier gebe ich SAR vollkommen Recht. Wenn man schon etwas kritisiert, sollte man es auch vernünftig begründen.
Wir sind aber Alle nur Menschen.

Kurze Erklärung (nüchtern).
Dein Tool wird mehr Speicher belegen. Für Jeden Aufruf wird ein teil des Speichers reserviert.
In deinem Fall hat es allerdings kaum eine Bedeutung, das Tool wird mal gestartet und dann beendet. Es läuft nicht permanent und ruft die Procedur/Funktion nur ein mal auf.
Beim Neustart des Rechners wird Alles wieder frei sein.
Man lernt allerdings als erstes. Bei Objekt orientierter Programmierung sollte man keine Objekte offen lassen.
Mehr hätte ich wohl nicht schreiben sollen.