brauche verbesserungsvorschläge für minispiel bot

[MAXPLO]

Hallo community!
Ich mich heutemal in autoit eingearbeitet und erstelle grade einen bot für das holzhack minigame.
das ist mein code:

$versatz=1
while 1=1
	$coor = PixelSearch( 591 + $versatz, 516, 631 + $versatz, 530, 14328109, 20, 3 )
	While @error = 0
		send("{Left}")
		$coor = PixelSearch( 591 + $versatz, 516, 631 + $versatz, 530, 14328109, 20, 3 )
	Wend
		$coor = PixelSearch( 591 + $versatz, 621, 631 + $versatz, 636, 15247933, 20, 3 )
	While @error = 0
		send("{right}")
		$coor = PixelSearch( 591 + $versatz, 621, 631 + $versatz, 636, 15247933, 20, 3 )
	Wend
	$i=0
	while $i <= $versatz
		$versatz = $versatz + 0.0005
		$i = $i + 1
	Wend
WEND

Am anfang klappt es super, dch danach beginnt er immer mehr auszulassen bis er keine mehr trifft.
Was kann ich dagegen machen?
mfg MAXPLO

PS: es muss nicht unbedingt ein neuer code sein ich brauche eigentlich nur einen denkanstoß

[~Freaky~]

Wieso so umstendlich?
Und für was ist dein $versetzt ?
benutz isarray bei pixelsearch und such dir feste punkte

[MAXPLO]

ok, hab jez net ganz verstanden... das $versatz steht dafür, dass er mir den balken in dem überprüft wird immer weiter nach rechts schiebt, weil die dinger ja auch immer schneller werden und ich immer früher drücken lassen muss.
was macht isarray?

[~Freaky~]

benutz die hilfe (F1)
damit kanst du gucken ob pixelsearch was gefunden hat oder nicht

[uni3]

warum so umständlich
welche auflösung haste

[uni3]

HotKeySet ( "{F1}" , "Start" ) ;Startet mit F1
HotKeySet ( "{ESC}" , "Stop" ) ;Schliesst mit ESC
Func Start () ;Hier Start
While 1
$FS = PixelSearch(315, 174, 317, 176, 0x36363D, 2)
If @error Then
$FS = PixelSearch(450,419,465,426,0xBB7A2A,9) ;Links Scannen den ändern
If not @error Then
Send("{left}")
EndIf
EndIf
$GS = PixelSearch(315, 174, 317, 176, 0x36363D, 2)
If @error Then
$GS = PixelSearch(450,520,460,529,0xBB7A2A,9) ;Rechts scannen den ändern
If not @error Then
Send ("{right}")
EndIf
EndIf
WEnd
EndFunc


While 1
sleep (1)
WEnd

Func Stop () ;Ausschalter
Exit
EndFunc


cordinaten und farbe muste selber suchen keine lust rauszusuchen

[~Freaky~]

"if not @error" ist müll
mach es so

$pixel = pixelsearch.....
if isarray ($pixel) then 
....
endif

[uni3]

Originally Posted by ~Freaky~

"if not @error" ist müll
mach es so

$pixel = pixelsearch.....
if isarray ($pixel) then 
....
endif

meinste meienbots laufenalle aleine dn ganzen tage habe 48 lvl21 charas xD *GRINS*

[MAXPLO]

Originally Posted by uni3

warum so umständlich
welche auflösung haste

ich hab in dem fall mal mit 1280:1024 gearbeitet

Originally Posted by uni3

HotKeySet ( "{F1}" , "Start" ) ;Startet mit F1
HotKeySet ( "{ESC}" , "Stop" ) ;Schliesst mit ESC
Func Start () ;Hier Start
While 1
$FS = PixelSearch(315, 174, 317, 176, 0x36363D, 2)
If @error Then
$FS = PixelSearch(450,419,465,426,0xBB7A2A,9) ;Links Scannen den ändern
If not @error Then
Send("{left}")
EndIf
EndIf
$GS = PixelSearch(315, 174, 317, 176, 0x36363D, 2)
If @error Then
$GS = PixelSearch(450,520,460,529,0xBB7A2A,9) ;Rechts scannen den ändern
If not @error Then
Send ("{right}")
EndIf
EndIf
WEnd
EndFunc


While 1
sleep (1)
WEnd

Func Stop () ;Ausschalter
Exit
EndFunc


cordinaten und farbe muste selber suchen keine lust rauszusuchen

was meinst du mit "rechts scannen den ändern"?
was ist denn der unterschied zwischen isarray und @error?
@Uni3: hast du jez einfach random irgendwelche oordinaten eingefügt oder is das ein coe den du selber benutzt?
Also eigentlich wollte ich keinen code vorgeworfen bekommen, ich wollte den selber schreiben, deswegen würd ich halt einfahc wissen, ob ich es einfach komplett falsch angegeangen bin oder ob ich einfahc nur was verbesern. muss.

[~Freaky~]

Originally Posted by uni3

meinste meienbots laufenalle aleine dn ganzen tage habe 48 lvl21 charas xD *GRINS*

Es ist aber sogesagt falsch. (Ich brauche nur 1 Acc und spiele genauso offt wie du nur etwas schneller *GRINS*)

[dpjeezy89]

if not @error then

ist überhaupt kein müll
isarray und @error gehen da beide aber das eine ist nicht weniger wert als das andere
oder soll ich sagen beides ist müll

$Pixel = PixelSearch(0, 0, 5, 5, 0xFFFFFF, 10)
If UBound($Pixel) >= 1 Then MsgBox(0, "omg", "Es gibt viele möglichkeiten")

hm

[Trøjan]

Ich glaub Freaky meinte nur @error is müll, weil es ja sein kann das andere Funktionen voher einen anderen Wert auf @error schreiben...

[dpjeezy89]

@error
bezieht sich immer auf die vorherige function
außer wenn man das so macht
if $var = @error then oder
if test() = @error then