je n'arrive à extraire un élément d'un ArrayList j'obtiens toujours le dernier élément

Bonjour

j'ai créé une classe Donne :

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 27 28 29 30 31 32 33

public class Donne {   //proprietes private static Integer numeroDonne; private static String donneur; private static String vulnerable;   public Donne(Integer numeroDonne, String donneur, String vulnerable) { this.numeroDonne = numeroDonne; this.donneur = donneur; this.vulnerable = vulnerable; }   public String getVulnerable() { return vulnerable; } public void setVulnerable(String vulnerable) { Donne.vulnerable = vulnerable; } public String getDonneur() { return donneur; } public void setDonneur(String donneur) { Donne.donneur = donneur; } public Integer getNumeroDonne() { return numeroDonne; } public static void setNumeroDonne(Integer numeroDonne) { Donne.numeroDonne = numeroDonne; } }

---

je crée une ArrayList de plusieurs donnes dans une classe Mouvement:

Code:

---

1 2 3 4 5 6 7 8

public static ArrayList<Donne> lesDonnesMouvement; public static ArrayList<Donne> getLesDonnesMouvement() { return lesDonnesMouvement; } public static void setLesDonnesMouvement(ArrayList<Donne> lesDonnesMouvement) { Mouvement.lesDonnesMouvement = lesDonnesMouvement; }

---

Dans la classe Controle j'extrait les donnes d'un JsonArray :

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 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82

public void afficheTournoi(String strMessage1, String strMessage2){   try { //transforme les données de message1 en un tableau : leMouvement JSONArray leMouvementJsonArray = new JSONArray(strMessage1); Log.d("controle.affiTourn", "le mvnt_numero "+leMouvementJsonArray.toString()); for (int k = 0; k < leMouvementJsonArray.length(); k++) { JSONObject leMouvement = new JSONObject(leMouvementJsonArray.get(k).toString()); mvnt_numero = leMouvement.getInt("mvnt_numero"); Mouvement.setMvnt_numero(mvnt_numero); table_numero = leMouvement.getInt("table_numero"); Mouvement.setTable_numero(table_numero); equipeNS = leMouvement.getInt("equipe_NS"); Mouvement.setEquipeNS(equipeNS); equipeEO = leMouvement.getInt("equipe_EO"); Mouvement.setEquipeEO(equipeEO); nomJ1 = leMouvement.getString("nomJ1"); Tournoi.setNomJ1(nomJ1); prenomJ1 = leMouvement.getString("prenomJ1"); Tournoi.setPrenomJ1(prenomJ1); nomJ2 = leMouvement.getString("nomJ2"); Tournoi.setNomJ2(nomJ2); prenomJ2 = leMouvement.getString("prenomJ2"); Tournoi.setPrenomJ2(prenomJ2); } } catch (JSONException e) { Log.d("controle", "***************" + "conversion JSON impossible " + e.toString()); // e.printStackTrace(); } try { //transforme les données de message2 en un tableau : leMouvement JSONArray lesDonnesMouvementJsonArray = new JSONArray(strMessage2); Log.d("controle.affiTourn", "les donnes du Mouvemengt "+ lesDonnesMouvementJsonArray.toString()); for (int k = 0; k < lesDonnesMouvementJsonArray.length(); k++) { JSONObject donneMouvementOject = new JSONObject(lesDonnesMouvementJsonArray.get(k).toString());   numeroDonne = donneMouvementOject.getInt("numero_donne"); donneur = donneMouvementOject.getString("donneur"); vulnerable = donneMouvementOject.getString("vulnerable"); //crée une donne du mouvement Donne donneMouvement = new Donne(numeroDonne, donneur, vulnerable); Log.d("control.affich","une donne du mouvement index : "+k+" ** "+donneMouvement+" ***numeroDonne : "+numeroDonne); Integer numero3Donne = donneMouvement.getNumeroDonne(); Log.d("control.affich","une donne du mouvement index : "+k+" ** "+donneMouvement+" ***numero3Donne : "+numero3Donne);   //ajoute les donnes à la liste des donnes du Mouvement lesDonnesMouvement.add(donneMouvement); Mouvement.setLesDonnesMouvement(lesDonnesMouvement); Integer numero0Donne = lesDonnesMouvement.get(0).getNumeroDonne(); Log.d("control.affich","une donne du mouvement index : "+k+" ** "+donneMouvement+" **numero0Donne : "+numero0Donne); Donne donne0 = lesDonnesMouvement.get(0); Integer numero0 = donne0.getNumeroDonne(); Log.d("control.affich","une donne0 du mouvement index : "+0+" ** "+donne0+" **numero0 : "+numero0); } } catch (JSONException e) { Log.d("controle", "***************" + "conversion JSON impossible " + e.toString()); // e.printStackTrace(); }   Log.d("control.affich", "mouvement.getdonne a jouer : "+Mouvement.getDonneAJouer()); if(Mouvement.getDonneAJouer()!=0){ Mouvement.setDonneAJouer(Mouvement.getDonneAJouer()+1); ; } Log.d("control.affich", "mouvement.getdonne a jouer : "+Mouvement.getDonneAJouer()); lesDonnesMouvement = Mouvement.getLesDonnesMouvement(); Log.d("control.affichetour","lesDonnesMouvements : "+lesDonnesMouvement.toString());   for (int k = 0; k < lesDonnesMouvement.size(); k++) { Integer numero2Donne = lesDonnesMouvement.get(k).getNumeroDonne(); Log.d("control.affichetour", "numero2Donne : " + numero2Donne.toString()); Donne uneDonne = Mouvement.getUneDonneMouvement( k); Integer numero5Donne = uneDonne.getNumeroDonne(); Log.d("control.affichetour", "numero5Donne : " + numero5Donne.toString()); } Mouvement.setDonneAJouer(numeroDonne);   Intent intent = new Intent(contexte, MouvementActivity.class); intent.setFlags(Intent.FLAG_ACTIVITY_NEW_TASK | Intent.FLAG_ACTIVITY_CLEAR_TASK); Log.d("controle.affiTourn", "le mvnt_numero => MouvementActivity"); contexte.startActivity(intent); }

---

Le problème je n'arrive pas à avoir le numéro de la première donne de l'arrayList "lesDonnesMouvement", une fois l'arraylist constitué, j'obtiens toujours le numéro de la dernière donne enregistrée :

Code:

---

1 2 3 4 5 6 7 8

//rappel du code ci-dessus for (int k = 0; k < lesDonnesMouvement.size(); k++) { Integer numero2Donne = lesDonnesMouvement.get(k).getNumeroDonne(); Log.d("control.affichetour", "numero2Donne : " + numero2Donne.toString()); Donne uneDonne = Mouvement.getUneDonneMouvement( k); Integer numero5Donne = uneDonne.getNumeroDonne(); Log.d("control.affichetour", "numero5Donne : " + numero5Donne.toString()); }

---

alors que lors de la création les numéro de la donne sont correct :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

//rappel du code ci-dessus for (int k = 0; k < lesDonnesMouvementJsonArray.length(); k++) { JSONObject donneMouvementOject = new JSONObject(lesDonnesMouvementJsonArray.get(k).toString()); numeroDonne = donneMouvementOject.getInt("numero_donne"); donneur = donneMouvementOject.getString("donneur"); vulnerable = donneMouvementOject.getString("vulnerable"); //crée une donne du mouvement Donne donneMouvement = new Donne(numeroDonne, donneur, vulnerable); Log.d("control.affich","une donne du mouvement index : "+k+" ** "+donneMouvement+" ***numeroDonne : "+numeroDonne); Integer numero3Donne = donneMouvement.getNumeroDonne(); Log.d("control.affich","une donne du mouvement index : "+k+" ** "+donneMouvement+" ***numero3Donne : "+numero3Donne);   //ajoute les donnes à la liste des donnes du Mouvement lesDonnesMouvement.add(donneMouvement);

---

Du bridge!

Ça tombe bien, j'adore ce jeu.

Je n'ai pas fait d'Androïd depuis longtemps, mais des choses me semblent étranges.

Pour commencer, les propriétés sont propre à la donne.

Elles ne peuvent donc pas être static.

Soit:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

public class Donne {   //proprietes private final Integer numeroDonne; private final String donneur; private final String vulnerable;   public Donne(Integer numeroDonne, String donneur, String vulnerable) { this.numeroDonne = numeroDonne; this.donneur = donneur; this.vulnerable = vulnerable; }   //Getter seulement

---

Mieux, toujours fonctionnellement parlant, on n'a pas n'importe quoi en joueur et team.
Je propose d'utiliser les enums ( https://www.jmdoudoux.fr/java/dej/chap-enums.htm ):

Code:

---

1 2 3 4

public enum Joueur{ NORD,SUD,EST,OUEST; }

---

Code:

---

1 2 3 4

public enum Equipe{ NORD_SUD,EST_OUEST; }

---

Ce qui donnerait:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

public class Donne {   //proprietes private final Integer numeroDonne; private final Joueur donneur; private final Equipe vulnerable;   public Donne(Integer numeroDonne, Joueur donneur, Equipe vulnerable) { this.numeroDonne = numeroDonne; this.donneur = donneur; this.vulnerable = vulnerable; }   //Getter seulement

---

Si on est en Java 17 ou +, je recommande d'utiliser les records ( https://www.jmdoudoux.fr/java/dej/chap-records.htm ):

Code:

---

1 2	public record Donne(Integer numeroDonne, Joueur donneur, Equipe vulnerable){}

---

Pour le mouvement, en utilisant les record:

Code:

---

1 2	public record Mouvement(List<Donne>lesDonnesMouvement){}

---

Notez que j'utilise les interfaces (le contrat) et non les implémentations (ici, ArrayList).
Là aussi, le static est un non sens.

Je te laisse faire ta boucle, mais là aussi, je recommande l'iteration/for each.
Soit par exemple:

Code:

---

1 2 3 4 5

for(Donne donne:lesDonnesMouvement){ //Par exemple, si on a des record, toString est de base redéfinie System.out.println(donne.toString()); }

---

Normalement, tu devrais voir la bonne donne au fur et à mesure.
Pour le JSON,il existe des librairies:
https://www.baeldung.com/java-org-json

Bonjour

Merci pour ta réponse.

Citation:

---

Mieux, toujours fonctionnellement parlant, on n'a pas n'importe quoi en joueur et team.

---

je ne vois pas bien ce que ça m'apporte ? le donneur et l'équipe c'est réglé.

Citation:

---

Notez que j'utilise les interfaces (le contrat) et non les implémentations (ici, ArrayList).
Là aussi, le static est un non sens.

---

je ne comprends les interfaces(le contrat) ?

J'ai beaucoup recopié d'exemples qui fonctionnent pour ça que j'ai mis les "static" (sans trop bien comprendre les différences).

Mais je crois que mon problème n'est pas là!

Une fois l'ArrayList constitué normalement

Code:

---

1 2	uneDonne = lesMouvementsDonne.get(0);

---

me donne bien la donne N° 0
mais lorsque je veux extraire le numéro de "uneDonne"

Code:

---

1 2	numeroDonne = uneDonne.getNumeroDonne ;

---

ça me donne toujours le numéro de la dernière donne enregistrée dans l'Arraylist.

c'est pour ça que dans mon code j'ai fait plein de Log.d à toutes les étapes pour voir les résultats.
pendant la constitution de l'arrayList pas de problème mais si je veux extraire après l'arrayList toujours le problème.

Je suis d'accord avec toi que mon code n'est pas très académique

Je viens de prendre le temps pour voir l'explication de "static"

Citation:

---

C'est une variable qui appartient à la classe et non à l'objet (instance).
Les variables statiques ne sont initialisées qu'une seule fois, au début de l'exécution. Ces variables seront initialisées en premier, avant l'initialisation de toute variable d'instance.
Une seule copie à partager par toutes les instances de la classe.
Une variable statique est accessible directement par le nom de la classe et ne nécessite aucun objet.
Syntaxe:Class.variable

---

et moi c'est bien des variables de l'instance.

je révise tout ça et je crois que ça devrait fonctionner.

Je vais répondre à l'histoire de l'interface.

Une bonne pratique de codage consiste au "masquage de l'information".
C'est la loi de Demeter ( https://fr.wikipedia.org/wiki/Loi_de_D%C3%A9m%C3%A9ter )

Une classe ne voit que les méthodes qu'elle peut appeler (les interfaces), elle ne doit en aucune façon voir l'implémentation des méthodes qu'elle est sensé appeler (implémenté dans une classe).

Dans le cas ici, l'interface est List:
https://docs.oracle.com/en/java/java...util/List.html

L'implémentation utilisée est ArrayList (https://docs.oracle.com/en/java/java...ArrayList.html ) mais il y en a d'autres.

Ainsi, le programme marche encore, si d'aventure on s'amuse à changer l'implémentation car le programme utilise le contrat (l'interface).

De fait, il faut écrire:

Code:

---

1 2	List<String> maListe = ....

---

Plutôt que

Code:

---

1 2	ArrayList<String> maListe = ...

---