Each case takes about 400 bytes with each instruction with rotary encoder in coordinate position which lead to roughly 4000 bytes. I want to reduce to just 800 bytes with default if it is possible.
Here's example:
case 2
case 3
default while no any button is pressed so it reads the rotary encoder which read left or right at coordinate position.
Is that possible in C ?
Robin2
August 15, 2015, 7:31pm
2
Why won't the same 400 bytes deal with all the different cases with one or two variables to distinquish one from the other.
Post your code.
...R
Do you mean something like this?
Here!
switch ( P_13_4 ) {
case 1:
if ( P_13_4 == 11 ) { waypoint_point_x = waypoint_01_x; point_xplus(); waypoint_01_x = waypoint_point_x; }
if ( P_13_4 == 12 ) { waypoint_point_x = waypoint_01_x; point_xminus(); waypoint_01_x = waypoint_point_x; }
if ( P_13_4 == 13 ) { waypoint_point_y = waypoint_01_y; point_yplus(); waypoint_01_y = waypoint_point_y; }
if ( P_13_4 == 14 ) { waypoint_point_y = waypoint_01_y; point_yminus(); waypoint_01_y = waypoint_point_y; }
break;
case 2:
if ( P_13_4 == 11 ) { waypoint_point_x = waypoint_02_x; point_xplus(); waypoint_02_x = waypoint_point_x; }
if ( P_13_4 == 12 ) { waypoint_point_x = waypoint_02_x; point_xminus(); waypoint_02_x = waypoint_point_x; }
if ( P_13_4 == 13 ) { waypoint_point_y = waypoint_02_y; point_yplus(); waypoint_02_y = waypoint_point_y; }
if ( P_13_4 == 14 ) { waypoint_point_y = waypoint_02_y; point_yminus(); waypoint_02_y = waypoint_point_y; }
break;
case 3:
if ( P_13_4 == 11 ) { waypoint_point_x = waypoint_03_x; point_xplus(); waypoint_03_x = waypoint_point_x; }
if ( P_13_4 == 12 ) { waypoint_point_x = waypoint_03_x; point_xminus(); waypoint_03_x = waypoint_point_x; }
if ( P_13_4 == 13 ) { waypoint_point_y = waypoint_03_y; point_yplus(); waypoint_03_y = waypoint_point_y; }
if ( P_13_4 == 14 ) { waypoint_point_y = waypoint_03_y; point_yminus(); waypoint_03_y = waypoint_point_y; }
break;
case 4:
if ( P_13_4 == 11 ) { waypoint_point_x = waypoint_04_x; point_xplus(); waypoint_04_x = waypoint_point_x; }
if ( P_13_4 == 12 ) { waypoint_point_x = waypoint_04_x; point_xminus(); waypoint_04_x = waypoint_point_x; }
if ( P_13_4 == 13 ) { waypoint_point_y = waypoint_04_y; point_yplus(); waypoint_04_y = waypoint_point_y; }
if ( P_13_4 == 14 ) { waypoint_point_y = waypoint_04_y; point_yminus(); waypoint_04_y = waypoint_point_y; }
break;
case 5:
if ( P_13_4 == 11 ) { waypoint_point_x = waypoint_05_x; point_xplus(); waypoint_05_x = waypoint_point_x; }
if ( P_13_4 == 12 ) { waypoint_point_x = waypoint_05_x; point_xminus(); waypoint_05_x = waypoint_point_x; }
if ( P_13_4 == 13 ) { waypoint_point_y = waypoint_05_y; point_yplus(); waypoint_05_y = waypoint_point_y; }
if ( P_13_4 == 14 ) { waypoint_point_y = waypoint_05_y; point_yminus(); waypoint_05_y = waypoint_point_y; }
break;
case 6:
if ( P_13_4 == 11 ) { waypoint_point_x = waypoint_06_x; point_xplus(); waypoint_06_x = waypoint_point_x; }
if ( P_13_4 == 12 ) { waypoint_point_x = waypoint_06_x; point_xminus(); waypoint_06_x = waypoint_point_x; }
if ( P_13_4 == 13 ) { waypoint_point_y = waypoint_06_y; point_yplus(); waypoint_06_y = waypoint_point_y; }
if ( P_13_4 == 14 ) { waypoint_point_y = waypoint_06_y; point_yminus(); waypoint_06_y = waypoint_point_y; }
break;
case 7:
if ( P_13_4 == 11 ) { waypoint_point_x = waypoint_07_x; point_xplus(); waypoint_07_x = waypoint_point_x; }
if ( P_13_4 == 12 ) { waypoint_point_x = waypoint_07_x; point_xminus(); waypoint_07_x = waypoint_point_x; }
if ( P_13_4 == 13 ) { waypoint_point_y = waypoint_07_y; point_yplus(); waypoint_07_y = waypoint_point_y; }
if ( P_13_4 == 14 ) { waypoint_point_y = waypoint_07_y; point_yminus(); waypoint_07_y = waypoint_point_y; }
break;
case 8:
if ( P_13_4 == 11 ) { waypoint_point_x = waypoint_08_x; point_xplus(); waypoint_08_x = waypoint_point_x; }
if ( P_13_4 == 12 ) { waypoint_point_x = waypoint_08_x; point_xminus(); waypoint_08_x = waypoint_point_x; }
if ( P_13_4 == 13 ) { waypoint_point_y = waypoint_08_y; point_yplus(); waypoint_08_y = waypoint_point_y; }
if ( P_13_4 == 14 ) { waypoint_point_y = waypoint_08_y; point_yminus(); waypoint_08_y = waypoint_point_y; }
break;
case 9:
if ( P_13_4 == 11 ) { waypoint_point_x = waypoint_09_x; point_xplus(); waypoint_09_x = waypoint_point_x; }
if ( P_13_4 == 12 ) { waypoint_point_x = waypoint_09_x; point_xminus(); waypoint_09_x = waypoint_point_x; }
if ( P_13_4 == 13 ) { waypoint_point_y = waypoint_09_y; point_yplus(); waypoint_09_y = waypoint_point_y; }
if ( P_13_4 == 14 ) { waypoint_point_y = waypoint_09_y; point_yminus(); waypoint_09_y = waypoint_point_y; }
break;
case 10:
if ( P_13_4 == 11 ) { waypoint_point_x = waypoint_10_x; point_xplus(); waypoint_10_x = waypoint_point_x; }
if ( P_13_4 == 12 ) { waypoint_point_x = waypoint_10_x; point_xminus(); waypoint_10_x = waypoint_point_x; }
if ( P_13_4 == 13 ) { waypoint_point_y = waypoint_10_y; point_yplus(); waypoint_10_y = waypoint_point_y; }
if ( P_13_4 == 14 ) { waypoint_point_y = waypoint_10_y; point_yminus(); waypoint_10_y = waypoint_point_y; }
break;
}
void point_xplus() {
if ( waypoint_point_x >= point_min && waypoint_point_x <= point_max) { ++waypoint_point_x; stepper_5plus16(); } else { portf(); } }
void point_xminus() {
if ( waypoint_point_x >= point_min && waypoint_point_x <= point_max) { --waypoint_point_x; stepper_5minus16(); } else { portf(); } }
void point_yplus() {
if ( waypoint_point_y >= point_min && waypoint_point_y <= point_max) { ++waypoint_point_y; stepper_5plus4(); } else { portf(); } }
void point_yminus() {
if ( waypoint_point_y >= point_min && waypoint_point_y <= point_max) { --waypoint_point_y; stepper_5minus4(); } else { portf(); } }
As you see, I have done something with case 1, 2, ... 10 that the button is same as a button and it connects to the exact rotary encoders until other button is pressed, then it connects to two specific rotary encoder while the first is connected off automatic, it is for to save pins while it is possible than reading 50 pins in array which takes a lot of code size.
Something not right here,
If P_13_4 is have value of "1", all the if statement in case 1: will be false, it will do nothing in it.
please the SwitchCase Doc
First you have to use different variables in the switch() and if(), else the entire switch will never do anything.
But before all you should explain what you want to do. Your approach looks not very promising
Robin2
August 16, 2015, 7:30am
8
Have you considered turning the logic on its head and doing theif ( P_13_4 == 11 ) {
That way you would only have 4 of those rather than 10 sets of 4
...R
Robin2:if ( P_13_4 == 11 ) {
That way you would only have 4 of those rather than 10 sets of 4
...R
Now I am drawing a sketch how I am thinking.
That means you are not thinking correctly and logically.
Robin2
August 17, 2015, 10:18am
12
Sorry but I can't make sense of your diagram.
Maybe you can describe what you want in English, step by step like this
Button1 pressed
Light LED
Button2 pressed
Button3 pressed
...R
Hi,
Tom....
Qwseyvnd:
Qwseyvnd:
Well there is not much coherent thinking going on in that diagram.
The closest it looks like is a switch matrix.
Is this what you are looking for?
Isn't it possible to making that the previous button turns off automatic if I turn on the next button something similar to the smoke starter in a car if it runs too hot?
Robin2
August 18, 2015, 6:57am
17
Qwseyvnd:
Most button switches turn off as soon as you lift your finger off them. How you use the Arduino to interpret what the buttons mean is up to you.
You can get button switches that latch in the on position and need a second push to switch them to the off position - don't use them.
...R
Robin2:
...R
You said that. Why don't use them ?
Robin2
August 18, 2015, 6:29pm
20
Qwseyvnd:
This is getting ridiculous ...
The only reason I mentioned the latching switches was so you knew the difference between them and non-latching switches. I was not / am not sure which sort you have.
You did not previously say
In Reply #14 you asked
I can only go on what you write - not what you would might have intended to write.
...R
