Использование switch case с 5-позиционным переключателем и последовательным выходом

В следующий раз выровняйте свой код с помощью Ctrl-K (после выбора).

И да, вы можете значительно оптимизировать свой код (для удобства сопровождения) с помощью массивов:

Вместо

const int buttonPin1 = 2;
const int buttonPin2 = 3;
const int buttonPin3 = 4;
const int buttonPin4 = 5;
const int buttonPin5 = 6;

использовать

const int buttonPins[] = { 2, 3, 4, 5, 6 };

В разработке программного обеспечения обычно начинают с индекса 0 (например, индекс массива).

Для состояний кнопки:

int buttonState1 = 0; // переменная для чтения статуса кнопки
int buttonState2 = 0;
int buttonState3 = 0;
int buttonState4 = 0;
int buttonState5 = 0;

вы можете использовать

boolean buttonStates[5];
for (uint8_t buttonIndex = 0; buttonindex < 5; buttonIndex++)
{
    buttonStates[buttonIndex]= false;
}

Вместо того, чтобы устанавливать режимы вывода:

pinMode(buttonPin1, INPUT); pinMode(кнопкаPin2, ВХОД); pinMode(кнопкаPin3, ВХОД); pinMode(кнопкаPin4, ВХОД); pinMode(кнопкаPin5, ВВОД);

Используйте цикл for:

for (uint8_t buttonIndex = 0; buttonindex < 5; buttonIndex++)
{
   pinMode(buttonPins[buttonIndex], INPUT);
}

Вы можете объединить два цикла for выше с одним.

Чтобы прочитать состояния кнопки:

buttonState1 = digitalRead(buttonPin1) == 2;
buttonState2 = digitalRead(buttonPin2) == 3;
buttonState3 = digitalRead(buttonPin3) == 4;
buttonState4 = digitalRead(buttonPin4) == 5;
buttonState5 = digitalRead(buttonPin5) == 6;

Использовать

for (uint8_t buttonIndex = 0; buttonindex < 5; buttonIndex++)
{
   buttonStates[buttonIndex] = digitalRead(buttonPins[buttonIndex]) == buttonPinds[buttonIndex]) ? true : false;
}

Что касается переключателя, я думаю, вам нужно оставить его, так как он отличается для каждого контакта.

И вместо использования значения 5 используйте, например:

const uint8_t NR_OF_BUTTONS = 5;

и везде используйте константу NR_OF_BUTTONS.

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

Кроме того, вместо нескольких массивов (каждый длиной NR_OF_BUTTONS) вы можете использовать массив структур, содержащих номер контакта и состояние, но это, возможно, выходит за рамки ваших знаний в области программирования. Использование массивов уже является огромным улучшением.

Майкл уже дал вам несколько отличных советов по очистке вашего кода.

Ваше выражение switch не имеет особого смысла. Вы переключаетесь на основе индекса вне допустимого диапазона в массив констант.

После того как вы прочитаете состояния всех кнопок, описанные Майклом, ваш цикл должен решить, что делать с состоянием каждого переключателя.

Это переключатель типа джойстика, когда пользователь может задействовать только одно направление за раз? Или можно нажать 2 переключателя сразу?

Если это ромб из 5 переключателей с переключателем посередине, пользователь может одновременно нажимать любую комбинацию переключателей.

Я бы предложил такой код:

void loop() {
    if buttonStates[0] == HIGH {
       //код для обработки нажатия кнопки 0 (контакт 2)
    }

    if buttonStates[1] == HIGH {
       //код для обработки нажатия кнопки 0 (контакт 3)
    }
    //И так далее для остальных ваших коммутаторов.
}

Если вы хотите выполнять действия в ответ на нажатие кнопок, следует позаботиться о реагировании на края сигнала, а не на уровни. Другими словами, вы выполняете действие только тогда, когда сигнал изменяется с От LOW до HIGH, а не каждый раз, когда вы видите HIGH.

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

#include <Bounce2.h>

const uint8_t pinDim    = 3;
const uint8_t pinMedium = 4;
const uint8_t pinBright = 5;

Bounce buttonDim, buttonMedium, buttonBright;

void setup() {
    buttonDim.attach(pinDim, INPUT);
    buttonMedium.attach(pinMedium, INPUT);
    buttonBright.attach(pinBright, INPUT);
}

void loop() {
    buttonDim.update();
    buttonMedium.update();
    buttonBright.update();

    if (buttonDim.rose()) {
        Serial.println("dim");
    }
    if (buttonMedium.rose()) {
        Serial.println("Medium");
    }
    if (buttonBright.rose()) {
        Serial.println("bright");
    }
}

Вы также можете, как предложено в ответе Мишеля Кейзерса, использовать массивы для выводов и объекты Bounce. Хотя это помогло бы рационализировать вызовы attach() и update(), они не помогут с мясистым часть кода, т.е. реагирующая на нажатия. Там я бы предпочел кнопка должна иметь описательные имена.