help not to repeat the same case of a switch inside a for (c#)

Question

I am in the first period of an SI course and I need to play a truce and I came across a situation that I am not able to solve. I need that during the loop I can not repeat the same case in the truco plays. Follow the method:

static void jogo()
{

    for (int jogada = 0; jogada<3; jogada++)
    {            
        if(jogador1jogada == 2 || jogador2jogada == 2)
        {
            break;
        }

        Console.WriteLine("Qual carta vc ira jogar? ");
        escolha = int.Parse(Console.ReadLine());
        switch (escolha)
        {
            case 1:
                Console.WriteLine("Voce jogou a carta: " + jogador1[0]);
                IAjogo();
                Console.WriteLine("A IA jogou: " + jogador2[escolhaIA - 1]);
                verificavitoria();                                              
            break;
            case 2:
                Console.WriteLine("Voce jogou a carta: " + jogador1[1]);
                IAjogo();
                Console.WriteLine("A IA jogou: " + jogador2[escolhaIA - 1]);
                verificavitoria();                    
            break;
            case 3:
                Console.WriteLine("Voce jogou a carta: " + jogador1[2]);
                IAjogo();
                Console.WriteLine("A IA jogou: " + jogador2[escolhaIA - 1]);
                verificavitoria();                    
            break;
        }

    }

}

Answer 1

Create a string array, and add value if the option has been chosen, before the switch make an if to check if the option has already been chosen, if it has already been, enter Else giving error message and putting the value of play -1 not to increment the move and keep repeating until the value is valid.

using System;

class MainClass {
  public static void Main (string[] args) {

    var test = new string[3];
    Console.WriteLine("Atribuir o valor quando o valor for escolhido dentro do case");
    Console.WriteLine(test[0] = "1");
    Console.WriteLine(test[1] = "2");
    Console.WriteLine(test[2]);

    Console.WriteLine("Ver se é nulo, se for, não há valores");
    Console.WriteLine(test[0] == null);
    Console.WriteLine(test[1] == null);
    Console.WriteLine(test[2] == null);
  }
}

In the example above I put the string 1 and 2 but did not put 3, which means that the player has already chosen 1 and 2, 3 is missing

Atribuir o valor quando o valor for escolhido dentro do case
1
2

Ver se é nulo, se for, não há valores
False
False
True

Then to see if you have already chosen the option just check if it is null, if it is not yet chosen, at the end of the for just put everything null again.

EDIT: Switch must be inside the IF, Else after switch.

Answer 2

Realize that the only thing that changes in each case is the value of X in Console.WriteLine("Voce jogou a carta: " + jogador1[X]). So just use only once this snippet of putting code escolha - 1 in place of X.

With this the case was eliminated. Later, you can fill the position of the vector whose card was downloaded with "-1" and check this before discarding.

for (int jogada = 0; jogada<3; jogada++)
{            
    if(jogador1jogada == 2 || jogador2jogada == 2)
    {
        break;
    }
    Console.WriteLine("Qual carta vc ira jogar? ");
    escolha = int.Parse(Console.ReadLine());

    //  evita a reutilização da carta
    if (jogador1[escolha - 1] == "-1") 
    {
        jogada--;
        continue;
    }

    Console.WriteLine("Voce jogou a carta: " + jogador1[escolha -1]);
    jogador1[escolha - 1] = "-1" // marca a carta como já utilizada
    IAjogo();
    Console.WriteLine("A IA jogou: " + jogador2[escolhaIA - 1]);
    verificavitoria();                                              
}