Форум arduino.ua

Vitalik_sce

Учасник

Зареєстрований: 2018-07-24

Повідомлень: 1

проверка if переписывает переменную

Здравствуйте
Когда выполняется функция save_set();  значение переменной alarm встает в 255
Если убрать проверку if  ((alarm != 0) && (set[8] == 1))  тогда работает верно
если ее переделать на if  ((set[8] == 1) && (alarm != 0)) тогда тоже этого глюка нет
В чем проблема?

volatile int set[9] = {0, 0, 0, 0, 0, 0, 0, 0, 0};
volatile int alarm = 0;
/*
....
*/

void setup() 
{
/*
....
*/
save_set(); 
}//setup

void turnbip_call(int turnbip_num, int turnbip_long, int turnp_cou)
{
  if (!turnb_s)
  {
  turnb_n = turnbip_num;
  turnb_l = turnbip_long;
  turnb_c = ((turnb_l * 2) * turnb_n);
  turnp_c = turnp_cou;
  digitalWrite(turns_pin, LOW);
  
  if  ((alarm != 0) && (set[8] == 1))  {digitalWrite(sirena_pin, LOW); } 
  
  }
}

void save_set()
{
  for (int i = 0; i < 10; i++)
  {set[i] = EEPROM.read(31+i);}
}

void loop () 
{
/*
....
*/
}

renoshnik

Учасник

Зареєстрований: 2017-04-03

Повідомлень: 1,025

Re: проверка if переписывает переменную

Проверь строку 127 .

Остання редакція renoshnik (2018-07-24 09:02:09)

Green

Учасник

Зареєстрований: 2015-11-08

Повідомлень: 593

Re: проверка if переписывает переменную

Из еепром считывается больше размера массива.
Руки бы поотрывать за такую писанину.

Kaka

Учасник

Зареєстрований: 2018-03-21

Повідомлень: 411

Re: проверка if переписывает переменную

Если уж у тебя set[9], так и пиши в цикле <9, чего ты <10 вляпал?

Остання редакція Kaka (2018-07-25 09:35:28)

Green

Учасник

Зареєстрований: 2015-11-08

Повідомлень: 593

Re: проверка if переписывает переменную

Там не 9 надо, а sizeof(set). Только это офигенно сложно. И не только для чайников.)))

renoshnik

Учасник

Зареєстрований: 2017-04-03

Повідомлень: 1,025

Re: проверка if переписывает переменную

Понятно, что чтение из памяти кривокосячное. Но как оно влияет на переменную alarm о которой спрашивает ТС.

Вячеслав Азаров

Учасник

Зареєстрований: 2017-05-25

Повідомлень: 1,732

Re: проверка if переписывает переменную

... В чем проблема?

В С нет проверки границ массивов. У вас массив размерности 9 а пишите вы 10 значений. 10-е значение массива попадает на область памяти переменной alarm.

renoshnik

Учасник

Зареєстрований: 2017-04-03

Повідомлень: 1,025

Re: проверка if переписывает переменную

... В чем проблема?

В С нет проверки границ массивов. У вас массив размерности 9 а пишите вы 10 значений. 10-е значение массива попадает на область памяти переменной alarm.

" если ее переделать на if  ((set[8] == 1) && (alarm != 0)) тогда тоже этого глюка нет  "  как это объяснить ?

Вячеслав Азаров

Учасник

Зареєстрований: 2017-05-25

Повідомлень: 1,732

Re: проверка if переписывает переменную

... В чем проблема?

В С нет проверки границ массивов. У вас массив размерности 9 а пишите вы 10 значений. 10-е значение массива попадает на область памяти переменной alarm.

" если ее переделать на if  ((set[8] == 1) && (alarm != 0)) тогда тоже этого глюка нет  "  как это объяснить ?

Это так кажется. Он проявится, потом, на другом состоянии данных.

Kaka

Учасник

Зареєстрований: 2018-03-21

Повідомлень: 411

Re: проверка if переписывает переменную

Там не 9 надо, а sizeof(set).

Ну, уж если по гамбургскому, то там надо sizeof(set)/sizeof(set[0])

Kaka

Учасник

Зареєстрований: 2018-03-21

Повідомлень: 411

Re: проверка if переписывает переменную

" если ее переделать на if  ((set[8] == 1) && (alarm != 0)) тогда тоже этого глюка нет  "  как это объяснить ?

Элементарно, Ватсон. ТС не уточнил, но очевидно что глюка нет не всегда, а тогда, когда set[8] == 1 истинно (видимо, при его тесте так и было). Тогда вторая часть условия (после &&) вообще не проверяется. Это т.н. "ленивая логика"

Green

Учасник

Зареєстрований: 2015-11-08

Повідомлень: 593

Re: проверка if переписывает переменную

Там не 9 надо, а sizeof(set).

Ну, уж если по гамбургскому, то там надо sizeof(set)/sizeof(set[0])

Там вообще каличная EEPROM, которая читает байтами!

renoshnik

Учасник

Зареєстрований: 2017-04-03

Повідомлень: 1,025

Re: проверка if переписывает переменную

" если ее переделать на if  ((set[8] == 1) && (alarm != 0)) тогда тоже этого глюка нет  "  как это объяснить ?

Элементарно, Ватсон. ТС не уточнил, но очевидно что глюка нет не всегда, а тогда, когда set[8] == 1 истинно (видимо, при его тесте так и было). Тогда вторая часть условия (после &&) вообще не проверяется. Это т.н. "ленивая логика"

... такую ленность я бы понял при условии ИЛИ но там условие И
Скорее условие set[8] == 1 должно быть ложным тогда лентяйство засчитывается...

Остання редакція renoshnik (2018-07-25 14:42:51)

Kaka

Учасник

Зареєстрований: 2018-03-21

Повідомлень: 411

Re: проверка if переписывает переменную

Скорее условие set[8] == 1 должно быть ложным тогда лентяйство засчитывается...

Да, конечно, больно быстро писал. Но идею-то ты понял - для того и писалось.