[The Lucas]

Originally Posted by dboisvert

I am trying to accomplish a simple button task. When someone pushes a button I want it to keep doing the action (IE In this case "Relay Forward" & "Relay Reverse") until the microswitch is activated. Right now it stops as soon as you release the button.


What the code looks like right now is this

if button1 == 1 & microswitch1 = 0
  Relay Forward
else if button2 == 1 & microswitch2 = 0
  Relay Reverse
else
  Relay Off

Any suggestions?

Try this:

if (button1 == 1 && microswitch1 == 0)
  Relay Forward
else if (button2 == 1 && microswitch2 == 0)
  Relay Reverse
else (0 == microswitch1 || 0 == microswitch2)
  Relay Off

= is the assignment operator and "microswitch = 0" sets microswitch to the value of 0 is always is true. You need to use == which is conditional equals

It is a best practice to write the statement 0 == microswitch so the compiler can catch the mistake (0=microswitch is an error)

Also it is better to use a logial and (&&) rather than a bitwise and (&) since the bitwise and will not work in all situations (2 & 1 =false)