Цитата Сообщение от SBorovkov Посмотреть сообщение
Вообще все нормально.

Я бы маленько подправил, но это на работу не повлияет.

Строку
if (RealTemp==130);
можно убрать, все равно ничего не делает.

Строку
int T = 75;
если значение менять в коде не будешь, лучше заменить на
#define T 75
а еще лучше - сразу назвать понятно, что-то вроде
#define DestinationEngineTemp 75
обрати внимание, что в строке с #define точку с запятой в конце не ставят!
Таким образом можно объявить константу.

Объявление
int i;
перенеси в то место, где эта переменная используется - в loop()

Еще совет - не заводи переменных, отличающихся регистром.
int t; //Температура
int T = 75;
Запаришься искать ошибку, если ошибешься регистром при их использовании.
Лучше всего все переменные, смысл которых сложнее, чем переменная цикла называть так, чтобы было понятно что в ней хранится. Хотя бы 2-3 символа, если использование локальное. Если же переменная или константа используется в нескольких функциях, то полноценное имя, иначе потом забудешь что за переменная, или будешь путаться.

И еще - все, что касается выводов МК лучше складывать в #define
То есть вместо AnalogRead(0)
надо написать примерно так, как ты писал выше:
в начале
#define EngineTempSensorPin 0

А дальше везде
AnalogRead(EngineTempSensorPin)

Связано это с тем, что если вдруг тебе надо будет потом поменять выводы ардуино, не придется искать все упоминания вывода, а достатоно тупо поменять константу.
Спасибо за дельные советы. Буду учиться делать код красиво и правильно.