Single Digit Seven Segment Display Code Question

Projects Programming
1

I have a R/G Seven Segment Display, well one I salvaged from a giant E-waste pile. I have identified the pins and it is common anode. Green Anode is on pin 1 of the display, and Pin 6 is the Red Anode. so 2,3,4,5,7,8,9,10 are the Cathodes. My Question is I have this code for a common cathode display:

int a = 2;  //For displaying segment "a"
int b = 3;  //For displaying segment "b"
int c = 4;  //For displaying segment "c"
int d = 5;  //For displaying segment "d"
int e = 6;  //For displaying segment "e"
int f = 8;  //For displaying segment "f"
int g = 9;  //For displaying segment "g"
void setup() {               
  pinMode(a, OUTPUT);  //A
  pinMode(b, OUTPUT);  //B
  pinMode(c, OUTPUT);  //C
  pinMode(d, OUTPUT);  //D
  pinMode(e, OUTPUT);  //E
  pinMode(f, OUTPUT);  //F
  pinMode(g, OUTPUT);  //G
}
void displayDigit(int digit)
{
 //Conditions for displaying segment a
 if(digit!=1 && digit != 4)
 digitalWrite(a,HIGH);
 
 //Conditions for displaying segment b
 if(digit != 5 && digit != 6)
 digitalWrite(b,HIGH);
 
 //Conditions for displaying segment c
 if(digit !=2)
 digitalWrite(c,HIGH);
 
 //Conditions for displaying segment d
 if(digit != 1 && digit !=4 && digit !=7)
 digitalWrite(d,HIGH);
 
 //Conditions for displaying segment e 
 if(digit == 2 || digit ==6 || digit == 8 || digit==0)
 digitalWrite(e,HIGH);
 
 //Conditions for displaying segment f
 if(digit != 1 && digit !=2 && digit!=3 && digit !=7)
 digitalWrite(f,HIGH);
 if (digit!=0 && digit!=1 && digit !=7)
 digitalWrite(g,HIGH);
 
}
void turnOff()
{
  digitalWrite(a,LOW);
  digitalWrite(b,LOW);
  digitalWrite(c,LOW);
  digitalWrite(d,LOW);
  digitalWrite(e,LOW);
  digitalWrite(f,LOW);
  digitalWrite(g,LOW);
}


void loop()
{
 
  int i;
  for( i = 0; i< 10; i++);
 {
   displayDigit( i );
   delay(1000);
   turnOff();
 }
}

Now the question is: to make my display work with this code^^^ Above, Don't I need to change all the functions of HIGHS to LOWS and The LOWS to HIGHS, Right??? It is like an active low component right??? I am worried IF I don't Change it I will short the pins or Even WORSE my arduino!

2

You don't come across as confused. Just insecure. I believe your logic is perfectly sound.

3

Yes TY ken, I aminsecure when it comes to electronics as I have been into the hobbyist level for lil less than a year! But now want to become an engineer. So to answer my question I do need to change the Highs to lows or 1-0's and 0-1's RIGHT??

4

I'd say so, Just go for it. The nice thing about LEDs is, even if you wire them up back to front they don't damage anything. They simply don't light.

5

Great point Duh moment they are a diode. LOL TY just a lil more logical and I had it. WELL TY SIR I AM GOIN FOR IT

6

BTW.
You know you can write to all 7 bits at the same time by writing direct to a port. (If you select your pins carefully.)