Hi
In my opinion there is no right and wrong as long as you get the result you want when you want.
Essentially you have created the same scenarios that the Component Macro does for you but with one possibly very important omission. When you added the two push buttons to your chart and set their Properties (e.g. connection = A1 etc) you will see underneath the Polarity setting a Debounce value (default = 10mS). This feature is a software debounce value that used if and only if you use the associated Component Macro to read the status of the switch.
Without using the Component Macro(s) you will need to debounce your switch some other way. When you press a button the physical mechanics of it changing state is not a clean changeover. It literally bounces on/off many times before settling in it's final state. This bouncing causes the microcontroller to receive a chain of pulses (high / low / high / low etc). Unless you are addressing this externally I think your chart would work in simulation but not in reality. There are many ways to eliminate bounce. Some use software as described, other may create a R/C solution whilst some may use a dedicated chip to eliminate, but irrespective of what works best for you, you do need to eliminate it. A search of the forum will give much advice regarding this.
As I mention above I don't believe in a right and wrong way if it's working, but it
could always be a bit better, well mine probably a lot better
![Smile :)](./images/smilies/icon_smile.gif)
Once mine is working I try to see if I can streamline any part(s) of it.
Going from the chart your scenario is that you have two inputs that will potentially do two separate things so the Switch would seem better than Decisions if only to reduce the number of icons.
I would definitely use the Component Macros to help with debounce (as mentioned above) and rather than using two separate push buttons I'd probably use the Button Array set for two buttons as this makes it a bit simpler (I can read both buttons from a single Component Macro rather than one at a time).
In the attached example I used the Button Array but I made it latching to help illustrate.
Hope this helps,
Regards