Discussion :

[Phenomenium]

Salut, je voudrais ajouter une ligne qui me permette de vérifier la bonne saisi d'un entier par l'utilisateur. Pour cela je voudrais faire un truc du genre : tant que nb n'est pas un entier, afficher erreur .... Seulement je ne sais pas comment faire , je peu pas ecrire (ligne 19):

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

(nb = "")

voila le debut de mon code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
using System;
 
namespace ConsoleApplication1
{
	/// <summary>
	/// Description résumée de Class1.
	/// </summary>
	class Class1
	{
		const int Max = 9;
 
		static void remplTab(ref int[]Tab, ref int taille)
		{
			int i;
			int nb;
			i = 0;
			Console.Write("Entrez un nombre : ");
			nb = Convert.ToInt32(Console.ReadLine());
			while (nb = ?????????)
			{
				Console.Write("Saisie incorrecte, veuillez retapez votre nombre.");
				Console.Write("Entrez un nombre : ");
				nb = Convert.ToInt32(Console.ReadLine());
			}
 
			while (i < Max && nb != 0)
			{
				Tab[i] = nb;
				i = i + 1;
				Console.Write("Entrez un nombre : ");
				nb = Convert.ToInt32(Console.ReadLine());
			}
			taille = i - 1;
		}
 
		static void Afficher(ref int[]Tab, ref int taille, ref int Numero)
		{
			int i;
			i = 0;
			Console.WriteLine("Tableau numéro {0}", Numero);
			for (i = 0; i <= taille;i++)
			{
				Console.WriteLine("{0} : {1}", i, Tab[i]);
			}
		}

merci de votre aide

[Ditch]

l'utilisation d'un try/catch qui englobe un Convert.ToInt32 peut t'aider. Ca doit exister une fonction qui verifie qu'un texte est un nombre (hum). Dans l'affirmative, il vaut mieux l'utiliser car les blocs try/catch pesent lourd à l'execution

[Phenomenium]

ba ouhai moi aussi j'aimerais mieux ...

[justice007]

Essai ceci.


soit b la valeur (string) entrée par l'utilisateur
int i = 0;
try
{
i = int.Parse(b);
}
catch(FormatException ao)
{
Console.WriteLine(ao.Message);
}

[Phenomenium]

ok merci, mais j'aimerais comprendre comment ca fonctionne. Est ce que ca remplace un While, ou est ce que je doit mettre les deux ? Si oui comment ?

[Piotrek]

La solution standard c'est Double.TryParse (ca peut verif un Int)

http://www.developpez.net/forums/viewtopic.php?t=235779

Mais le probleme c'est qu'une exception est levee tout de meme, et ca c'est pas bien! Une exception c'est tres lourd et ca doit etre declanche uniquement en cas de problemes, Ms le crie sur tous les toits, voici un bench pour le prouver:

http://aspalliance.com/articleViewer.aspx?aId=80&pId=3

Donc le mieux pour verifier frequement (ton cas precis) si une str est un entier, c'est de verifier que chaque char de la string est un nombre

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
    Public Function Numerik(ByVal tastring As String) As Boolean
 
        Dim i As Int32
        Dim isNum As Boolean = True
 
        For i = 0 To tastring.Length - 1
 
            If Not Char.IsDigit(tastring, i) Then
 
                isNum = False
                i = tastring.Length - 1
 
                MessageBox.Show("le char a la position " + i.ToString + " n'est pas un chiffre")
 
            End If
 
        Next
 
        Return isNum
 
    End Function

[Phenomenium]

euh merci beaucoup, le problème c'est que j'ai deja pas mal de mal avec C# mais alors de passer de ta solution en C a C#... jsuis un peu perdu ...

[Piotrek]

c'est du vb.net nondidjiuuuuu!

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
		public bool Numerik(string tastring)
		{
			Int32 i;
			bool isNum = true;
			for (int i = 0; i <= tastring.Length - 1; i++) {
				if (!char.IsDigit(tastring, i)) {
					isNum = false;
					i = tastring.Length - 1;
					MessageBox.Show("le char a la position " + i.ToString + " n'est pas un chiffre");
				}
			}
			return isNum;
		}

(traduit en 10 nanosecondes grace au lien de ma signature)

[Thomas Lebrun]

C'est pas un peu gourmand de faire la vérif sur tous les caractères, un par un....

[Piotrek]

etrangement non, c'est meme ce qu'il y a de plus rapide

http://aspalliance.com/articleViewer.aspx?aId=80&pId=3

Si on regarde dans System.Double.TryParse(String, NumberStyles, IFormatProvider, Double&) : Boolean avec Reflector: on n'as pas d'appel direct a la machinerie interne du framework: c'est du code .net comme un autre. Donc il est possible de faire mieux

[Thomas Lebrun]

En effet

pourtant, il est vrai que ma première réaction est celle que tout le monde pourrait avoir....

[Ditch]

Je me suis posé la meme question tout à l'heure...

[KINKIN]

Voici ce que je propose à mes élèves pour vérifier la saisie de données numériques :

1) créer une fonction qui vérifie la véracité de l'information saisie :

static int SaisieInt(string question)
{
int num =0;
string maLigne;
bool err;
do
{ try
{
err=false;
Console.WriteLine(question);
maLigne =Console.ReadLine();
num = int.Parse(maLigne);
}
catch (System.FormatException )
{
Console.WriteLine("votre saisie est incorrecte !");
err=true;
}
} while (err);

return num;

}


2) appeler cette fonction

L’appel pour vérifier la validité de la saisie d’un entier se fera ainsi :

// saisie du nombre d'étudiants
int nbEtu;
libQuestion="saisir le nombre d’étudiants : ";
nbEtu =SaisieInt(libQuestion);

// saisie du nombre de profs
int nbProf;
libQuestion="saisir le nombre de professeurs : ";
nbProf =SaisieInt(libQuestion);


Il est bien sûr possible de créer une fonction SaisieFloat pour vérifier la saisie de float, de double...

Puis, pour être encore plus "pro", vous pouvez créer une bibliothèque Utilitaire contenant une classe regroupant toutes ces méthodes de classe. Vous n'aurez plus qu'à ajouter une référence à cette bibliothèque dans tous les projets où vous souhaitez vérifier votre saisie ; vous pourrez alors appeler toutes vos méthodes de classe vérifiant la saisie depuis votre projet !!

Bon courage !

[Piotrek]

Tout a fait d'accord pour ta methode KINKIN

Mais le probleme est justement d'eviter de lancer une exception (elles sont tres couteuses en terme de performances) pour une raison autre que celles preconisees par microsoft
http://msdn.microsoft.com/library/default.asp?url=/library/en-us/cpguide/html/cpconexceptionsoverview.asp

C'est a dire: un simple test et non pas une erreur

Il suffit d'imaginer qu'on ait un formulaire avec une quinzaine de champs a valider ou tout autre traitement en boucle

[EDITION]
je me suis trompe de lien
http://msdn.microsoft.com/library/en-us/dnpag/html/scalenetchapt05.asp?frame=true#scalenetchapt05_topic24
je laisse l'autre, il reste tout de meme interessant

[BrItneY]

Juste une petite correction du code de Piotrek, pour ceux que cela intéresse i était définit 2 fois :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
public bool Numerik(string tastring)
		{
			bool isNum = true;
			for (Int32 i = 0; i <= tastring.Length - 1; i++) {
				if (!char.IsDigit(tastring, i)) {
					isNum = false;
					i = tastring.Length - 1;
					MessageBox.Show("le char a la position " + i.ToString + " n'est pas un chiffre");
				}
			}
			return isNum;
		}

En tout cas merci beaucoup pour ce topic, très intéressant