Tableau Java

bonsoir les amis j'ai un petit souci:

bon soit un objet Algogéne.convertseq(d, o, i, k); qui contient k tableau a chaque foie il récupère des valeurs différentes selon K.

et voici mon code :

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
sym=Algogéne.convertseq(d, o, i, k);
    for( i=1;i<394;i++){
        if(k==1){
            tal1[i]=sym[i].getName();
 
        }
        if(k==2){
            tal2[i]=sym[i].getName();
 
        }
        if(k==3){
            tal3[i]=sym[i].getName();
 
        }
        if(k==4){
            tal4[i]=sym[i].getName();
 
        }
 
        if((k<=2)){
 
        }
 
                System.out.print(tal1[1]);
 
    }

et l'affichage de l’exécution:

Code:

1
2
3
4
5
6
 
1cytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosinecytosine 
2nullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnull 
3nullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnull 
4nullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnullnull 
GÉNÉRATION TERMINÉE (durée totale* 4 secondes)

pourquoi il affiche les autre ligne null normalement il affiche seulement la première ligne

Aucune idée. Le code que tu nous as montré ne permet pas de comprendre de quoi tu parles.

Si tu veux savoir pourquoi plusieurs lignes sont affichées alors qu'il ne devrait y en avoir qu'une, il suffit de débugguer. Par exemple mettre un println() ou un breakpoint devant chaque affichage de ligne, ce qui permet de remonter à ce qui a conduit à l'affichage de cette ligne.

Ne pouvant t'aider pour le moment sur le font de ton problème. Je voudrais de faire quelles remarques sur ton code, qui t'aideront peut-être à l'avenir.
En premier lieu, je pense qu'il est possible d'introduire une structure switch à la place de tes "if"
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 switch (k) { case 1: tal1[i] = sym[i].getName(); break; case 2: tal2[i] = sym[i].getName(); break; case 3: tal3[i] = sym[i].getName(); break; case 4: tal4[i] = sym[i].getName(); break; default: break; }
Je ne sais pas si les "tal"*1 sont déclarés localement ou non. Mais il est préférable d'avoir un tableau de tableaux que des noms de variables incrémentale.
Si on suppose que tes "tals" ont la déclaration suivante :

Code:

String[] tal1, tal2, tal3, tal4;

Il est possible de faire la déclaration suivante :

Code:

String[][] tals;

Et de transformer le switch précédent en :

Code:

tail[k][i] = sym[i].getName();

Ce qui est beaucoup plus simple, d'après moi.

Il est aussi possible que ton rendu ne soit pas celui que tu voudrait à cause de cette ligne (se trouvant dans le for each) :

Code:

System.out.print(tal1[1]);

D'après moi, elle devrait ressemblé à :

Code:

System.out.print(tail[k][i]);

En effet, si tu passe un k différent de 1, alors le tableau affiché n'est pas celui rempli.
Et le fait que tu ai systématiquement "cytosine", me fait penser que tu n'affiche pas la valeur courante mais la seconde valeur du tableau.*2

Cordialement,
Patrick Kolodziejczyk.

Note :
1. Change tes noms de variable en des noms explicites. C'est une manière de s'y retrouver et de pouvoir communiquer avec son code.
2. Les tableaux commence à partir de l'index 0 et non 1.

Citation:

Envoyé par kolodz

Note :
1. Change tes noms de variable en des noms explicites. C'est une manière de s'y retrouver et de pouvoir communiquer avec son code.
2. Les tableaux commence à partir de l'index 0 et non 1.

+1
Si tu veux parcourir tout le tableau, utilise son longueur mais pas le borner en hardcoded

Code:

for( i=0;i<sym.length;i++)

ou si tu veux limiter le parcours jusqu'à l'indice 394, tu dois éviter l'IndexOutOfBoundsException dans le cas où la taille du tableau est inférieur à 394

Code:

for( i=0 ; i<sym.length && i<394 ; i++)

A+.

Celà est très moche et il manque un truc.

Citation:

Envoyé par anoir19

Code:

1
2
3
4
5
 
        if((k<=2)){
 
        }        
                System.out.print(tal1[1]);

voulais-tu faire un

Code:

1
2
3
4
5
6
 
        if((k<=2)){
 
        }else{        
                System.out.print(tal1[1]);   
        }

?