CLA Elevator&ATM Practice, Please review and comment (OOP Style)

[mabolas]

Hello,

Recently I'm trying to get my CLA and this is my third time practicing this example.

I took about 3 hour and 20 minutes to finish it.
(Actually I planned spent 3 hour to finish it , it's already the third time, I barely analyzed it , just coding)

The first hour I took was for establishing ancestor classes, set up public data, sharing mechanism / VI
(open/close queue references, access data base, main running VI framework.....etc)

The second hour I spent on modules (error handler, user console, display console and sensor )

The third hour is implementation of the main module : Controller 

and I took another 20 minutes because some requirement tags I have no idea where to put.


I know lots of people suggest do not use OOP in CLA, because it takes too much time.
But I've been used to OOP style and not very familiar with tradition QMH programming.

Something Still Confusing me , it's appreciated if leave some comments or suggestions to me.


1. I still think it's not fast enough, did I put too much code or redundant comments in it?
(when I took my CLD, lots things slowed me down, like nervous,misunderstanding questions, strong AC, noise...etc , I think those will happen again in the CLA, )

2. Did I comment in the proper way ? Did I comment clear enough for developers?
(which is the NI examiner), if I don't build any child message class (for saving time),
just leave the comments to tell how to build them, is that OK? I'm not sure whether I leave enough comments for understanding 

 

3. I didn't build the timer FGV module, instead, I encapsulate a "local timer" in to controller.lvclass and directly manipulate it in the loop, is that an acceptable method in CLA?

(I use this method in real world a lot, since I just need a local timer in one loop, I do not need set up another timer FGV to confuse me in the project)

4.Also the example has a data base module, I do not either. It's more flexible to me let the ancestor class 

handle the data base then share it to every child class.

Elevator.zip

Edited October 7, 2015 by mabolas

[mabolas]

I've made ATM practice too. 
I think it's little easier than Elevator.

I don't know why, maybe I've practiced at CLD.

 

ATM.zip

Edited October 7, 2015 by mabolas

[mabolas]

I made anther practice, this time I try typing more words and less coding work.

ATM.zip

Elevator.zip

Edited October 21, 2015 by mabolas

[eberaud]

There is rarely feedback on practices on LAVA.

 

You'll have better chances here: http://forums.ni.com/t5/Certification/Sample-Exam-Solutions-for-Review/td-p/1824703/page/27

[mabolas]

Thanks the advice, I posted there and seems not enough time to get feedback.

 

(I'll take exam next week, so, this style would be my final framework and architecture,

the rest I can do just more practices and pray......)

[mabolas]

I think this kind of architect is OK to NI.

 

I got 95.7 points at my CLA exam .

 

Architeture development could still do a little bit better.

And I didn't tag the requirement tags perfectly.

Anyway, this result is good enough to me.

 

I'll tag my last practice examples as the solution.

Edited December 5, 2015 by mabolas