Solenoid control

[Grizzlie]

I am trying to program the solenoids this year to wherewhen i hold the trigger it activates and when i let go it deactivates. Same as the reverse motion. I wrote this code to do this but it wont work. When i press the trigger the solonoid activates but when i let go it doesnt stop. And it WILL NOT go back when i press the rev button. heres the code i wrote :

 if (p3_sw_trig);

{              
      relay3_fwd = 1;
}

if (!p3_sw_trig);

{
      relay3_fwd = 0;
}

if (p3_sw_top);

{
      relay3_rev = 1;
}

if (!p3_sw_trig);

{
      relay3_rev = 0;
}

It would be great if someone could explain to me what i did wrong here because i'm fairly new to the C language. But it would be even better if someone could post the exact code to make this work. Thank You for even taking your time to read this post. And also good luck this year!

[jgannon]

Originally Posted by Grizzlie

I am trying to program the solenoids this year to wherewhen i hold the trigger it activates and when i let go it deactivates. Same as the reverse motion. I wrote this code to do this but it wont work. When i press the trigger the solonoid activates but when i let go it doesnt stop. And it WILL NOT go back when i press the rev button. heres the code i wrote :

 if (p3_sw_trig);

{              
      relay3_fwd = 1;
}

if (!p3_sw_trig);

{
      relay3_fwd = 0;
}

if (p3_sw_top);

{
      relay3_rev = 1;
}

if (!p3_sw_trig);

{
      relay3_rev = 0;
}

It would be great if someone could explain to me what i did wrong here because i'm fairly new to the C language. But it would be even better if someone could post the exact code to make this work. Thank You for even taking your time to read this post. And also good luck this year!

Well, I see three immediate problems. One, all of your if statements end in semicolons. That is bad. Two, your last if statement checks !p3_sw_trig instead of !p3_sw_top. Three, I would suggest using else statements, rather than the structure of if statements that you've chosen. Try this:

if (p3_sw_trig)             
      relay3_fwd = 1;
else
      relay3_fwd = 0;
if (p3_sw_top)
      relay3_rev = 1;
else
      relay3_rev = 0;

If you wanted, you could simplify this even further:

relay3_fwd=p3_sw_trig?1:0;
relay3_rev=p3_sw_top?1:0;

Simpler, though not quite as easy to read.

<edit>
Hm... now that I think about it, you might even be able to do:

relay3_fwd=p3_sw_trig;
relay3_rev=p3_sw_top;

I don't have an RC with which to test that, but I know the other two methods will work just fine.
</edit>

[reilly]

Here is the way to do this:

if (p3_sw_trig) //no semi colons for functions

{
relay3_fwd = 1;
}

else (!p3_sw_trig)

{
relay3_fwd = 0;
}

else if (p3_sw_top) //or if is fine here also

{
relay3_rev = 1;
}

else

{
relay3_rev = 0;
}

[Grizzlie]

[quote] Two, your last if statement checks !p3_sw_trig instead of !p3_sw_topQUOTE]

o yea that isnt the problem because i didnt put that there i kinda just that laswt bit of code from memory. but the semicolon would explainthe syntax errors . Ok im gonna try your code and hope it works. Thanks!

[Grizzlie]

where in the user_routines.c would i put this code? or is there a diiferent file where this code would belong?