Arduino.ru

Вы бы хоть весь скетч выложили что-ли.

Ну.... каГ-бЕ в общем случае:

"Значения локальных переменных не сохраняются после выхода из функции."

Поэтому значения переменных time и time_light могут приниматься от совершеннейшей балды.

смысл вас грузить ~500 строками, не относящимися к данной части кода, кроме, заданных ранее, глобальных переменных?

конструктивно. попробую зделать их глобальными.

зачем DS1307 для отсчета времени работы вентилятора и тп? еще время показывать?

да, просто вывод на LCD текущего времени, ну а раз DS1307 уже стоит, то почему бы его и не использовать для таймера?

действительно, это оказалось так.

Хм теперь не хочет выключаться, как пройдет интервал работы.Ну думаю тут уж сам разберуь.

Всем спасибо!

при входе в функцию "fan" значение переменной time_light берется от фонаря. При переходе на глобальные переменные не забудьте сделать начальную инициализацию флагов. 

картинку в студию.
Я бы сделал конечный автомат. Типа

...
void loop()
{
    doFan();
}

byte state; // состояние автомата

unsigned long myTimer;
unsigned long delayMs=60000;

void doFan()
{
  // включили свет
  if(state==0 && СветВключён()==true)
  {
	state =  1;
	myTimer = millis();
  }

  // выключили свет когда минута ещё не прошла
  if(state==1 && СветВключён()==false && 
      myTimer + delayMs > millis() )
  {
    state = 0;
  }

  // минута прошла и свет всё ещё горит
  if(state==1 && СветВключён()==true && 
      myTimer + delayMs < millis() )
  {
    state = 2;  // "приготовились"
  }
  
  // свет выключили
  if(state==2 && СветВключён()==false )
  {
    state = 3;  
	myTimer = millis();
	ВентиляторВключить();
  }
  
  if(state==3 &&
      myTimer + delayMs < millis() )
  {
    state = 0;  
	if(ВлажностьНижеАварии()==true)
      ВентиляторВыключить();
  }
}

жаль что я упустил из вида теорию автоматов. Очень удобно и прозрачно.

Сейчас сам пишу код для контроллера солнечного коллектора и уже ноги можно поломать в условиях, флагах. 

...
void loop()
{
    doFan();
}

byte state; // состояние автомата

unsigned long myTimer;
unsigned long delayMs=60000;

void doFan()
{
  // включили свет
  if(state==0 && СветВключён()==true)
  {
	state =  1;
	myTimer = millis();
  }

  // выключили свет когда минута ещё не прошла
  if(state==1 && СветВключён()==false && 
      myTimer + delayMs > millis() )
  {
    state = 0;
  }

  // минута прошла и свет всё ещё горит
  if(state==1 && СветВключён()==true && 
      myTimer + delayMs < millis() )
  {
    state = 2;  // "приготовились"
  }
  
  // свет выключили
  if(state==2 && СветВключён()==false )
  {
    state = 3;  
	myTimer = millis();
	ВентиляторВключить();
  }
  
  if(state==3 &&
      myTimer + delayMs < millis() )
  {
    state = 0;  
	if(ВлажностьНижеАварии()==true)
      ВентиляторВыключить();
  }
}

Как вариант. Надо запесать в примеры. Спасибо.

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

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

RainMan, моя "реализация" не соответствует "тех.заданию"* товарища инженера Соловьёва? ;) 

* я прочитал только текстовую часть первого сообщения. Код не осилил, сорри.

Не берите на свой счет. Это просто мысли в слух.

Доброго времени суток !

Я бы пробовал так:

typedef enum {
          off,
          onlight1l,
          onlight1h,
          offlight
} StatType;

StatType status = off;
volatile unsigned long tim;

void loop()
{
        switch(status) {
        case off:
                do_off();
                break;
        case onlight1l:
                do_light1l();
                break;
        case onlight1h:
                do_light1h();
                break;
        case offlight:
                do_offlight();
                break;
        }
}
void do_off()
{
        if (свет включен)  {
            tim = millis();
            status = onlight1l;}
}
void do_light1l()
{
unsigned long tim2;

         if (свет включен) {
                  tim2 = millis();
                  tim2 -= tim;
                  if (tim2 > 60000) status = onlight1h;
         } else status = off;
}
void do_light1h()
{
        if (свет погас) {
               вентилятор_включить();
               tim = millis();
               status = offlight;
        }
}
void do_offlight()
{
unsigned long tim2;

        tim2 = millis();

        tim2 -= tim;
        if (tim2 > интервал) {
                   if (авария) tim = mills(); else {ветилятор_выключить();status = off;}
         }
}