Есть идеи, почему мой светодиод не работает с оператором if

Question

Есть идеи, почему мой светодиод не работает с оператором if

Я попробовал несколько обходных путей, но безуспешно.

#include <LiquidCrystal.h>
float tempC;
int tempPin = 0;// BS E D4 D5 D6 D7
int buzzer = 4;
int LED =2;
LiquidCrystal lcd(7, 8, 9, 10, 11, 12);
unsigned char i;
void setup()
{
lcd.begin(16, 2);
lcd.setCursor(0, 0);
lcd.print("Temp        C ");
pinMode(LED, OUTPUT);
pinMode(buzzer,OUTPUT);//инициализируем вывод зуммера как выход
}
void loop()
{
tempC = getTemperature();
lcd.setCursor(6, 0); // Отображение температуры в градусах Цельсия
lcd.print(tempC);
delay(500);
}
float getTemperature()
{
int tempReading = analogRead(tempPin); // Хорошо
double tempK = log(1000000.0 * ((1024.0 / tempReading - 1)));
//Температура в Кельвинах
tempK = 1 / (0.001129148 + (0.000234125 + (0.0000000876741 * tempK * tempK )) * tempK );
return (tempK - 273.15);

if (tempC>0)
digitalWrite(LED,HIGH);
else
digitalWrite(LED, LOW);
}

@Dermaine Whittaker, 👍1

Обсуждение

две возможные причины... 1. светодиод подключен неправильно и он не горит, что бы вы ни делали... 2. оператор if дает результат, который не включает светодиод, @jsotola

Я отредактировал ваш вопрос и использовал кнопку {} для форматирования кода, но в вашем коде нет отступов. Вам необходимо отредактировать свой код, чтобы использовать правильные отступы. Без этого читать очень тяжело., @Duncan C

Ваш код для просмотра температуры испорчен. У вас есть глобальная переменная tempC. Функция getTemperature() не меняет значение tempC, но использует его, чтобы решить, должен ли светодиод гореть или нет. Это означает, что эта функция примет решение на основе предыдущего значения tempC., @Duncan C

Вам следует переписать функцию getTemperature(), чтобы поместить температуру Цельсия во временную переменную, использовать ее, чтобы решить, что делать со светодиодом, а затем вернуть временное значение. Таким образом, getTemperature() будет устанавливать светодиод на основе только что рассчитанного значения температуры, а не на старом, уже устаревшем показании температуры., @Duncan C

Также обратите внимание, что если tempReading равен 0, ваш код выйдет из строя с ошибкой деления на ноль. Вам следует добавить код обработки ошибок, чтобы справиться с этим случаем., @Duncan C

1 ответ

Лучший ответ:

@*VE7JRO* · Accepted Answer · 2019-03-20 01:10-00:00

Светодиод не работает с оператором if, поскольку вызывается оператор return Завершает функцию и возвращает значение из функции вызывающей функции, если это необходимо. Попробуйте свой скетч с оператором return как «последним» в функции.

#include <LiquidCrystal.h>

float tempC;
int tempPin = 0; // БС Е D4 D5 D6 D7
int buzzer = 4;
int LED = 2;
LiquidCrystal lcd(7, 8, 9, 10, 11, 12);
unsigned char i;

void setup() { 
  lcd.begin(16, 2); 
  lcd.setCursor(0, 0); 
  lcd.print("Temp C "); 
  pinMode(LED, OUTPUT); 
  pinMode(buzzer,OUTPUT);//инициализируем вывод зуммера как выход
} 

void loop() { 
  tempC = getTemperature(); 
  lcd.setCursor(6, 0); // Отображение температуры в градусах Цельсия
  lcd.print(tempC); 
  delay(500); 
} 

float getTemperature() { 
  int tempReading = analogRead(tempPin); // Хорошо
  double tempK = log(1000000.0 * ((1024.0 / tempReading - 1))); //Температура в Кельвинах
  tempK = 1 / (0.001129148 + (0.000234125 + (0.0000000876741 * tempK * tempK )) * tempK ); 
  if (tempC>0) digitalWrite(LED,HIGH); else digitalWrite(LED, LOW); 
  return (tempK - 273.15);
}