Ein ... "Winziges" Problem

Hey

@Edit: Thanks an alle Programm im anhang (64 und 32 bit version :D)

MfG
Tutnix

Erstmal Thanks an alle

Zitat von PainTain

1. Der Button ... du hast die falsche Variable genommen.

2. Drück nächstes Mal mal Strg + T. Dann macht Tidy deinen Code einigermaßen lesbar

3. Deine Fehler waren, das du teilweise 2x EndFunc geschrieben hast oder es einfach vergessen.

Zudem deklarierst du auch immer wieder die Funktion Rechnen() neu...

Auch Zeile 115 - Wo sind die Klammern?

Vielleicht hab ich noch ein paar Fehler übersehen.
Ich würde dir sehr, sehr chessi's [Tutorial] Sauber Programmieren ans Herz legen

Und mithilfe der Zeilennummern, welche der Syntax-Checker ausspuckt, solltest du deine Fehler selbst finden können.
AutoIt macht die Fehlersuche sowieso schon sehr leicht, das es immer Zeilennummern anzeigt.

//Edit: Das größte Problem ist mir gerade eben erst aufgefallen. Es wird niemals eine Funktion aufgerufen - daher rattert der Interpreter einfach drüber und fertig...

Alles anzeigen

@Edit stimmt XD das war nur nen versuch diese blöden false rauszubekommen XD sorry vergessen rauszunehmen

zu 1. -.- ich bin doof ... sorry

zu 2. ähm ... was zum Teufel is Tidy?????? (bring mir das grad selber bei drum frag ich mal ... hab noch nich alle funktionen entdeckt)

zu 3. Wie soll ich das mit der Func Rechner () denn machen?? sie is ja in nem Func eingebettet also dürfte da eigentlich ja nix passiren
und ... mal wieder doofheit XD (z.115)

ok ... werd mal deinen skript so überarbeiten ... und [Tutorial] Sauber Programmieren is schon als Lesezeichen XD

Zitat von alpines

4. Nehm (wenn du schon für jede Form eine neue Variable machst) auch für die Controls eine neue Variable z.B. $AdditionInput, damit ist dein Code besser verständlich.
5. Übertreib es nicht mit den Funktionen!, du hättest die GUIs genauso gut auch in die Switch-Case Abfrage packen können, aber das ist Geschmackssache. Ein weiterer Grund für die Switch-Case wäre die zu vermeidende Rekursion, auch wenn man das natürlich beheben könnte.
6. GUIDelete($hGUI) verwenden und nicht GUIDelete $hGUI ($hGUI durch dein Formhandle ersetzen), da die Funktion den Parameter braucht und nicht wie z.B. Exit 1 anzuwenden ist.

zu 4. ok ... das könnte vill helfen XD

zu 5. dem switch code ??? und was is ne Rekusion ... bitte nicht erschlagen ich lern auch ich lern auch

zu 6. wo findet man diese tricks ... ich bekomm das kein stück hin XD breucht ne erklärung für sowas ... also nehm ich mal mein GUIDelete ... da weis ich wie es geht XD

Zitat von RR04

Hi,

[autoit]

Case $Butten2 ; Zeile 28
; besser:
Case $Button2

[/autoit]

XD ja ... ich hab keine Brille auf gehabt (irgend ne ausrede)

Zitat von Andy

Hi,

Ein Doppelklick auf die in der Scite-Console angezeigte Zeile lässt den Cursor auf den fehlerhaften Code springen.

°-° bitte deutschen das nochmal ein XD (so wie ich das verstanden hab ... mit nem Doppelklick auf das komische korektur ding und is man an der stelle mit dem fehler?? richtig?)

Thanks leute ihr seit wirklich gut (alles ausgebildete programmirer/informatiker oder sind hier noch nen paar junge hobby programmirer mit der aussicht auf so nen job??)

MfG
Tutnix
PS: Werd den skript ausbessern wenn er fertig is ... hoffe bekomm das blöde ding auf gelöst XD.

Zitat von alpines

Im Source-Code stand

[autoit]

GuiDelete $hGui

[/autoit]

oder so ähnlich, das ist aber nicht der richtige Syntax, mittlerweile hast du das glaub ich schon ergänzt.
Tidy ist ein Programm, (ist in SciTE per HotKey STRG+T glaub ich verfügbar), welches dein Script mit sinnvollen Code-Einrückungen versieht, damit es besser lesbar ist.
Aus:

[autoit]

Func Addition($fValue1, $fValue2)
Local $fReturn
$fReturn = $fValue1 + $fValue2
Return $fReturn
EndFunc

[/autoit]

wird sowas wie:

[autoit]

Func Addition($fValue1, $fValue2)
Local $fReturn
$fReturn = $fValue1 + $fValue2
Return $fReturn
EndFunc ;==> Addition

[/autoit]

Mit Rekursion mein ich folgendes: Ich würde nicht für jede Form unbedingt eine eigene Funktion machen, da wenn du die Funktion selbst in der Funktion aufrufst eine Rekursion (sprich Wiederholung) hervorrufst.
AutoIt streikt ab knapp 1899 in sich wiederholten Funktionsaufrufen und beendet das Programm, deshalb vermeid Funktionen solange sie nicht von Nöten sind.

Desweiteren kannst du statt Funktionen auch einfach die GUIs am Anfang initialisieren und den GUISetState auf @SW_HIDE stellen, und immer wenn du die GUI brauchst, nimmst du einfach GUISetState($hGui, @SW_SHOW),
dabei musst du aber jeder Form natürlich unterschiedliche Variablen zuweisen!

Alles anzeigen

hmm ok jetzt weis ich wenigstens was Tidy is (direkt testen XD)
Nich so viele Funktionen ... Gespeichert.
°-° mit gui initalisiren meinst du doch das ich die gui in ner au3 speichern soll und dann wenn ich sie haben möchte GuiSetState ($hGui,@sw_show) eingebe ... dann is sie wieder sichtbar? Cool (kann man über diese funktions whiles aktiviren/deaktiviren die mit der Gui verknüpft sind?)

XD alle discutiren über diesen Komischen Recusse Lv ... bei mir Win8 32bit (x86) is es auch 3899 nur um daruaf zu kommen ...

aber jetzt nochmal zu meiner frage ...

°-° mit gui initalisiren meinst du doch das ich die gui in ner au3 speichern soll und dann wenn ich sie haben möchte (vorher per #include initalisiren) GuiSetState ($hGui,@sw_show) eingebe ... dann is sie wieder sichtbar? Cool (kann man über diese funktions whiles aktiviren/deaktiviren die mit der Gui verknüpft sind?)

achso meinst du das ... alle guis untereinader und nur die hauptgui sichtbar machen und denn rest nur sichtbar wenn man auf denn buttendrückt ... ouhh ... damit könnte der bug gefixt sein denn ich mom hab XD thanks

@Edit: Thanks an alpines, aber auch an die Anderen alle bugs gefixt Thanks .... HAPPPYYYY (wer will kann mal testen aber net enteuscht sein ... anfänger do it )