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

Это все делаю на ардуино уно

шоу ми код

шоу ми код

Скрином или текстом?

на пастебин лучше

Ты ж в атмел студии работал что пошло не так?)

Ты ж в атмел студии работал что пошло не так?)

я в ней и работаю

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

Чёт мне схема магнитного пускателя вспоминается

В текстовичке boards.txt появилось много новых записей

Ты ж в атмел студии работал что пошло не так?)

мне под марлин нужна ардуинка

и для некоторых тестов

на пастебин лучше

https://pastebin.com/3i3yfRJq

а все зашел

61

Строка

Перепроверь

unsigned long currentMillis = millis(); объяви глобально, и сделай инициализацию в setup то же самое для previousMillis1

а вторая и так глобально

но инициализацию ей сделай начальную

Ща буду разбираться с этим, спасибо большое

(ledState1 == LOW) можно просто писать: (!ledState1) (ledState1 == HIGH) можно просто писать: (ledState1)

хотя скорей от платформы зависит, стоит проверить

То бишь инвертировать

нет

проверка в условии

а, равно и не равно

надо идти спать, я уже туплю

! Пока не буду использовать, надо вникнуть получше во все

у тебя состояние светодиодов - их всего два?

А таки видел сегодня, что инвертирование это и знаю, что не равно

! Пока не буду использовать, надо вникнуть получше во все

смотри

Arduino.h #define HIGH 0x1 #define LOW 0x0

у тебя состояние светодиодов - их всего два?

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

!0 = TRUE !1= FALSE 0 = FALSE 1 = TRUE

Пара реле вчера и сегодня через делей работала нормально, но ничего допом тогда не получится прикрутить

Параллельно

if (ledState1 == LOW) то же самое if (ledState1 == 0) или if (!ledState1) или if (!0)

if (ledState1 == HIGH) то же самое if (ledState1 == 1) или if (ledState1) или if (1)

Ну это на будущее, чтоб знать)

ладн это между делом )

но лучше пиши как пишешь, наглядней для новичка ;)

вот еще смотри

if((ledState2 == HIGH) && (currentMillis - previousMillis2 >= OffTime2)) { } else if ((ledState2 == LOW) && (currentMillis - previousMillis2 >= OnTime2))

зачем ты одно и то же условие проверяешь дважды?

if (currentMillis - previousMillis2 >= OffTime2) { if (ledState2 == HIGH) { } else { } }

красифшей?

процессинг под линукс )

зачем?

Нет скобки не везде красиво

зачем?

использую для графики

int main(){ }

Так не красиво

красифшей?

Мне надо понятнее пока что))

Так не красиво

это предрассудки

int main() { }

но инициализацию ей сделай начальную

В сетап им присвоить 0? Правильно я понял?

а вот так красиво)

Мне надо понятнее пока что))

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

а вот так красиво)

неа

В сетап им присвоить 0? Правильно я понял?

зачем? можешь сразу загрузить текущее значение милис и присвоить

int main() { }

да вы эстет

да вы эстет

не эстет а зомби

с портянками кода в которых охуеешь читать

В сетап им присвоить 0? Правильно я понял?

так, какие симптомы, опиши еще раз внятно

блядь пиздец

меня на стиль олмана студия подсадила

ERROR: S client not available

больше никак не хочу

Вот ты тоже зомби))

https://youtu.be/uyBePpPyd5c?t=16m9s

в ассемблере скобки не нужны

// конечный автомат для 1-го светодиода if (currentMillis - previousMillis1 >= OffTime1) { if (ledState1 == HIGH) ledState1 = LOW; // выключаем else ledState1 = HIGH; // выключаем previousMillis1 = currentMillis; // зап. время digitalWrite(ledPin1, ledState1); // реализ. сост. } // конечный автомат для 2-го светодиода if (currentMillis - previousMillis2 >= OffTime2) { if (ledState2 == HIGH) ledState2 = LOW; // выключаем else ledState2 = HIGH; // выключаем previousMillis2 = currentMillis ; // зап. время previousMillis1 = currentMillis ; digitalWrite(ledPin2, ledState2); // реализ. сост. digitalWrite(ledPin1, ledState2); }

Вот ты тоже зомби))

ты просто не писал пока сложных условий, которые не оптимизируются Жегалкиным и дискреткой

В сетап им присвоить 0? Правильно я понял?

ай

у тебя previousMillis2 не инициализирована

ты во второе условие никогда не попадешь

а не попадешь

ох блеадь

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

А надо, чтоб оно вместе с ним отключалось

// конечный автомат для 1-го светодиода if (currentMillis - previousMillis1 >= OffTime1) { if (ledState1 == HIGH) ledState1 = LOW; // выключаем else ledState1 = HIGH; // выключаем previousMillis1 = currentMillis; // зап. время digitalWrite(ledPin1, ledState1); // реализ. сост. } // конечный автомат для 2-го светодиода if (currentMillis - previousMillis2 >= OffTime2) { if (ledState2 == HIGH) ledState2 = LOW; // выключаем else ledState2 = HIGH; // выключаем previousMillis2 = currentMillis ; // зап. время previousMillis1 = currentMillis ; digitalWrite(ledPin2, ledState2); // реализ. сост. digitalWrite(ledPin1, ledState2); }

?

Сек

// конечный автомат для 1-го светодиода if (currentMillis - previousMillis1 >= OffTime1) { ledState1 ^= HIGH; // переключаем previousMillis1 = currentMillis; // зап. время digitalWrite(ledPin1, ledState1); // реализ. сост. } // конечный автомат для 2-го светодиода if (currentMillis - previousMillis2 >= OffTime2) { ledState2 ^= HIGH; // переключаем // зап. время previousMillis1 = previousMillis2 = currentMillis; digitalWrite(ledPin2, ledState2); // реализ. сост. digitalWrite(ledPin1, ledState2); }

setup(){ previousMillis1=previousMillis2=currentMillis=millis(); }

Сек

спрашивай )

Arduino: 1.8.5 (Windows 10), Плата:"Arduino/Genuino Uno" C:\Users\MS\Desktop\ARD\sketch_may06c\sketch_may06c.ino: In function 'void setup()': sketch_may06c:25: error: 'currentMillis' was not declared in this scope previousMillis1=previousMillis2=currentMillis=millis(); ^ exit status 1 'currentMillis' was not declared in this scope Этот отчёт будет иметь больше информации с включенной опцией Файл -> Настройки -> "Показать подробный вывод во время компиляции"

Arduino: 1.8.5 (Windows 10), Плата:"Arduino/Genuino Uno" C:\Users\MS\Desktop\ARD\sketch_may06c\sketch_may06c.ino: In function 'void setup()': sketch_may06c:25: error: 'currentMillis' was not declared in this scope previousMillis1=previousMillis2=currentMillis=millis(); ^ exit status 1 'currentMillis' was not declared in this scope Этот отчёт будет иметь больше информации с включенной опцией Файл -> Настройки -> "Показать подробный вывод во время компиляции"

я же сказал глобально объяви

unsigned long currentMillis;

вверху перед setup()

Я запутался просто где и что менял, дописал уже

из loop() убери

Уже

// конечный автомат для 1-го светодиода if (currentMillis - previousMillis1 >= OffTime1) { ledState1 ^= HIGH; // переключаем previousMillis1 = currentMillis; // зап. время digitalWrite(ledPin1, ledState1); // реализ. сост. } // конечный автомат для 2-го светодиода if (currentMillis - previousMillis2 >= OffTime2) { ledState2 ^= HIGH; // переключаем // зап. время previousMillis1 = previousMillis2 = currentMillis; digitalWrite(ledPin2, ledState2); // реализ. сост. digitalWrite(ledPin1, ledState2); }

? тут у тебя включается первое реле, и тут же выключается вторым условием, так?

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

а не

состояние же не меняется

Теперь вообще ничего не включается

Вот так сделай // конечный автомат для 1-го светодиода if (currentMillis - previousMillis1 >= OffTime1) { ledState1 ^= HIGH; // переключаем previousMillis1 = currentMillis; // зап. время digitalWrite(ledPin1, ledState1); // реализ. сост. } // конечный автомат для 2-го светодиода if (currentMillis - previousMillis2 >= OffTime2) { ledState2 ^= HIGH; // переключаем // зап. время previousMillis2 = currentMillis; digitalWrite(ledPin2, ledState2); // реализ. сост. }

Теперь вообще ничего не включается

а ты добавил в loop() загрузку милисекунд в currentMillis?