CLD ATM + Car Wash

[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 677 razy

Dzięki!

[dajpanspokój]

Ladnie, wyglada jak komplecik

[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.

[Krzysztof Szewczyk]

Dzięki za uwagi, mam jeszcze ponad miesiąc do CLD, chciałbym wszystkie przykłady zrobic.

[Krzysztof Szewczyk]

Cześć,

No teraz kolej na Car Wash...

Car Wash.zip

(394.39 KiB) Pobrany 653 razy

[Krzysztof Szewczyk]

cześć
udało się 81%

pozdrawiam

[micard]

Gratulacje!

[PiDi]

Gratulacje! To teraz jeszcze CLA i jazda na CLA Summit w przyszłym roku