Hallo werter Lernender,
erstmal vorweg zu deinem Problem. Der Code ist soweit richtig bzw. er funktioniert bei mir auch wie er soll. Die Ini wird erstellt nachdem ich mit dem Durchsuchen fertig bin (FileDialog) oder wenn sie dann noch nicht existiert auch beim Drücken des Start Buttons. Das das bei dir nicht gehen soll wundert mich, am FileDialog kann es nicht liegen.
Zur Sauberkeit und Sauberkeit des Codes:
Ich habe mir nicht wirklich viel durchgeguckt, aber mir ist sofort eines aufgefallen.
Zitat
$Datei = FileOpenDialog("Wähle dein Firefox aus", "C:\", "Alles (*.*)")
Guictrlsetdata($Inputfirefox,$Datei)
$firefoxpfad=GuictrlRead($Inputfirefox)
IniWrite("settings.ini","Eingaben","Firefox",$firefoxpfad)
Die rot markierte Zeile ist überflüssig. Du weißt doch, was in dem Input steht, und das ist in $Datei gespeichert. Zudem solltest du darauf achten Abstände zu lassen. Sowohl Leerzeichen als auch Absätze machen einen Quellcode ungemein attraktiver, weil man sich nicht durch einen unübersichtlichen Klumpen welzen muss. Lass vor und hinter dem = ein Leerzeichen. Mach Absätze, wenn der folgende Codeabschnitt nichts mehr direkt mit dem vorherigen zu tun hat, lass Absätze vor und hinter Schleifen und If-Abfragen, z.B:
Zitat
Func Terminate()
For $i = 1 To 1 Step -1
MsgBox(0, "Schließt in...", $i & " Sekunden", 1)
Next
MsgBox(0, "", "Script beendet!", 1)
Exit 0
EndFunc ;==>Terminate
Alles anzeigen
zu
Zitat
Func Terminate()
For $i = 1 To 1 Step -1
MsgBox(0, "Schließt in...", $i & " Sekunden", 1)
Next
MsgBox(0, "", "Script beendet!", 1)
Exit 0
EndFunc ;==>Terminate
Alles anzeigen
(Nur ein Beispiel)
Mach Kommentare, gerade bei Code, bei dem du eventuell später Verstänsnisprobleme haben könntest.
Lass Absätze zwischen einzelnen Funktionen.
Nutze eventuelle Regionen:
Zitat
#region Loop
GUI Code blabla
#endregion Loop
Versuche Variablen mit einem eindeutigen Namen zu hinterlegen.
Ansonsten ist es gut, dass das Tidy-Tool nutzt um die Einrückung zu gewährleisten.
Sieht für jemanden, der gerade erst angefangen hat, insgesamt ganz gut aus.
Die Einrückung ist jetzt aufgrund der Quote's verloren gegangen, hätt ma die Code-Felder nehmen sollen...