Was ist daran falsch gescriptet

02/14/2009 21:26 Operation Walküre#1
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
02/15/2009 10:08 BlackFog#2
Wie wäre es mit einer Fehlerbechreibung?
02/15/2009 10:50 Operation Walküre#3
kommt keine aber trozdem machts net das was es machen soll,weil es passiert rein gar nix
02/15/2009 10:53 BlackFog#4
$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
02/15/2009 10:54 schlurmann#5
Ich habe zwar keine Ahnung von AutoIT, aber
Code:
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...
02/15/2009 10:54 Operation Walküre#6
also ich will halt wenn in den berreich der angegebener pixel gefunden wird die aktion ausgeführt wird
02/15/2009 11:01 BlackFog#7
Dann muss es heißen if not @ error.
02/15/2009 11:01 DamageBreaker#8
Quote:
Originally Posted by Operation Walküre View Post
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?
02/15/2009 11:39 BlackFog#9
Coord wird niemals "true" zurückgeben. Coord gibt ein integer Array mit den x und y Koordinaten des gefundenen Pixels zurück.
02/15/2009 12:29 DamageBreaker#10
Quote:
Originally Posted by BlackFog View Post
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
02/15/2009 12:34 BlackFog#11
Wenn du's nicht kannst und noch weniger Ahnung von Funktionen und deren Rückgabewerte hast, dann solltest du Spekulationen einfach lassen ;).

Mfg
Blackfog
02/15/2009 14:16 DamageBreaker#12
Quote:
Originally Posted by BlackFog View Post
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>.<
02/15/2009 14:27 Computerfreek#13
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.
02/16/2009 21:52 Bot_interesierter#14
Quote:
Originally Posted by Computerfreek View Post
....
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.
02/16/2009 23:02 verT!c4L#15
Sag ich doch... ;)