Discussion :

[bandit_debutant]

Bonjour à tous. Je suis débutante et j'ai un problème avec une classe très simple: créer un objet Livre et afficher titre et auteur. Le problème vient lors de l'affichage. Si je ne déclare pas mes variables dans la class Livre, il m'affiche "null". Elles n'apparaissent que si je les initialise dans cette classe. L'objet est créé avec les paramètres déclarés dans le constructeur, mais je n'arrive pas à voir où est l'erreur. C'est certainement très simple, mais j'ai besoin d'aide.
Voici les deux classes (je ne sais pas encore utiliser les balises CODE, désolée)

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
package gestionmateriel;
 
public class Livre {
 
	String auteur="Jean";
	String titre="Techniques";
 
	public Livre (String titre, String auteur)
	{
		titre=this.titre;
		auteur=this.auteur;
	}
 
	public String getAuteur()
	{
		return auteur;
	}
 
	public String getTitre()
	{
		return titre;
	}
 
}

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
package gestionmateriel;
 
public class EssaiLivre {
 
	public static void main(String[] args) {
		Livre monLivre = new Livre("t", "e");
 
		String monAuteur=monLivre.getAuteur();
		System.out.println(monAuteur);
		String monTitre=monLivre.getTitre();
		System.out.println(monTitre);
 
 
	}
 
}

J'ai beau changer les paramètres de mon constructeur, il affichera toujours "Jean" et "Technique".
Merci d'avance pour votre aide.

[Napalm51]

Salut, pour les balises code il suffit de mettre ton code entre [kode] blabla [/kode] en remplaçant le "k" par un "c"

Ensuite ton problème vient de ton constructeur, au lieu de faire :

Code :

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

this.auteur = auteur;

tu inverses avec :

Code :

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

auteur=this.auteur;

Forcément, tu penses mettre "t" à la place de "Jean" alors que d'après ton code, tu fais l'inverse ...

[bandit_debutant]

Merci beaucoup pour la réponse

Je vais réviser le mot clé "this"

[Mooneer]

en fait, dans ton contructeur, quand tu fais:

Code :

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

1
2
3
4
5
public Livre (String titre, String auteur)
	{
		titre=this.titre;
		auteur=this.auteur;
	}

explicitement, cela donne

Code :

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

1
2
3
4
5
public Livre (String titre, String auteur)
	{
		this.titre=this.titre;
		this.auteur=this.auteur;
	}

Donc, this.titre prend la valeur de this.titre, or avant cette opération, on a this.titre = "Techniques"
C'est pour cela que quelque soit l'argument, this.titre prendra la valeur "Techniques"

Tu peux effectivement mettre this.titre = titre, mais le plus simple, pour éviter tout confusion serait de faire :

Code :

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

1
2
3
4
5
public Livre (String nom, String ecrivain)
	{
		titre=nom;
		auteur=ecrivain;
	}

[bandit_debutant]

Encore merci pour une réponse claire.

[dingoth]

en fait, dans ton contructeur, quand tu fais:

Code :

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

1
2
3
4
5
public Livre (String titre, String auteur)
	{
		titre=this.titre;
		auteur=this.auteur;
	}

explicitement, cela donne

Code :

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

1
2
3
4
5
public Livre (String titre, String auteur)
	{
		this.titre=this.titre;
		this.auteur=this.auteur;
	}

C'est faux.

Le premier constructeur assigne la valeur de la variable d'instance titre à la variable locale titre (pareil pour auteur).

Pour vous en convaincre :

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
public Livre (String titre, String auteur)
	{
                System.out.println("Avant l'attribution des valeurs");
                System.out.println("titre = " + titre);
                System.out.println("this.titre = " + this.titre);
                System.out.println("auteur = " + auteur);
                System.out.println("this.auteur = " + this.auteur);
 
		titre=this.titre;
		auteur=this.auteur;
 
                System.out.println("Après l'attribution des valeurs");
                System.out.println("titre = " + titre);
                System.out.println("this.titre = " + this.titre);
                System.out.println("auteur = " + auteur);
                System.out.println("this.auteur = " + this.auteur);
	}

[Napalm51]

J'avais raison donc, merci dingoth

[Mooneer]

Autant pour moi ^^
Mais le plus simple étant toujours de différencier le nom des varibles

[dingoth]

Le plus simple pour éviter ce problème, oui.

Le plus simple pour bien comprendre le code, cela dépend des cas.

Dans la doc, si on dit qu'on a une variable de classe appelée auteur, il est plus aisé de comprendre qu'on la réécrit en écrivant setAuteur (String auteur).

Mais bon, chacun a son point de vue. Personnellement, je n'écris this.auteur = auteur; que dans un constructeur. Mes setters ont une autre règlementation.