Почему кнопка конфликтует с двигателем на Arduino?

Question

[Макс Darkleviathan]

Вот мой скетч:

int analogPin = A0;
int val = 0;
#include "GyverMotor.h"

GMotor motor(DRIVER3WIRE, 7, 8, 9, HIGH);

unsigned long time_work;
int tester;

void setup() {
  Serial.begin(9600);
  motor.setLevel(HIGH);

  //motor.setSmoothSpeed(40);
  // ключ на старт!
  motor.setMode(FORWARD);
  //motor.setMinDuty(50);

}

void loop() {
  val = analogRead(analogPin);
  //Serial.println(val);

  if (val < 700) {tester = 1;} // кнопка включена
  if (val >= 700) {tester = 3;} // кнопка выключена
  
  if (tester == 1 && millis() - time_work > 5000) {
      time_work = millis();
      tester = 0;
      Serial.println("step1");
      motor.smoothTick(255);
    }

    if (tester == 0 && millis() - time_work > 5000) {
      time_work = millis();
      tester = 1;
      Serial.println("step2");
      motor.smoothTick(0);
    }

}

Проблема такого рода:
Если кнопка выключена, переменная tester = 3,
Двигатель срабатывает, после нажания кнопки, что меняет значение переменной на 1, и после отрабатывания условия меняется на 0 и так по кругу. Для наглядности в консоль я пишу step1, step2.
Так вот если кнопка подключена, отрабатывает все время только step1, что не понятно почему, переменная то меняется.

Если я игнорирую кнопку и устанавливаю изначально tester = 1 , весб алгоритм с запуском\остановкой отрабатывает как надо. Где может быть проблема?

Comments

[Армянское Радио]

Кнопка. На аналоговом пине. Это стабильно работать не будет..

[Макс Darkleviathan]

Бобби Шифер, Так на цифровом та же картина)))

[Армянское Радио]

Макс Darkleviathan, Сначала убираем явные косяки. Потом разбираемся с тем, что такое tester = 3 и на бумажке проходим программу, понимая, что с таким значением, ни первое, ни второе условие никогда не сработают.

[Макс Darkleviathan]

Бобби Шифер, Я ж написал в самом посте при каком условие сработает. К чему эта демагогия?

Если угодно, условие при котором сработает.

#define button_one 3
#include "GyverMotor.h"

GMotor motor(DRIVER3WIRE, 7, 8, 9, HIGH);

unsigned long time_work;
int tester = 1;

void setup() {
  Serial.begin(9600);
  //pinMode(button_one, INPUT_PULLUP);

  motor.setLevel(HIGH);

  //motor.setSmoothSpeed(40);
  // ключ на старт!
  motor.setMode(FORWARD);
  //motor.setMinDuty(50);

}

void loop() {

  
  if (tester == 1 && millis() - time_work > 5000) {
      time_work = millis();
      tester = 0;
      Serial.println("step1");
      motor.smoothTick(255);
    }

    if (tester == 0 && millis() - time_work > 5000) {
      time_work = millis();
      tester = 1;
      Serial.println("step2");
      motor.smoothTick(0);
    }

}

[Армянское Радио]

Макс Darkleviathan,

Первое.

Не используйте термины, которые не знаете ("Демагогия").

Второе.

У вас при нажатии кнопке переменная tester становится равна тройке, а условие сравнивает ее с нулем. Теперь я доступно написал для вашего понимания?

[Макс Darkleviathan]

Бобби Шифер,
1. Не сомневаюсь что вы понимаете смысл этого слова, раз этим успешно пользуетесь.
2. Прочтите код повнимательней. При нажатии кнопки tester = 1, при отпущенной 3. Но это не было моим вопросом.
Кнопку можно приравнять к булевской переменной и читать digital pin, но это толку не дает. Происходит конфликт, возможно из-за использования сторонней библиотеки, возможно из-за железа. Я ожидал получить вменяемый ответ, а не переливание воды.
Если хочеться дальше заниматься пустословием, проходите мимо. А если действитедьно есть желание помочь, буду оч благодарен.

[Армянское Радио]

Макс Darkleviathan, пробую объяснить еще раз.

1) tester = 1, вы держите кнопку, срабатывает условие, пишет в тестер НОЛЬ
2) вы отпускаете кнопку, в tester пишется три
3) Дальше программа может вертеться в цикле сколько угодно, второе условие не сработает, мотор не остановится.

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

[Макс Darkleviathan]

Бобби Шифер, Стоп. Мы про разные кнопки говорим. Я понял какую кнопку вы имели в виду. Кнопка по такому принципу устроена.


Задествую пока только один пин.

[Армянское Радио]

Макс Darkleviathan,

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

потом цикл прокручивается, в переменную пишется 1 или 3, что исключает срабатывание условия остановки мотора.

[Макс Darkleviathan]

Бобби Шифер, Ага. дошло. Тоесть мне нужно прописать первое условие где не будет таймера, а потом уже прописвать вход в цикличную работу?

[Армянское Радио]

Макс Darkleviathan, у вас в вопросе не написано, что должна делать программа, так что я могу объяснить только то, что она делает в том виде, в каком вы ее написали.

Answer 1

[evgeniy_lm]

А что у вас за изврат с подключением кнопки?
В принципе подключить к аналоговому входу 4-5 кнопок можно (но не нужно). Для одной кнопки это идиотизм.
Посмотрите значение val в плоттере, думаю будет интересно

в сктче у вас есть if (tester == 1 и if (tester == 0 про tester == 3 ваш скетч ничего не знает

Comments

[Макс Darkleviathan]

Изврат такой, потому что думал возможно проблема в кнопке.

При такой конфиграуии так же у меня не переходит на условие if (tester==1 && millis() - time_work >= 5000)

Вот код.

#define button_one 3
#include "GyverMotor.h"
 
GMotor motor(DRIVER3WIRE, 7, 8, 9, HIGH);

unsigned long time_work;
int tester = 4;
boolean butt_breath;

void setup() {
  Serial.begin(9600);
  pinMode(button_one, INPUT_PULLUP);

  motor.setLevel(HIGH);

  //motor.setSmoothSpeed(40);
  // ключ на старт!
  motor.setMode(FORWARD);
  //motor.setMinDuty(50);

}

void loop() {

  butt_breath = !digitalRead(button_one);

  if (butt_breath==1) {
    motor.setSpeed(255); 
    tester = 1; 
    Serial.println((String)"start: "+tester);
 
  }
  if (butt_breath==0) {
    motor.setSpeed(0);
  }


  if (tester==1 && millis() - time_work >= 5000) {
      time_work = millis();
      motor.setSpeed(0);
      tester = 1;
      Serial.println((String)"stop: "+tester);
    }
  
}

[evgeniy_lm]

как я понимаю у вас кнопка подключена стандартно на массу. Тогда чтобы сработал
if (tester==1 && millis() - time_work >= 5000) нужно держать кнопку 5сек
с другой стороны начальное значение time_work неопределённо и что там у вас будет не понятно.

Напишите что вы хотите попробую подредактировать код.

ЗЫ А вы что ни будь про "дребезг контактов" слышали?

[Макс Darkleviathan]

evgeniy_lm, Про дребезг слышал. Понял в чем проблемма, но не знаю пока решения.
Кнопка по типу такой:


И выходит что кнопка всегда зажата и выполняет условие 1.

А необходимо всего лишь нажать кнопку, после чего будет смена включения\выключением по кругу 5000 милисекунд каждый. И выключить принудительно после нажатия кнопки в режим 0.

[evgeniy_lm]

1. Это не кнопка, а выключатель. Разница в том что кнопка замкнута только на время удержания, а выключатель (ну в общем выключатель)

2. код должен быть примерно такой

#define button_one 3
#include "GyverMotor.h"

GMotor motor(DRIVER3WIRE, 7, 8, 9, HIGH);

unsigned long time_work;
int tester = 4;
int btn = LOW,
old_btn = LOW;
unsigned char mot=0, mot1=0;

void setup() {
Serial.begin(9600);
pinMode(button_one, INPUT_PULLUP);
motor.setLevel(HIGH);
motor.setMode(FORWARD);
}

void loop()
{
butt_breath = !digitalRead(button_one);
if (butt_breath != old_btn) //состояние кнопки изменилось
{
old_btn = butt_breath;
if (butt_breath == HIGH)
{
mot = 255;
time_work = millis();
} else mot = 0;
mot1 = mot;
motor.setSpeed(mot1);
if (millis() - time_work >= 5000) // время истекло
{
time_work = millis();
if (mot == 255) && mot == mot1) mot1 = 0; else mot1 = mot;
motor.setSpeed(mot1);
}

[evgeniy_lm]

не проверял, возможно есть ошибки

[Макс Darkleviathan]

evgeniy_lm, Спасибо большое, вроде работает.

А у меня вот так стработало. Правда код немного не логичный получился.

#define button_breath_pin 3 
#include "GyverMotor.h"

GMotor motor(DRIVER3WIRE, 7, 8, 9, HIGH);

// кнопка режим 1
boolean butt_breath;
boolean butt_flag = 0;

long mytime = 2000;


//блок таймера и цикла режим 1
long time_work;
int tester = 3;

void setup() {  
  pinMode(button_breath_pin, INPUT_PULLUP);     
  
  Serial.begin(9600); 
  motor.setLevel(HIGH);
  motor.setMode(FORWARD);
  
}

void loop() {
  
  butt_breath = !digitalRead(button_breath_pin);   
 

    // блок включения режим 1
    if (butt_breath == 1 && butt_flag == 0) {time_work = millis(); tester = 1; Serial.println("start"); motor.setSpeed(255); butt_flag = 1;}    
    if (digitalRead(3)==HIGH && butt_flag == 1) {tester = 3;Serial.println("stop"); motor.setSpeed(0); butt_flag = 0;}
    // блок включения режим 1


    // цикл режим 1
    if (tester == 1 && millis() - time_work > mytime) {
      time_work = millis();
      tester = 0;
      Serial.println("step1");
      motor.setSpeed(0);
    }

    if (tester == 0 && millis() - time_work > mytime) {
      time_work = millis();
      tester = 1;
      Serial.println("step2");
      motor.setSpeed(255);
    }
    // цикл режим 1

}

Answer 2

[Антон]

а чо не так-то?
в цикле после ненажатия кнопки tester=1 -> и 5с if(tester==1 -> timework=milis() и все повторяется,
если нажата, вооще ничего не делаем. второй if никогда не сработает.