Discussion :

[IBKAM]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
private void classementCroissante(){
        for(int j=0; j== UserName.size();j++) {
 
 
            for (int i=j+1; i== ScoreUser.size();i++) {
                if (ScoreUser.get(j).compareTo(ScoreUser.get(i))<0) {
                    score =  ScoreUser.get(j);                       user= UserName.get(j);
 
                    ScoreUser.set(j, ScoreUser.get(i));     UserName.set(j,UserName.get(i));
                    ScoreUser.set(i, score);                        UserName.set(i, user);
                }
 
            }
        }

Quand je fait appel à cette fonction, le programme tourne mais rien ne change alors que ne utiisant collections.sort le trie marche bien

[thelvin]

Hello,

je te suggère de revoir l'usage de == dans tes boucles.

Dans une boucle, on fait des comparaisons d'inégalité, pas des comparaisons d'égalité. Regarde à nouveau toutes les boucles que tu as faites jusqu'à présent. Elles ne ressemblent pas à ça.

(Au passage, si la raison pour laquelle tu n'utilises pas simplement sort() et que tu as décidé d'écrire le tri toi-même, c'est que tu as deux listes à trier, d'une part les scores et d'autre part les noms des joueurs... La bonne chose à faire était de faire une classe Player qui contient le score et le nom du joueur, et de n'avoir qu'une seule liste, une liste qui contient les Player. Il n'y a plus qu'à la trier par score.)

[IBKAM]

Merci cher Mr telvin pour votre reponse,
J'ai réussit a faire mon tri sans utilisé collections.sort, voilà:
J'ai créé u e class score, un getter et un setter permettant de chargé les données.
Ensuite j'ai reprit ma fonction classement en respectant les obligations comme tu les a notifié ==
L'objectif est que en même temps quand je permute les score je permute aussi les nom enfin de garder à chacun des nom sin score.
Bien sa marche. Et j'ai implémenté la méthode comparable. Merci bien

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
private void classementCroissante(){
        for(int j=0; j<ScoreUser.size();j++) {
 
 
            for (int i=j+1; i<ScoreUser.size();i++) {
                if (ScoreUser.get(j).compareTo(ScoreUser.get(i))<0) {
                    score =  ScoreUser.get(j);                       user= UserName.get(j);
 
                    ScoreUser.set(j, ScoreUser.get(i));     UserName.set(j,UserName.get(i));
                    ScoreUser.set(i, score);                        UserName.set(i, user);
                }
 
            }
        }

Mais j'aurais aimer que vous le montrez votre solution , là ou il faut créé une classe player et trié par score.
Merci

[thelvin]

L'objectif est que en même temps quand je permute les score je permute aussi les nom enfin de garder à chacun des nom sin score.

Oui, et je t'ai dit que le plus simple pour faire ça c'est d'avoir une seule liste au lieu d'en avoir deux.

Mais bon, tu as tout le temps de progresser.