A few questions on programming the Control System

[rdsideresistanc]

Originally Posted by Joe Ross

you need to pass relayfwd and relayrev as pointers so that the modifications you make don't get "thrown out" when you exit the function.

You are correct about this.....hmm....could you possibly tell me how to pass them as pointers?

[Raven_Writer]

Originally Posted by Mike Soukup

If they don't work, we're all in trouble since there's an enum in 'ifi_utilities.h'. Our current code has multiple enum's and we haven't had any problems with them.

enum's work. I've tried the code out on the compiler. It compiles perfectly and such.

[Joe Ross]

Here is how I would implement the code (from which you should be able to figure out how to implement your code with pointers)

void toggle_single_solenoid(unsigned char button, unsigned char *prev_button, unsigned char *relay_fwd)
{
   if (button && !prev_button)     //don't keep toggling if they hold the button
      *relay_fwd = !(*relay_fwd);

   *prev_button = button;
}

call it like this:

 toggle_single_solenoid(p1_sw_trig, &prev_p1_sw_trig, &relay1_fwd);

Remember to declare prev_p1_sw_trig and change the variables to whichever switches and relays you want.

[seanwitte]

I haven't tried it, but can you have a pointer to a bit field? relay1_fwd is a macro with the value LATDbits.LATD0. Just curious.

[Avarik]

What exactly is prev_button?

And thanks...I've been at this for too long...

Shouldn't take me this long!

[Avarik]

Alright, I finally figured it out! Here is how I did it.

if (p1_sw_trig && rel1==0)
{

	rel1++;
	if (rel2==0)
	{
	relay1_fwd=1;
	rel2++;
	}
	else
	{		
	relay1_rev=1;
	rel2--;
	}
						
}

if (!p1_sw_trig)
{
	rel1=0;
	relay1_fwd=0;
	relay1_rev=0;
}

rel1 and rel2 are simply declared as ints, where rel1 is initialized to 0, and rel2 to 1. This makes it go forward at first, then backwards the next time.

[Adam Shapiro]

Originally Posted by Avarik

Alright, I finally figured it out! Here is how I did it.

if (p1_sw_trig && rel1==0)
{

	rel1++;
	if (rel2==0)
	{
	relay1_fwd=1;
	rel2++;
	}
	else
	{		
	relay1_rev=1;
	rel2--;
	}
						
}

if (!p1_sw_trig)
{
	rel1=0;
	relay1_fwd=0;
	relay1_rev=0;
}

rel1 and rel2 are simply declared as ints, where rel1 is initialized to 0, and rel2 to 1. This makes it go forward at first, then backwards the next time.

Essentially my method.