0

Estou tentando desenvolver um código no meu jogo mas estou com este erro.

...

case "cpsuser":
    Console.Write("Digite o apelido do usuário: ");

    string Nickname = Console.ReadLine();

    Console.Write("Digite a quantidade de Cupons");

    string Cupons = Console.ReadLine();

    Console.Write("Digite a mensagem a ser informada para o player. Por exemplo: Parabéns você recebeu 10000 Cupons.");

    string Message = Console.ReadLine();

    GamePlayer Player = WorldMgr.GetAllPlayers()
                                .FirstOrDefault(p => p.PlayerCharacter
                                                      .NickName == Nickname);

    if (Player == null)
    {
        Console.WriteLine("O usuário informado não existe ou está offline!");
    }
    else
    {
        int num6 = (object)Cupons;
        foreach (GamePlayer gamePlayer in WorldMgr.GetAllPlayers())
        {
            gamePlayer.AddMoney(num6);
            gamePlayer.SendMessage("({0})", Message);

        }
    }
    break;

...

No caso é aqui que estou com o erro:

...

else
{
    int num6 = (object)Cupons;
    foreach (GamePlayer gamePlayer in WorldMgr.GetAllPlayers())
    {
        gamePlayer.AddMoney(num6);
        gamePlayer.SendMessage("({0})", Message);

        ...
5
  • 1
    Você está tentando colocar um object dentro de um int. Imagine que o object é pai de todos os objetos, e int é filho de object, em uma analogia você está tentando colocar o pai dentro do filho. Commented 18/05/2016 às 14:29
  • Como eu poderia fazer para tentar corrigir?
    – user39571
    Commented 18/05/2016 às 14:40
  • 1
    use o Convert.ToInt32, tem que ser um objeto que pode ser convertido para int, como por exemplo um string ou um decimal. O que esse objeto cupons tem de propriedade? De que tipo ele é? Commented 18/05/2016 às 14:46
  • Já resolvido obrigado.
    – user39571
    Commented 18/05/2016 às 15:15
  • @Vinicius na verdade não foi resolvido, eu dei uma resposta porque as respostas que tinham ensinam errado.
    – Maniero
    Commented 18/05/2016 às 17:35

3 Respostas 3

6

Resolvi responder porque a resposta aceita é um erro e a outra fala em boas práticas, quando é justamente o oposto. Ou funciona direito ou não funciona, não existe isso de boa prática, e só uma das soluções apresentadas funciona direito.

É importante notar que testar dentro do uso normal e dizer que funciona não é programar corretamente. O certo é fazer algo que funcione sempre. Então com dados que vêm de fontes externas que o programador não tem controle, não é possível fazer conversões diretas e considerar que isto é correto. Tem que tentar fazer a conversão e se der algum problema decidir que ação tomar. Por exemplo, é possível definir que se um texto inválido foi digitado, considere o número como 0 (como fiz no exemplo abaixo). Não estou dizendo que isto é o mais adequado para este código. Pode ser que seja outro número ou pode ser que o código precise ter uma outra ação, talvez avisar do problema para o usuário e solicitar outro número, depende do contexto.

Seria algo assim:

int num6;
num6 = int.TryParse(cupons, out num6) ? num6 : 0;

Veja funcionando no ideone. E no .NET Fiddle. Também coloquei no GitHub para referência futura.

Para saber se algo funciona precisa testar com todas possibilidades. Tem que testar coisas até mesmo estapafúrdias. Tem que entender todo o mecanismo que está usando e testar por todos os caminhos que ele pode executar. Tem que saber tudo o que pode retornar de um método, inclusive as exceções que ele pode lançar. Aí deve tratar cada situações adequadamente ou mudar o método escolhido se tiver outro mais adequado, como é o caso acima.

Sem ler e interpretar toda documentação do que está usando adequadamente não dá para fazer nada certo e não pode-se afirmar que algo funciona. Muito menos pode-se falar em boa prática. Acreditar em boas práticas é uma má prática e vai dar um tiro no próprio pé. Ou a coisa é certa ou errada.

Pode testar as duas opções erradas. Primeiro coloque uma palavra, depois tente de novo colocando um número correto e uma palavra no segundo pedido. Verá a exceção.

Documentação.

Outra pergunta com mais detalhes sobre o assunto.

2
  • Poderia ainda sim, simplificar somente para int.TryParse(cupons, out int num6);. O int já tem valor padrão 0. Claro, isso dependendo da versão do c# que está sendo utilizada, mas não que esteja errado. Commented 14/01/2020 às 14:40
  • Agora pode, na época da resposta não dava ou era muito recente pra responder isso.
    – Maniero
    Commented 14/01/2020 às 14:45
3

@Vinicius, Não é uma boa prática fazer um cast (aka. unboxing).

Uma das melhores práticas, diz que você precisa fazer uma conversão ou parse e para fazer isso você tem algumas opções.

  • Int32 Convert.ToInt32(string) - Recebe uma string como parâmetro e tenta converter para Int32 (que é um alias para Int32), se não conseguir retorna um erro
  • Int32 Int32.Parse(string) - Faz o mesmo do item acima
  • bool Int32.TryParse(string, out Int32) - Recebe uma string como parâmetro e um Int32 marcado com a keyword out (indicando que a variável passada terá um valor no final da execução do método). Caso consiga converter, a variável terá o valor convertido e o método retorna true; Caso contrário, retorna false e a variável terá o valor default para Int32, que é 0.


Na prática ficaria algo assim:

Int32 Convert.ToInt32(string)

int num6 = Convert.ToInt32(Cupons);

Int32 Int32.Parse(string)

int num6 = Int32.Parse(Cupons);

bool Int32.TryParse(string, out Int32)

int num6;
bool converteu = Int32.TryParse(Cupons, out num6);

Espero ter ajudado \o/

1
  • 1
    Obrigado por ajudar :)
    – user39571
    Commented 18/05/2016 às 14:54
1

Não podes fazer um cast para um formato de objecto e tentares guardar em int... Tens de fazer assim com o Convert.ToInt32()

{
   int num6 = Convert.ToInt32(Cupons);
   foreach (GamePlayer gamePlayer in WorldMgr.GetAllPlayers())
   {
      gamePlayer.AddMoney(num6);
      gamePlayer.SendMessage("({0})", Message);
5
  • Agora está dando erro: Cannot convert type 'string' to 'int'
    – user39571
    Commented 18/05/2016 às 14:39
  • você verificou o que sua variável ta recebendo pelo Watch? a variável esta sendo uma string vazia? Commented 18/05/2016 às 14:43
  • Agora já dará :) Commented 18/05/2016 às 14:44
  • Funcionou obrigado Pedro Luzio.
    – user39571
    Commented 18/05/2016 às 14:44
  • @Vinicius funcionou em um teste simples e direcionado, sem considerar que o usuário pode digitar algo errado. Na verdade esta é a resposta mais errada. O próprio comentário acima já indica algo errado. Só funciona quando a pessoa digita certo. Não é o programador que deve exigir o usuário fazer o certo, é o código que deve funcionar corretamente. Note que sua intuição foi fazer o errado. Se vai insistir em fazer errado, pelo menos coloque uma mensagem para o usuário alertando que ele não pode fazer nada errado.
    – Maniero
    Commented 19/05/2016 às 11:16

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