NI Home
Cart Cart | Help
Hello Events Academic NI Developer Zone Support Solutions Products & Services Contact NI MyNI
You are here: 
NI Home > NI Developer Zone > NI Discussion Forums


Reply
Member
Mike_Le
Posts: 71
0 Kudos

Security System Solution for Review (LV8.0+)

Hey all,

 

I'm back with another CLD practice exam. I'd really appreciate any feedback.

 

I found this exam to be substantially easier than the other samples (Boiler, ATM, Car Wash). It was functional after 2 hours. The total amount of time, including commenting, was less than 3 hours.

 

I'd really appreciate any thoughts on my solution. I've attached a back-converted 8.0 version and a 2011 version. If anyone has problems loading either project, please let me know.

 

Thanks,

Mike

-----

CLD 2012
Member
zakic
Posts: 20

Re: Security System Solution for Review (LV8.0+)

This security scenario was the development section of the CLD when it was a two section exam consisting of a multiple choice section (1-hour) and a development section (3-hours). The difficulty was set appropriately for the allocated time. This security exam is not a representative of the level of the current CLD exams.

 

- Zaki

___________________________________________________________________________________________________________

Zaki Chasmawala | Certification Development Group Manager | Training & Certification | National Instruments | certification@ni.com

 

 

Member
Mike_Le
Posts: 71
0 Kudos

Re: Security System Solution for Review (LV8.0+)

That makes sense. Thanks for the information! I hope the prep I did with the other exams is sufficient.

-----

CLD 2012
Proven Zealot
Jeff·Þ·Bohrer
Posts: 5,358
0 Kudos

Re: Security System Solution for Review (LV8.0+)

Mike,

 

I would like to add one other point that you have overlooked.  Practicing in 8.0 may not be the best idea.  You want to be able to employ new features in the actual test. 

 

For all others: If you can employ a feature of the latest LabVIEW version do so.:smileywink:


"All shall be well and all shall be well and all manner of things shall be well" -Julian
Member
Mike_Le
Posts: 71
0 Kudos

Re: Security System Solution for Review (LV8.0+)

[ Edited ]

Hey Jeff,

 

Thanks for the tip! I am using LabVIEW 2011, but after finishing the test, I "saved for previous version" to 8.0 so the maximum number of people could review my sample exams. I didn't get replies on some of my earlier review requests and figured that using 2011 was preventing people from providing feedback.

 

Mike

-----

CLD 2012
Proven Zealot
Jeff·Þ·Bohrer
Posts: 5,358
0 Kudos

Re: Security System Solution for Review (LV8.0+)

It is more likley that many of us are very busy. A few more CLDs on the market would help free up the time to review practice exams. Better question, what do you wonder about your code? Is there a particular query other than general aassessment? We can address that without in depth code review. you might be surprised that the "any comments" queries take time to evaluate "anything" while what about this? is much easier to answer and any good candidate for CLD would ask "why did the vi analyzer ding this?"

Your style is not the problem. And, for the most part posters here can sling some wires but, a specific question will get more responses.


"All shall be well and all shall be well and all manner of things shall be well" -Julian
Active Participant
Freelance_LV
Posts: 354
0 Kudos

Re: Security System Solution for Review (LV8.0+)

[ Edited ]

Hi

 

I want to know how long you took to code this. i liked the organization. the sub vi icons are bright and clear.

 

i opened the main vi and saw the data cluster constant expanded to full size. it was overlapping the 'Initialize: Macro' enum.  image attached

would suggest you to place the type def control in place of this cluster constant and have a sub vi in the init case, which writes a constant value to this cluster.

 

the 'error status' cluster element is overlapping the event structure in the producer loop. almost all the cluster constants seem to be overlapping their neighbours in all the sub vis that i opened. suggest you check them once

 

since you use the enqueue function in for all the events, you can have one enque function outside the event structure and pass the enum and variant data to it from each event case. i would do that, what you are doing right now is good enough

 

Front Panel: would suggest to make the boolean array backgrounds transparent. only the boolean controls will be visible, though they are in an array. image attached. you can aligh them to be equally distant.  the labels also could be in the same level. the indicator label is lower than that of the controls. want to know why you used the colors for 'Alarm', 'Tamper' etc in the top right corner. suggest to color 'Green' as green and 'blue' as the shade of blue you used for indicators. Keep the texts 'Alarm', 'Tamper' etc in black.

 

 In the 'Exit' loop of the consumer, you could build an array of references for switches and pass them to the For loop, to clear references, instead of using 4 close reference vis in series.

 

In the 'Initialize: Macro' state, you could have had an array constant of enum value, with 4 elements selected.

 

Initialzie Front panel data : you have three boolean array constants where all are equal. wire the same constant to all the three cluster elements, instead

 

i made a change in some of your cases, and straightened some wires.

 

your sub vis have huge FP and BDs. i made some changes.  

 

hmmm... seems like i had got real free time!

 

best of luck, hope you have not yet taken the exam (before reading my post)

 

 

Edit: Forgot to sign off!! here i go..

 

EX-CLD (born: 21-12-2008 expired: 21-12-2010) 

 

By using this web site, you accept the Terms of Use for this web site. Please read these Terms of Use carefully before using any part of this site. Please go here for information on ni.com's copyright infringement policy.
My Profile | Privacy | Legal | Contact NI © 2011 National Instruments Corporation. All rights reserved.    |    E-Mail this Page E-Mail this Page