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.