Код с множественными условиями

Непонятно, что на самом деле должен делать код (какого поведения вы пытаетесь добиться), поэтому я рассмотрю только очевидные проблемы программирования.

1. Следующие объявления в глобальной области действия кажутся номерами контактов:

   int x = 45;
   int y = 47;
   int z = 41;
   int w = 43;
   
   int WaterIn =  40;
   int WaterOut = 42;
   
   int Pump1 = 25;
   int Pump2 = 24;
   int Pump3 = 27;
   

   Но в функции Valves() вы обращаетесь с ними так, как будто они представляли значение этих выводов. Это не относится к делу. Если вы хотите прочитать состояние контакта, вы не можете сделать

   if ((x == HIGH) && (y == HIGH) && (z == HIGH) && (w == LOW))
   

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

   if( digitalRead(x) == HIGH && digitalRead(y) == HIGH && digitalRead(z) == HIGH && digitalRead(w) == HIGH)
   

   И это можно сократить до:

   if( digitalRead(x) && digitalRead(y) && digitalRead(z) && digitalRead(w) )
   

2. В упомянутых операторах if вы устанавливаете переменную несколько раз, ничего с ней не делая. Любая последовательная запись в эту переменную перезапишет предыдущее значение. После блока кода

   pinToTurnHigh = WaterIn;
   pinToTurnHigh = Pump1;
   pinToTurnHigh = Pump2;
   pinToTurnHigh = Pump3;
   

   Переменная pinToTurnHigh равна 27 (Pump3). Значения, которые вы присвоили непосредственно перед этой строкой, теряются. Вам действительно нужно что-то делать с переменной, иначе первые 3 строки этого блока не имеют смысла.

3. В большинстве случаев не рекомендуется использовать переменные с изменяющимися значениями для обработки номеров выводов, поскольку это затруднит понимание кода. Схема в основном зашита, она не изменится. Таким образом, нет необходимости присваивать номера выводов изменяющейся переменной. Вместо этого вы можете напрямую делать то, что вам нужно, с постоянным номером контакта. (На самом деле оптимизатор кода, идущий в комплекте с компилятором, может оптимизировать неизменяемые переменные с номером контакта, чтобы вы прописали их в своем коде, но в самом коде их может и не быть.) Тут вроде бы и нужно чтобы записать значение HIGH для всех упомянутых контактов. Вы можете сделать это напрямую:

   digitalWrite(WaterIn, HIGH);
   digitalWrite(Pump1, HIGH);
   digitalWrite(Pump2, HIGH);
   digitalWrite(Pump3, HIGH);
   

   Здесь важно понимать, что digitalWrite() одновременно работает только с одним выводом. Если вы хотите установить с ним несколько контактов, вам придется вызывать его несколько раз с соответствующими номерами контактов.

4. Теперь время вашего кода: функция Calendar() заблокирует остальную часть кода примерно на 4 секунды, потому что вы использовали так много delay(). В это время Arduino ни на что не реагирует. Переключатели тоже ничего не сделают (поскольку Arduino занят ожиданием в delay() вместо проверки переключателей). Код, который вызывается после функции Calendar(), не содержит delay(), поэтому он будет работать очень быстро. В функции Valves() вы сначала сбрасываете все насосы и клапаны на LOW. Затем вы пишете HIGH на один контакт (как упоминалось выше, я предполагаю, что вы хотели написать их все при условии). После этого программа вернется к функции loop() и снова выполнит функцию resetAllPumpValve(), которая установит для всех выходов значение LOW.

   В целом это означает, что Arduino возится с функцией Calendar(), не делая ничего другого. Затем он будет сбрасывать, устанавливать и снова сбрасывать контакты. Поскольку там нет временного кода, это произойдет очень быстро. Таким образом, контакты должны быть LOW большую часть времени и включаться примерно каждые 4 секунды в течение небольшого промежутка времени (небольшое количество миллисекунд, так как это примерно время для выполнения digitalWrite). () или его братья и сестры). При этом моторы или ценности вряд ли успеют сдвинуться с места.

   Чтобы решить эту проблему, вы можете изучить пример BlinkWithoutDelay, поставляемый с Arduino IDE. Он показывает неблокирующий стиль кодирования, который будет что-то делать, только если пришло время это сделать. Так же, как когда вы печете пиццу. Вы не будете сидеть перед духовкой и терпеливо ждать, пока пицца полностью пропечется. Вы поставите ее в духовку и регулярно (может быть, каждые 5 минут) смотрите, готова ли пицца к употреблению. Тем временем вы займетесь другими делами. Это то, что там делается с помощью функции millis(), которая вернет количество миллисекунд, прошедших с момента запуска программы. Вы можете изучить пример и узнать больше о нем в Google. Это неоднократно обсуждалось в Интернете.

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

Развлекайтесь во время обучения :-)

Я бы добавил к ответу Крисла, сказав, что если вы хотите сделать код более компактным, вы можете представить 4 логические переменные одним байтом,

например, w HIGH + x HIGH + y LOW + z LOW = 1100.
например w HIGH + x HIGH + y LOW + z HIGH = 1101.

Тогда было бы проще реализовать оператор switch вместо if( digitalRead(x) == HIGH && digitalRead(y) == HIGH && digitalRead (z) == HIGH && digitalRead(w) == HIGH) и т. д.

Например:

byte state;

void setup(){}

void loop(){
 // вставляем код для чтения состояния
 switch (state)
 {
     case 1: 
         // вставляем код для запуска, если state = 0b0001;
     break;

     case 13: 
         // вставляем код для запуска, если state = 0b1101;
     break;

     default: // код для запуска, если состояние не соответствует ни одному из случаев
 }
 // вставляем другой код
}

Если вы хотите использовать цифровые контакты 30-37 ATmega, вы можете использовать команду порта AVR PINC для одновременного считывания всех состояний HIGH/LOW с помощью одной инструкции. Я оставляю это вам в качестве упражнения!