Overroll UDF

Achte mal darauf das du innerhalb der Func die Variablen deklarierst, so beugst du Fehlern vor. Wenn du innerhalb einer Func auf Variablen zugreifst, die du nicht Local definiert hast ,werden Werte aus der im Hauptscript definierten Variablen des selben Namens geändert . Und das Script benutzt auch die Werte aus den Global definierten Variablen des gleichen Namens . Es wird niemand nachsehen welche Variablennamen du innerhalb deiner Function benutzt .Außerdem setzt du innerhalb deiner Func voraus, das die Variable $Objektstatus bereits definiert ist . Das ist kein guter Programmierstil .Desweiteren rufst du aus deiner Func _Overroll weitere Func´s auf die aber Benutzerspezifisch und nicht allgemeingültig sind . Im Augenblick ist das ganze nichts weiter als eine Func in einem Script und keine UDF.

Edit : Gewöhne dir am besten an Opt("MustDeclareVars", 1) in deinen Scripten zu verwenden, da wirst du dir viele Fehlersuchen ersparen. Ich weiß wovon ich rede, hab am Anfang auch den Fehler gemacht Variablen nicht zu deklarieren und dann Stundenlang nach Fehlern gesucht .

Zitat von Jam00

Ja, kanst du mir das mal sagen wie das geht?

Auf was bezieht sich die Frage jetzt ? Was genau willst du wissen ?

Was mir auch aufgefallen ist: _Overroll($fGui,$fObjekt,$fTitel,$fFunc)
Warum soll der Titel übergeben werden, das ist überflüssig, da du schon $fGui übergibst und dieses ebenso als Titel verwendet werden kann.
Übrigens machst du dir viel zu viel Arbeit - warum die Mausposition mit der Controlposition vergleichen? Hast du dir schon mal angeschaut, was GUIGetCursorInfo() zurückgibt? ...$array[4] = ID of the control that the mouse cursor is hovering over
Du brauchst also nur die Ctrl-ID mit $array[4] vergleichen.

P.S. Nicht verzagen, wenn wir hier dein Skript zerlegen. Das ist positive Kritik und hilft dir beim Lernen.

Edit Falls du schon mitten im Skript steckst und nachträglich die Variablen deklarieren möchtest, ist dieses eine kleine Hilfe: Variablendeklaration automatisiert - Opt("MustDeclareVars", 1)

Also hier mal ein Beispiel :

Func _Overroll($fGui,$fObjekt,$fFunc)
Local $fPos = GUIGetCursorInfo ($fGui)
If $fPos[4] = $fObjekt Then
If $Objektstatus = False Then
Call ($fFunc,"1")
$Objektstatus = True
EndIf
Else
If $Objektstatus = True Then
Call ($fFunc,"2")
$Objektstatus = False
EndIf
EndIf
If $Objektstatus <> False And $Objektstatus <> True Then $Objektstatus = False
EndFunc

Also die Variable $fPos solltest du als Local deklarieren . So wird es keine Komplikationen mit einer Variable geben die unter dem gleichen Namen Global deklariert ist .
Wenn ich später Zeit habe werde ich dein Script mal so anpassen , das du nicht innerhalb der Function auf außerhalb der Funktion deklarierte Variablen zugreifen mußt. Hab jetzt keine Zeit dazu , bin auf den Sprung.

Edit :
Dein Beispielscript

Deine Overroll.au3

Schau dir das ganze mal an . Wenn du noch Fragen hast , dann kannst du mir auch eine PN schicken .

Local heißt , das die Variable nur innerhalb der Func existiert und nach dem verlassen der Func nicht mehr .Return ist der Rückgabewert der Func. Du kannst eigentlich alle Variablentypen für die Rückgabe benutzen . Würdest du z.B. in deiner _Overroll Funktion Return $fPos schreiben, dann würde das das Ergebnis deines GUIGetCursorInfo ($fGui) zurückgeben. Das mit dem neusetzen des Bildes haste recht. Man kann auch noch abfragen ob des aktuelle Pic das anzuzeigende Pic ist .Ich wollte dir nur ein Beispiel für eine Universell einsetzbare Func geben. Deine Func _Overroll kann man auch ohne irgendwelche vorher definierte Variablen oder Funktionen nutzen .

Edit: Normalerweise kann man auch im Hauptscript Variablen Local definieren , auf diese kann man dann nicht innerhalb von Funktionen zugreifen. Eigentlich sollte das so sein, aber Autoit hat da eine Bug. Alle Local deklarierte Variablen werden im Hauptscript als Global deklariert .
Folgendes Beispiel sollte eigentlich in der Messagebox gar nicht´s ausgeben ,macht es aber. En Bug halt .