[SOLUCIONADO] [C#] Cortar por lo sano

REHome · #1 (**permalink**) 03/12/2015, 17:25

Hola:

Tengo este código hecho en C#.

Código:

                switch (comando)
                {
                    case "1=ON":
                        pictureBox1.Image = Properties.Resources.Led_rojo_encendido;
                        Recibidos = "";
                        break;

                    case "1=OFF":
                        pictureBox1.Image = Properties.Resources.Led_rojo_apagado;
                        Recibidos = "";
                        break;

                    case "2=ON":
                        pictureBox1.Image = Properties.Resources.Led_rojo_encendido;
                        Recibidos = "";
                        break;

                    case "2=OFF":
                        pictureBox1.Image = Properties.Resources.Led_rojo_apagado;
                        Recibidos = "";
                        break;

                    case "3=ON":
                        pictureBox1.Image = Properties.Resources.Led_rojo_encendido;
                        Recibidos = "";
                        break;

                    case "3=OFF":
                        pictureBox1.Image = Properties.Resources.Led_rojo_apagado;
                        Recibidos = "";
                        break;

                    case "4=ON":
                        pictureBox1.Image = Properties.Resources.Led_rojo_encendido;
                        Recibidos = "";
                        break;

                    case "4=OFF":
                        pictureBox1.Image = Properties.Resources.Led_rojo_apagado;
                        Recibidos = "";
                        break;
                }

En una variable recibidos me llega todo junto esta cadena de carácteres.

1=ON 2=ON 3=OFF 4=OFF

Como está todo junto, no consigo ni mirando aquí la solución para que me detecte algún case.

¿Alguna idea?

Un cordial saludo.

agleiva · #2 (**permalink**) 03/12/2015, 19:11

No entiendo para qué semejante switch si solo estás usando 2 posibles imagenes?

Además, como decís que te viene todo junto pero estás modificando solo 1 PictureBox? tu código y tu explicación no tienen mucho sentido, la verdad.

Más detalles?

REHome · #3 (**permalink**) 03/12/2015, 19:58

Hola:

Deja ver si me expreso mejor.

Hice una interfaz.

Donde pone Mensajes desde Arduino: Como vez en la imagen no le hagas caso, ahora pone:

1=ON 2=ON 3=OFF 4=OFF

En la variable Recibidos me sale toda esa cadena y no puedo entrar en el switch porque debo partirlo y no se como hacerlo por mucho que use split, no se usarlo como quiero.

El código completo del programa es este:

Código:

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

using System.IO.Ports; // No olvidar.

namespace Entrada_Arduino_AWF_3_CS
{
    public partial class Form1 : Form
    {
        // Utilizaremos un string como buffer de recepción.
        string Recibidos;

        public Form1()
        {
            InitializeComponent();

            if (!serialPort1.IsOpen)
            {
                try
                {
                    serialPort1.Open();
                }
                catch (System.Exception ex)
                {
                    MessageBox.Show(ex.ToString());
                }

                serialPort1.DataReceived += new SerialDataReceivedEventHandler(Recepcion);
            }
        }
        // Al recibir datos.
        private void Recepcion(object sender, SerialDataReceivedEventArgs e)
        {
            // Acumula los caracteres recibidos a nuestro 'buffer' (string).
            Recibidos += serialPort1.ReadExisting();

            // Invocar o llamar al proceso de tramas.
            Invoke(new EventHandler(Actualizar));
        }

        // Procesar los datos recibidos en el bufer y extraer tramas completas.
        private void Actualizar(object sender, EventArgs e)
        {

            //// Asignar el valor de la trama al richTextBox.
            //richTextBox1.Text = Recibidos + "\n";

            //// Selecciona la posición final para leer los mensajes entrantes.
            //richTextBox1.SelectionStart = richTextBox1.Text.Length;

            //// Mantiene el scroll en la entrada de cada mensaje.
            //richTextBox1.ScrollToCaret();

                switch (Recibidos)
                {
                    case "1=ON":
                        pictureBox1.Image = Properties.Resources.Led_rojo_encendido;
                        Recibidos = "";
                        break;

                    case "1=OFF":
                        pictureBox1.Image = Properties.Resources.Led_rojo_apagado;
                        Recibidos = "";
                        break;

                    case "2=ON":
                        pictureBox1.Image = Properties.Resources.Led_rojo_encendido;
                        Recibidos = "";
                        break;

                    case "2=OFF":
                        pictureBox1.Image = Properties.Resources.Led_rojo_apagado;
                        Recibidos = "";
                        break;

                    case "3=ON":
                        pictureBox1.Image = Properties.Resources.Led_rojo_encendido;
                        Recibidos = "";
                        break;

                    case "3=OFF":
                        pictureBox1.Image = Properties.Resources.Led_rojo_apagado;
                        Recibidos = "";
                        break;

                    case "4=ON":
                        pictureBox1.Image = Properties.Resources.Led_rojo_encendido;
                        Recibidos = "";
                        break;

                    case "4=OFF":
                        pictureBox1.Image = Properties.Resources.Led_rojo_apagado;
                        Recibidos = "";
                        break;
            }
        }

        private void Form1_FormClosing(object sender, FormClosingEventArgs e)
        {
            if (serialPort1.IsOpen) // ¿El puerto está abierto?
            {
                serialPort1.Close(); // Puerto cerrado.
            }
        }
    }
}

Un cordial saludo y disculpen por las molestias.

agleiva · #4 (**permalink**) 03/12/2015, 21:03

Ok, lo primero que haría es crear una lista de los pictureboxes

Código C#:

Ver originalpublic partial class Form1 : Form
    {
        // Utilizaremos un string como buffer de recepción.
        string Recibidos;
 
        private List<PictureBox> leds;
 
        public Form1()
        {
            InitializeComponent();
 
            //Inicializar la lista, esto debe ocurrir DESPUES de InitializeComponent()
            this.leds = new[]{ this.pictureBox1, this.pictureBox2, this.pictureBox3, this.pictureBox4 }.ToList();

despues, solo es cuestion de un poco de LINQ:

Código C#:

Ver originalprivate void Actualizar(object sender, EventArgs e)
        {
 
            //// Asignar el valor de la trama al richTextBox.
            //richTextBox1.Text = Recibidos + "\n";
 
            //// Selecciona la posición final para leer los mensajes entrantes.
            //richTextBox1.SelectionStart = richTextBox1.Text.Length;
 
            //// Mantiene el scroll en la entrada de cada mensaje.
            //richTextBox1.ScrollToCaret();
 
            var si = Properties.Resources.Led_rojo_encendido;
            var no = Properties.Resources.Led_rojo_apagado;
 
            var valores = Recibidos.Split(' ')
                                   .Select(x => x.Split('='))
                                   .Select(x => new
                                                {
                                                   Index = int.Parse(x[0]),
                                                   Value = x[1] == "ON"
                                                });
                          
            foreach (var v in valores)
            {
                this.leds[v.Index].Image = v.Value ? si : no;
            }
       }

No hace falta un switch ni ninguna cosa por el estilo.

REHome · #5 (**permalink**) 04/12/2015, 00:08

Hola:

Tu código es corto, el mio que hice es largo y por fin funciona aunque a veces se le va la oya.

Código:

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

using System.IO.Ports; // No olvidar.

namespace Entrada_Arduino_AWF_3_CS
{
    public partial class Form1 : Form
    {
        // Utilizaremos un string como buffer de recepción.
        string Recibidos;

        public Form1()
        {
            InitializeComponent();

            if (!serialPort1.IsOpen)
            {
                try
                {
                    serialPort1.Open();
                }
                catch (System.Exception ex)
                {
                    MessageBox.Show(ex.ToString());
                }

                serialPort1.DataReceived += new SerialDataReceivedEventHandler(Recepcion);
            }
        }
        // Al recibir datos.
        private void Recepcion(object sender, SerialDataReceivedEventArgs e)
        {
            // Acumula los caracteres recibidos a nuestro 'buffer' (string).
            Recibidos += serialPort1.ReadExisting();

            // Invocar o llamar al proceso de tramas.
            Invoke(new EventHandler(Actualizar));
        }

        // Procesar los datos recibidos en el bufer y extraer tramas completas.
        private void Actualizar(object sender, EventArgs e)
        {

            // Asignar el valor de la trama al richTextBox.
            richTextBox1.Text += Recibidos;

            // Selecciona la posición final para leer los mensajes entrantes.
            richTextBox1.SelectionStart = richTextBox1.Text.Length;

            // Mantiene el scroll en la entrada de cada mensaje.
            richTextBox1.ScrollToCaret();

            char[] Delimitador = { ' ', '\r', '\n' };

            string[] Palabras = Recibidos.Split(Delimitador);


            foreach (string Comandos in Palabras)
            {
                switch (Comandos)
                {
                    case "1=ON":
                        pictureBox1.Image = Properties.Resources.Led_rojo_encendido;
                        Recibidos = "";
                        break;

                    case "1=OFF":
                        pictureBox1.Image = Properties.Resources.Led_rojo_apagado;
                        Recibidos = "";
                        break;

                    case "2=ON":
                        pictureBox2.Image = Properties.Resources.Led_rojo_encendido;
                        Recibidos = "";
                        break;

                    case "2=OFF":
                        pictureBox2.Image = Properties.Resources.Led_rojo_apagado;
                        Recibidos = "";
                        break;

                    case "3=ON":
                        pictureBox3.Image = Properties.Resources.Led_rojo_encendido;
                        Recibidos = "";
                        break;

                    case "3=OFF":
                        pictureBox3.Image = Properties.Resources.Led_rojo_apagado;
                        Recibidos = "";
                        break;

                    case "4=ON":
                        pictureBox4.Image = Properties.Resources.Led_rojo_encendido;
                        richTextBox1.Text += " " + DateTime.Now.ToString();
                        Recibidos = "";
                        break;

                    case "4=OFF":
                        pictureBox4.Image = Properties.Resources.Led_rojo_apagado;
                        richTextBox1.Text += " " + DateTime.Now.ToString();
                        Recibidos = "";
                        break;
            }
          }
        }

        private void Form1_FormClosing(object sender, FormClosingEventArgs e)
        {
            if (serialPort1.IsOpen) // ¿El puerto está abierto?
            {
                serialPort1.Close(); // Puerto cerrado.
            }
        }

        void Actualizar()
        {
            byte[] mBuffer = Encoding.ASCII.GetBytes("ACTUALIZAR"); // Envía comando ACTUALIZAR por el puerto.
            serialPort1.Write(mBuffer, 0, mBuffer.Length);
        }

        private void button_Actualizar_Click(object sender, EventArgs e)
        {
            Actualizar();
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            Actualizar();
        }

        private void button2_Click(object sender, EventArgs e)
        {
            richTextBox1.Clear(); // Limpiar contenido del richTextBox.
            Recibidos = "";
        }
    }
}

Saludos.

Malenko · #6 (**permalink**) 04/12/2015, 08:32

Que tiene que ver la longitud del código con que sea correcto?

Yo también cuando ví los switch me dieron ganas de tirarme de los pelos. Primero por la cantidad de repetición de código, el mal uso de los "cases", la poca mantenibilidad del código y un largo etc.

En cambio la versión de agleiva es mucho mejor. Y esto se ve en detalles como que le da igual si hay 4, 1 o 20 "interruptores" que no va a tener que tocar el método "actualizar". Si buscas por internet verás que el switch podrás ver las diferentes formas de hacer refactorizarlo.

REHome · #7 (**permalink**) 04/12/2015, 09:04

Hola:

Probando el código.

Código C++:

Ver originalvar si = Properties.Resources.Led_rojo_encendido;
            var no = Properties.Resources.Led_rojo_apagado;
 
            var valores = Recibidos.Split(' ')
                                   .Select(x => x.Split('='))
                                   .Select(x => new
                                   {
                                       Index = int.Parse(x[0]),
                                       Value = x[1] == "ON"
                                   });
 
            foreach (var v in valores)
            {
                this.leds[v.Index].Image = v.Value ? si : no;
            }

Me sale este error.

Cita:

Error CS1061 'Form1' no contiene una definición para 'leds' ni se encuentra ningún método de extensión 'leds' que acepte un primer argumento del tipo 'Form1' (¿falta alguna directiva using o una referencia de ensamblado?) Entrada_Arduino_AWF_3_CS C:\Users\Usuario\Documents\Visual Studio 2015\Projects\Entrada_Arduino_AWF_3_CS\Entrada_Ard uino_AWF_3_CS\Form1.cs 125

Código completo.

Código C++:

Ver originalusing System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
 
using System.IO.Ports; // No olvidar.
 
namespace Entrada_Arduino_AWF_3_CS
{
    public partial class Form1 : Form
    {
        // Utilizaremos un string como buffer de recepción.
        string Recibidos;
 
        public Form1()
        {
            InitializeComponent();
 
            if (!serialPort1.IsOpen)
            {
                try
                {
                    serialPort1.Open();
                }
                catch (System.Exception ex)
                {
                    MessageBox.Show(ex.ToString());
                }
 
                serialPort1.DataReceived += new SerialDataReceivedEventHandler(Recepcion);
            }
 
            //Inicializar la lista, esto debe ocurrir DESPUES de InitializeComponent()
            this.leds = new[] { this.pictureBox1, this.pictureBox2, this.pictureBox3, this.pictureBox4 }.ToList();
        }
        // Al recibir datos.
        private void Recepcion(object sender, SerialDataReceivedEventArgs e)
        {
            // Acumula los caracteres recibidos a nuestro 'buffer' (string).
            Recibidos += serialPort1.ReadExisting();
 
            // Invocar o llamar al proceso de tramas.
            Invoke(new EventHandler(Actualizar));
        }
 
        // Procesar los datos recibidos en el bufer y extraer tramas completas.
        private void Actualizar(object sender, EventArgs e)
        {
 
            // Asignar el valor de la trama al richTextBox.
            richTextBox1.Text += Recibidos;
 
            // Selecciona la posición final para leer los mensajes entrantes.
            richTextBox1.SelectionStart = richTextBox1.Text.Length;
 
            // Mantiene el scroll en la entrada de cada mensaje.
            richTextBox1.ScrollToCaret();
 
            //  char[] Delimitador = { ' ', '\r', '\n' };
 
            //  string[] Palabras = Recibidos.Split(Delimitador);
 
 
            //  foreach (string Comandos in Palabras)
            //  {
            //      switch (Comandos)
            //      {
            //          case "1=ON":
            //              pictureBox1.Image = Properties.Resources.Led_rojo_encendido;
            //              Recibidos = "";
            //              break;
 
            //          case "1=OFF":
            //              pictureBox1.Image = Properties.Resources.Led_rojo_apagado;
            //              Recibidos = "";
            //              break;
 
            //          case "2=ON":
            //              pictureBox2.Image = Properties.Resources.Led_rojo_encendido;
            //              Recibidos = "";
            //              break;
 
            //          case "2=OFF":
            //              pictureBox2.Image = Properties.Resources.Led_rojo_apagado;
            //              Recibidos = "";
            //              break;
 
            //          case "3=ON":
            //              pictureBox3.Image = Properties.Resources.Led_rojo_encendido;
            //              Recibidos = "";
            //              break;
 
            //          case "3=OFF":
            //              pictureBox3.Image = Properties.Resources.Led_rojo_apagado;
            //              Recibidos = "";
            //              break;
 
            //          case "4=ON":
            //              pictureBox4.Image = Properties.Resources.Led_rojo_encendido;
            //              Recibidos = "";
            //              break;
 
            //          case "4=OFF":
            //              pictureBox4.Image = Properties.Resources.Led_rojo_apagado;
            //              Recibidos = "";
            //              break;
            //  }
            //}
 
            var si = Properties.Resources.Led_rojo_encendido;
            var no = Properties.Resources.Led_rojo_apagado;
 
            var valores = Recibidos.Split(' ')
                                   .Select(x => x.Split('='))
                                   .Select(x => new
                                   {
                                       Index = int.Parse(x[0]),
                                       Value = x[1] == "ON"
                                   });
 
            foreach (var v in valores)
            {
                this.leds[v.Index].Image = v.Value ? si : no;
            }
 
            //richTextBox1.Text += "        " + DateTime.Now.ToString() + "\r";
        }
 
        private void Form1_FormClosing(object sender, FormClosingEventArgs e)
        {
            if (serialPort1.IsOpen) // ¿El puerto está abierto?
            {
                serialPort1.Close(); // Puerto cerrado.
            }
        }
 
        void Actualizar()
        {
            byte[] mBuffer = Encoding.ASCII.GetBytes("ACTUALIZAR"); // Envía comando ACTUALIZAR por el puerto.
            serialPort1.Write(mBuffer, 0, mBuffer.Length);
        }
 
        private void button_Actualizar_Click(object sender, EventArgs e)
        {
            Actualizar();
        }
 
        private void Form1_Load(object sender, EventArgs e)
        {
            Actualizar();
        }
 
        private void button2_Click(object sender, EventArgs e)
        {
            richTextBox1.Clear(); // Limpiar contenido del richTextBox.
            Recibidos = "";
        }
    }
}

Saludos.

Malenko · #8 (**permalink**) 04/12/2015, 09:06

Quizás tendrías que leer mejor el comentario que hizo Agleiva y así verás de donde sale esa variable...

Aprende a que cuando a comprender el código que copies o te dará muchos problemas ;)

REHome · #9 (**permalink**) 04/12/2015, 09:18

En esta ocasión la salidad la he cambiado por esta otra.

1=ON 2=ON 3=OFF 4=OFF\r\n

En Recibidos hay que quitar el \r y el \n.

Saludos.