Помогите исправить код.

Vetrinus · 18 авг 2016

lav522 сказал(а): ↑

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

Вы не успеваете попасть в тот мизерный интервал, порядка 20 микросекунд, когда опрашивается состояние кнопки.

lav522 · 18 авг 2016

Vetrinus сказал(а): ↑

Например, вот так:

Код (C++):

#define BUZZER 5
#define LED 7
#define SWITCH 8
#define ECHO 11
#define TRIG 12
#define TIMEOUT 5
#define DIST_MIN 45
#define DIST_MAX 60

#define SWITCH_ON (!digitalRead(SWITCH)) // макрос для удобства

enum{DISABLED, READY, ENABLED, ALARM}; // возможные режимы работы
byte state; // текущий режим работы
int freequency=400;
bool buzzer_mode=true;

void setup()
{
pinMode(BUZZER, OUTPUT);
pinMode(LED, OUTPUT);
pinMode(TRIG, OUTPUT);
pinMode(ECHO, INPUT);
pinMode(SWITCH, INPUT_PULLUP);
state = DISABLED;
}

void loop() // основная логика работы
{
switch(state)
{
case DISABLED: // спящий режим. Все отключено, проверяется только состояние переключателя.
if(SWITCH_ON) state = READY;
break;

case READY: // режим готовности. Отсчет 5 сек, затем переход в режим охраны.
for(byte i = 0; i < TIMEOUT; i++)
{
digitalWrite(LED, HIGH);
tone(BUZZER, 1200);
delay(500);
digitalWrite(LED, LOW);
noTone(BUZZER);
delay(500);
}
state = ENABLED;
break;

case ENABLED: // режим охраны. Постоянный опрос сонара и состояния переключателя.
if(!SWITCH_ON) state = DISABLED;
else
{
int distance = readSensor();
if (distance < DIST_MIN || distance < DIST_MAX) state = ALARM;
}
break;

case ALARM: // режим тревоги. Звучит сирена и проверяется только состояние переключателя.
/*for(int i = 400; i < 1600; i=+10)
{
tone(BUZZER, i);
delay(20);
noTone(BUZZER);
}*/
if(buzzer_mode){
tone(BUZZER, freequency);
delay(20);
noTone(BUZZER);
freequency=freequency+10;
} else {
tone(BUZZER, freequency);
delay(20);
noTone(BUZZER);
freequency=freequency-10;
}
if(freequency > 1600 || freequency <400 ) buzzer_mode=!buzzer_mode;
digitalWrite(LED, !digitalRead(LED));
if(!SWITCH_ON)
{
state = DISABLED;
digitalWrite(LED, LOW);
freequency=400;
buzzer_mode=true;
}
break;

}
}

int readSensor()
{
int duration, cm;
digitalWrite(TRIG, LOW);
delayMicroseconds(2);
digitalWrite(TRIG, HIGH);
delayMicroseconds(10);
digitalWrite(TRIG, LOW);
duration = pulseIn(ECHO, HIGH);
cm = duration / 58;
return cm;
}

Нажмите, чтобы раскрыть...

теперь просто супер, именно то, что хотел, еще только поиграюсь с мелодией) Спасибо огромное за оперативную помощь, писал на офф сайте, проигнорили. Амперка рулит

Tomasina · 18 авг 2016

а корректно ли обрабатывается конструкция if(!SWITCH_ON) ?
Попробуй заменить if(!SWITCH_ON) на if(digitalRead(SWITCH))

Vetrinus · 18 авг 2016

lav522 сказал(а): ↑

Амперка рулит
Нажмите, чтобы раскрыть...

Ну, с чем не поспоришь, с тем не поспоришь, форум реально полезен. И амперка очень грамотно поступила, сделав его дополнением к магазину.

Помогите исправить код.

Vetrinus Гик

lav522 Нуб

Tomasina Сушитель лампочек Модератор

Vetrinus Гик