Code optimization

[zezendapuss]

Hey guys,

So i have this super simple code to time 3 led in sequence. I am quite sure that there is a much simpler and elegant way to do it. 

your suggestions are much appreciated.

[Bob_Schor]

Are you teaching yourself LabVIEW?  [I'm guessing this is the case, as you (a) use a Frame Sequence structure, almost never necessary, (b) use three separate indicators instead of an Array,  and (c) don't use an Event Sequence to handle the button pushes and (optionally) the timing.

Note that it takes a long time to react to the Stop button -- it can take more than 10 seconds (quiz yourself to figure out why).

I strongly urge you to spend several hours with the LabVIEW tutorials, including those that I think are listed the Getting Started With LabVIEW box on the first page of the LabVIEW Forums.

Bob Schor

[altenbach]

You have four states: all LEDs off, LED1 on, LED2 on, LED3 On.

Transition between states depend on the state of the start button, the current state, and the elapsed time in the current state. All you need is a very simple state machine. No need for sequence frames and absolutely no need for value property nodes.

Make sure that the program can instantly react to user input at any time, so don't trap it inside long sequences.

See how far you get.