Jump to content

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


Recommended Posts

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 by mabolas
Link to comment
  • 2 weeks later...
  • 1 month later...

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 by mabolas
Link to comment

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.