Discussion :

[abysr]

Bonjour à tous j'aurai besoin de votre avis sur une "bonne pratique" à avoir,
il est mieux de faire comme ça :

Code :

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

1
2
3
4
5
6
 
        Utilisateur utilisateur;
        .
        .
        .
        utilisateur = new Utilisateur();

ou, comme ça :

Code :

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

Utilisateur utilisateur = new Utilisateur();

ou autre???

[guatto]

Bonsoir abysr,

étant "débutant" en Java ma réponse pourrait te paraitre bizarre, mais c'est juste une question de manière de coder, et je ne pense pas que ça puisse affecter la mémoire, mais j'ai remarqué que parfois en utilisant Netbeans, il me demande de déclarer, avant d'instancier. comme le 1er exemple :

Code :

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

1
2
3
4
 
Utilisateur utilisateur;   // déclaration.
 
 utilisateur = new Utilisateur();  // instanciation.

-Pour ma part le 1er c'est mieux et me parait plus clean !

[francky74]

Bonjour,

cela dépend de ton besoin :
si tu déclare ton instance à l'intérieur d'une méthode, elle ne vivra que dans cette méthode et te rendra null si tu l'appelle en dehors.
par exemple :

Code :

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

1
2
3
public void maMéthode() {
	MaClasse maclasse = new Maclasse();
}

Si tu déclare ton instance en dehors d'une méthode, et que tu l'instancie dans le constructeur, cette instance sera valable pour ton la durée de vie de l'instance construite ....

par exemple :

Code :

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

1
2
3
4
5
6
public class MaClasse() {
	MaClasse maclasse;
	public MaClasse() {
		maclasse = new Maclasse();
	}
}

et cela vaut aussi pour les try catch !!

exemple :

Code :

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

1
2
3
4
5
6
7
8
9
public void maMethode{
		try {
			MaClasse maclasse = new MaClasse();
		} catch (ArrayIndexOutOfBoundsException e) {
			e.printStackTrace();
		}
 
		System.out.println(maclasse); //ici maclasse vaut null car en dehors du try / catch
}

Pour pouvoir utiliser l'instance en dehors du try / catch, il faut plutôt faire cela (si tu en as besoin en dehors du try / catch) :

Code :

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

1
2
3
4
5
6
7
8
9
10
public void maMethode{
		MaClasse maclasse;
                try {
			maclasse = new MaClasse();
		} catch (ArrayIndexOutOfBoundsException e) {
			e.printStackTrace();
		}
 
		System.out.println(maclasse.toString); //ici maclasse ne vaut plus null
}

Voilà, j'espère ne pas raconter de bêtises et j'espère que l'ont me corrigera si je dit faux !

[thelvin]

Mouais, pas très convaincant tout ça -_-°.

En termes d’exécution du programme, bien sûr, ça ne change rien : les deux donnent la même chose après compilation.

En termes de propreté, par contre, il vaut mieux viser le deuxième. Ce n'est pas toujours possible, parfois on ne peut pas, mais il vaut mieux viser le deuxième et se rabattre sur le premier quand le deuxième ne convient pas.
En fait, le premier, avec ses ". . ." semble indiquer qu'il y a beaucoup de lignes entre la déclaration et l'initialisation, et ça... Ce n'est jamais intéressant. Même si on fait des lignes séparées, elles doivent être groupées ensemble.

La raison de viser le deuxième, c'est que, de manière générale, il vaut mieux commencer à créer des variables, au plus proche de l'endroit où elles vont servir. Donc dans le même genre, si une variable ne sert que dans une boucle, il vaut mieux la déclarer dans cette boucle qu'en dehors.
Pourquoi, eh bien tout simplement parce qu'en déclarant les variables ailleurs que là où elles vont servir, on se rend capable de s'en servir ailleurs. C'est à dire à d'autres endroits que le bon endroit. C'est à dire au mauvais endroit. Et ainsi naissent les bugs, plus difficiles à retrouver parce qu'on a pas remarqué que cette variable était déclarée et utilisée ailleurs que là où elle le devait.
C'est pas plus compliqué que ça.

C'est le principe même des langages à typage statique : s'arranger pour ne pas pouvoir faire ce qu'il ne faut pas faire, histoire de gagner du temps en évitant certaines erreurs.

Pour pouvoir utiliser l'instance en dehors du try / catch, il faut plutôt faire cela (si tu en as besoin en dehors du try / catch) :

Code :

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

1
2
3
4
5
6
7
8
9
10
public void maMethode{
		MaClasse maclasse;
                try {
			maclasse = new MaClasse();
		} catch (ArrayIndexOutOfBoundsException e) {
			e.printStackTrace();
		}
 
		System.out.println(maclasse.toString); //ici maclasse ne vaut plus null
}

Voilà, j'espère ne pas raconter de bêtises et j'espère que l'ont me corrigera si je dit faux !

Ben, en fait, cela ne marche pas du tout -_-°. La variable ne vaut pas null, elle n'a jamais valu null.
Mais en dehors du try/catch elle n'est pas garantie d'avoir été initialisée (la ligne maclasse = new MaClasse(); peut échouer à cause d'une Exception.) Si elle n'est pas garantie d'être initialisée, elle ne peut pas être lue, et donc ce code ne compile pas.

[joel.drigo]

Code :

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

1
2
3
4
5
6
7
8
9
public void maMethode{
		try {
			MaClasse maclasse = new MaClasse();
		} catch (ArrayIndexOutOfBoundsException e) {
			e.printStackTrace();
		}
 
		System.out.println(maclasse); //ici maclasse vaut null car en dehors du try / catch
}

On peut même ajouter que le code ci-dessus ne compile pas non plus : la variable maclasse existe dans le bloc du try et pas en dehors. Il ne vaut pas null en dehors, ni autre chose, elle n'existe simplement pas en dehors du bloc, donc valloir quelque chose n'a aucun sens (à moins qu'il ne s'agisse d'une variable de la classe où se trouve la méthode maMethode(), auquel cas, ce ne sont pas les mêmes variables (elles n'ont en commun que leur nom, et c'est une pratique à éviter pour éviter toutes difficultés, à déboguer le code, liées à la confusion évidente entre les deux).

[tchize_]

Allez, je démonte les troisième bout de code.

cette instance sera valable pour ton la durée de vie de l'instance construite ....

par exemple :

Code :

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

1
2
3
4
5
6
public class MaClasse() {
	MaClasse maclasse;
	public MaClasse() {
		maclasse = new Maclasse();
	}
}

En admettant que ça compile (c'est à dire en corrigeant le problème du C majuscule dans Maclasse), ce code va juste partir en StackOverflowError. On ne s'instancie pas soi même dans son constructeur!

[FaridM]

Bonjour,

cela dépend de ton besoin :
si tu déclare ton instance à l'intérieur d'une méthode, elle ne vivra que dans cette méthode et te rendra null si tu l'appelle en dehors.
par exemple :

Code :

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

1
2
3
public void maMéthode() {
	MaClasse maclasse = new Maclasse();
}

Et l'accent dans le nom de la méthode on en parle? (Oui Java l'autorise, mais bon, tant qu'on est dans les bonnes pratiques...)