Zwischen Funktionen wechseln?

[Requi]

Func _palla()
		$palla = Not $palla
		If $palla Then
			While 1
				Global $pPalla = PixelSearch(0, 0, @DesktopHeight, @DesktopWidth, "0xECFBF7")
				If IsArray($pPalla) Then
					WinWaitActive("www.DarkOrbit.com")
					MouseMove($pPalla[0], $pPalla[1])
					MouseClick("LEFT")
					Sleep(4000)
				Else
					_fliegen()
				EndIf
			WEnd
		Else
			_pause()
		EndIf
	EndFunc

	Func _fliegen()
		While 1
			If Not IsArray($pPalla) Then
				WinWaitActive("www.DarkOrbit.com")
				$randomx = Random(0, GuiCtrlRead($Input1))
				$randomy = Random(0, GuiCtrlRead($Input2))
					If (Hex(PixelGetColor($randomx, $randomy),6) <> 000000) Then
						MouseMove($randomx, $randomy)
					EndIf
				MouseClick("LEFT")
				Sleep(4000)
			Else
				_palla()
			EndIf
		WEnd
	EndFunc

Der soll fliegen, aber wenn der Pixel auftaucht dahin klicken.
Wenn der Pixel nicht verfügbar ist, soll er wieder fliegen, bis der Pixel wieder da ist.

Es geht um diesen Pixel:
0xECFBF7

Danke :*

[butter123]

While Sleep(10)
While PixelGetColour($x,$y) <> $Colour
Fliegen
WEnd
MouseClick("left",$x,$y)
WEnd

[YatoDev]

pixelsearch()
If not @error
nimm das mal für auf box klicken

[Lawliet]

Würde das ganze aus ner dritten Funktion aufrufen, rekursion sollte man vermeiden.

Func DoStuff
$isPalaInRange = FindPalaPixel()
if(isPalaInRange)
     _collectPala()
_flieg()
endfunc

_collectPala() sollte so lang laufen, bis nichts mehr zu sehnen ist. _flieg immer nur einmal durchlaufen lassen.

[Achat]

Rekursion führt zum Absturz, , ist also unbedingt zu vermeiden, wie Lawliet! schon sagte.

MfG

[BladeTiger12]

Ich glaube Lawliet meint einfach nur, dass du mehrere Funktionen
erstellen sollst und die dann Aufrufen sollst.

[lolkop]

Originally Posted by Requi

Ja das weiß ich auch .

Hab es sogar geschafft indem ich ein Funktion gelöscht habe.

If Not isarray() Then
Fliegenteil
Else
Pallateil
EndIf

mal zur info... rekursion bedeutet, das die funktion sich selbst x mal aufruft. es gibt also IMMER eine abbruch bedingung, welche unendliche rekursion vermeiden muss.

rekursion ist gegenüber iterativer alternativen immer ineffizienter, dabei aber immer deutlich einfacher formulierbar. sie ist dadurch keinesfalls wie oben geschrieben immer zu vermeiden.
insbesondere weil in autoit das parsen des quellcodes in der regel durchaus die meisten ressourcen frisst.

kleines beispiel anhand einer funktion, welche hex strings umkehrt:

Func IterativeReverseHex($hexString)
	Local $return
	For $i = StringLen($hexString) + 1 To 1 Step -2
		$return &= StringMid($hexString, $i, 2)
	Next
	Return '0x'&$return
EndFunc

Func RecursivReverseHex($hexString, $return="")
	If $hexString = "" Then Return '0x'&$return
	Return RecursivReverseHex(StringTrimLeft($hexString,2), StringLeft($hexString,2)&$return)
EndFunc

ConsoleWrite(RecursivReverseHex('05000000')&@CRLF)
ConsoleWrite(IterativeReverseHex('05000000')&@CRLF)

hier ist schön erkennbar, das sich viele probleme rekursiv als zweizeiler lösen lassen, während iterativ deutlich mehr logik und oft auch deutlich ineffizientere methoden notwendig sind (im beispiel stringmid, welches immer ineffizienter ist als stringleft).
soviel zum thema der rekursion...

was dein eigentlicher code aber sichtlich macht ist, das du allgemein ein sehr schlechtes verständnis von scriptsprachen hast. da scriptsprachen in der regel nicht multithreading fähig sind, müssen wir für einen ersatz sorgen, indem wir uns der einfachen automatenlogik bedienen.

diese besagt nichts anderes, als das sich unser programm zu jedem beliebigen zeitpunkt in einem bestimmten zustand befindet. je nach zustand reagieren wir dann unterschiedlich.
sobald du diese aussage wirklich verstanden hast, wirst du in einem programm NIEMALS mehr als eine endlosschleife (iterativ natürlich) benötigen.

nichts anderes hast du dort übrigens intuitiv in deiner eigenen lösung gemacht. da ich davon ausgehe, das dein code aber mehr als nur diese 2 funktionen beinhaltet, solltest du das ganze noch weiter vereinfachen, bis du wirklich nurnoch eine hauptschleife hast, in der dein ganzes programm läuft.