Vérification erreur/éléments inutiles programme

**jerdel** · 26/08/2009, 14h38

Bonjour,

Quelqu'un pourrait-il regarder s'il y a des erreurs ou des éléments inutiles dans le programme qui suit.

Merci

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
46
47
48
49
50
51
52
53
54
55
56
57
class Personne3{
	int age;
	String nom,ntel;
	Adresse adr;//champ qui se réfère à adresse
	//le constructeur:
	Personne3(int a,String n,String t)
		{
		age=a;
		nom=n;
		ntel=t;
		}
	Personne3(int a,String n,String t,int num,String r,String vil){
		Personne3(a,n,t);
		adr=new Adresse(num,r,vil);
		}
	class Adresse{
		int numero;
		String rue;
		String ville;
		Adresse(){
			}
		Adresse(int num,String r,String vil){
			numero=num;
			rue=r;
			ville=vil;
			}
	}
 
	//le modifieur:
	void setAge(int ag)
		{
		if(age<0) System.out.println("age négatif, impossible");
		else age=ag;
		}
	//l'accesseur:
    	int getAge()
		{
		return age;
		}
	String getNom()//rendre une valeur accessible en lecture
		{
		return nom;
		}
	public String toString()	
		{
		return("l'age est de "+age);
		}
	boolean memeNom(Personne3 p)
			{//fonction utilisée pour comparer les noms
			if(nom.equals(p.nom)){
			return true;
			}
			else{
			return false;
			}
			}
}

**dinobogan** · 26/08/2009, 14h53

Quel est le problème ?

**Rei Ichido** · 27/08/2009, 11h55

1. Quand tes setters sont non-triviaux (ie, comportent des tests ou des fonctions en plus du simple set), je te conseille fortement de les utiliser dans ton constructeur plutôt que de faire des affectations.
Parce que là, tu peux créer une personne d'âge négatif via le constructeur.

2. Est-il vraiment utile que ta classe Adresse soit interne ? Elle a probablement des utilités ailleurs

En plus ça te permettrait de créer un constructeur qui prenne une adresse en argument, c'est moins horrible que 3 arguments en plus pour un total de 6 arguments.

3. Si certaines données membre sont assorties de getters/setters, il faut les mettre en private, sinon tu peux bypasser ces getters/setters, ce qui en diminue fortement l'intéret !

**jerdel** · 27/08/2009, 13h18

Ok merci. Ainsi j'évite les choses inutiles dans mes programmes.

**jerdel** · 27/08/2009, 13h39

Et une variable (je ne sais pas si l'on appelle ça ainsi) qui passe par un getteur est-elle utilisable dans une classe externe (fichier externe)?
Mes questions sont peut être idotes mais je débute.

Merci

**tchize_** · 27/08/2009, 13h47

a partir du moment ou tu récupère une valeur de retour après avoir appelé une méthode, t'en fais ce que tu veux.

**Torg666** · 27/08/2009, 16h16

Tu devrais eviter le toString pour afficher le champ age et plutot declarer un fonction propre genre:

Code :

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

1
2
3
4
 
public String quelAge(){
return("l'age est de "+age);
}

note: si tu as juste besoin d'afficher l'age tu peu aussi faire:

Code :

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

1
2
3
4
 
public void quelAge(){
system.out.println("l'age est "+age);
}

La surcharge de toString serai plus logique pour faire ca:

Code :

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

1
2
3
4
 
public String toString(){
return (nom+" est agé de "+age+", son num de tel est: "+ntel+",son adresse est "+adr.toString);
}

En redefinissant biensur le toString de ton objet adresse.
Toutefois c'est a eviter car certain objet, notement les element graphique utilise,par defaut, le toString pour l'affichage de l'objet, exemple les JTables.

donc mieux vaut declarer un fonction propre plutot que de surcharger un fonction dont tu sais au final si elle sera pas utiliser ailleur par defaut..

**ANoki** · 27/08/2009, 17h48

bonjour
tu peux peut etre simplifier ca

Code :

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

1
2
3
4
5
6
7
8
9
boolean memeNom(Personne3 p)
			{//fonction utilisée pour comparer les noms
			if(nom.equals(p.nom)){
			return true;
			}
			else{
			return false;
			}
			}

par

Code :

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

1
2
3
4
 
boolean memeNom(Personne3 p){
	return nom.equals(p.getnom())
}

**ipingu** · 27/08/2009, 18h00

Envoyé par Rei Ichido

1. Quand tes setters sont non-triviaux (ie, comportent des tests ou des fonctions en plus du simple set), je te conseille fortement de les utiliser dans ton constructeur plutôt que de faire des affectations.
Parce que là, tu peux créer une personne d'âge négatif via le constructeur.

Le problème n'est pas résolu si tu utilises le setters dans le constructeur. Si tu mets le setter dans le constructeur et que tu donnes un age négatif, alors la variable age ne sera pas initialisée, ce qui est problématique.

Solution à ca :

Donner une valeur à age par défaut au cas ou l'age est négatif
Ou, je préfère, créer une méthode statique de création de l'objet Personne, qui renvoie un objet uniquement si les paramètres sont valides, et renvoie null si invalides, et mettre le constructeur en private.
Ou balancer dans le constructeur des IllegalArgumentException si les arguments sont invalides

**tchize_** · 27/08/2009, 18h23

Voilà une correction que je considère personnellement comme "propre". Idéallement il faudrait aussi des getters/setters sur la classe Adresse

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
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
 
public class Adresse{
    public int numero;
    public String rue;
    public String ville;
    Adresse(){ }
    Adresse(int num,String r,String vil){
        numero=num;
        rue=r;
        ville=vil;
    }
}
 
public class Personne3{
    private int age;
    private String nom,tel;
    Adresse adr;//champ qui se réfère à adresse
    //le constructeur:
    Personne3(int a,String n,String t){
        setAge(a);
        setNom(n);
        setTel(t);
    }
    Personne3(int a,String n,String t,Adresse adresse){
        Personne3(a,n,t);
        setAdresse(adresse);
    }
    public Adresse getAdresse() { return adr;}
    public void setAdresse(Adresse adresse) {
        this.adr=adresse;
    }
    public String getTel() { return tel;}
    public void setTel(String tel) {
        this.tel=tel;
    }
    public void setAge(int age){
        if(age<0) 
            throw new IllegalArgumentException("age négatif impossible");
        else 
            this.age=age;
    }
    //l'accesseur:
    public int getAge(){
        return age;
    }
    String getNom() {
        //rendre une valeur accessible en lecture
        return nom;
    }
    public void setNom(String nom) {
        this.nom=nom;
    }
    /* Bon pour du debug seulement
    public String toString(){
        return("l'age est de "+age);
    } */
    public boolean memeNom(Personne3 p){//fonction utilisée pour comparer les noms
        if (p.nom==nom==null)
            return true; // les deux noms sont à null
        if (p.nom==null || nom==null)
            return false; // un seul des noms est à null
        return p.nom.equals(nom); // aucun nom à null
    }
}

correction effectuée

précision de la portée des champs / méthodes (public / private)
indentation
correction des erreurs de logique (test d'équalité, test de l'âge)
systématisation des getters/setters
correction des noms de champs
Utilisation du IllegalArgumentException
Retrait de la classe interne

edit: code corrigé (cf adresse et setter incomplet)

**jerdel** · 27/08/2009, 19h04

Ok merci pour vos réponses.

**sovo** · 27/08/2009, 19h30

J'ai quelques incompréhensions/questions avec ton code tchize_

1. Pourquoi ne pas faire un constructeur avec une variable de type Adresse, vue que tu as déjà fait sortir la classe Adresse ??

Code :

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

1
2
 
Personne3(int a,String n,String t,Adresse add){...}

2. Je vois

Code :

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

setAdresse(new Adresse(num,r,vil));

mais je ne vois nulle part une méthode SetAdresse

3. Comment ce fait que ton setter n'a pas de paramètre, il modifie donc quoi ??

Code :

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

1
2
3
public void setTel() {
        this.tel=tel;
    }

**tchize_** · 27/08/2009, 21h53

Envoyé par sovo

J'ai quelques incompréhensions/questions avec ton code tchize_

1. Pourquoi ne pas faire un constructeur avec une variable de type Adresse, vue que tu as déjà fait sortir la classe Adresse ??

Code :

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

1
2
 
Personne3(int a,String n,String t,Adresse add){...}

J'ai péché par paresse, je suis d'accord avec toi, autant faire aussi sortie le new

mais je ne vois nulle part une méthode SetAdresse

3. Comment ce fait que ton setter n'a pas de paramètre, il modifie donc quoi

La fatigue, je corrige mon post

Vérification erreur/éléments inutiles programme

avec Java

Discussions similaires

Partager

Partager