Відповісти

Введіть повідомлення і натисніть Надіслати
Параметри

Назад

Огляд теми (нові повідомленні вгорі)

Kaka
2018-07-25 19:13:12
renoshnik пише:

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

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

renoshnik
2018-07-25 14:40:25
Kaka пише:
renoshnik пише:

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

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

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

Green
2018-07-25 14:21:51
Kaka пише:
Green пише:

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

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

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

Kaka
2018-07-25 14:07:46
renoshnik пише:

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

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

Kaka
2018-07-25 14:04:12
Green пише:

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

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

Вячеслав Азаров
2018-07-25 12:00:23
renoshnik пише:
Вячеслав Азаров пише:
Vitalik_sce пише:

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

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

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

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

renoshnik
2018-07-25 11:52:51
Вячеслав Азаров пише:
Vitalik_sce пише:

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

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

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

Вячеслав Азаров
2018-07-25 11:36:32
Vitalik_sce пише:

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

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

renoshnik
2018-07-25 10:56:04

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

Green
2018-07-25 10:18:22

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

Kaka
2018-07-25 09:35:13

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

Green
2018-07-24 08:26:40

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

renoshnik
2018-07-24 06:56:08

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

Vitalik_sce
2018-07-24 00:16:16

Здравствуйте
Когда выполняется функция 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 () 
{
/*
....
*/
}

Підвал форуму