Discussion :

[sami_trab]

Bonjours tout le monde,

je viens de commencer avec c# , j'ai besoin d'une méthode qui retourne True si la chaine d'entré ne contient pas des lettre.

Mais ca ne fonctionne pas , je ne vois pas ou sa bloque

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
public Boolean SeulementChiffre(string chaine)
        {
            Boolean result = true;
            int i=0;            
            while(i<chaine.Length && result)
            {
                if (chaine[i].Equals("1") | chaine[i].Equals("2") | chaine[i].Equals("3") | chaine[i].Equals("4") |
                    chaine[i].Equals("5") | chaine[i].Equals("6") | chaine[i].Equals("7") | chaine[i].Equals("8") |
                    chaine[i].Equals("9") | chaine[i].Equals("0") | chaine[i].Equals("") | chaine[i].Equals("+") |
                    chaine[i].Equals("-") | chaine[i].Equals("(") | chaine[i].Equals(")") | chaine[i].Equals(":") |
                    chaine[i].Equals("/") | chaine[i].Equals("."))
                 i++; 
                else
                 result = false; 
            }
 
            return result;
        }

Merci d'avance de votre aide

[Jean-Michel Ormes]

Bonjour,

C'est pas plutôt des || pour signifier le 'OR' ?

[StormimOn]

Le problème vient de

Code :

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

chaine[i].Equals("1")

chaine[i] est un caractère, hors tu le testes par rapport à une chaîne.

Ceci marchera mieux

Code :

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

chaine[i].Equals('1')

Pour ce qui est des opérateurs OU, il ne faut pas confondre | qui représente le OU logique et || qui représente le OU conditionnel. Les 2 fonctionnent, mais le second est adapté pour les conditions, en n'évaluant l'expression qui si nécessaire.

Sinon, pour avoir un code plus clair, passer par une expression régulière est aussi bien

Code :

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

1
2
3
4
public bool ChaineValide(string chaine)
{
    return Regex.IsMatch(chaine, @"^[\d\+\-\(\):/\.]*$");
}

Enfin, plus clair si on comprend les expressions régulières évidemment ^^

Si tu préfères tout tester, tu peux au moins supprimer la boucle est passer par la méthode Contains de la classe String.

[sami_trab]

Merci Jmichou972 et StormimOn de vos reponse,

Votre reponse StormimOn m'a resolu le probleme et la methode fonctionne correctement

Mais j'ai pa bien compris votre fonction

Code :

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

Regex.IsMatch(chaine, @"^[\d\+\-\(\):/\.]*$");

[StormimOn]

Comme je l'ai dit, c'est une expression régulière. Après on connait ... ou pas. Et si on ne connait pas, et bien cherche sur ce sujet si cela t'intéresse

Cela permet de faire de la recherche, du remplacement ou de la validation de chaîne de caractères par exemple. Pour résumer très vite évidemment ^^

[sevyc64]

Bien compliqué tout ça.

Essaye simplement de convertir en nombre.

Ex, si ta chaine est succeptible de représenter un Int :

Code :

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

1
2
3
4
5
public Boolean SeulementChiffre(string chaine)
        {
        int i = 0;
        return int.TryParse(chaine, out i); 
        }

[StormimOn]

Bien compliqué sauf qu'il veut plus que des chiffres, contrairement à ce qu'indique le titre du sujet. Il veut les chiffres, des opérateurs ainsi que des parenthèses (cf. les tests effectués dans son code)

[sami_trab]

Merci tout le monde Pour vos reponses,

Effectivement , Je suis interessé par les chiffres, les parentheses, les operateurs....

[roster]

Les regex sont bien pratiques et parfois même nécessaires. Par contre, c'est souvent d'une lenteur affligeante pour le traitement de données volumineuses.

C'est un comportement normal ou bien c'est moi qui ne sais pas les utiliser?

PS: désolé pour le HS...