erreur bete de java

**r0nsync** · 23/08/2007, 10h38

j ai une erreur je sais que c est bête mais j arrive pas la trouve en faite, j ai
un tableau d objet Personne(Id, Name)

Code :

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

1
2
3
4
5
6
7
8
9
10
int index=0;
            Personne[] TabPersonne = new Personne[10000];
            Personne tempPersonne = new Personne();
            tempPersonne.setName("HELP");
            tempPersonne.setId("MERCI");
 
            TabPersonne[index]=tempPersonne;
            index++;
tempPersonne.setNAme("****");
TabPersonne[index]=tempPersonne;

après j ai mon tableau remplie que par ******* ???????????????????????????????????

Code :

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

1
2
3
4
5
int i=0;
while(i<TabPersonne.length){
system.out(TabPersonne[i].getName);
i++
}

*****
*****

**MarkH** · 23/08/2007, 10h43

Salut r0nsync,

à première vue, ton erreur est due au fait que tu ne crées qu'une seule instance de Personne que tu affectes à tempPersonne.

Ensuite, tu affectes à tous les éléments de ton tableau cette même instance.
Pas étonnant que lorsque tu modifies cette instance, c'est l'ensemble des éléments du tableau qui sont apparamment modifiés.

Pour t'en sortir, il te faudrait écrire autant de fois

Code :

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

1
2
 
temPersonne=new Personne();

que tu as de personnes à caser dans le tableau.

Mes 2€ centimes.

MarkH

**r0nsync** · 23/08/2007, 10h54

en faite moi je veux stock les tempPersonne dans mon tableau afin de les utilises plus tard .

j ai essayé de crée une instance de personne a chaque fois mais j ai le meme résultat

**eclesia** · 23/08/2007, 11h05

Code :

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

1
2
3
4
5
6
7
8
9
 
Personne[] TabPersonne = new Personne[10000];
 
for(int i=0;i<10000;i++){
            Personne tempPersonne = new Personne();
            tempPersonne.setName("HELP " + i);
            tempPersonne.setId("MERCI " + i); 
            TabPersonne[i]=tempPersonne;
}

... ce n'est pas une bonne solution (surtout un tableau de 10000)
utilises plutot une liste :

Code :

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

1
2
 
ArrayList<Personne> personnes = new ArrayList<Personne>();

**MarkH** · 23/08/2007, 11h08

Pour fixer les choses, voici un bout de code qui remplit un tableau de trois personnnes et qui tente de rester proche de ce que tu as écrit :

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
 
Personne[] tabPersonne=new Personne[3];
 
int index=0;
Personne tempPersonne=new Personne();
tempPersonne.setName("Dupont");
tempPersonne.setId("12323");
tabPersonne[index]=tempPersonne;
 
index++;
tempPersonne=new Personne();
tempPersonne.setName("Durand");
tempPersonne.setId("45654");
tabPersonne[index]=tempPersonne;
 
index++;
tempPersonne=new Personne();
tempPersonne.setName("Martin");
tempPersonne.setId("567657");
tabPersonne[index]=tempPersonne;
 
index=0;
while (index<tabPersonne.length) {
   System.out.println(tabPersonne[index].getName());
   index++;
}

Je doute fort que l'affichage produise trois fois le même nom....

MarkH

P.S. (pour les puristes Java qui liraient ce code) : dans une *vraie* appli, je n'écrirais sûrement pas ça comme ça

**r0nsync** · 23/08/2007, 11h10

je suis oblige d utilise java 1.4 donc je pence que je ne peux pas utilise les List

si non enfaite je ne connais pas la taille de mon TAbpersonne a lavance donc je peux pas faire un for ???
vous aurez pas une autre solution

**Deadpool** · 23/08/2007, 11h15

Les Lists sont là depuis Java 1.2 alors tu peux les utiliser sans problèmes.

D'autant plus si tu ne connais pas le nombre d'instances de Personnes que tu dois stocker.

**crawling5** · 23/08/2007, 11h17

Il me semble que les ArrayList existent bien dans le java 1.4.

Parce que lorsque tu ne connais pas le nombre d'éléments que tu dois sauvegarder, je te recommande les listes (ou les hashmap si tu dois pouvoir retrouver facilement un de ces objets).

**r0nsync** · 23/08/2007, 11h24

Personne[] Personnexml = new Personne[10000]

voila mon Code, enfaite il est un peux plus compliqué
j ai des personnes que je lire dans un fichier xml et je veux les stock ds un tableau

List listPersonne = XPath.selectNodes(document, "//value");
Iterator i = listPersonnes.iterator();
while (i.hasNext()) {
Element courant = (Element) i.next();
Personne temp_Personne = new Personne();
temp_Personne.setMPersonneName(courant.getChildText("name"));
Personnexml[indexPersonne]=temp_Personne;
System.out.println( Personnexml[indexPersonne].getMPersonneName()+
"---"+Personnexml[0].getMPersonneName());
indexPersonne=indexPersonne+1;

}

et j ai comme affichage

dadazda---dadazda
ddddddd---ddddddd
JPC_99---JPC_99
TOTO4---TOTO4
TOTO5---TOTO5

**r0nsync** · 23/08/2007, 11h28

voila l ereur que j ai quand j utilise ArrayList comme ça
"""ArrayList<Personne> list_personne = new ArrayList<Personne>();""""

Erreur de syntaxe. Les types paramétrés sont disponibles uniquement si le niveau du source est 5.0.

**crawling5** · 23/08/2007, 11h40

Ces ArrayList n'existent que depuis la 5.0 c'est vrai.
Donc il faut utiliser ceci :

Code :

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

ArrayList listePersonne = new ArrayList();

Pour ajouter une personne :

Code :

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

listePersonne.add(personne);

Et pour récupérer un objet :

Code :

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

Personne pers = (Personne)listePersonne.get(0);

**r0nsync** · 23/08/2007, 12h08

j ai change mon code et j ai toujour le meme blem
ArrayList list_Personne = new ArrayList()
List listPersonnes = XPath.selectNodes(document, "//value");
Iterator i = listPersonnes.iterator();
while (i.hasNext()) {
Element courant = (Element) i.next();
Personne temp_Personne = new Personne();
temp_Personne.setMPersonneName(courant.getChildText("name"));
list_Personne.add(temp_Personne);
System.out.println(temp.getMPersonneName()+" : "+indexPersonne );
indexPersonne=indexPersonne+1;

}

Iterator j = list_Personne.iterator();
while(j.hasNext() ){
Personne temp = (Personne)j.next();
System.out.println("personnexml "+temp.getMPersonneName()+" de j :");
}

resultat
dadazda : 0
ddddddd : 1
....
TOTO4 : 55
TOTO5 : 56

et ????

personnexml TOTO5 de j :0
personnexml TOTO5 de j :1
...............
personnexml TOTO5 de j :55
personnexml TOTO5 de j :56

**crawling5** · 23/08/2007, 12h44

Ca doit venir de la lecture de ton xml alors.
T'as a débugé ton programme ? Pour essayer de voir ce qui est réellement injecté dans ton objet Personne ?

**elmor** · 23/08/2007, 13h57

Hum... je dois avouer que ton problème me laisse perplexe

Le seul cas que je vois possible est que les champs de ta classe Personne sont statiques.

Est-ce que tu pourrais nous envoyer le code de la classe Personne.

Si le code est :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
Personne {
static String name;
Personne(){}
 
public void setName (String n){
this.name=n;
}
 
public String getName (){
return name;
}
}

Alors ton problème est normal, il suffit d'enlever le static pour que tout fonctionne

**r0nsync** · 23/08/2007, 14h32

BRAVo

chapo

en fait sa change koi pour le code !!!!!
je voi pas ou il est le blem

**Caster** · 23/08/2007, 15h14

en mettant ta variable en static, ce sera la meme pour toutes tes instances, hors ce n'est pas ce que tu veux et c'est pourquoi tu avais toujours le meme nom de Personne

**elmor** · 24/08/2007, 09h12

Comme l'a dit Caster, en object on distingue deux types de variables:

Les variables de classes
les variables d'instances

Les variables de classes sont stockées au niveau de la classe et sont donc communues à toutes instances de la classe.

Par contre les variables d'instances ne sont visibles que par l'instance elle même. Les autres instances de la classe ne les voient pas.

Pour en revenir au "static", lorsque tu mets static devant le nom d'un membre d'une classe. Ce membre est un "variable de la classe".

J'espère que c'est clair.

**yarf** · 27/08/2007, 14h49

Envoyé par eclesia

Code :

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

1
2
3
4
5
6
7
8
9
 
Personne[] TabPersonne = new Personne[10000];
 
for(int i=0;i<10000;i++){
            Personne tempPersonne = new Personne();
            tempPersonne.setName("HELP " + i);
            tempPersonne.setId("MERCI " + i); 
            TabPersonne[i]=tempPersonne;
}

... ce n'est pas une bonne solution (surtout un tableau de 10000)
utilises plutot une liste :

Code :

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

1
2
 
ArrayList<Personne> personnes = new ArrayList<Personne>();

Pourquoi est-ce que ce n'est pas une bonne solution d'utiliser un tableau de 10000 éléments plutôt qu'une liste ?

**adiGuba** · 27/08/2007, 14h58

Salut,

Envoyé par yarf

Pourquoi est-ce que ce n'est pas une bonne solution d'utiliser un tableau de 10000 éléments plutôt qu'une liste ?

Les tableaux ne sont pas manipulable aussi facilement que les List, et à moins de n'avoir qu'un nombre d'élément fixe ou d'autre impératif pour utiliser un tableau (parce que doit le passer à une méthode par exemple), il est généralement plus simple d'utiliser une List...

Surtout avec les Generics qui simplifie leurs utilisations...

Je rappelle quand même :

Que les ArrayList sont implémenté en utilisant des tableaux, et propose donc grosso-modo les mêmes performances qu'un tableau.
Qu'il est possible de définir la taille initial d'une List pour éviter de trop multiple redimensionnement lorsqu'on connait la taille finale (la plupart des implémentations propose un tel constructeur) :
Code : Sélectionner tout - Visualiser dans une fenêtre à part
```
ArrayList<Personne> personnes = new ArrayList<Personne>( 10000 );
```

a++

erreur bete de java

Langage Java

Discussions similaires

Partager

Partager