Discussion :

[the_lord_sith]

Bonjours, comme je ne suis pas très expérimenté en programmation, j'ai probablement encore certaines mauvaises pratiques dont j'aimerais me débarasser.

En ce moment par exemple, pour vérifier si un caractère est une lettre, j'ai un test à rallonge dans ce genre là.

Code :

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

while ( !((choixC  >= 'a' && choixC <= 'z') || (choixC >= 'A' && choixC <= 'Z')) )

Bref, compliqué et pas très lisible le truc. Je suis sur qu'il y a moyen de faire beaucoup plus simple, mais comment ?

Merci

[Robin56]

Je dirais d'utiliser ceci après un tour sur la Javadoc :

Code :

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

Character.isLetter(choixC)

[thelvin]

Ceci étant dit, Character.isLetter() accepte aussi les accents, les lettres arabes, grecques et hébreuses, les idéogrammes et phonèmes japonais, etc.

C'est pas forcément ce qu'on veut, quand on veut juste vérifier si c'est de a à z en majuscules ou minuscules. Mais dans ce cas-là, il n'y a pas beaucoup plus simple que faire ce test à rallonge. Ca peut se simplifier comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
static boolean isLetter(char ch) {
  return (ch >= 'a' && ch <= 'z') || (ch >= 'A' && ch <= 'Z');
}
 
public Truc autreMethode() {
  // début
  while (!isLetter(choixC)) {
    // boucle
  }
  // fin
}

[Laine]

Avez vous essayer les expressions régulières ?
Peut être que ce bout de code va vous aider:

Code :

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

1
2
3
Pattern pattern = Pattern.compile("[a-zA-Z]");
Matcher matcher = pattern.matcher("a");
boolean respons = matcher.matches();

Ligne 1: compile l'expression régulière (regex) pour en faire un objet Java
Ligne 2: appliquer l'expression à une chaîne de caractère
Ligne 3: teste si la chaîne correspond au regex.

[Catsoumoto]

Plus simple la Class String possède une méthode matches a lequel on donne un regex.

Code :

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

1
2
3
4
 
static boolean isLetter(char ch) {
     return Character.toString(ch).matches("[a-zA-Z]");
}

[Laine]

Encore mieux en effet !

[Robin56]

Avec tout ça, s'il n'a pas de solution à son problème, je ne comprendrais pas

[the_lord_sith]

Tu n'aura pas besoin de ne pas comprendre quoi que ce soit Dans mon cas, la meilleur solution est ce fameux isLetter, car il est tout à fait possible qu'il y ait des accents. Donc mon premier test, en plus de faire mal aux yeux n'était même pas fonctionnelle.

[Catsoumoto]

Peux-tu mettre résolu sur ton post, car je pense que c'est le cas^^