Need help

im working on a car seat controller and im using 2 arduino nano boards to controll the seat motors. but i ran into a problem with the code. i only get 1V from D2,D4,D5 if i use this code. but if i load another code the pins give 5V and my cirquit only draws 8.5 mA. can some body help.

 //B1-B6 Buttons 
 //D2-D7 output pins
 
int BI = 14;
int B2 = 15;
int B3 = 16;
int B4 = 17;
int B5 = 18;
int B6 = 19;

int D2 = 2;
int D3 = 3;
int D4 = 4;
int D5 = 5;
int D6 = 6;
int D7 = 7;

void setup() {
  
  pinMode(B1, INPUT);
  pinMode(B2, INPUT);
  pinMode(B3, INPUT);
  pinMode(B4, INPUT);
  pinMode(B5, INPUT);
  pinMode(B6, INPUT);

  pinMode(D2, OUTPUT);
  pinMode(D3, OUTPUT);
  pinMode(D4, OUTPUT);
  pinMode(D5, OUTPUT);
  pinMode(D6, OUTPUT);
  pinMode(D7, OUTPUT);
  
}

void loop() {
  
  if (digitalRead(BI) == HIGH && digitalRead(B2) == LOW) {
    
  digitalWrite(D2, HIGH);
  digitalWrite(D3, LOW);
}
  if (digitalRead(B2) == HIGH && digitalRead(BI) == LOW) {
    
  digitalWrite(D3, HIGH);
  digitalWrite(D2, LOW);
}
  else{
  digitalWrite(D2, LOW);
  digitalWrite(D3, LOW);
}

  if (digitalRead(B3) == HIGH && digitalRead(B4) == LOW) {
    
  digitalWrite(D4, HIGH);
  digitalWrite(D5, LOW);
}
  if (digitalRead(B4) == HIGH && digitalRead(B3) == LOW) {
    
  digitalWrite(D5, HIGH);
  digitalWrite(D4, LOW);
}
  else{
  digitalWrite(D4, LOW);
  digitalWrite(D5, LOW);
}

  
  if (digitalRead(B5) == HIGH && digitalRead(B6) == LOW) {
    
  digitalWrite(D6, HIGH);
  digitalWrite(D7, LOW);
}
  if (digitalRead(B6) == HIGH && digitalRead(B5) == LOW) {
    
  digitalWrite(D7, HIGH);
  digitalWrite(D6, LOW);
}
  else{
  digitalWrite(D6, LOW);
  digitalWrite(D7, LOW);
}
}

Top Replies

Parents

masa98 over 3 years ago

i made some changes to the code but now i get 5v from D2 and only 2.9v from D3. the funktion im after is, if B1 is HIGH then D2 must be HIGH and D3 must be LOW. and if B2 is HIGH then D2 must be LOW and D3 must be HIGH

 //B1-B6 Buttons 
 //D2-D7 output pins
 
int BI = 14;
int B2 = 15;
int B3 = 16;
int B4 = 17;
int B5 = 18;
int B6 = 19;

int D2 = 2;
int D3 = 3;
int D4 = 4;
int D5 = 5;
int D6 = 6;
int D7 = 7;

void setup() {
  
  pinMode(B1, INPUT);
  pinMode(B2, INPUT);
  pinMode(B3, INPUT);
  pinMode(B4, INPUT);
  pinMode(B5, INPUT);
  pinMode(B6, INPUT);

  pinMode(D2, OUTPUT);
  pinMode(D3, OUTPUT);
  pinMode(D4, OUTPUT);
  pinMode(D5, OUTPUT);
  pinMode(D6, OUTPUT);
  pinMode(D7, OUTPUT);
  
}

void loop() {
  
  if (digitalRead(BI) == HIGH && digitalRead(B2) == LOW) {
    
  digitalWrite(D2, HIGH);
  digitalWrite(D3, LOW);
}
  else{
  digitalWrite(D2, LOW);
  digitalWrite(D3, LOW);


if (digitalRead(B2) == HIGH && digitalRead(BI) == LOW) {
    
  digitalWrite(D3, HIGH);
  digitalWrite(D2, LOW);
}
  else{
  digitalWrite(D2, LOW);
  digitalWrite(D3, LOW);
}
}
}

genebren over 3 years ago in reply to masa98

You still have an issue with your B1 and BI. BI is not being set as input, but B1 is.
Cancel
Vote Up +3 Vote Down

Sign in to reply

Cancel

Reply

genebren over 3 years ago in reply to masa98

You still have an issue with your B1 and BI. BI is not being set as input, but B1 is.
Cancel
Vote Up +3 Vote Down

Sign in to reply

Cancel

Children

masa98 over 3 years ago in reply to genebren

it had on effect when i changed it
Cancel
Vote Up 0 Vote Down

Sign in to reply

Cancel
Jan Cumps over 3 years ago in reply to masa98

Did you change all occurrences of BI ?
Cancel
Vote Up 0 Vote Down

Sign in to reply

Cancel
masa98 over 3 years ago in reply to Jan Cumps

yes
Cancel
Vote Up 0 Vote Down

Sign in to reply

Cancel
Jan Cumps over 3 years ago in reply to masa98

This one

int BI = 14;

this one:

if (digitalRead(BI) == HIGH && digitalRead(B2) == LOW) {

and this:

if (digitalRead(B2) == HIGH && digitalRead(BI) == LOW) {

?
Cancel
Vote Up +1 Vote Down

Sign in to reply

Cancel

masa98 over 3 years ago in reply to Jan Cumps

the code looks like this and i still have the problem with D3 it gives only 2.9v when HIGH

 //B1-B6 Buttons 
 //D2-D7 output pins
 
int Button1 = 14;
int Button2 = 15;
int Button3 = 16;
int Button4 = 17;
int Button5 = 18;
int Button6 = 19;

int D2 = 2;
int D3 = 3;
int D4 = 4;
int D5 = 5;
int D6 = 6;
int D7 = 7;

void setup() {
  
  pinMode(Button1, INPUT);
  pinMode(Button2, INPUT);
  pinMode(Button3, INPUT);
  pinMode(Button4, INPUT);
  pinMode(Button5, INPUT);
  pinMode(Button6, INPUT);

  pinMode(D2, OUTPUT);
  pinMode(D3, OUTPUT);
  pinMode(D4, OUTPUT);
  pinMode(D5, OUTPUT);
  pinMode(D6, OUTPUT);
  pinMode(D7, OUTPUT);
  
}

void loop() {
  
  if (digitalRead(Button1) == HIGH && digitalRead(Button2) == LOW) {
    
  digitalWrite(D2, HIGH);
  digitalWrite(D3, LOW);
}
  else{
  digitalWrite(D2, LOW);
  digitalWrite(D3, LOW);


if (digitalRead(Button2) == HIGH && digitalRead(Button1) == LOW) {
    
  digitalWrite(D3, HIGH);
  digitalWrite(D2, LOW);
}
  else{
  digitalWrite(D2, LOW);
  digitalWrite(D3, LOW);
}
}
}

ntewinkel over 3 years ago in reply to masa98

Good call on using better names for the buttons "ButtonX" instead of "BX". It makes your code much more readable, and in the future you'll thank your past self

I mentioned it in your other post just a moment ago - it will be very helpful for you to make use of the Serial Console by adding Serial.println() debug statements into each of the if/else blocks.
Cancel
Vote Up +2 Vote Down

Sign in to reply

Cancel