CLD ATM + Car Wash

Tematy związane z tworzeniem dużych aplikacji. Zaganiednia dotyczące architektury oraz zasad tworzenia optymalnych rozwiązań.
Awatar użytkownika
Krzysztof Szewczyk
Posty: 37
Rejestracja: 19 maja 2011 02:48
Wersja środowiska: LabVIEW 2017
Lokalizacja: Koszyce
Kontakt:

CLD ATM + Car Wash

Post 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 519 razy
Dzięki!
Ostatnio zmieniony 04 maja 2018 12:11 przez Krzysztof Szewczyk, łącznie zmieniany 1 raz.
Obrazek
Awatar użytkownika
dajpanspokój
Posty: 145
Rejestracja: 05 lis 2015 08:15
Wersja środowiska: LabVIEW 2016

Re: CLD ATM

Post autor: dajpanspokój »

Ladnie, wyglada jak komplecik :o)
Obrazek
PiDi
Posty: 641
Rejestracja: 31 gru 2010 01:36
Wersja środowiska: LabVIEW 2017
Lokalizacja: Katowice

Re: CLD ATM

Post 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.
ObrazekObrazekObrazekObrazek
Awatar użytkownika
Krzysztof Szewczyk
Posty: 37
Rejestracja: 19 maja 2011 02:48
Wersja środowiska: LabVIEW 2017
Lokalizacja: Koszyce
Kontakt:

Re: CLD ATM

Post autor: Krzysztof Szewczyk »

Dzięki za uwagi, mam jeszcze ponad miesiąc do CLD, chciałbym wszystkie przykłady zrobic.
Obrazek
Awatar użytkownika
Krzysztof Szewczyk
Posty: 37
Rejestracja: 19 maja 2011 02:48
Wersja środowiska: LabVIEW 2017
Lokalizacja: Koszyce
Kontakt:

Re: CLD ATM + Car Wash

Post autor: Krzysztof Szewczyk »

Cześć,

No teraz kolej na Car Wash...

Car Wash.zip
(394.39 KiB) Pobrany 504 razy
Obrazek
Awatar użytkownika
Krzysztof Szewczyk
Posty: 37
Rejestracja: 19 maja 2011 02:48
Wersja środowiska: LabVIEW 2017
Lokalizacja: Koszyce
Kontakt:

Re: CLD ATM + Car Wash

Post autor: Krzysztof Szewczyk »

cześć
udało się 81%

pozdrawiam
Obrazek
Awatar użytkownika
micard
Posty: 207
Rejestracja: 30 wrz 2011 11:28
Wersja środowiska: LabVIEW 2017
Kontakt:

Re: CLD ATM + Car Wash

Post autor: micard »

Gratulacje!
PiDi
Posty: 641
Rejestracja: 31 gru 2010 01:36
Wersja środowiska: LabVIEW 2017
Lokalizacja: Katowice

Re: CLD ATM + Car Wash

Post autor: PiDi »

Gratulacje! To teraz jeszcze CLA i jazda na CLA Summit w przyszłym roku :)
ObrazekObrazekObrazekObrazek
ODPOWIEDZ