Was ist daran falsch gescriptet

[Operation Walküre]

wie die überschrift schon sagt,was ist daran falsch (soll i-wan mal ein fischöffner werden)xD

#Region ;**** Directives created by AutoIt3Wrapper_GUI ****
#AutoIt3Wrapper_Res_Language=1031
#EndRegion ;**** Directives created by AutoIt3Wrapper_GUI ****

HotKeySet ( "{Numpad7}" , "Start" )
HotKeySet ( "{Numpad8}" , "Ende" )

While (1)
Sleep (1000)
Wend

Func Start ()
While (1)

$coord= PixelSearch (854, 411,1010, 694, 0x970000 )
If @error Then
Sleep (500)
MouseClick("right",$coord[0],$coord[1])
Sleep (500)
EndIf
ExitLoop
Wend
Sleep(2000)
Endfunc

Func Ende ()
Exit
EndFunc

[BlackFog]

Wie wäre es mit einer Fehlerbechreibung?

[Operation Walküre]

kommt keine aber trozdem machts net das was es machen soll,weil es passiert rein gar nix

[BlackFog]

$coord= PixelSearch (854, 411,1010, 694, 0x970000 )
If @error Then
Sleep (500)
MouseClick("right",$coord[0],$coord[1])
Sleep (500)
EndIf
ExitLoop
Wend
Sleep(2000)
Endfunc


Du hast das stehen
if @error then blabla

Ist es richtig, dass nur etwas passieren soll, wenn ein Fehler da ist ? Wenn nicht müsste es

if not @error then balalbla

heißen.


Mfg
Blackfog

[schlurmann]

Ich habe zwar keine Ahnung von AutoIT, aber

While (1)
Sleep (1000)
Wend

sieht mir ziemlich nach ner Endlosschleife aus, in dem Fall würde die Pixelsearch - Mausklick Schleife nie erreicht.

Correct me if I'm wrong...

[Operation Walküre]

also ich will halt wenn in den berreich der angegebener pixel gefunden wird die aktion ausgeführt wird

[BlackFog]

Dann muss es heißen if not @ error.

[DamageBreaker]

Originally Posted by Operation Walküre

wie die überschrift schon sagt,was ist daran falsch (soll i-wan mal ein fischöffner werden)xD

#Region ;**** Directives created by AutoIt3Wrapper_GUI ****
#AutoIt3Wrapper_Res_Language=1031
#EndRegion ;**** Directives created by AutoIt3Wrapper_GUI ****

HotKeySet ( "{Numpad7}" , "Start" )
HotKeySet ( "{Numpad8}" , "Ende" )

While (1)
Sleep (1000)
Wend

Func Start ()
While (1)

$coord= PixelSearch (854, 411,1010, 694, 0x970000 )
If ($coord= true) Then
Sleep (500)
MouseClick("right",$coord[0],$coord[1])
Sleep (500)
EndIf
Wend
Endfunc

Func Ende ()
Exit
EndFunc

gehts vllt so?

[BlackFog]

Coord wird niemals "true" zurückgeben. Coord gibt ein integer Array mit den x und y Koordinaten des gefundenen Pixels zurück.

[DamageBreaker]

Originally Posted by BlackFog

Coord wird niemals "true" zurückgeben. Coord gibt ein integer Array mit den x und y Koordinaten des gefundenen Pixels zurück.

aber wenn er an dem punkt den pixel findet heißt es doch das wahr ist oder net?

ich kann kein autoit XD

[BlackFog]

Wenn du's nicht kannst und noch weniger Ahnung von Funktionen und deren Rückgabewerte hast, dann solltest du Spekulationen einfach lassen .

Mfg
Blackfog

[DamageBreaker]

Originally Posted by BlackFog

Wenn du's nicht kannst und noch weniger Ahnung von Funktionen und deren Rückgabewerte hast, dann solltest du Spekulationen einfach lassen .

Mfg
Blackfog

stimmt schon sry>.<

[Computerfreek]

Wie schon gesagt wurde, du musst If not @Error then reinschreiben, ansonsten macht er nur weiter wenn eben ein error (den Pixel nicht gefunden) zurückgegeben wird.
Damit hängt sich das dann zu einer Endlosschleife auf und wenn man es nicht bemerkt haut das deine Geschwindigkeit vom PC ganz schön runter.

[Bot_interesierter]

Originally Posted by Computerfreek

....
Damit hängt sich das dann zu einer Endlosschleife auf und wenn man es nicht bemerkt haut das deine Geschwindigkeit vom PC ganz schön runter.

Nein tut es nicht, er verlässt die Schleife nach dem IF clause und er hat überall sleeps drinne, die Whileschleife in seiner Funktion wird bei jeder Ausführung abgebrochen, so könnte er sie sich auch gleich Sparen, die ist völlig Sinnlos.

[verT!c4L]

Sag ich doch...