Sample Exam Solutions for Review

[JÞB]

Overall no too bad however I'll make a few observations:

1. You are not using a project correctly!  The only item in your project is "Sprinkler.vi" every thing else is in "Dependancies"  Except for the configuration file which isn'e anywhere except on disc.  Put everything in the project under the My Computer target (Subvis, controls, configuration and requirements)
2. No VI documentation for any *.ctl or the main vi.
3. Missing / wrong functionality
   1. With run selector in "Single" status remains "Running" when the zone array completes
   2. Timing does not pause- you need a pausable timer and you simply do not have one
   3. config file is never written.  You need to write to it when the user wants to save new zone setup.
   4. Clicking setup enables the zone setup array forever.
4. Why does "Zone.ctl" not appear in the hierarchy? you made the type def why not use it or remove it.

[littlett]

Hi Jeff,

Thanks for reviewing. Thanks for the comments on project and documentation.

Regarding the missing and wrong functionality, I am confused. I feel I have serious problem in understanding the spec now:

1. With run selector in "Single" status remains "Running" when the zone array completes

the spec says: "The Controller Status indicator should continue to display Running even after the single run is complete." page 5

I thought that means to display "Running" after zone array completes. Could you help me with that?

2. Timing does not pause- you need a pausable timer and you simply do not have one

the spec says: "If it stops raining, and the Run Selector switch is in the Single position, the controller should make a single pass and wait for user input. If the Run Selector switch is in the Continuous position, the controller should start servicing the zone beginning with the first zone." (top of page 6)

It explicitly said to start with the first zone if Run Selector is in Continuous position. It didn't say what if in Single position. I therefore assumed also the same behavior, to start over. I can't read "pause" out of this. Could you help me with that too?

3. config file is never written.  You need to write to it when the user wants to save new zone setup.

I can't find any place in the spec that says to over write the configuration file either. ( page 4)

4. Clicking setup enables the zone setup array forever.

I am confused. After "Start" is clicked on my side, the zone array is disabled. Or do you mean there should be a time out function? I can't find that fromt the spec either.

I attached the specs I have, too.

[JÞB]

OK, Now that you exposed the requirements document I'll concede the wrong/missing functionality points 1-4.  I went of previous views of that exam scenario back when it was a much harder example. 

Of course you also exposed a missed requirement: from pg. 4 "The Timing setting should be in seconds (no decimals)." you should have changed the display format and data entry properties.

But, the point remains, would it have been so darned hard to add that pdf to the project?

[littlett]

Hi Jeff,

Thanks again! Sorry that I didn't know the requirement was different.

I will change the rest. 

[littlett]

Hi, 

I made up a Microwave Oven question based on the microwave oven at my home.

I think it may help people who haven't thought through how a microwave oven works. Btw, I discovered some new functions of my own microwave oven.

I believe this is more complicated than real quesiton, and 4 hours will not be enough. 

But it definitely help me understand how a microwave oven works 🙂

[J_M_M]

Dear Community. I am planning on taking the CLD exam in the next couple of weeks and I am wondering if you could provide your feedback on the attached sample exam: ATM. Thanks!!!

[Manushak]

Hi

I think you need add descriptions for controls in main window
and be more tidy with wires, crossed or hidden wires can decrease your points

[J_M_M]

Thanks Manushak, I will keep those things in mine while practing the other sample exams. 

Best Regards,

Joel Minier Matar

[J_M_M]

Dear Community member,

I just finished the sprinkler controller example for the CLD exam. I would appreciate your comments on it. I tried to avoid crossed wires as much as I could, but sometimes it is difficult especially when you have property nodes (see Initialize state); I did my best in 4 hours. If anyone have a feedback on how to avoid those, I would really appreciate it,

Thanks!

[Jacobson-ni]

The functionality seems to work well but you could probably figure out approximately how many points you would get yourself. A few things that I saw.

• There were a few wires hiding under the case structure (controller status and zone indicator wires)
• You are missing tip strips for all controls as far as I could tell
• Personally I would suggest putting some higher level documentation. Most of your documentation focuses on very specific logic in each state (which can be helpful) but there isn't ever documentation on what each state is supposed to do. 
• Style wise I think there is still some room to clean up a bit. You avoided all of the big mistakes like crossing wires when you don't have to or wiring backwards but there are some easy places where you could remove unnecessary wire bends or very long wires. For instance, you could move the case selector for your state to be in line with the shift register or move the index function and tunnel it connects to in the initialize case to be in line with the Zone Setup Array. I'm not sure if you'll get many points off for things like that but I think you'll find that your code will be a lot easier to read when you don't have to trace where wires are going.