Discussion :

[julien.63]

salut,
J'ai une classe qui stocke des objets dans un vecteur.
Mais j'ai un problème à l'exécution.

Voici le code simplifié, ça plante sur le addElement. N'est ce pas la bonne méthode pour ajouter un objet?

Code :

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

1
2
3
4
5
6
7
8
9
10
public abstract class AgendaTel {
 
	private Vector<AdresseTel> carnet;
 
	public void newContact(String n, String p, String ph){
		AdresseTel a = new AdresseTel(n,p,ph);
		System.out.println(a.getName() + " " + a.getFirstName() + " " + a.getPhone());
		carnet.addElement(a);
	}
}

[mavina]

Salut,

Je te conseille de remplacer Vector par ArrayList et d'utiliser la méthode add de cette derniere.

L'avantage d'ArrayList est qu'elle est plus récente et non synchronisée.

F.

[Amine_sas]

Salut,

L'avantage d'ArrayList est qu'elle est plus récente et non synchronisée.

+1
et plus rapide par conséquent.

[julien.63]

merci pour vos conseils, j'ai modifié le code et noté pour la prochaine fois.
Mais j'ai toujours le même problème.
En fait j'ai une java.lang.NullPointerException

Code :

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

1
2
3
4
5
6
7
8
9
10
public abstract class AgendaTel {
 
	private ArrayList<AdresseTel> carnet;
 
	public void newContact(String n, String p, String ph){
		AdresseTel a = new AdresseTel(n,p,ph);
		System.out.println(a.getName() + " " + a.getFirstName() + " " + a.getPhone());
		carnet.add(a);
	}
}

[jibbi]

Salut

est-ce que la collection carnet à été initialisé ?

vérifie les paramètres si n, p ou ph == null.

[mavina]

Salut,

En effet, il faut initialiser ton ArrayList.

Par souci de performances et si tu sais que tu vas mettre beaucoup d'éléments dedans, initialise le via le constructeur prenant une taille de base et mets là assez grande : http://javasearch.developpez.com/j2s...ArrayList(int)... En effet, le resize interne de la liste est relativement couteux, donc si il est fait régulièrement, il peut entacher les performance de ton application.

Sinon, le constructeur par défaut l'initialise à 10.

et plus rapide par conséquent.

Ceci est de moins en moins vrai, plus on avance dans les versions de Java et moins la synchronisation est couteuse. Le fait est que Vector soit synchronisé n'est pas la seule raison de ne pas l'utiliser : elle est vieille (elle date de la version 1.1 de Java, et donc n'est pas apparue avec l'API des collections. Elle est venue "se greffer" à cet API. Il est préférable si tu veux une liste synchronisée d'appeller Collections.synchronizedList(List), ce qui te permet d'avoir la même liste en synchronisée, et donc d'éviter d'utiliser Vector.
Vector est encore dans l'API par souci de rétro-compatibilité : beaucoup (énormémént même) de programmes utilisent cette classe au lieu d'utiliser ArrayList, j'ai moi même vu dans des programmes 1.4 des programmeurs utiliser Vector sans contexte multi-Threadés.

F.

[julien.63]

Merci pour vos conseils,
En fait, en voulant initialiser ArrayList, je me suis rendu compte que j'avais un peu oublier de créer l'instance....
Je déclarais la variable seulement....

Donc ça marche maintenant. Merci pour toutes ces infos.

@+

Julien