Arduino.ru

1) будут, именно это условие написано правильно. Это не касается иных ошибок или логического не соответствия желаемого с описанным.

2) что именно нужно оптимизировать?
3) немного громоздко, но лучше сперва создать переменные, а потом эти переменные сравнивать

Меньше нуля часов не бывает. Как и больше 24. Даже 24 не бывает, а больше 24 и подавно.

Верно, не подумал

Подумал что этот код можно сделать лучше
 

с ds.getTempC(sensor1) могут возникнуть проблемы (не обязательно). избавьтесь от функции и используйте переменную в условии

Еще вопрос, для проверки на перегрев использую такое условие:
 if (ds.getTempC(sensor1) == 70){peregrev1 = false;}  if (ds.getTempC(sensor1) == 40) {peregrev1 = true;}

Но условие отрабатывает не верно, вот этот кусок кода (ds.getTempC(sensor1) < 70), проходит, не ждет пока температура упадет до 40

есть мысли?

такие строгие условия (выделил) скорее всего не будут выполняться. опишите словами логику работы по этим условиям

 if (ds.getTempC(sensor1) >= 70){peregrev1 = false;}  if (ds.getTempC(sensor1) <= 40) {peregrev1 = true;}
так сделал и заработало

вообще смыл такой был, если темп достигнет 70 градусов, отключить прибор и ждать пока не упадет до 40, потом включить

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

если <40C включаем нагрев

если >70C выключаем нагрев

называется гистерезис.

Загуглю гестерезис, спасибо

Думаю тут просто true и false перепутаны.

если >70C "мотор" перегрелся, выключаем.

если < 40C "мотор" остыл, включаем.

В данном условии ночью результат не зависит от температуры. Это так и задумано? Если нет, пользуйтесь скобками. 

Так и задумано, работать 20 часов в сутки, не превышая 70 градусов, если достиг 70, отключить устройство и ждать пока температура остынет до 40

Смотрите что будет в час ночи при температуре 71 градус.

const uint8_t pinLed = 13;

void setup() {
	pinMode(pinLed, 1); digitalWrite(pinLed, 0);
	uint8_t hour = 1; // час
	uint8_t tempC = 71; // температура
	bool peregrev1 = false;

	if (tempC >= 70) peregrev1 = true; // проверяем на перегрев

	if (hour >= 0 && hour <= 5 || hour >= 9 && hour <= 24 && tempC < 70 && peregrev1 == false) {
		digitalWrite(pinLed,1); // зажигаем светодиод, если выражение true
	}
}

void loop() {}

Я без скобок логику условия вообще не понимаю....

Это ваш плюс или "их" минус?

Это факт )))

С таким условием с 0 до 5 часов всегда будет true, с 6 до 8 всегда false, с 9 до 23 будет зависеть от температуры и флага. Вроде, так.

Как вариант можно и без скобок, но будет не оптимально.

Я неправильно выразился, без дополнительных скобок записать выражение, которое хочет ТС. В моём понимании чего он хочет.

А вот в понимании компилятора:

Согласен, был не прав.

Но можно ещё и по нескольким if-ам раскидать.