Skip to main content
se añadieron 38 caracteres en el cuerpo
Origen Enlace
Luis
  • 1.6k
  • 9
  • 9
static string validoOpcion(string Opcion)
{
    int numero;
    if (Int32.TryParse(Opcion, out numero))
    {
        // haces lo que quieras hacer con el número, quedó guardado en la variable numero...
        Mensaje = "Opcion Valida";
    }
    else
    {
        Mensaje = "No es un numero lo que ingresaste!";
    }    

    return Mensaje;  
}
static string validoOpcion(string Opcion)
{
    int numero;
    if (Int32.TryParse(Opcion, out numero))
    {
        // haces lo que quieras hacer con el número...
        Mensaje = "Opcion Valida";
    }
    else
    {
        Mensaje = "No es un numero lo que ingresaste!";
    }    

    return Mensaje;  
}
static string validoOpcion(string Opcion)
{
    int numero;
    if (Int32.TryParse(Opcion, out numero))
    {
        // haces lo que quieras hacer con el número, quedó guardado en la variable numero...
        Mensaje = "Opcion Valida";
    }
    else
    {
        Mensaje = "No es un numero lo que ingresaste!";
    }    

    return Mensaje;  
}
se añadieron 1692 caracteres en el cuerpo
Origen Enlace
Luis
  • 1.6k
  • 9
  • 9
 return Mensaje();
 return Mensaje;

Aparte, cuando llamas al método 'validoOpcion', te falta pasarle el parámetro que definiste:

 Opcion = Int32.Parse(Console.ReadLine());
 Console.WriteLine(validoOpcion());

Debería ser:

 Opcion = Int32.Parse(Console.ReadLine());
 Console.WriteLine(validoOpcion(out Opcion ));

Aun así no funcionará, te explico, en el cuerpo del programa, solicitas la opción:

 Opcion = Int32.Parse(Console.ReadLine());

Eso está bien, bueno, mas o menos, porque si escribes una letra te va a dar un error, pero bueno, la cosa es que una vez que llamas al método 'validoOpcion', vuelves a solicitar la opción, es decir, de nada sirve que lo pidas antes de llamar al método si dentro del método la vuelves a pedir.

Yo lo haría así, en el cuerpo del programa:

Opcion = Console.ReadLine();

Luego llamaría al método así:

Console.WriteLine(validoOpcion(Opcion));

Y el método lo modificaría así:

    static string validoOpcion(string Opcion)
    {
        boolean Numerico = false;
        int numero;
        Numerico = Int32.TryParse(Opcion, out numero);

        if (!Numerico)
        {
            Mensaje = "No es un numero lo que ingresaste!";
        } 
        else
        {
            Mensaje = "Opcion Valida";
        }

        return Mensaje;
    }

O refactorizado para que quede mas pequeño:

static string validoOpcion(string Opcion)
{
    int numero;
    var Numerico = Int32.TryParse(Opcion, out numero);
        
    return Numerico ?? "Opcion Valida" : "No es un numero lo que ingresaste!";  
}

O, si vas a utilizar el número dentro de validoOpcion para aprovechar de hacer alguna operación:

static string validoOpcion(string Opcion)
{
    int numero;
    if (Int32.TryParse(Opcion, out numero))
    {
        // haces lo que quieras hacer con el número...
        Mensaje = "Opcion Valida";
    }
    else
    {
        Mensaje = "No es un numero lo que ingresaste!";
    }    

    return Mensaje;  
}

Como ves, hay varias formas de hacerlo.

return Mensaje();
return Mensaje;
 return Mensaje();
 return Mensaje;

Aparte, cuando llamas al método 'validoOpcion', te falta pasarle el parámetro que definiste:

 Opcion = Int32.Parse(Console.ReadLine());
 Console.WriteLine(validoOpcion());

Debería ser:

 Opcion = Int32.Parse(Console.ReadLine());
 Console.WriteLine(validoOpcion(out Opcion ));

Aun así no funcionará, te explico, en el cuerpo del programa, solicitas la opción:

 Opcion = Int32.Parse(Console.ReadLine());

Eso está bien, bueno, mas o menos, porque si escribes una letra te va a dar un error, pero bueno, la cosa es que una vez que llamas al método 'validoOpcion', vuelves a solicitar la opción, es decir, de nada sirve que lo pidas antes de llamar al método si dentro del método la vuelves a pedir.

Yo lo haría así, en el cuerpo del programa:

Opcion = Console.ReadLine();

Luego llamaría al método así:

Console.WriteLine(validoOpcion(Opcion));

Y el método lo modificaría así:

    static string validoOpcion(string Opcion)
    {
        boolean Numerico = false;
        int numero;
        Numerico = Int32.TryParse(Opcion, out numero);

        if (!Numerico)
        {
            Mensaje = "No es un numero lo que ingresaste!";
        } 
        else
        {
            Mensaje = "Opcion Valida";
        }

        return Mensaje;
    }

O refactorizado para que quede mas pequeño:

static string validoOpcion(string Opcion)
{
    int numero;
    var Numerico = Int32.TryParse(Opcion, out numero);
        
    return Numerico ?? "Opcion Valida" : "No es un numero lo que ingresaste!";  
}

O, si vas a utilizar el número dentro de validoOpcion para aprovechar de hacer alguna operación:

static string validoOpcion(string Opcion)
{
    int numero;
    if (Int32.TryParse(Opcion, out numero))
    {
        // haces lo que quieras hacer con el número...
        Mensaje = "Opcion Valida";
    }
    else
    {
        Mensaje = "No es un numero lo que ingresaste!";
    }    

    return Mensaje;  
}

Como ves, hay varias formas de hacerlo.

Origen Enlace
Luis
  • 1.6k
  • 9
  • 9

En el return del método 'validoOpcion' estás regresando la variable mensaje como si fuese otro método:

return Mensaje();

cuando debería ser:

return Mensaje;