Can you please help me with my code?

Projects Programming
1

So for my little project I created a thermostat using an active buzzer, LM35 temp sensor, and 1602 lcd liquid crystal screen. Everything is going smoothly except the active buzzer portion. I want the buzzer to activate when it reaches 75 degrees F and to stop when it goes below but it wants to keep buzzing forever?
Sorry if code is sloppy. I'm still a beginner.

#include <LiquidCrystal.h>
float temp;
float temp2;


LiquidCrystal lcd(7,8,9,10,11,12);

void setup() {
  Serial.begin(9600);
  pinMode(13, OUTPUT);
  lcd.begin(16, 2);
  lcd.print("The temp is  ");

}

void loop() {
temp= analogRead(1)*5*100/1024.00;
temp2 = temp*1.8+32;
Serial.print("The tempeture is ");


  lcd.setCursor(0, 1);

  lcd.print(temp2);
  delay(3000);
  
  if(temp2 > 75){
        digitalWrite(13,HIGH);
        Serial.print("The tempeture is ");
        lcd.setCursor(0, 1);
        lcd.print(temp2);
        delay(1000);
 
    
  }
}
2

Less 'sloppy' code

#include <LiquidCrystal.h>
float temp;
float temp2;

LiquidCrystal lcd(7,8,9,10,11,12);

void setup() 
{
    Serial.begin(9600);
    pinMode(13, OUTPUT);
    lcd.begin(16, 2);
    lcd.print("The temp is  ");
}

void loop() 
{
    temp= analogRead(1)*5*100/1024.00;
    temp2 = temp*1.8+32;
    Serial.print("The tempeture is ");
    lcd.setCursor(0, 1);
    lcd.print(temp2);
    delay(3000);

    if(temp2 > 75)
    {
        digitalWrite(13,HIGH);
        Serial.print("The tempeture is ");
        lcd.setCursor(0, 1);
        lcd.print(temp2);
        delay(1000);
    }
}

You digitalWrite(13,HIGH); when temp2 > 75, but nowhere do you turn it off. What happens when temp2 <= 75?

3

oh ok. I thought that since the code is constantly looping it turn off on its own. So it would work if I put in an else if statement saying that the buzzer is to turn off if < 75???

4

above reply is the main answer in your question.
But also take note that "serial.print" , "lcd" and "delay" is happening 2 times

5

No need for else if, just else. If temp2 is not >75, then what is it?

6

Thank you for the replies and letting me know how to make my code look cleaner. The circuit is working exactly how I want it to now!=)

7

fettersp:
oh ok. I thought that since the code is constantly looping it turn off on its own.

Remember that code only does what you tell it to do. Take also the advice offered by demkat1. Your entire if statement, after removing the duplicate prints, can be reduced to

if(temp2 > 75) digitalWrite(13,HIGH);
else digitalWrite(13,LOW);