Button betätigen nach Ereignissänderung

Frost0071

Juhu ich habe es hinbekommen Sogar mit einer zusätzlichen Farbe, für mich schon als Erfolg zu werten

Allerdings eins bekomm ich noch nicht hin und zwar das ganze in einer Endlosschleife laufen zu lassen. Bekomme immer einen Error.
Kann mir da vielleicht nochmal jemand helfen?

Hier mal das Script:

AutoIt

;-- TIME_STAMP   2017-01-22 21:16:57


#include <GUIConstants.au3>
#include <GUIConstantsEx.au3>
#include <WindowsConstants.au3>


Global $g_aCheckPos


; $g_iColor = Hintergrundfarbe des SciTE-Window
Global $g_iColor = 0x87c323, $g_iColor1 = 0x34302d, $g_iCheckCount = 1, $g_iSleep = 100, $g_iTimeout = 3000, $g_x = $g_aCheckPos, $g_y = $g_aCheckPos


Global $g_ixPosClick = 86, $g_iyPosClick = 508


Global $g_aArray = [[$g_x + 290, $g_y + 198, $g_iColor], _
				    [$g_x + 290, $g_y + 198, $g_iColor], _
				    [$g_x + 290, $g_y + 198, $g_iColor], _
					 [$g_x + 325, $g_y + 631, $g_iColor1]]


Global $g_bAdlib = AdlibRegister('_CheckColor', $g_iTimeout)


While $g_bAdlib = True
	Sleep(100)
WEnd


_Exit()


Func _CheckColor()
	Local Static $iCheckCount = 0


		If _PixelArrayCheckColor($g_aArray, $g_iSleep) = True Then
			MouseClick("left", $g_ixPosClick, $g_iyPosClick, 1, 1)
		EndIf


	If _PixelArrayCheckColor($g_aArray, $g_iSleep) = True Then
		ConsoleWrite('+ Bingo!!!' & @CRLF)
	Else
		ConsoleWrite('! Ops...' & @CRLF)
	EndIf


	$iCheckCount += 1
	If $iCheckCount = $g_iCheckCount Then $g_bAdlib = False
EndFunc


Func _PixelArrayCheckColor($aArray, $iSleep = 1)
	Local $iColor
	For $i = 0 To UBound($aArray) -1 Step 1
		$iColor = PixelGetColor($aArray[$i][0], $aArray[$i][1])
		If $iColor <> $aArray[$i][2] Then Return False
		ConsoleWrite('- $iColor(' & _Hex($iColor) &') = PixelGetColor('& _Hex($aArray[$i][2]) & ')' & @CRLF)
		Sleep($iSleep)
	Next


	Return True
EndFunc


Func _Hex($iDec)
	Return '0x' & Hex($iDec, 6)
EndFunc


Func _Exit()
	AdlibUnRegister('_CheckColor')
	Exit
 EndFunc

Alles anzeigen

Danke

**alpines**

Zitat von Frost0071

Bekomme immer einen Error.

Es wäre nicht schlecht wenn du uns den Error auch mitposten würdest.

**Oscar**

Ein Array-Fehler offensichtlich!
In Zeile 48 und 49 wird auf ein drittes Array-Element ($aArray[$i][2]) zugegriffen, obwohl es nur 2 Elemente gibt. Hier wäre also nur 0 oder 1 erlaubt.

Frost0071

Naja ich weis ja nicht mal so richtig was ich alles in eine While Schleife setzen soll.

wenn ich diese Funktion hier in die schleife setzt:

AutoIt

Func _CheckColor()
   While 1
	Local Static $iCheckCount = 0


		If _PixelArrayCheckColor($g_aArray, $g_iSleep) = True Then
			MouseClick("left", $g_ixPosClick, $g_iyPosClick, 1, 1)
		EndIf


	If _PixelArrayCheckColor($g_aArray, $g_iSleep) = True Then
		ConsoleWrite('+ Bingo!!!' & @CRLF)
	Else
		ConsoleWrite('! Ops...' & @CRLF)
	EndIf


	$iCheckCount += 1
	If $iCheckCount = $g_iCheckCount Then $g_bAdlib = False
	   WEnd
	EndFunc

Alles anzeigen

Läuft es zwar einmal durch aber dann become ich folgenden Fehler:

>"C:\Program Files (x86)\AutoIt3\SciTE\..\autoit3.exe" /ErrorStdOut "D:\Users\\Desktop\uu.au3"
! Ops...
"D:\Users\\Desktop\uu.au3" (28) : ==> Cannot make existing variables static.:
Local Static $iCheckCount = 0
Local Static ^ ERROR
>Exit code: 1 Time: 3.19

**alpines**

Schau doch mal was als Fehlermeldung angegeben wird. "Cannot make existing variables static".
Du hast die $iCheckCount schon vorher deklariert und deklarierst sie hier nochmal als statische Variable neu, das kann natürlich nicht klappen.

Entweder gibts du ihr einen neuen Namen oder nimmst die Deklaration vorher raus.

Frost0071

Super Danke! Jetzt läuft es und macht was ich will.

Vielen Dank für die Unterstützung und sorry wegen den vielen "doofen" Fragen.

**alpines**

Zitat von Frost0071

Vielen Dank für die Unterstützung und sorry wegen den vielen "doofen" Fragen.

Dafür sind wir schließlich da.

Static wird im Normalfall in AutoIt leider fast von niemandem verwendet obwohl es ziemlich praktisch ist.
Es erstellt die Variable einmal mit dem Wert den man ihr übergibt und danach wird der Wert verwendet der drinne eingespeichert wird.

Sozusagen globale Variablen für eine Funktion.

**Bitnugger**

@Frost0071
In dem geänderten Script in Post #21 sehe ich, dass du mich nicht verstanden hast!

AutoIt

Global $g_aCheckPos

Hier reicht es nicht, wenn du die Array-Variable deklarierst - du musst ihr auch Werte zuweisen! Bei mir hatte ich die Koordinaten der Maus verwendet, du solltest hier die Position des Windows verwenden, in dem du mit PixelGetColor() die Farben prüfen willst und vor allem natürlich auch testen, ob das Programm gestartet wurde und das Window des Programms auch sichtbar ist.

AutoIt

Global $g_iColor = 0x87c323, $g_iColor1 = 0x34302d, $g_iCheckCount = 1, $g_iSleep = 100, $g_iTimeout = 3000, $g_x = $g_aCheckPos, $g_y = $g_aCheckPos

Denn $g_x und $g_y bekommen hier dann natürlich auch keinen Wert zugewiesen und sind somit absolut nutzlos/überflüssig.

AutoIt

Global $g_aArray = [[$g_x + 290, $g_y + 198, $g_iColor], _
		    [$g_x + 290, $g_y + 198, $g_iColor], _
		    [$g_x + 290, $g_y + 198, $g_iColor], _
		    [$g_x + 325, $g_y + 631, $g_iColor1]]

Hier gibst du gleich 3x hintereinander dieselben Posititionen an... das ist natürlich Quatsch.

In diesem Array sollen die (relativen) Positionen und Farben der "gemalten Controls" innerhalb des Windows "Mach3 CNC Demo" gespeichert werden. Dazu musst du natürlich wissen, an welcher Position sich dieses Window gerade befindet und ob es sichtbar ist, denn anderfalls liefert dir PixelGetColor() falsche Werte, wenn es z. B. minimiert wurde oder von anderen Fenstern verdeckt wird!

Morgen mal schauen, was mein Zeitplan sagt... evtl. schreibe ich dir ein Demo-Script...