Help using Relays with greenhouse

Projects Programming
15

Here we go :

//Library for SainSmart LCD 20x4 I2C 
#include <FastIO.h>
#include <I2CIO.h>
#include <LCD.h>
#include <LiquidCrystal.h>
#include <LiquidCrystal_I2C.h>
#include <LiquidCrystal_SR.h>
#include <LiquidCrystal_SR2W.h>
#include <LiquidCrystal_SR3W.h>

#define lcdAddr 0x27

LiquidCrystal_I2C lcd(lcdAddr, 2, 1, 0, 4, 5, 6, 7, 3, POSITIVE);

//Library for DHT Sensor
#include "DHT.h"
#define DHTTYPE DHT11
#define DHTPIN 2
DHT dht(DHTPIN, DHTTYPE);


//Library for Button 
#include <Button.h>

//Library for Datalogger
#include <SD.h>
#include <SPI.h>
const int chipSelect = 10;

//Basic Libraries
#include <Wire.h>

//Analog Sensors
const int WaterLevelPin = A0;
int sensorValue = 0;

//Relays
const int relay1Pin = 12;
const int relay2Pin = 11;
const int relay3Pin = 10;
const int relay4Pin = 9;
int relay1State = LOW;
int relay2State = LOW;
int relay3State = LOW;
int relay4State = LOW;

void setup() {

//Setup for DHT
  Serial.begin(9600);
  Serial.println("DHT Readings");

dht.begin();

//Setup for Datalogging

Serial.print("Initializing SD card...");
if (!SD.begin(chipSelect)) {
    Serial.println("Card failed, or not present");
    return;
  }
  Serial.println("card initialized.");
//Setup for LCD Display
  
  lcd.begin(20,4);

//Relay pins

  pinMode(relay1Pin, OUTPUT);
  pinMode(relay2Pin, OUTPUT);
  pinMode(relay3Pin, OUTPUT);
  pinMode(relay4Pin, OUTPUT);
}
void loop() {
  
//Loop for DHT Sensor
  unsigned long currentdataTime = millis();
  static unsigned long startdataTime = 0;
  
  float h, t, f;

  if (startdataTime == 0){
   
   startdataTime = currentdataTime;
   
  h = dht.readHumidity();
  t = dht.readTemperature();
  f = dht.readTemperature(true);

   if (isnan(h) || isnan(t) || isnan(f)) {
    Serial.println("Failed to read from DHT sensor!");
    return;
  }
  
  float hif = dht.computeHeatIndex(f, h);
  float hic = dht.computeHeatIndex(t, h, false);

  Serial.print("Humidity: ");
  Serial.print(h);
  Serial.print(" %\t");
  Serial.print("Temperature: ");
  Serial.print(t);
  Serial.print(" *C ");
  Serial.print(f);
  Serial.print(" *F\t");
  Serial.print("Heat index: ");
  Serial.print(hic);
  Serial.print(" *C ");
  Serial.print(hif);
  Serial.println(" *F");  
  
  String dataString = "";
   for (int analogPin = 0; analogPin < 3; analogPin++) {
    int sensor = analogRead(analogPin);
    dataString += String(sensor);
    if (analogPin < 2) {
      dataString += ",";
      
  File dataFile = SD.open("datalog.txt", FILE_WRITE);
  if (dataFile) {
    dataFile.println(dataString);
    dataFile.close();
    // print to the serial port too:
    Serial.println(dataString);
    }
  else {
    Serial.println("error opening datalog.txt");
  }
  }
 else if(currentdataTime-startdataTime > 5000) {
    startdataTime = 0;
  }
//Loop for datalogger

 
    }
}    



//Water Level Analog Sensor
  
//Relay loop

  unsigned long currentcycleTime = millis();
  static unsigned long startcycleTime = 0;

if (startcycleTime == 0) {
  currentcycleTime = startcycleTime;

  relay1State = digitalRead(relay1Pin);
  relay2State = digitalRead(relay2Pin);
  
  if (t<=28) {
    digitalWrite(relay1State, HIGH);
    digitalWrite(relay2State, HIGH);
  }
  else if (currentcycleTime - startcycleTime > 300000) {
    startcycleTime = 0;
  }
}

//LCD Display

  lcd.setBacklight(1);
  
  lcd.home();
  lcd.print("Temperature");
  lcd.setCursor(0,1);
  lcd.print(t);
  lcd.print("*C");
  
  lcd.setCursor(0,2);
  lcd.print("Humidity");
  lcd.setCursor(0,3);
  lcd.print(h);
  lcd.print("*%");

}

I have change names for some timers but I am not comprehending why it is going erratic like that