Verifica stringa contenente solo numeri in c#

[Reayes]

Ciao a tutti vorrei chiedervi come si fa a controllare se una stringa contiene solo numeri e se non li contiene richiedere di nuovo il numero

[wingman87]

Per controllare se contiene solo numeri puoi usare un'espressione regolare: "^[0-9]*$"
Per l'utilizzo delle espressioni regolari cerca su internet "C# Regex Match".
Per richiedere il numero, se stai facendo un'applicazione console, puoi usare un ciclo do-while che fa la richiesta fino a quando l'input non è accettabile.

[Reayes]

ok sono andato a vedere come funziona ma per fare quello che mi dici tu ci vorrebbe un' espressione regolare che verifichi se non ci sono numeri ecco il
codice:
do
{
Console.Clear();
tmp = "";
numero = 0;
ris = 0;
do
{

Console.WriteLine("Inserisci un numero intero positivo minore di 21");
tmp = Console.ReadLine();
Regex regex = new Regex("^[0-9]*$");
Match match = regex.Match(tmp);
if (match.Success)
{
Console.WriteLine(match.Value);
int a = 1;
Console.WriteLine("Inserisci un numero corretto");
}
numero = Convert.ToInt16(tmp);
if (numero > 21)
Console.WriteLine("Inserisci un numero corretto");
} while (a == 1);

} while (numero >= 21 || numero <= 0);
come si potrebbe modificare per utilizzare l'espressione "^[0-9]*$" ?

[wingman87]

Per scrivere codice qui sul forum usa il tag code che mantiene l'indentazione.
Ti scrivo la parte interessante:

int MAX_VALUE = 20;
numero = MAX_VALUE + 1;
Regex regex = new Regex("^[0-9]+$"); //La stringa deve essere composta da sole cifre (e almeno una)
do 
{
   Console.WriteLine("Inserisci un numero intero positivo minore di 21");
   tmp = Console.ReadLine();
   Match match = regex.Match(tmp);
   if (match.Success)
   {
      numero = Convert.ToInt16(tmp);
   }    
} while (numero >= MAX_VALUE);
// Quando arrivi qui hai il numero corretto

[Reayes]

ho gia risolto da solo bastava utilizzare "[^0-9]" grazie per la dritta comunque

[Reayes]

abbiamo inviato quasi allo stesso tempo

[wingman87]

Originariamente inviato da Reayes

ho gia risolto da solo bastava utilizzare "[^0-9]" grazie per la dritta comunque

Prego, comunque se noti la regex che avevo pensato io usava la logica inversa.

[gugoXX]

Io getto questo.
2 metodi diversi
uno che fa uso dello statico char.IsDigit

l'altro che usa direttamente il int.TryParse, di gia' che poi mi sembra tu debba comunque convertire in intero, cosi' in una botta sola fai il check che sia valido e lo converti pure.

 class Program
    {
        static void Main(string[] args)
        {
            string test1 = "1414534593449499";
            var test1AllDigits = HasOnlyDigits(test1);

            string test2 = "141234324a234";
            var test2AllDigits = HasOnlyDigits(test2);

            Console.WriteLine("{0} - {1}", test1, test1AllDigits);
            Console.WriteLine("{0} - {1}", test2, test2AllDigits);
            Console.WriteLine();
            Console.ReadKey();

            string test3 = "14242";
            int test3val;
            var test3AllDigits = int.TryParse(test3, out test3val);

            string test4 = "1434f242";
            int test4val;
            var test4AllDigits = int.TryParse(test4, out test4val);

            Console.WriteLine("{0} - {1} - {2}", test3, test3AllDigits, test3val);
            Console.WriteLine("{0} - {1} - {2}", test4, test4AllDigits, test4val);
            Console.ReadKey();
        }

        private static bool HasOnlyDigits(string str)
        {
            return str.All(char.IsDigit);
        }
    }

1414534593449499 - True
141234324a234 - False

14242 - True - 14242
1434f242 - False - 0