Не происходит выключение светодиода на 6 пине. Что не так в моем скетче?

Question

[Brenli]

При каждом нажатии кнопки зажигается светодиод на соответствующем пине.
Как только счетчик led_counter достигает значение 9, его значение устанавливается = 4 и при кахдом нажатии кнопки
включенные светодиоды поочередно выключаются, но этого не происходит на пине 6.
Инверсия состояния светодиода реализована digitalWrite(led_counter, !digitalRead(led_counter)); , но почему то значение digitalRead(led_counter) на 6 пине при включенном светодиоде равно = 0 и при инверсии он остается включенным.
Вопрос к знатокам. Что не так?

boolean but1     = 0;
boolean led_flag = 0;
boolean flag     = 0;
unsigned long last_press;

byte led_counter = 4;

void setup() {
   pinMode(A5,INPUT_PULLUP);
   pinMode(4,OUTPUT);
   pinMode(5,OUTPUT);
   pinMode(6,OUTPUT);
   pinMode(7,OUTPUT);
   pinMode(8,OUTPUT);
   pinMode(9,OUTPUT);


   digitalWrite(4,0);
   digitalWrite(5,0);
   digitalWrite(6,0);
   digitalWrite(7,0);
   digitalWrite(8,0);
   digitalWrite(9,0);
   
   Serial.begin(9600);
}

void loop() {
  boolean but1 = !digitalRead(A5);
  if (but1==1 && flag==0 && millis()-last_press > 50){
    flag = !flag;
    Serial.println("Button pressed");
    last_press = millis();
  }

  if (but1==0 && flag==1){
    flag = !flag;
    //Serial.println("Button released"+ String(led_counter));
    led_flag = !led_flag;
    digitalWrite(led_counter, !digitalRead(led_counter));
    led_counter++;
    if(led_counter>9) led_counter = 4;
  }
}

Answer 1

[Roman K]

Ты пытаешься прочитать значение пина в OUTPUT моде.

Comments

[Brenli]

А так делать нельзя? Прочитать текущее значение которое выводится... высокий уровень или низкий...
В любом случае спасибо попробую переделать. Только странно что все остальные светодиоды работают корректно.

[kalapanga]

Brenli, Большинство сходится на том, что так делать можно, и на всех "классических" ардуинах будет получено реальное состояние пина. Но я бы всё-таки считал эту возможность недокументированной и не сильно на неё полагался. Загуглите "digitalread output pin", сами почитаете. Заодно и альтернативные варианты решения найдёте.

[Brenli]

kalapanga, спасибо, в моем случае видимо на 6 пине что то не так и значение не верное.

[Borys Latysh]

DDRxn – это регистр направления порта. Порт в конкретный момент времени может быть либо входом либо выходом (но для состояния битов PINxn это значения не имеет. Читать из PINxn реальное значение можно всегда).
DDRxy = 0 – вывод работает как ВХОД.
DDRxy = 1 – вывод работает на ВЫХОД.

Получается "Читать" что на "вЫходе" можно всегда. Акцентирую внимание: мы читаем НЕ состояние порта а ЧТО на ноге какой логический уровень. Не совсем понимаю зачем это надо. Ведь программа собственно и управляет состоянием порта. Нужен высокий уровень - устанавливаем высокий, как переключатель, все просто.

[Brenli]

Не совсем понимаю зачем это надо.

Не понимаете зачем я пытаюсь получить значение на выходе пина? Я пытаюсь сделать это для того чтобы последовательно выключать зажжённые диоды, или включать погашенные.

[Roman K]

Brenli, тебе умный дядя объясняет, как делать не надо, ты возражаешь. Какой смысл? Ты научиться чему-то хочешь или просто поспорить? Храни желаемое состояние пина в коде и меняй его каждый цикл.

Answer 2

[Borys Latysh]

digitalWrite(led_counter, !digitalRead(led_counter));

Это конструкция несколько спорная.
Я бы так сделал:

led = !led ;
digitalWrite ( led_counter, led );

И на каждом проходе состояние будет меняться на противоположное.