Почему даже default не срабатывает в switch?

Question

[Amogus )-]

Есть вот такой свич:

switch(state){
    case DHT_OK:
       hRTemperature =  "T: " + (String)sensor.tem + " C";
       screen.print(hRTemperature, OLED_C, 25);
       int circleX = 64 + hRTemperature.length() / 2 * 12 - 12 - 5;

       if (circleX != prevCircleX) screen.drawCircle(prevCircleX, 12, 3, false, 0);
       screen.drawCircle(circleX, 12, 3, false, 1);
   
       screen.print("H: " + (String)sensor.hum + "%", OLED_C, 48);
       prevCircleX = circleX;
       Serial.println("rp");
       break;
    case DHT_ERROR_NO_REPLY:
      Serial.println("no rp");
      screen.print("Check", OLED_C, 25);
      screen.print("sensor", OLED_C, 48);
      break;
    default:
      screen.print("!!ERROR!!", OLED_C, 35);
  }

Срабатывает либо первый case, либо вообще ничего, даже default(на примере нет вывода в консоль, но был). Почему так может происходить?

Comments

[Amogus )-]

Василий Банников, в неработающем примере у state значение -3, у DHT_ERROR_NO_REPLY - тоже -3.
3. Мискликнул

[Василий Банников]

Андрей Семейкин, это числа?

[Василий Банников]

Андрей Семейкин, тогда скорее всего ты просто не перекомпилировал свой код, а по тому на плату загрузилась какая-нибудь старая версия.

[Amogus )-]

Василий Банников, нашел старый кусок, а он работает спокойно.

switch(state){
    case DHT_OK:
       screen.print("T: " + (String)sensor.tem + " C", OLED_C, 25);
       screen.drawCircle(107, 12, 3, false, 1);
       screen.print("H: " + (String)sensor.hum + "%", OLED_C, 48);
       break;
    case DHT_ERROR_NO_REPLY:
      screen.print("Waiting", OLED_C, 25);
      screen.print("for data", OLED_C, 48);
      break;
    default:
      screen.print("!!ERROR!!", OLED_C, 35);
  }

[Василий Банников]

Андрей Семейкин, теперь добавляй части из нового кода, но всё ещё не вижу ничего, из-за чего могло поломаться.
Могу только посоветовать:
1. Пиши заново, а не ctrl+c -> ctrl+v
2. Не используй однострочные if-ы. Всегда используй {}
3. Всегда пиши break; в конце case/default-веток.

[Amogus )-]

Василий Банников, ничего почему-то не работало из за того, что я объявил circleX локально... Ардуино очень странный продукт местами.
А за рекомендации спасибо, хоть я и пишу все сам.

[Василий Банников]

Андрей Семейкин,

хоть я и пишу все сам.

Имел в виду конкретно в этом случае, чтобы найти проблему. Чтобы случайная опечатка в оригинале не помешала.

[mayton2019]

Андрей Семейкин, ты лучше вноси изменения по чуть-чуть. Тоесть добавил строку - проверил.

Я не знаю есть ли в Arduino дебаггер. Но желательно-бы им воспользоваться. И есть ли доказательство
успешной компилляции? Возможно у тебя она просто не проходит и ты видишь старый бинарник.
Добавь логгирования побольше.

Вобщем тащи в топик побольше доказательств.

[Amogus )-]

mayton2019, Доказательство успешной компиляции есть, вопрос уже решен, смотрите комментарий выше. Если вы не верите, что проблема была из за переменной, то ничем помочь не могу. А насчет писать по одной строке, очень неудобно ждать компиляции по 30-50 секунд, да и плюсы я вполне неплохо знаю.

[mayton2019]

Андрей Семейкин, 3 часа назад ты опубликовал этот пост. За это время ты мог уже наводнить код логгированием и понять на какой строке у тебя процесс аварийно выпадает в kernel.

Но мы до сих пор топчемся на месте при том что ты "неплохо" знаешь.

[Amogus )-]

mayton2019, попробуйте перечитать первую часть комментария, специально для вас выделил жирным нужный фрагмент

[mayton2019]

Андрей Семейкин, как будет угодно. Разбирайся.

[Евгений Шатунов]

Андрей Семейкин, у switch единая область видимости для всех case. И ты в этой области определяешь переменные.
Вот давай подумаем с тобой вместе, что произойдет если управление со switch прыгнет в case, в котором видна переменная, объявленная выше этого case?
Определяет ли стандарт C++ поведение в такой ситуации?

Answer 1

[Василий Банников]

1. Какое значение у state и констант в case-ах?
Если кейсы не срабатывают, то default ОБЯЗАН сработать, если мы не рассматриваем сценарий бага в компиляторе ил и вмешательства потусторонних сил. (https://onlinegdb.com/VTqDzMc8D)
Вполне возможно, что ты изначально не писал default-ветку, а после добавления забыл перекомпилировать.
2. Попробуй написать минимально воспроизводимый пример, тк я хз что у тебя за библиотеки используются и по тому не имею возможности воспроизвести (а так выглядит нормально)

Comments

[Amogus )-]

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

[Александр Ананьев]

default может не отработать, если программа упала с segfault

[Василий Банников]

Александр Ананьев, ага, но где тут сегфолт то может произойти?

[freeExec]

Василий Банников, да хоть в screen. Вот если бы принт стоял первой строкой.

[Евгений Шатунов]

Василий Банников

Если кейсы не срабатывают, то default ОБЯЗАН сработать

Подскажу. Это так только в том случае, если дальнейшее поведение определено стандартом.

[Василий Банников]

Евгений Шатунов, изначально был тег C#, по тому я предполагал, что тут C# :)

Answer 2

[d-stream]

Напрашивается что выполнение уходит в Serial.println("no rp"); и не возвращается... ну например ждёт готовности вечно.
Думаю стоит перед ним поставить screen.print(... и если будет видимый результат - точка проблемы понятна