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
Has thanked: 4 times
Been thanked: 2 times
Kontakt:

CLD ATM + Car Wash

Post autor: Krzysztof Szewczyk » 18 kwie 2018 14:49

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 91 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
Has thanked: 4 times
Been thanked: 5 times

Re: CLD ATM

Post autor: dajpanspokój » 18 kwie 2018 16:08

Ladnie, wyglada jak komplecik :o)
Obrazek

PiDi
Posty: 626
Rejestracja: 31 gru 2010 01:36
Wersja środowiska: LabVIEW 2017
Lokalizacja: Katowice
Has thanked: 3 times
Been thanked: 3 times

Re: CLD ATM

Post autor: PiDi » 18 kwie 2018 20:48

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
Has thanked: 4 times
Been thanked: 2 times
Kontakt:

Re: CLD ATM

Post autor: Krzysztof Szewczyk » 19 kwie 2018 09:35

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
Has thanked: 4 times
Been thanked: 2 times
Kontakt:

Re: CLD ATM + Car Wash

Post autor: Krzysztof Szewczyk » 04 maja 2018 12:12

Cześć,

No teraz kolej na Car Wash...

Car Wash.zip
(394.39 KiB) Pobrany 68 razy
Obrazek

Awatar użytkownika
Krzysztof Szewczyk
Posty: 37
Rejestracja: 19 maja 2011 02:48
Wersja środowiska: LabVIEW 2017
Lokalizacja: Koszyce
Has thanked: 4 times
Been thanked: 2 times
Kontakt:

Re: CLD ATM + Car Wash

Post autor: Krzysztof Szewczyk » 13 cze 2018 11:03

cześć
udało się 81%

pozdrawiam
Obrazek

Awatar użytkownika
micard
Posty: 178
Rejestracja: 30 wrz 2011 11:28
Wersja środowiska: LabVIEW 2016
Has thanked: 6 times
Been thanked: 6 times
Kontakt:

Re: CLD ATM + Car Wash

Post autor: micard » 14 cze 2018 08:04

Gratulacje!

PiDi
Posty: 626
Rejestracja: 31 gru 2010 01:36
Wersja środowiska: LabVIEW 2017
Lokalizacja: Katowice
Has thanked: 3 times
Been thanked: 3 times

Re: CLD ATM + Car Wash

Post autor: PiDi » 14 cze 2018 21:46

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

ODPOWIEDZ