Discussion :

[Morby]

Salut tout le monde, je suis grand débutant en Java et je m'arrache les cheveux sur mon 1er projet : j'ai créé une fenetre avec une série de JTextfields pour saisir les coordonnées de clients, avec un écouteur sur le bouton de validation.

public class actionNewFicheClient implements ActionListener{
public void actionPerformed(ActionEvent e) {

/////////// test de validité des infos saisies ////////////

// pas de n° de client
if(numClient.getText().length()==0){
System.out.println("ERREUR : pas de n° de client");
}
// pas de prenom
else if(pname.getText().length()==0){
System.out.println("ERREUR : pas de prénom");
}
// pas de nom
else if(name.getText().length()==0){
System.out.println("ERREUR : pas de nom");
}
// pas d'adresse
else if(adresse.getText().length()==0){
System.out.println("ERREUR : pas d'adresse");
}
// pas de code postal
else if(codePostal.getText().length()==0){
System.out.println("ERREUR : pas de code postal");
}
// pas de ville
else if(ville.getText().length()==0){
System.out.println("ERREUR : pas de ville");
}

else{
System.out.println("ajout du nouveau client");

// création du nouvel objet CLIENT
Client cl1 = new Client (
Integer.parseInt(numClient.getText()),
pname.getText(),
name.getText(),
adresse.getText(),
Integer.parseInt(codePostal.getText()),
ville.getText()
);

FenSaisieClient.setVisible(false); // fermeture de la fenetre
new Fenetre(); // retour à la fenetre d'accueil
}

----------------

mon problème est le suivant : je voudrais aussi vérifier que pour le n° client et le code postal on a bien mis des chiffres et pas des lettres.
en fouinant sur le net j'ai trouvé du code qui fait ça très bien :

// le numero client doit etre constitué de chiffres
try {int i = Integer.parseInt(numClient.getText());}
catch (Exception exCL1) {
System.out.println("ERREUR : le n° client doit etre constitué de chiffres");
}

// le code postal doit etre constitué de chiffres
try {int i = Integer.parseInt(codePostal.getText());}
catch (Exception exCP2) {
System.out.println("ERREUR : le n° client doit etre constitué de chiffres");
}
// le code postal doit contenir UNIQUEMENT 5 chiffres
if(codePostal.getText().length()!=5) {
System.out.println("ERREUR : un code postal doit comporter 5 chiffres");
}

le soucis est que je ne parviens pas à intégrer ce code au milieu de ma série de "else if"


d'autre part ya bcp de répétition de code (if(xxxxxxx.length()==0)), ça fait pas très "DRY", surtout que j'ai aussi une fenetre de saisie "Produit" à faire où je devrais effectuer les mêmes vérifications.
faire un appel de méthode dans une classe extérieure "Verif" serait peut être plus simple, mais je ne maitrise pas du tout cette technique...

help please

[deathness]

Tu peux faire une méthode verifieChamp ou tu mettra ton xxx.length==0 pour le factoriser. ça peut avoir un intérêt si après tu décides de mettre d'autre critères (comme xxx!=null && ...).

Quand a tes bouts de code, tu a juste a les rajouter dans ta série de vérification, je ne vois pas très bien le problème?

[Morby]

merci mais ça ne répond pas à mes questions :
- comment j'insère les "try" parmis mes "else if"
- comment je construis et appelle ma méthode externe "vérif"
pensez bien que je n'ai eu que 20h de cours de Java, je débute

[Robin56]

merci mais ça ne répond pas à mes questions

J'ai pourtant l'impression que deathness répond à ta question moi.

comment j'insère les "try" parmis mes "else if"

Tu ne les insères pas, tu les mets à la suite.

Code :

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

1
2
3
4
5
6
7
8
9
if (toto.length() == 0) {
  ... // Erreur
} 
try {
  Integer.parseInt(toto);
} catch (Exception e) {
  ... // Erreur
}
// TODO : Etc

comment je construis et appelle ma méthode externe "vérif"

Quelque chose dans le genre :

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 void actionPerformed(ActionEvent e) {
  if (this.verifTailleChamp(toto) == false) {
    // Erreur
  }
  // TODO : Etc
}
 
private bool verifTailleChamp(String nom) {
  if (nom == 0) {
    return false;
  } else {
    return true;
  }
}

[Morby]

merci pour ta réponse Robin, ça m'aide
j'essaye ça demain

[Morby]

finalement tes indications m'ont aidé à me sortir de mon bourbier pour parvenir à ce que je voulais en le faisant à ma sauce. si jamais quelqu'un tombe sur le même prb que moi, voici la soluce que j'ai utilisé :


- avant ma batterie de tests, j'ai créé une variable booléenne que j'ai initialisée sur "true"

boolean verif = true;

- lors de chaque test de type "if", si le JTextfield est vide mon booléen passe en "false"

// pas de No de client
if(numClient.getText().length()==0){
verif = false;
JOptionPane.showMessageDialog(null,"ERREUR : pas de No de client");
}

- suite à quoi j'enchaine les tests avec les "try" et sans mettre de "else"

// le numero client doit etre constitue de chiffres
try {
int i = Integer.parseInt(numClient.getText());
}
catch (Exception exCL1) {
if(verif){
verif = false;
JOptionPane.showMessageDialog(null,
"ERREUR : le numero client doit etre constitue uniquement de chiffres");
}
}

- et enfin si toutes les conditions de verifications sont remplies (verif toujours sur "true"), on cree le client avec un appel de méthode

if( verif){
creaClient();
}

j'ai pas encore fait la factorisation de la requête if(xxxxx.getText().length()==0), on verra ça à la fin quand j'aurais résolu mes autres prbs de conception.
encore merci pour le forumo-guidage