No me salen algunas cosas con C#

[Meta]

Buenas:

Ahora funciona tal como lo quiero.

¿Qué es lo que has hecho exactamente?

Muchísimas gracias.

[El Benjo]

Lo que te mencionaba sobre cambiar los colores en la asignación. Es decir cambiar la parte por ejemplo de:

Código (csharp) [Seleccionar] Expandir

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

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

Por:

Código (csharp) [Seleccionar] Expandir

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

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

Y creo que la parte del if (Color_5x7 == true) antes estaba en 'false'.

Eso fue todo, que bueno que ya te funciona. Saludos

[Meta]

Si,a ntes if (Color_5x7 == true) estaba en false. Estaba probando tantas cosas que ni m di cuenta.

[engel lex]

me meto solo para ayudar a reducir el codigo

Código (csharp) [Seleccionar] Expandir

if (Color_p1_5x7)
{
    pictureBox1.BackColor = Color.White;
}
else
{
    pictureBox1.BackColor = Color.Blue;
}
Color_p1_5x7 = !Color_p1_5x7;

y no se si soporta la sintaxis "?" de c/c++ (no se nada de c# )

pero en ese caso

Código (csharp) [Seleccionar] Expandir


pictureBox1.BackColor = Color_p1_5x7 ?  Color.White : Color.Blue;
Color_p1_5x7 = !Color_p1_5x7;

tambien recomendaría esos colores meterlos como un array y en lugar de alargar tanto el código se usa for

sorry si mi conocimiento no sirve aquí

[El Benjo]

Claro que sirve. El conocimiento siempre es bienvenido. De hecho ya le hice las recomendaciones para reducir el código. De hecho le había comentado que se pueden eliminar la mayoría de los 'if' si se utilizan dos variables para los colores y se asigna el valor según el estado del checkbox.

[Meta]

Buenas de nuevo:

Ahora que por ahora está bien, he añadido dos botones más. Rellenar y Limpiar. Lo que hace es rellenar de golpe todos los pictureBox, y limpiar vaciar todos los pictureBox. Su código va en lo último del principal.

Código (csharp) [Seleccionar] Expandir

private void button_Rellenar_Click(object sender, EventArgs e)
        {
            if (Color_5x7 == true)
            {
                Rellenar_negro();
            }

            else
            {
                Rellenar_blanco();
            }
        }

        private void button_Limpiar_Click(object sender, EventArgs e)
        {
            if (Color_5x7 == true)
            {
                Rellenar_verde();
            }

            else
            {
                Rellenar_azul();
            }
        }

        public void Rellenar_blanco()
        {
            pictureBox1.BackColor = Color.White;
            pictureBox2.BackColor = Color.White;
            pictureBox3.BackColor = Color.White;
            pictureBox4.BackColor = Color.White;
        }

        public void Rellenar_azul()
        {
            pictureBox1.BackColor = Color.Blue;
            pictureBox2.BackColor = Color.Blue;
            pictureBox3.BackColor = Color.Blue;
            pictureBox4.BackColor = Color.Blue;
        }

        public void Rellenar_negro()
        {
            pictureBox1.BackColor = Color.Black;
            pictureBox2.BackColor = Color.Black;
            pictureBox3.BackColor = Color.Black;
            pictureBox4.BackColor = Color.Black;
        }

        public void Rellenar_verde()
        {
            pictureBox1.BackColor = Color.Lime;
            pictureBox2.BackColor = Color.Lime;
            pictureBox3.BackColor = Color.Lime;
            pictureBox4.BackColor = Color.Lime;
        }

Los botones de entrada funciona muy bien, exceptuando que al pulsar Invertir hace cosas raras. Hagan pruebas y lo notarán. Quiero corregir esa parte pero no lo consigo.

Un saludo.

[El Benjo]

Eso es porque sólo modificas el color del fondo pero no haces nada con las variables de estado.

Código (csharp) [Seleccionar] Expandir

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

Esas variables también debes modificarlas (ponerlas todas a true o false según corresponda).

[Meta]

Hola:

Lo hice así pero no me cuadra.

Código (csharp) [Seleccionar] Expandir

        private void button_Rellenar_Click(object sender, EventArgs e)
        {
            if (Color_5x7 == true)
            {
                Rellenar_negro();
            }

            else
            {
                Rellenar_blanco();
            }
        }

        private void button_Limpiar_Click(object sender, EventArgs e)
        {
            if (Color_5x7 == true)
            {
                Rellenar_verde();
            }

            else
            {
                Rellenar_azul();
            }
        }

        public void Rellenar_blanco()
        {
            Color_p1_5x7 = false;
            Color_p2_5x7 = false;
            Color_p3_5x7 = false;
            Color_p4_5x7 = false;
            pictureBox1.BackColor = Color.White;
            pictureBox2.BackColor = Color.White;
            pictureBox3.BackColor = Color.White;
            pictureBox4.BackColor = Color.White;
        }

        public void Rellenar_azul()
        {
            Color_p1_5x7 = true;
            Color_p2_5x7 = true;
            Color_p3_5x7 = true;
            Color_p4_5x7 = true;
            pictureBox1.BackColor = Color.Blue;
            pictureBox2.BackColor = Color.Blue;
            pictureBox3.BackColor = Color.Blue;
            pictureBox4.BackColor = Color.Blue;
        }

        public void Rellenar_negro()
        {
            Color_p1_5x7 = false;
            Color_p2_5x7 = false;
            Color_p3_5x7 = false;
            Color_p4_5x7 = false;
            pictureBox1.BackColor = Color.Black;
            pictureBox2.BackColor = Color.Black;
            pictureBox3.BackColor = Color.Black;
            pictureBox4.BackColor = Color.Black;
        }

        public void Rellenar_verde()
        {
            Color_p1_5x7 = true;
            Color_p2_5x7 = true;
            Color_p3_5x7 = true;
            Color_p4_5x7 = true;
            pictureBox1.BackColor = Color.Lime;
            pictureBox2.BackColor = Color.Lime;
            pictureBox3.BackColor = Color.Lime;
            pictureBox4.BackColor = Color.Lime;
        }

Un saludo.

[Meta]

Ya lo hice funcionar, gracias.

Cambié los true por los false y al revés.

[El Benjo]

Sí, exactamente así.