2

Tenho um cartão com números gerados aleatoriamente que quando são iguais ao número gerado no meio, tornam as labels laranja.

No entanto, eu queria fazer algo para quando todas as labels com números ficassem laranjas, aparecia uma MessageBox dizendo algo. Mas não sei verificar se estas estão todas a laranja. Como é que faço isso?

Só tenho isto para quando o texto das labels é igual ao número do meio.

foreach (Label lblCor in panel1.Controls)
    if (lblNum.Text == lblCor.Text)
        lblCor.BackColor = Color.Orange;

Os cartões ão estes.

12
  • Não seria porque você esta tentado colocar a sua lógica dentro de if (lblNum.Text == lblCor.Text), ou seja, só vai verificar quando o número do meio for igual. Commented 2/02/2019 às 12:40
  • Talvez seja isso... tentei também fazer de outra forma, mas não consegui. Tentei criar um int com um tamanho de 15 e sempre que uma label ficava laranja tirava 1 na int. Logo, quando tivesse 0, aparecerima uma MessageBox, mas não resultava Commented 2/02/2019 às 12:46
  • 1
    Você pode criar uma flag fora do for com o nome mostraMensagem = true e dentro do for você coloca o seguinte if (lblCor.BackColor != Color.Orange), se ele entrar neste if você seta a flag para falso, e embaixo do for você coloca outro if verificando a flag, ou seja, se ele entrar no if dentro do for é porque não foram preenchida todas as labels com laranja. Commented 2/02/2019 às 12:56
  • 1
    Flag é só um nome "bonitinho", flag é uma variável só isso Commented 2/02/2019 às 13:06
  • 1
    acho que a resposta de vik ajudou! Commented 2/02/2019 às 14:01

2 Respostas 2

3

Caso o seu painel contenha apenas Labels, pode fazer o seguinte para contar quantas labels tem o BackColor a laranja:

int totalDeLaranjas = panel1.Controls.Cast<Label>().Count(lbl => lbl.BackColor == Color.Orange);
8
  • Acho que com isto vou conseguir o que quero, obrigado! Commented 2/02/2019 às 14:01
  • Vik, só tenho um problema, eu fiz um if(totalDeLaranjas == 15) e funciona. Só que quando uso uma MessageBox lá dentro, a mensagem não desaparece. Ou seja, mesmo depois de clicar em "Ok" na mensagem, ou tentar fechá-la, ela volta a aparecer. Eu percebo qual é o problema, mas não consigo resolvê-lo Commented 2/02/2019 às 14:41
  • Sem ver o codigo fica dificil saber qual o problema, pode postar o codigo do botão "novo numero", ou editar o post..
    – vik
    Commented 2/02/2019 às 15:02
  • Outra coisa que você poderia melhorar era a criação dos cartões, por exemplo, cada coluna ter apenas numeros dessa dezena, e cada coluna ter pelo menos um numero.
    – vik
    Commented 2/02/2019 às 15:15
  • 1
    Sem saber o que está antes e onde se chama esse trecho de codigo fica dificil... o que podes pensar em fazer é criar um metodo com os valores iniciais "SetDefaultValues()", lá inicias / fazes reset de todas variavies e controlos e quando recebes "Yes" na mensagem chamas esse metodo, não sendo necessario estar a fechar e abrir o form de novo...
    – vik
    Commented 2/02/2019 às 16:02
2

Baseando-me na resposta de @vik, outro LINQ possível de usar:

continuaWhile = 
    !panel1.Controls
        .OfType<Label>()
        .All(label => label.BackColor == Color.Orange);

e daí, no while, vai a condição:

bool continuaWhile = true;

while (continuaWhile)
{
    // Faz o que tem que fazer, aqui.

    // Por último, verifica se continua iterando o laço.
    continuaWhile =
        !panel1.Controls
            .OfType<Label>()
            .All(label => label.BackColor == Color.Orange);
}

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .