[vhcook]

It's generally not good practice to set the same motor in multiple places. When you tell the same motor to go forward and stop at the same time, it could do anything. And if someone presses buttons 4 and 5 at the same time, you're asking for full forward and full reverse at the same time.

To get your intended output, I would use two select statements (one triggered by each button) to produce one value to feed to a single Motor Set Output like Mark's example above. The logic being If Button4, then 1, elseif Button5 then -1, else 0.