Mejorar el código.

Hola:

Hice un código por parte que vino en un libro de C para Arduino. Este código dice que se puede mejorar que es este. Para ser de un libro, la gente no le gusta un pelo.

Lo que hace el programa es recibir comandos por puerto serie. También recibe datos.

Código

#include <LiquidCrystal.h>
 
// Inicializa la librería con sus pines indicados.
// RS, RW, Enable, D4, D5, D6, D7.
LiquidCrystal lcd(8, NULL, 9, 4, 5, 6, 7);
// LiquidCrystal lcd(8, 9, 4, 5, 6, 7);
 
// Pin 10 para saber que es luz de fondo.
const byte LuzFondo = 10;
 
const byte Led = 13;              // Declaramos la variable pin del Led.
char caracter;
String comando;
 
void setup()
{
  pinMode(Led, OUTPUT);           // Inicializa el pin del LED como salida:
  Serial.begin(115200);           // Puerto serie 115200 baudios.
  lcd.begin(16, 2);               // Formato de pantalla.
  lcd.clear();                    // Borra la pantalla y su posición superior izquierda.
  lcd.print("    Arduino     ");
  delay(1000);
}
 
void loop()
{
  /*
    Voy leyendo carácter a carácter lo que se recibe por el canal serie
    (mientras llegue algún dato allí), y los voy concatenando uno tras otro
    en una cadena. En la práctica, si usamos el "Serial monitor" el bucle while
    acabará cuando pulsamos Enter. El delay es conveniente para no saturar el
    canal serie y que la concatenación se haga de forma ordenada.
  */
  while (Serial.available() > 0)
  {
    caracter = Serial.read();
    comando.concat(caracter);
    delay(10);
  }
 
  /*
    Una vez ya tengo la cadena "acabada", compruebo su valor y hago que
    la placa Arduino reacciones según sea este. Aquí podríamos hacer lo
    que quisiéramos: si el comando es "tal", enciende un Led, si es cual,
    mueve un motor... y así.
  */
 
  // Si los carácteres es recibido y verdadero.
  if (comando.equals("Luz_ON") == true)
  {
    digitalWrite(Led, HIGH); // Enciende el Led 13.
    Serial.write("ON - Led encendido.");    // Envía este mensaje a C++.
    lcd.setCursor(0, 1);
    lcd.print("Luz ON.         "); // Mostrar en el LCD.
  }
 
 
  if (comando.equals("Luz_OFF") == true)
  {
    digitalWrite(Led, LOW); // Apaga el Led 13.
    Serial.write("OFF - Led apagado. ");  // Envía este mensaje a C++.
    lcd.setCursor(0, 1);
    lcd.print("Luz OFF.        "); // Mostrar en el LCD.
  }
 
    if (comando.equals("Salir") == true)
  {
    digitalWrite(Led, LOW); // Apaga el Led 13.
    Serial.write("Ha salido de C#.");  // Envía este mensaje a C++.
    lcd.setCursor(0, 1);
    lcd.print("Ha salido de C#."); // Mostrar en el LCD.
  }
 
  // Limpiamos la cadena para volver a recibir el siguiente comando.
  comando = "";
}

He hecho aquí abajo algunos consejos que me dieron por ahí y por allá.

Código

#include <LiquidCrystal.h>
 
// Inicializa la librería con sus pines indicados.
// RS, RW, Enable, D4, D5, D6, D7.
LiquidCrystal lcd(8, NULL, 9, 4, 5, 6, 7);
// LiquidCrystal lcd(8, 9, 4, 5, 6, 7);
 
// Pin 10 para saber que es luz de fondo.
const byte LuzFondo = 10;
 
const byte Led = 13;  // Declaramos la variable pin del Led.
char caracter;
String comando;
 
void setup() 
{
  pinMode(Led, OUTPUT);  // Inicializa el pin del LED como salida:
  Serial.begin(115200);  // Puerto serie 115200 baudios.
  lcd.begin(16, 2);      // Formato de pantalla.
  lcd.clear();           // Borra la pantalla y su posición superior izquierda.
  lcd.print("    Arduino     ");
  delay(1000);
  Serial.setTimeout(5);  //Tiempo maximo de espera en ms para el dato en el puerto.
}
 
void loop() 
{
  /*
    Voy leyendo carácter a carácter lo que se recibe por el canal serie
    (mientras llegue algún dato allí), y los voy concatenando uno tras otro
    en una cadena. En la práctica, si usamos el "Serial monitor" el bucle while
    acabará cuando pulsamos Enter. El delay es conveniente para no saturar el
    canal serie y que la concatenación se haga de forma ordenada.
  */
  while (Serial.available() > 0) 
  {
    comando = Serial.readString();  // Lees el stream directamente, la lectura termina segun el timeOut establecido.
    comando.trim();                 // Limpia tabulaciones, espacios en blanco, \r, \n, etc.
  }
 
  /*
    Una vez ya tengo la cadena "acabada", compruebo su valor y hago que
    la placa Arduino reacciones según sea este. Aquí podríamos hacer lo
    que quisiéramos: si el comando es "tal", enciende un Led, si es cual,
    mueve un motor... y así.
  */
 
  // Si los carácteres es recibido y verdadero.
  if (comando.startsWith(F("Luz_ON")) == true) 
  {
    digitalWrite(Led, HIGH);                 // Enciende el Led 13.
    Serial.print(F("ON - Led encendido."));  // Envía este mensaje a C++.
    lcd.setCursor(0, 1);
    lcd.print(F("Luz ON.         "));        // Mostrar en el LCD.
  }
 
 
  if (comando.startsWith(F("Luz_OFF")) == true) 
  {
    digitalWrite(Led, LOW);                  // Apaga el Led 13.
    Serial.print(F("OFF - Led apagado. "));  // Envía este mensaje a C++.
    lcd.setCursor(0, 1);
    lcd.print(F("Luz OFF.        "));        // Mostrar en el LCD.
  }
 
  if (comando.startsWith(F("Salir")) == true) 
  {
    digitalWrite(Led, LOW);                  // Apaga el Led 13.
    Serial.print(F("Ha salido de C++.  "));    // Envía este mensaje a C++.
    lcd.setCursor(0, 1);
    lcd.print(F("Ha salido de C++"));        // Mostrar en el LCD.
  }
 
  // Limpiamos la cadena para volver a recibir el siguiente comando.
  comando = "";
}

Todavía me dice que es muy horrible. A pesar de que funciona bien, pues no es la mejor manera.

Entonces hago esta pregunta.

¿Cuál es la mejor manera?

Ninguno me dice como hacerlo o ponen ejemplos para coger ideas.

El programa de abajo es una interfaz que comunica desde el PC por puerto serie/USB con Arduino bajo Visual Studio Community 2022. Está hecho con C++ nativo.

Puedes enviar comandos pulsando la tecla 1, 2 y 3.

Código

#include <iostream>
#include <fstream>
#include <conio.h>      // Controla el _getch.
#include <Windows.h>    // Para mostrar texto en el título de la ventana.
#include <stdio.h>      // Cambio color de fondo y letras.
#include "SerialClass.h"
 
using namespace std;
//using std::cout;
//using std::cin;
 
// Función posición del cursor.
void gotoxy(int ancho_x, int alto_y)
{
    HANDLE hcon;
    hcon = GetStdHandle(STD_OUTPUT_HANDLE);
    COORD dwPos{};
    dwPos.X = ancho_x;
    dwPos.Y = alto_y;
    SetConsoleCursorPosition(hcon, dwPos);
}
 
int main()
{
 
#pragma region "Configuración ventana."
    // Mostrar caracteres correctamente en pantalla y título de la ventana.
    SetConsoleOutputCP(65001);
    wchar_t titulo[128];
    MultiByteToWideChar(CP_UTF8, 0, "Led Arduino C++ nativo", -1, titulo, 128);
    SetConsoleTitleW(titulo);
 
    // Tamaño de la pantalla. Se cambia en los dos últimos dígitos.
    SMALL_RECT r = { 0, 0, 80, 20 }; // X = 80, Y = 20.
    SetConsoleWindowInfo(GetStdHandle(STD_OUTPUT_HANDLE), TRUE, &r);
 
    // Cambio color de 6 (naranja / amarillo claro), color letras 0 (negro).
    system("color 60");
 
    // Ocultar cursor.
    CONSOLE_CURSOR_INFO cci;
    GetConsoleCursorInfo(GetStdHandle(STD_OUTPUT_HANDLE), &cci);
    cci.bVisible = 0; // 0 oculta. 1 muestra cursor.
    SetConsoleCursorInfo(GetStdHandle(STD_OUTPUT_HANDLE), &cci);
#pragma endregion
 
 
 
    // Comandos para Arduino.
    char Luz_ON[] = "Luz_ON";       // Envía "Luz_ON" al puerto serie.
    char Luz_OFF[] = "Luz_OFF";
    char salir[] = "Salir";
    char limpiar7[] = "       ";    // Siete espacios en blanco para limpiar un área seleccionado.
    char lectura[50] = "\0";        // Guardan datos de entrada del puerto.
    char tecla;                     // Guarda un 1, 2 o un 3 tipo entero que introduces desde la consola.
 
    // Puerto serie.
    Serial* Puerto = new Serial("COM4");
 
    gotoxy(0, 0);                   // Posición de la pantalla.
    cout << endl;                   // Retorno.
    cout << "Introduzca la opción deseada: " << endl;
    cout << "Pulse 1 para encender el Led y 2 para apagar." << endl << endl;
 
    while (Puerto->IsConnected())
    {
        tecla = _getch();           // Aquí introduces un número, el 1, 2 o el 3.
 
        switch (tecla)              // Espera recibir un 1, 2 o el 3.
        {
        case '1':
            // Encener luz.
            gotoxy(10, 5);
            cout << limpiar7;
            gotoxy(0, 5);
            cout << "Enviando: " << Luz_ON << endl;         // Muestra en pantalla textos.
            Puerto->WriteData(Luz_ON, sizeof(Luz_ON) - 1);  // Envía al puerto el texto "Luz_ON".
            gotoxy(0, 9);
            cout << "                               ";      // Borrado del mensaje si haz pulsado distinto del 1 o la tecla 2.
            break;
 
        case '2':
            // Apagar luz.
            gotoxy(10, 5);
            cout << limpiar7;
            gotoxy(0, 5);
            cout << "Enviando: " << Luz_OFF << endl;
            Puerto->WriteData(Luz_OFF, sizeof(Luz_OFF) - 1);
            gotoxy(0, 9);
            cout << "                               ";
            break;
 
        case '3':
            // Salir.
            gotoxy(10, 5);
            cout << limpiar7;
            gotoxy(0, 5);
            cout << "Enviando: " << salir << endl;
            Puerto->WriteData(salir, sizeof(salir) - 1);
            gotoxy(0, 9);
            cout << "                               ";
            break;
 
        default: // Si haz pulsado otro número distinto del 1 y 2, muestra
            gotoxy(0, 9);
            cout << "Pulse las teclas el 1, el 2 y el 3 para salir.";  // este mensaje.
        }
 
        Sleep(500);
        int numeros = Puerto->ReadData(lectura, 49);            // Recibe datos del puerto serie.
        if (numeros > 0)
        {
            lectura[numeros + 1] = '\0';                        // Limpia de basura la variable.
            gotoxy(0, 7);
            cout << "Recibido: " << lectura << endl;            // Muestra en pantalla dato recibido.
            cout << "-----------------------------" << endl;
        }
    }
    return 0;
}

¿Algún consejo?

Saludos camaradas.