Problem z zadaniem domowym w AutoIt

[pro201]

Witam otóż mam mały problem chodzę do 4 klasy informatycznej i dostaliśmy zadanie dodatkowe otóż dostaliśmy taki kod w autoit :

#include <GuiConstantsEx.au3>
#include <AVIConstants.au3>
#include <TreeViewConstants.au3>

HotKeySet("{F6}", "Praca")
HotKeySet("{F7}", "Pauza")
GUICtrlCreateLabel("F6 start" & @CR &"F7 stop", 10, 10, 180, 50)

GUISetState(@SW_SHOW)

Pauza()

func Pauza()
	While 1
		$Msg = GUIGetMsg()
		Switch $Msg
			Case $GUI_EVENT_CLOSE
				Exit
		EndSwitch
	WEnd
endfunc

func Praca()

	for $i = 0 to 9999999
	GuiCreate("Program", 150, 150)
	$lewo     =	1
	$gora     =	1
	$prawo    = 1600
	$dol      = 1600
	$kolor    = 0xB9525F
while 1
$a = PixelSearch($lewo, $gora, $prawo, $dol, $kolor )
If Not @ Then
EndIf
MouseClick("left",$a[0],$a[1])
WEnd

	Next
EndFunc

Ma on szukać koloru na pulpicie a potem go kliknąć i otworzyć plik. Nie umiem jednak go naprawić. Nauczyciel powiedział, że coś jest w nim źle lecz nie umiem się tego doszukać według mnie wszystko jest dobrze. Umie mi ktoś wskazać ten błąd ?

[Hans Kloss PL]

Ten fragment jest skopany:

while 1
$a = PixelSearch($lewo, $gora, $prawo, $dol, $kolor )
If Not @ Then
EndIf
MouseClick("left",$a[0],$a[1])
WEnd

Kod powtarza się w nieskończoność, MouseClick jest poza instrukcja warunkową.

Jakbyś chociaż uruchomił ten kod to byś sam znalazł przynajmniej drugi błąd.

[MistrzMaster]

    for $i = 0 to 9999999
    GuiCreate("Program", 150, 150)
    $lewo =    1
    $gora =    1
    $prawo = 1600
    $dol = 1600
    $kolor = 0xB9525F

Czy ja dobrze rozumiem ten kod? Chcesz stworzyć gui 9999999 razy? Poza tym chyba nie zamknąłeś tej pętli for tak jak zamierzałeś, kod z pętli while 1 (kliknięcie) wykona się nawet w przypadku niepowodzenia, bo masz pustego ifa a mouseclick jest poza tą instrukcją warunkową.

 

PS. Takie rzeczy robi się w 4 klasie informatycznej?

[JDaniel1]

@pro201

Ty się w szkole uczysz AutoIT? Ciekawe

#include <GuiConstantsEx.au3>
#include <AVIConstants.au3>
#include <TreeViewConstants.au3>

HotKeySet("{F6}", "Praca")
HotKeySet("{F7}", "Pauza")
GuiCreate("Program", 150, 150)
GUICtrlCreateLabel("F6 start" & @CR &"F7 stop", 10, 10, 180, 50)

GUISetState(@SW_SHOW)



	While 1
		$Msg = GUIGetMsg()
		Switch $Msg
			Case $GUI_EVENT_CLOSE
				Exit
		EndSwitch
	WEnd

func Praca()
	$lewo     =	1
	$gora     =	1
	$prawo    = 1600
	$dol      = 1600
	$kolor    = 0xB9525F
while 1
$a = PixelSearch($lewo, $gora, $prawo, $dol, $kolor )
If Not @ Then
	MouseClick("left",$a[0],$a[1])
EndIf
Sleep (100)
WEnd
EndFunc
Func Pauza()
	Sleep (100)
	EndFunc

Masz kod naprawiony oraz jest teraz bardziej wydajny. Jeżeli czegoś nie rozumiesz to pisz.

 

 

PS: Po co biblioteka od plików AVI w aplikacji która szuka tylko pixela?

[pro201]

Nasz nauczyciel chciał sprawdzić czy umiemy posługiwać się innymi programami niż C++ czyli ile pracy wkładamy w to są to podobne języki.

 

Hans Kloss PL

Co do Twojego postu to sprawdzałem i nie wykrywało mi żadnego błędu.

 

MistrzMaster

Niestety. Jak większość klasy nie rozumie o co chodzi to w kółko wałkowane jest to samo.

 

Dziękuje wszystkim za pomoc.

[Agent K]

Moim zdaniem w ogóle nie podobne