Strona 1 z 1

CLD ATM + Car Wash

: 18 kwie 2018 14:49
autor: Krzysztof Szewczyk
Cześć,

Czy mógłby któryś z kolegów sprawdzić mój przykład zadania na CLD?
Automatic Teller Machine.zip
(269.44 KiB) Pobrany 523 razy
Dzięki!

Re: CLD ATM

: 18 kwie 2018 16:08
autor: dajpanspokój
Ladnie, wyglada jak komplecik :o)

Re: CLD ATM

: 18 kwie 2018 20:48
autor: PiDi
Ujemne punkty:
- Brak ikon kontrolek
- Brak ikony głównego VI
- Brak description w VI
- Elementy na block diagram nachodzą na siebie (np. DeQueueHMI.vi)
- Kontrolki leżące swobodnie na panelu (umieścić w odpowiednich case w event structure)
- Wejście Action do Card State.vi powinno być typedefem (to raczej byłoby zakwalifikowane jako błąd - skoro jest enum używany w wielu miejscach, to typedefa mu)
- Wejście MSG Index do Prepare MSG.vi powinno być typedefowanym enumem (to niekoniecznie byłoby zakwalifikowane jako błąd, ale mogłoby jako trudne do utrzymania/edycji)

Działanie wygląda ok, architektura wygląda ok. Max punktów minus parę drobnych za powyższe. Jeśli z ręką na sercu zrobiłeś to w 4h, to spokojnie sobie poradzisz na rzeczywistym egzaminie (rzeczywisty jest zwykle nieco trudniejszy od przykładowych).

Z uwag niezwiązanych z oceną samego CLD: update GUI mógłbyś zrobić w event structure zamiast w oddzielnej pętli, zastępując kolejkę HMI user eventem. Dzięki temu miałbyś całą obsługę GUI w jednym miejscu.

Re: CLD ATM

: 19 kwie 2018 09:35
autor: Krzysztof Szewczyk
Dzięki za uwagi, mam jeszcze ponad miesiąc do CLD, chciałbym wszystkie przykłady zrobic.

Re: CLD ATM + Car Wash

: 04 maja 2018 12:12
autor: Krzysztof Szewczyk
Cześć,

No teraz kolej na Car Wash...

Car Wash.zip
(394.39 KiB) Pobrany 509 razy

Re: CLD ATM + Car Wash

: 13 cze 2018 11:03
autor: Krzysztof Szewczyk
cześć
udało się 81%

pozdrawiam

Re: CLD ATM + Car Wash

: 14 cze 2018 08:04
autor: micard
Gratulacje!

Re: CLD ATM + Car Wash

: 14 cze 2018 21:46
autor: PiDi
Gratulacje! To teraz jeszcze CLA i jazda na CLA Summit w przyszłym roku :)