Хотел бы, чтобы вы помогли исправить некоторые ошибки и улучшить код

Question

Я написал код, предназначенный для автоматического включения и выключения генератора в зависимости от того, заряжен ли аккумулятор или нет.

В lcd.begin () и lcd.clear () (есть ошибки)оба не работают). ошибка: неверное использование нестатической функции-члена. Спасибо!

#include <LiquidCrystal_I2C.h>
#include <Wire.h>
// Set the LCD address to 0x27 for a 16 chars and 2 line display
LiquidCrystal_I2C lcd(0x27, 16, 2);
bool Settings = false;
unsigned long time1;
#define RELAY_PORT 10
float Voltage = 0.00;
int VoltOn = 47;
int VoltOff = 56;
int upbutton = 1;
int downbutton = 2;
int okbutton = 3;
int backbutton = 4;
void setup() {
  pinMode(upbutton, INPUT);
  pinMode(downbutton, INPUT);
  pinMode(okbutton, INPUT);
  pinMode(backbutton, INPUT);
  pinMode(RELAY_PORT, OUTPUT);
  // initialize the LCD
  lcd.begin();

  // Turn on the blacklight and print a message.
  lcd.backlight();
  lcd.setCursor(0, 1);
  lcd.print ("generator: off");
}

void loop() {
  int analog_value = analogRead(A0);
  Voltage = ((analog_value * 5.0) / 1020) * 12;
  lcd.setCursor(0, 0);
  lcd.print("Volt: ");
  lcd.print(Voltage);
  lcd.print("V");

  if (Voltage <= VoltOn && digitalRead(RELAY_PORT) == LOW)
  {
    digitalWrite(RELAY_PORT, HIGH);
    lcd.setCursor(0, 1);
    lcd.print ("generator: on");
  }

  if (Voltage >= VoltOff && digitalRead(RELAY_PORT) == HIGH)
  {
    digitalWrite(RELAY_PORT, LOW);
    lcd.setCursor(0, 1);
    lcd.print ("generator: off");
  }





  if ((digitalRead(upbutton) == HIGH  && digitalRead(downbutton) == HIGH && digitalRead(okbutton) == HIGH && digitalRead(backbutton) == HIGH) || (Settings = true))
  {
    lcd.clear;
    lcd.print("Settings:");
    delay(2000);
    time1 = millis();
    bool Setting = false;
    while (digitalRead(upbutton) == LOW  && digitalRead(downbutton) == LOW && digitalRead(okbutton) == LOW && digitalRead(backbutton) == LOW)
    {
      if (Setting == false)
      {
        lcd.clear;
        lcd.scrollDisplayLeft();
        lcd.print("press Up to set turn on");
        lcd.setCursor(0, 1);
        lcd.print("press down to set turn off");
        Setting = true;
      }
      if (millis() > time1 + 60000)
      {
        loop();
      }
    }
    time1 = millis();
    while (millis() > time1 + 60000)
    {
      if (upbutton == HIGH)
      {
        time1 = millis();
        //lcd.clear;
        lcd.scrollDisplayLeft();
        lcd.print("press Up/ Down to up/ Down Voltage turn on");
        while (digitalRead(upbutton) == LOW  && digitalRead(downbutton) == LOW && digitalRead(okbutton) == LOW && digitalRead(backbutton) == LOW)
        {
          if (millis() > time1 + 60000)
          {
            loop();
          }
        }
        time1 = millis();
        while (millis() > time1 - 60000)
        {
          if (upbutton == HIGH)
          {
            int xdelay = 1000;
            time1 = millis();
            (VoltOn) = (VoltOn) + 1;
            lcd.setCursor(0, 1);
            lcd.print("Voltage-on: ");
            lcd.print(VoltOn);
            lcd.print("V");
            delay(xdelay);
            xdelay = xdelay / 1.5;
          }
          if (downbutton == HIGH)
          {
            int xdelay = 1000;
            time1 = millis();
            (VoltOn) = (VoltOn) - 1;
            lcd.setCursor(0, 1);
            lcd.print("Voltage-on: ");
            lcd.print(VoltOn);
            lcd.print("V");
            delay(xdelay);
            xdelay = xdelay / 1.5;
          }
          if (okbutton == HIGH)
          {
            loop();
          }
          if (backbutton == HIGH)
          { Settings = true;
          }
          if (millis() > time1 + 60000)
          {
            loop();
          }
        }

      }

      if (downbutton == HIGH)
      {
        time1 = millis();
        lcd.clear;
        lcd.scrollDisplayLeft();
        lcd.print("press Up/ Down to up/ Down Voltage turn off");

        while (digitalRead(upbutton) == LOW  && digitalRead(downbutton) == LOW && digitalRead(okbutton) == LOW && digitalRead(backbutton) == LOW)
        {
          if (millis() > time1 + 60000)
          {
            loop();
          }

        }
        time1 = millis();
        while (millis() > time1 - 60000)
        {
          if (upbutton == HIGH)
          {
            int xdelay = 1000;
            time1 = millis();
            (VoltOff) = (VoltOff) + 1;
            lcd.setCursor(0, 1);
            lcd.print("Voltage-off: ");
            lcd.print(VoltOff);
            lcd.print("V");
            delay(xdelay);
            xdelay = xdelay / 1.5;
          }
          if (downbutton == HIGH)
          {
            int xdelay = 1000;
            time1 = millis();
            (VoltOff) = (VoltOff) - 1;
            lcd.setCursor(0, 1);
            lcd.print("Voltage-off: ");
            lcd.print(VoltOff);
            lcd.print("V");
            delay(xdelay);
            xdelay = xdelay / 1.5;
          }
          if (okbutton == HIGH)
          {
            loop();;
          }
          if (backbutton == HIGH)
          {
            Settings = true;
          }
          if (millis() > time1 + 60000)
          {
            loop();
          }
        }

      }

      if (backbutton == HIGH)
      {
        loop();
      }

      if (okbutton == HIGH)
      {
        loop();
      }

    }

  }
}

Answer 1

Я думаю, вам не следует спрашивать в stackoverflow, чтобы кто-то просто выполнял вашу работу. Людям здесь не нравятся неопределенные вопросы, которые практически не показывают собственных исследований ..

Тем не менее, одна вещь, которая выделяется в том, что все ваши lcd.clear не содержат скобок, они должны быть lcd.clear(); Попробуйте разбить код на более мелкие разделы и протестировать отдельные компоненты, если что-то не работает.

Answer 2

1.Как заставить этот код (в начале цикла) запускаться только каждые полсекунды, не заклепывая остальную часть кода?

lcd.setCursor(0, 0);
  lcd.print("Volt: ");
  lcd.print(Voltage);
  lcd.print("V");

2. ЖК-дисплей отображает (что текущийПоступление в Arduino от аккумулятора ниже его пусковой мощности) "генератор: включен", а не "генератор: выключен":

lcd.print ("generator: off");
if (Voltage <= VoltOn && digitalRead(RELAY_PORT) == LOW)
  {
    digitalWrite(RELAY_PORT, HIGH);
    lcd.setCursor(0, 1);
    lcd.print ("generator: on");
  }

  if (Voltage >= VoltOff && digitalRead(RELAY_PORT) == HIGH)
  {
    digitalWrite(RELAY_PORT, LOW);
    lcd.setCursor(0, 1);
    lcd.print ("generator: off");