Boolean issue ??

Using Arduino Programming Questions
1

I have a pot that sends a voltage from 0-5 volts which I use to display data with LEDS. However the pot can rotate just past 360 deg and the voltage goes back to 0. I don't want to display 0 in this case. I set a flag OVER to check for this condition and keep 6 displayed. .
However my syntax is not correct and I can't get it right,

int analogin = A0;    // input pin for the potentiometer
int LEDa = 2;
int LEDb = A3; 
int LEDc = A1; 
int LEDd = 11;
int LEDe = 0;
int LEDf = 4;
int LEDg = A2;
int DIM = 5;
boolean OVER; 


int ATD = 0;   // variable to store the value coming from the sensor

void setup() {
  pinMode(LEDa, OUTPUT);  
  pinMode(LEDb, OUTPUT);  
  pinMode(LEDc, OUTPUT);  
  pinMode(LEDd, OUTPUT);  
  pinMode(LEDe, OUTPUT);  
  pinMode(LEDf, OUTPUT);  
  pinMode(LEDg, OUTPUT);   
  pinMode(DIM, INPUT);   
//  OVER = HIGH;
}

void loop()
   {
   ATD = analogRead(analogin);    
   if ((ATD <= 190) && (OVER == LOW))  //0
      {
       digitalWrite(LEDa, LOW);
       digitalWrite(LEDb, LOW);
       digitalWrite(LEDc, LOW);
       digitalWrite(LEDd, LOW);
       digitalWrite(LEDe, LOW);
       digitalWrite(LEDf, LOW);      
       digitalWrite(LEDg, HIGH);   
       }
    else if ((ATD >= 210) && (ATD <= 260))  //1
       {
       digitalWrite(LEDa, HIGH);
       digitalWrite(LEDb, HIGH);
       digitalWrite(LEDc, HIGH);
       digitalWrite(LEDd, HIGH);
       digitalWrite(LEDe, LOW);
       digitalWrite(LEDf, LOW);      
       digitalWrite(LEDg, HIGH);
       }
    else if ((ATD >= 340) && (ATD <= 400))  //2
       {
       digitalWrite(LEDa, LOW);
       digitalWrite(LEDb, LOW);
       digitalWrite(LEDc, HIGH);
       digitalWrite(LEDd, LOW);
       digitalWrite(LEDe, HIGH);
       digitalWrite(LEDf, LOW);      
       digitalWrite(LEDg, LOW);
       }
    else if ((ATD >= 480) && (ATD <= 540)) //3
      {
       digitalWrite(LEDa, LOW);
       digitalWrite(LEDb, HIGH);
       digitalWrite(LEDc, HIGH);
       digitalWrite(LEDd, LOW);
       digitalWrite(LEDe, LOW);
       digitalWrite(LEDf, LOW);      
       digitalWrite(LEDg, LOW);
       }
    else if ((ATD >= 620) && (ATD <= 675)) //4
      {
       digitalWrite(LEDa, HIGH);
       digitalWrite(LEDb, HIGH);
       digitalWrite(LEDc, LOW);
       digitalWrite(LEDd, HIGH);
       digitalWrite(LEDe, LOW);
       digitalWrite(LEDf, LOW);      
       digitalWrite(LEDg, LOW);
       OVER = LOW;       
      }
    else if ((ATD >= 760) && (ATD <= 815))  //5
       {
       digitalWrite(LEDa, LOW);
       digitalWrite(LEDb, HIGH);
       digitalWrite(LEDc, LOW);
       digitalWrite(LEDd, LOW);
       digitalWrite(LEDe, LOW);
       digitalWrite(LEDf, HIGH);      
       digitalWrite(LEDg, LOW);
       OVER = HIGH;
       }
    else if ((ATD >= 900) || ((ATD <= 100) && (OVER == HIGH)))  //6
       {
       digitalWrite(LEDa, LOW);
       digitalWrite(LEDb, LOW);
       digitalWrite(LEDc, LOW);
       digitalWrite(LEDd, LOW);
       digitalWrite(LEDe, LOW);
       digitalWrite(LEDf, HIGH);      
       digitalWrite(LEDg, LOW);
       }       
    else
       {
       digitalWrite(LEDa, HIGH);
       digitalWrite(LEDb, HIGH);
       digitalWrite(LEDc, HIGH);
       digitalWrite(LEDd, HIGH);
       digitalWrite(LEDe, HIGH);
       digitalWrite(LEDf, HIGH);      
       digitalWrite(LEDg, HIGH);
       }
   }
2

i don't see where you're updating the value of OVER

3

You are not intiasise default value for over. i.e low or high
During first time execution default find as LOw and IF ATD value less than 190. Surely display to Zero.

AS retronet_RIMBA1ZO saying your are no where updating OVER.

4

000 - 100 && over = 6
100 - 190 && over = undefined
000 - 190 && !over= 0
191-209 undefined
210-260 = 1
261-339 = undefined
340-400 = 2
401-479 = undefined
480-540 = 3
541-619 = undefined
620-675 = 4
676-759 = undefined
760-815 = 5
816-899 = undefined
900-1023 =6

There are lots values that are not defined and will there for have all LEDs HIGH

I can not imagine what criteria you are going to use to set over=high or over=low.

I suggest you use a flag to indicate increasing or decreasing.
So if lastValue < newValue increasing=true else increasing=false
and then if increasing=true & newValue <100 and lastValue>900 then ignore newValue.

Well there is probably a bit more you need to think about but that is a starting point.

5

In between ADC values are set to high, ie all LEDs off

I presume I can preset OVER to LOW in Setup()
void setup() {
pinMode(LEDa, OUTPUT);

pinMode(DIM, INPUT);
OVER = LOW;
}

OVER is only set when the pot goes through the
else if ((ATD >= 760) && (ATD <= 815)) //5

OVER=HIGH;

and reset when

else if ((ATD >= 620) && (ATD <= 675)) //4

OVER=LOW;

The problem is to do with my use of the boolean operators. The OVER is not being set or evaluated properly.

6

AMPS-N:
You are not intiasise default value for over. i.e low or high

The compiler and crt0 already initialised it to zero, aka LOW.