Остановка/запуск двигателя постоянного тока с концевыми выключателями и кнопкой включения/выключения

Проверьте, есть ли у ваших переключателей подтягивающие резисторы

Было бы полезно, если бы ваш вопрос включал схему. Есть ли у вас подтягивающие резисторы в соответствии с вашими активационными и концевыми выключателями? Если нет, вы можете обнаружить, что входные данные "плавающие", то есть они будут случайным образом возвращать либо ВЫСОКИЕ, либо НИЗКИЕ значения. Это, возможно, приведет к тому, что двигатель будет двигаться вперед и назад, но поскольку цикл, возможно, происходит сотни раз в секунду, вы можете не видеть ничего движущегося. Вам нужны повышающие или понижающие резисторы в схеме, чтобы Arduino мог видеть определенный НИЗКИЙ или ВЫСОКИЙ уровень. Видишь https://www.arduino.cc/en/Tutorial/Button для урока с резистором или даже лучше используйте https://www.arduino.cc/en/Tutorial/InputPullupSerial и используйте существующие бортовые импульсные резисторы на Arduino, как предложил Юрай ниже.

Потенциальная Логическая Ошибка

В этой части кода может быть логическая ошибка:

  if (onOffButtonState == HIGH) {
    if ( frontButtonState == LOW ) {
      drawMovesForward();
    }
    else if (backButtonState == LOW ) {
      drawMovesBackward();
    }
  }
  else {
    if ( frontButtonState == HIGH ) {
      stopDrawFromMoving();
    }
    else if (backButtonState == HIGH ) {
      stopDrawFromMoving();
    }
  }

Цикл №1: Допустим, состояние кнопки включения ВЕЛИКО (кто - то нажал переключатель в положение ВКЛ.). Ящик закрыт, поэтому передний концевой выключатель нажат, поэтому состояние передней кнопки ВЫСОКОЕ. Таким образом, программа выполнит stopDrawFromMovingи ничего не сделает. В настоящее время вы говорите, что ничего не происходит, так что это может быть вашей проблемой.

Что вам, вероятно, понадобится сделать, так это использовать флаги и циклы while. Например:

В начале цикла() объявите это

drawerMovingOutFlag = 1;

Затем измените цикл, чтобы он содержал:

if (onOffButtonState == HIGH) {
    if ( frontButtonState == HIGH ) {
        while(drawerMovingOutFlag == 1)
            drawMovesForward();
            if ( backButtonState == HIGH ) {
                drawerMovingOutFlag = 0;
                stopDrawFromMoving()
            }
        }
    }
    else ...

Я позволю тебе сделать все остальное!

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

// constants won't change. They're used here to set pin numbers:
const int buttonPin = 4;     // the number of the pushbutton pin
const int buttonPin3 = 11;     // the number of the pushbutton pin
const int buttonPin2 = 2;     // the number of the pushbutton pin
const int motorPin =  8;      // the number of the LED pin
const int motorPin2 =  7;      // the number of the LED pin

// variables will change:
boolean buttonState = 0;         // variable for reading the pushbutton status
boolean buttonState2 = 0;         // variable for reading the pushbutton status
boolean buttonState3 = 0;   // variable for reading the pushbutton status


int drawPosition = 2;  // Tells progam what position the draw is in


/**
 *  setup inputs for switches and outs for motor pins
 *  serial begin to read the switches to test for errors
 */
void setup() {
    // initialize the pin as an inputs:
    pinMode(motorPin, OUTPUT);
    pinMode(motorPin2, OUTPUT);
    // initialize the pushbutton pin as an input:
    pinMode(buttonPin, INPUT);
    pinMode(buttonPin2, INPUT);
    pinMode(buttonPin3, INPUT);
    Serial.begin(9600);
}

/**
 * this function turns motor foward 
 */
void drawForward() {
     // turn motor foward:
     digitalWrite(motorPin, LOW);
     digitalWrite(motorPin2, HIGH);
}

/**
 * this function turns motor backwards
 */
void drawBackward() {
     // turn motor in other direction:
     digitalWrite(motorPin, HIGH);
     digitalWrite(motorPin2, LOW);  
}

/**
 * Stop the motor form moving
 */
void stopDrawFromMoving() {
  digitalWrite(motorPin, LOW);
  digitalWrite(motorPin2, LOW);

}


/**
 * 
 */
void loop() {
   // read the state of the pushbutton value:
   buttonState = digitalRead(buttonPin);
   buttonState2 = digitalRead(buttonPin2);
   buttonState3 = digitalRead(buttonPin3);



   // check if the pushbutton is pressed. If it is, the buttonState is HIGH:
   if (buttonState == HIGH) {
      drawPosition = 2; 
      Serial.println(F("Front button state is high draw is open")); 
      stopDrawFromMoving();
   }else if (buttonState2 == HIGH) {
      drawPosition = 1;        
      stopDrawFromMoving();
  } 

  if (buttonState3 == HIGH) {
    if( drawPosition  == 1 ){
      drawBackward(); 
      delay(100);      
      drawPosition = 2;
      delay(100);  
    }else if (drawPosition == 2 ) {
      delay(100); 
      drawForward();
      delay(100); 
      drawPosition = 1;       
    }

  }
}

Это отличный проект. Отличная работа.

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

/* Allows the program to keep track of the last known position of the drawer */
enum class DrawerPositions : byte
{
  Closed,
  Open
} drawerPosition = DrawerPositions::Closed;

/* Keep track of what direction the drawer is moving, can one of the following:  Null | Closing | Opening */
enum class DrawerMovingDirections : byte
{
  Null,
  Closing,
  Opening
} drawerMovingDirection = DrawerMovingDirections::Null;
...
if (drawerPosition == DrawerPositions::Closed)
...
else if (drawerPosition == DrawerPositions::Open)
...
if (drawerMovingDirection == DrawerMovingDirections::Opening)
...

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

#include "Debouncer.h"
...
void loop()
{
  static Debouncer limitSwitch0(limitSwitchPin0);
  static Debouncer limitSwitch1(limitSwitchPin1);
  static Debouncer manualToggleButton(manualToggleButtonPin);
  
  limitSwitch0.Update();
  limitSwitch1.Update();
  manualToggleButton.Update();
  
  if (drawerIsMoving == true)
  {
    if (drawerMovingDirection == DrawerMovingDirections::Opening)
    {
      if (limitSwitch0.Output() == LOW)  // Get the debounced output of the switch.
      {
        drawerPosition = DrawerPositions::Open;
  ...
  if(manualToggleButton.Rise())   // For rising edge trigger.
  //if(manualToggleButton.Fall())  // For falling edge trigger.
  {
    ...
  }
  ...
}

Вот демо-версия дебюта, которую я поставил на Wokwi.