Vérification erreur/éléments inutiles programme

Version imprimable

Bonjour,

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

Merci

Code:

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;
			}
			}
}

26/08/2009, 14h53
dinobogan

Quel est le problème ?
27/08/2009, 11h55
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.

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 !
27/08/2009, 13h18
jerdel

Ok merci. Ainsi j'évite les choses inutiles dans mes programmes.
27/08/2009, 13h39
jerdel

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
27/08/2009, 13h47
tchize_

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.
27/08/2009, 16h16
Torg666
Tu devrais eviter le toString pour afficher le champ age et plutot declarer un fonction propre genre:
Code:

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:

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:

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..

bonjour
tu peux peut etre simplifier ca

Code:

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:

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

27/08/2009, 18h00
ipingu
Citation:

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

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

Code:

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)

27/08/2009, 19h04
jerdel

Ok merci pour vos réponses.
27/08/2009, 19h30
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:

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

Code:

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:

1 2 3 public void setTel() { this.tel=tel; }
27/08/2009, 21h53
tchize_
Citation:
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:

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 ;)

Citation:

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 :D