No me salen algunas cosas con C#

[Meta]

Hay dos objetivos que no me salen.

1) Cuando tengo algunos pictureBox activado, en esta caso solo uno. Quiero lograr de alguna manera que al pulsar "Color azul" las veces que quiera, no pierda los estados de los pictureBox que he marcado con el ratón pulsándolos. Si no entiendes lo que quiero decir sobre algo. Pregunta.

2) El botón "Invertir estados" lo que hace que los picturesBox activados y desactivados se inviertan, es decir, los que están activados se desactivan y los que están desactivados se activan.

El código fuente de lo que he hecho hasta ahora es esto:

Código (csharp) [Seleccionar] Expandir

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;

namespace Picture_Colores
{
   public partial class Form1 : Form
   {
       public Form1()
       {
           InitializeComponent();
       }

       bool Color_5x7 = true;
       bool Color_p1_5x7 = false;
       bool Color_p2_5x7 = false;
       bool Color_p3_5x7 = false;
       bool Color_p4_5x7 = false;

       private void checkBox_Color_5x7_CheckedChanged(object sender, EventArgs e)
       {
           if (Color_5x7 == false)
           {
               groupBox1.BackColor = Color.LawnGreen;
               groupBox1.ForeColor = Color.Black;
               Color_5x7 = true;
           }

           else
           {
               groupBox1.BackColor = Color.RoyalBlue;
               groupBox1.ForeColor = Color.White;
               Color_5x7 = false;
           }
       }

       private void pictureBox1_Click(object sender, EventArgs e)
       {
           if (checkBox_Color_5x7.Checked == false)
           {
               if (Color_p1_5x7 == true)
               {
                   pictureBox1.BackColor = Color.Lime;
                   Color_p1_5x7 = false;
               }

               else
               {
                   pictureBox1.BackColor = Color.Black;
                   Color_p1_5x7 = true;
               }
           }
           else
           {
               if (Color_p1_5x7 == true)
               {
                   pictureBox1.BackColor = Color.Blue;
                   Color_p1_5x7 = false;
               }

               else
               {
                   pictureBox1.BackColor = Color.White;
                   Color_p1_5x7 = true;
               }
           }
       }

       private void pictureBox2_Click(object sender, EventArgs e)
       {
           if (checkBox_Color_5x7.Checked == false)
           {
               if (Color_p2_5x7 == true)
               {
                   pictureBox2.BackColor = Color.Lime;
                   Color_p2_5x7 = false;
               }

               else
               {
                   pictureBox2.BackColor = Color.Black;
                   Color_p2_5x7 = true;
               }
           }
           else
           {
               if (Color_p2_5x7 == true)
               {
                   pictureBox2.BackColor = Color.Blue;
                   Color_p2_5x7 = false;
               }

               else
               {
                   pictureBox2.BackColor = Color.White;
                   Color_p2_5x7 = true;
               }
           }
       }

       private void pictureBox3_Click(object sender, EventArgs e)
       {
           if (checkBox_Color_5x7.Checked == false)
           {
               if (Color_p3_5x7 == true)
               {
                   pictureBox3.BackColor = Color.Lime;
                   Color_p3_5x7 = false;
               }

               else
               {
                   pictureBox3.BackColor = Color.Black;
                   Color_p3_5x7 = true;
               }
           }
           else
           {
               if (Color_p3_5x7 == true)
               {
                   pictureBox3.BackColor = Color.Blue;
                   Color_p3_5x7 = false;
               }

               else
               {
                   pictureBox3.BackColor = Color.White;
                   Color_p3_5x7 = true;
               }
           }
       }

       private void pictureBox4_Click(object sender, EventArgs e)
       {
           if (checkBox_Color_5x7.Checked == false)
           {
               if (Color_p4_5x7 == true)
               {
                   pictureBox4.BackColor = Color.Lime;
                   Color_p4_5x7 = false;
               }

               else
               {
                   pictureBox4.BackColor = Color.Black;
                   Color_p4_5x7 = true;
               }
           }
           else
           {
               if (Color_p4_5x7 == true)
               {
                   pictureBox4.BackColor = Color.Blue;
                   Color_p4_5x7 = false;
               }

               else
               {
                   pictureBox4.BackColor = Color.White;
                   Color_p4_5x7 = true;
               }
           }
       }
   }
}

Un cordial saludo.

[El Benjo]

Para cambiar el color y que los estados permanezcan es necesario que evalúes el estado al presionar el checkbox y realices el cambio del color:

Código (csharp) [Seleccionar] Expandir

if (Color_5x7 == false)
            {
                groupBox1.BackColor = Color.LawnGreen;
                groupBox1.ForeColor = Color.Black;
                if(bool Color_p1_5x7 == false)
                   pictureBox1.BackColor = // Aquí el color para el estado desactivado.
                else
                   pictureBox1.BackColor = // Aquí el color para el estado activado.
// Repetir el 'if' anterior para cada picturebox
                Color_5x7 = true;
            }

            else
            {
                groupBox1.BackColor = Color.RoyalBlue;
                groupBox1.ForeColor = Color.White;
                if(bool Color_p1_5x7 == false)
                   pictureBox1.BackColor = // Aquí el color para el estado desactivado.
                else
                   pictureBox1.BackColor = // Aquí el color para el estado activado.
// Repetir el 'if' anterior para cada picturebox
                Color_5x7 = false;
            }

Y para invertir los estados sería de una manera similar. Compruebas el estado de cada cuadro y cambias su valor, después inviertes el color.

Código (csharp) [Seleccionar] Expandir

if(bool Color_p1_5x7 == false)
{
   Color_p1_5x7 = true;
   pictureBox1.BackColor = // Color para el estado activo.
}else{
   Color_p1_5x7 = false;
   pictureBox1.BackColor = // Color para el estado inactivo.
}

Y repites los if para cada picturebox.

Como sugerencia, para evitarte tanto código deberías cambiar las condiciones y trabajar con variables del tipo 'Color' y realizar la asignación mediante éstas. Saludos.

[Meta]

Hola:

Estándo en la primera parte he intentado hacer el código usando las {} para tenerlo como más ordenado. A lo mejor me he equivocado, no me sale elefecto esperado.

Solo he cambiado el código del pictureBox 1, el primero.

Código (csharp) [Seleccionar] Expandir

        private void pictureBox1_Click(object sender, EventArgs e)
        {
         if (Color_5x7 == false)
         {
             groupBox1.BackColor = Color.LawnGreen;
             groupBox1.ForeColor = Color.Black;

             if (Color_p1_5x7 == false)
             {
                 pictureBox1.BackColor = Color.Lime;
             }

             else
             {
                 pictureBox1.BackColor = Color.Black;
             }
             Color_5x7 = true;
         }

         else
         {
             groupBox1.BackColor = Color.RoyalBlue;
             groupBox1.ForeColor = Color.White;

             if (Color_p1_5x7 == false)
             {
                 pictureBox1.BackColor = Color.Blue;
             }

             else
             {
                 pictureBox1.BackColor = Color.White;
             }

             Color_5x7 = false;
         }

        }



Vuelvo a decirlo de otra manera, cuando el estado del checkBox, en este caso llamado "Color_azul" está desactivado, el fondo es verde. El cuadrado desactivado es verde de otro tono y el negro es activado.

Cuando se activa el checbox "Color_azul" sin que se pierda los valores marcados, la base es azul, la base me refiero el grgoupBox, el picture box azul más fuerte cuando está desactivado y se pone blanco cuando es activado.

Saludos.

[El Benjo]

Ah, es que el primer código que yo puse no va en los picturebox sino en el código del checkbox. Y el segundo código en el botón que invierte los estados.

[Meta]

Entendido. Ahora funciona un poco mejor, aún así tiene algo que no concuerda, algún fallo de despiste.

Código (csharp) [Seleccionar] Expandir

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;

// http://social.msdn.microsoft.com/Forums/es-ES/6b834135-b0d1-4ec8-aa50-ba3316bfd5b5/conseguir-unos-objetivos-que-no-me-salen-con-c-2013?forum=vcses
// http://foro.elhacker.net/net/no_me_salen_algunas_cosas_con_c-t420178.0.html
// http://www.forosdelweb.com/f29/no-salen-algunas-cosas-con-c-1107200/

namespace Picture_Colores
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        bool Color_5x7 = true;
        bool Color_p1_5x7 = false;
        bool Color_p2_5x7 = false;
        bool Color_p3_5x7 = false;
        bool Color_p4_5x7 = false;

        private void checkBox_Color_5x7_CheckedChanged(object sender, EventArgs e)
        {
            if (Color_5x7 == false)
            {
                groupBox1.BackColor = Color.LawnGreen;
                groupBox1.ForeColor = Color.Black;

                if (Color_p1_5x7 == false)
                {
                    pictureBox1.BackColor = Color.Lime;
                }

                else
                {
                    pictureBox1.BackColor = Color.Black;
                }

                Color_5x7 = true;
            }

            else
            {
                groupBox1.BackColor = Color.RoyalBlue;
                groupBox1.ForeColor = Color.White;

                if (Color_p1_5x7 == false)
                {
                    pictureBox1.BackColor = Color.Blue;
                }

                else
                {
                    pictureBox1.BackColor = Color.White;
                }

                Color_5x7 = false;
            }
        }

        private void pictureBox1_Click(object sender, EventArgs e)
        {
            if (checkBox_Color_5x7.Checked == false)
            {
                if (Color_p1_5x7 == true)
                {
                    pictureBox1.BackColor = Color.Lime;
                    Color_p1_5x7 = false;
                }

                else
                {
                    pictureBox1.BackColor = Color.Black;
                    Color_p1_5x7 = true;
                }
            }
            else
            {
                if (Color_p1_5x7 == true)
                {
                    pictureBox1.BackColor = Color.Blue;
                    Color_p1_5x7 = false;
                }

                else
                {
                    pictureBox1.BackColor = Color.White;
                    Color_p1_5x7 = true;
                }
            }
        }

        private void pictureBox2_Click(object sender, EventArgs e)
        {
            if (checkBox_Color_5x7.Checked == false)
            {
                if (Color_p2_5x7 == true)
                {
                    pictureBox2.BackColor = Color.Lime;
                    Color_p2_5x7 = false;
                }

                else
                {
                    pictureBox2.BackColor = Color.Black;
                    Color_p2_5x7 = true;
                }
            }
            else
            {
                if (Color_p2_5x7 == true)
                {
                    pictureBox2.BackColor = Color.Blue;
                    Color_p2_5x7 = false;
                }

                else
                {
                    pictureBox2.BackColor = Color.White;
                    Color_p2_5x7 = true;
                }
            }
        }

        private void pictureBox3_Click(object sender, EventArgs e)
        {
            if (checkBox_Color_5x7.Checked == false)
            {
                if (Color_p3_5x7 == true)
                {
                    pictureBox3.BackColor = Color.Lime;
                    Color_p3_5x7 = false;
                }

                else
                {
                    pictureBox3.BackColor = Color.Black;
                    Color_p3_5x7 = true;
                }
            }
            else
            {
                if (Color_p3_5x7 == true)
                {
                    pictureBox3.BackColor = Color.Blue;
                    Color_p3_5x7 = false;
                }

                else
                {
                    pictureBox3.BackColor = Color.White;
                    Color_p3_5x7 = true;
                }
            }
        }

        private void pictureBox4_Click(object sender, EventArgs e)
        {
            if (checkBox_Color_5x7.Checked == false)
            {
                if (Color_p4_5x7 == true)
                {
                    pictureBox4.BackColor = Color.Lime;
                    Color_p4_5x7 = false;
                }

                else
                {
                    pictureBox4.BackColor = Color.Black;
                    Color_p4_5x7 = true;
                }
            }
            else
            {
                if (Color_p4_5x7 == true)
                {
                    pictureBox4.BackColor = Color.Blue;
                    Color_p4_5x7 = false;
                }

                else
                {
                    pictureBox4.BackColor = Color.White;
                    Color_p4_5x7 = true;
                }
            }
        }
    }
}


Me funciona en el pictureBox1, el pictureBox2 no hace el cambio en el checbox "Color azul".

Saludo.

[El Benjo]

Eso es porque no has escrito el código par los demás picturebox. Si te fijas en la primer respuesta que te dejé (en los comentarios escribí dentro del código) había instrucciones del código que debías escribir ahí. Sólo tienes que repetir el 'if' que hace el cambio en el picturebox1, pero ahora para cada uno de los demás picturebox.

[Meta]

Buenas:

Funciona mejor pero hay peros. Tiene comportamientos no deseados.

Si cambios todos a 1 desde el chekbox, lo pulso varias veces repetidas, luego pulso uno a  uno a los pictureBox, se comporta de modo raro.

Código (csharp) [Seleccionar] Expandir

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;

namespace Picture_Colores
{
   public partial class Form1 : Form
   {
       public Form1()
       {
           InitializeComponent();
       }

       bool Color_5x7 = true;
       bool Color_p1_5x7 = false;
       bool Color_p2_5x7 = false;
       bool Color_p3_5x7 = false;
       bool Color_p4_5x7 = false;

       private void checkBox_Color_5x7_CheckedChanged(object sender, EventArgs e)
       {
           if (Color_5x7 == false)
           {
               groupBox1.BackColor = Color.LawnGreen;
               groupBox1.ForeColor = Color.Black;

               if (Color_p1_5x7 == false)
               {
                   pictureBox1.BackColor = Color.Lime;
                   pictureBox2.BackColor = Color.Lime;
                   pictureBox3.BackColor = Color.Lime;
                   pictureBox4.BackColor = Color.Lime;
               }

               else
               {
                   pictureBox1.BackColor = Color.Black;
                   pictureBox2.BackColor = Color.Black;
                   pictureBox3.BackColor = Color.Black;
                   pictureBox4.BackColor = Color.Black;
               }

               Color_5x7 = true;
           }

           else
           {
               groupBox1.BackColor = Color.RoyalBlue;
               groupBox1.ForeColor = Color.White;

               if (Color_p1_5x7 == false)
               {
                   pictureBox1.BackColor = Color.Blue;
                   pictureBox2.BackColor = Color.Blue;
                   pictureBox3.BackColor = Color.Blue;
                   pictureBox4.BackColor = Color.Blue;
               }

               else
               {
                   pictureBox1.BackColor = Color.White;
                   pictureBox2.BackColor = Color.White;
                   pictureBox3.BackColor = Color.White;
                   pictureBox4.BackColor = Color.White;
               }

               Color_5x7 = false;
           }
       }

       private void pictureBox1_Click(object sender, EventArgs e)
       {
           if (checkBox_Color_5x7.Checked == false)
           {
               if (Color_p1_5x7 == true)
               {
                   pictureBox1.BackColor = Color.Lime;
                   Color_p1_5x7 = false;
               }

               else
               {
                   pictureBox1.BackColor = Color.Black;
                   Color_p1_5x7 = true;
               }
           }
           else
           {
               if (Color_p1_5x7 == true)
               {
                   pictureBox1.BackColor = Color.Blue;
                   Color_p1_5x7 = false;
               }

               else
               {
                   pictureBox1.BackColor = Color.White;
                   Color_p1_5x7 = true;
               }
           }
       }

       private void pictureBox2_Click(object sender, EventArgs e)
       {
           if (checkBox_Color_5x7.Checked == false)
           {
               if (Color_p2_5x7 == true)
               {
                   pictureBox2.BackColor = Color.Lime;
                   Color_p2_5x7 = false;
               }

               else
               {
                   pictureBox2.BackColor = Color.Black;
                   Color_p2_5x7 = true;
               }
           }
           else
           {
               if (Color_p2_5x7 == true)
               {
                   pictureBox2.BackColor = Color.Blue;
                   Color_p2_5x7 = false;
               }

               else
               {
                   pictureBox2.BackColor = Color.White;
                   Color_p2_5x7 = true;
               }
           }
       }

       private void pictureBox3_Click(object sender, EventArgs e)
       {
           if (checkBox_Color_5x7.Checked == false)
           {
               if (Color_p3_5x7 == true)
               {
                   pictureBox3.BackColor = Color.Lime;
                   Color_p3_5x7 = false;
               }

               else
               {
                   pictureBox3.BackColor = Color.Black;
                   Color_p3_5x7 = true;
               }
           }
           else
           {
               if (Color_p3_5x7 == true)
               {
                   pictureBox3.BackColor = Color.Blue;
                   Color_p3_5x7 = false;
               }

               else
               {
                   pictureBox3.BackColor = Color.White;
                   Color_p3_5x7 = true;
               }
           }
       }

       private void pictureBox4_Click(object sender, EventArgs e)
       {
           if (checkBox_Color_5x7.Checked == false)
           {
               if (Color_p4_5x7 == true)
               {
                   pictureBox4.BackColor = Color.Lime;
                   Color_p4_5x7 = false;
               }

               else
               {
                   pictureBox4.BackColor = Color.Black;
                   Color_p4_5x7 = true;
               }
           }
           else
           {
               if (Color_p4_5x7 == true)
               {
                   pictureBox4.BackColor = Color.Blue;
                   Color_p4_5x7 = false;
               }

               else
               {
                   pictureBox4.BackColor = Color.White;
                   Color_p4_5x7 = true;
               }
           }
       }
   }
}


Saludo.

[El Benjo]

Mmmmmmm, no. Definitivamente no me refería a eso sino ha esto:

Código (csharp) [Seleccionar] Expandir

if (Color_5x7 == false)
           {
               groupBox1.BackColor = Color.LawnGreen;
               groupBox1.ForeColor = Color.Black;
               if(bool Color_p1_5x7 == false)
                  pictureBox1.BackColor = // Aquí el color para el estado desactivado.
               else
                  pictureBox1.BackColor = // Aquí el color para el estado activado.
// Repetir el 'if' anterior para cada picturebox
                // ***********             OJO          ******************
               // *********** Repetir toda la condición if **************
                if(bool Color_p2_5x7 == false)
                  pictureBox2.BackColor = // Aquí el color para el estado desactivado.
               else
                  pictureBox2.BackColor = // Aquí el color para el estado activado.
               Color_5x7 = true;
           }

           else
           {
               groupBox1.BackColor = Color.RoyalBlue;
               groupBox1.ForeColor = Color.White;
               if(bool Color_p1_5x7 == false)
                  pictureBox1.BackColor = // Aquí el color para el estado desactivado.
               else
                  pictureBox1.BackColor = // Aquí el color para el estado activado.
// Repetir el 'if' anterior para cada picturebox
               // ***********             OJO          ******************
               // *********** Repetir toda la condición if **************
               if(bool Color_p2_5x7 == false)
                  pictureBox2.BackColor = // Aquí el color para el estado desactivado.
               else
                  pictureBox2.BackColor = // Aquí el color para el estado activado.
               Color_5x7 = false;
           }

El código es del checkbox.
Ahora sí has lo mismo que hice en el código anterior, donde a cada picturebox le corresponde un if Fíjate como en el ejemplo todavía me faltan dos picturebox por comprobar. Espero que ya con eso no te de problemas. Saludos.

[Meta]

Ahora si funciona. Por fiiiiiiiiiiiinnnnnnnnnnn. 

Código (csharp) [Seleccionar] Expandir

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;

namespace Picture_Colores
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        bool Color_5x7 = true;
        bool Color_p1_5x7 = false;
        bool Color_p2_5x7 = false;
        bool Color_p3_5x7 = false;
        bool Color_p4_5x7 = false;

        private void checkBox_Color_5x7_CheckedChanged(object sender, EventArgs e)
        {
            if (Color_5x7 == false)
            {
                groupBox1.BackColor = Color.LawnGreen;
                groupBox1.ForeColor = Color.Black;

                if (Color_p1_5x7 == false)
                {
                    pictureBox1.BackColor = Color.Lime;
                }

                else
                {
                    pictureBox1.BackColor = Color.Black;
                }

                if (Color_p2_5x7 == false)
                {
                    pictureBox2.BackColor = Color.Lime;
                }

                else
                {
                    pictureBox2.BackColor = Color.Black;
                }

                if (Color_p3_5x7 == false)
                {
                    pictureBox3.BackColor = Color.Lime;
                }

                else
                {
                    pictureBox3.BackColor = Color.Black;
                }

                if (Color_p4_5x7 == false)
                {
                    pictureBox4.BackColor = Color.Lime;
                }

                else
                {
                    pictureBox4.BackColor = Color.Black;
                }

                Color_5x7 = true;
            }

            else
            {
                groupBox1.BackColor = Color.RoyalBlue;
                groupBox1.ForeColor = Color.White;

                if (Color_p1_5x7 == false)
                {
                    pictureBox1.BackColor = Color.Blue;
                }

                else
                {
                    pictureBox1.BackColor = Color.White;
                }

                if (Color_p2_5x7 == false)
                {
                    pictureBox2.BackColor = Color.Blue;
                }

                else
                {
                    pictureBox2.BackColor = Color.White;
                }

                if (Color_p3_5x7 == false)
                {
                    pictureBox3.BackColor = Color.Blue;
                }

                else
                {
                    pictureBox3.BackColor = Color.White;
                }

                if (Color_p4_5x7 == false)
                {
                    pictureBox4.BackColor = Color.Blue;
                }

                else
                {
                    pictureBox4.BackColor = Color.White;
                }
                Color_5x7 = false;
            }
        }

        private void pictureBox1_Click(object sender, EventArgs e)
        {
            if (checkBox_Color_5x7.Checked == false)
            {
                if (Color_p1_5x7 == true)
                {
                    pictureBox1.BackColor = Color.Lime;
                    Color_p1_5x7 = false;
                }

                else
                {
                    pictureBox1.BackColor = Color.Black;
                    Color_p1_5x7 = true;
                }
            }
            else
            {
                if (Color_p1_5x7 == true)
                {
                    pictureBox1.BackColor = Color.Blue;
                    Color_p1_5x7 = false;
                }

                else
                {
                    pictureBox1.BackColor = Color.White;
                    Color_p1_5x7 = true;
                }
            }
        }

        private void pictureBox2_Click(object sender, EventArgs e)
        {
            if (checkBox_Color_5x7.Checked == false)
            {
                if (Color_p2_5x7 == true)
                {
                    pictureBox2.BackColor = Color.Lime;
                    Color_p2_5x7 = false;
                }

                else
                {
                    pictureBox2.BackColor = Color.Black;
                    Color_p2_5x7 = true;
                }
            }
            else
            {
                if (Color_p2_5x7 == true)
                {
                    pictureBox2.BackColor = Color.Blue;
                    Color_p2_5x7 = false;
                }

                else
                {
                    pictureBox2.BackColor = Color.White;
                    Color_p2_5x7 = true;
                }
            }
        }

        private void pictureBox3_Click(object sender, EventArgs e)
        {
            if (checkBox_Color_5x7.Checked == false)
            {
                if (Color_p3_5x7 == true)
                {
                    pictureBox3.BackColor = Color.Lime;
                    Color_p3_5x7 = false;
                }

                else
                {
                    pictureBox3.BackColor = Color.Black;
                    Color_p3_5x7 = true;
                }
            }
            else
            {
                if (Color_p3_5x7 == true)
                {
                    pictureBox3.BackColor = Color.Blue;
                    Color_p3_5x7 = false;
                }

                else
                {
                    pictureBox3.BackColor = Color.White;
                    Color_p3_5x7 = true;
                }
            }
        }

        private void pictureBox4_Click(object sender, EventArgs e)
        {
            if (checkBox_Color_5x7.Checked == false)
            {
                if (Color_p4_5x7 == true)
                {
                    pictureBox4.BackColor = Color.Lime;
                    Color_p4_5x7 = false;
                }

                else
                {
                    pictureBox4.BackColor = Color.Black;
                    Color_p4_5x7 = true;
                }
            }
            else
            {
                if (Color_p4_5x7 == true)
                {
                    pictureBox4.BackColor = Color.Blue;
                    Color_p4_5x7 = false;
                }

                else
                {
                    pictureBox4.BackColor = Color.White;
                    Color_p4_5x7 = true;
                }
            }
        }
    }
}


Ahora queda pulir el botón "Invertir".

Antes de seguir, horas y horas, con dolores en los ojos por mirar tanto el monitor y falta de sueño. Prefiero dejar el código justo abajo del botón Invertir.

Código (csharp) [Seleccionar] Expandir


        private void button_Invertir_Click(object sender, EventArgs e)
        {
            if (Color_5x7 == false)
            {
                if (Color_p1_5x7 == false)
                {
                    Color_p1_5x7 = true;
                    pictureBox1.BackColor = Color.Lime;
                }

                else
                {
                    Color_p1_5x7 = false;
                    pictureBox1.BackColor = Color.Black;
                }

                if (Color_p2_5x7 == false)
                {
                    Color_p2_5x7 = true;
                    pictureBox2.BackColor = Color.Lime;
                }

                else
                {
                    Color_p2_5x7 = false;
                    pictureBox2.BackColor = Color.Black;
                }

                if (Color_p3_5x7 == false)
                {
                    Color_p3_5x7 = true;
                    pictureBox3.BackColor = Color.Lime;
                }

                else
                {
                    Color_p3_5x7 = false;
                    pictureBox3.BackColor = Color.Black;
                }

                if (Color_p4_5x7 == false)
                {
                    Color_p4_5x7 = true;
                    pictureBox4.BackColor = Color.Lime;
                }

                else
                {
                    Color_p4_5x7 = false;
                    pictureBox4.BackColor = Color.Black;
                }

                Color_5x7 = true;
            }

            else
            {
                if (Color_p1_5x7 == false)
                {
                    Color_p1_5x7 = true;
                    pictureBox1.BackColor = Color.Blue;
                }

                else
                {
                    Color_p1_5x7 = false;
                    pictureBox1.BackColor = Color.White;
                }

                if (Color_p2_5x7 == false)
                {
                    Color_p2_5x7 = true;
                    pictureBox2.BackColor = Color.Blue;
                }

                else
                {
                    Color_p2_5x7 = false;
                    pictureBox2.BackColor = Color.White;
                }

                if (Color_p3_5x7 == false)
                {
                    Color_p3_5x7 = true;
                    pictureBox3.BackColor = Color.Blue;
                }

                else
                {
                    Color_p3_5x7 = false;
                    pictureBox3.BackColor = Color.White;
                }

                if (Color_p4_5x7 == false)
                {
                    Color_p4_5x7 = true;
                    pictureBox4.BackColor = Color.Blue;
                }

                else
                {
                    Color_p4_5x7 = false;
                    pictureBox4.BackColor = Color.White;
                }

                Color_5x7 = false;
            }
        }


En cuanto al paso 1) ya está completado y me falta el paso 2). El botón Invertir no se comporta como debería, hace cosas raras.

Muchísimas gracias por la ayuda.

[El Benjo]

Por nada, para eso estamos aquí. Con exactitud ¿qué es lo que hace el botón y que no debería hacer? Espero que además de explicar puedas poner imágenes.