Discussion :

[anoir19]

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 :

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
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 :

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

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

[thelvin]

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.

[kolodz]

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 :

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
		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 :

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

String[] tal1, tal2, tal3, tal4;

Il est possible de faire la déclaration suivante :

Code :

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

String[][]  tals;

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

Code :

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

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 :

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

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

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

Code :

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

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.

[andry.aime]

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 :

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

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 :

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

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

A+.

[supergeoffrey]

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

Code :

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

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

voulais-tu faire un

Code :

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

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

?