comparaison des elements d'un arraylist

bonjour!
j'aimerai comparer les elements de mon arraylist index a index et les changer une a une par les valeurs correspondants.
mon code ne me montre pas d'erreur mais ne fait pas le remplacement . est ce que vous pouvez m'aider a voir clair merci

Code:

---

1 2 3 4 5 6 7 8 9 10 11

public AcideRibonucleique complement(){ for(int i=0;i<this.size();i++) { if(get(i)==Nucleotide.A) { Nucleotide.set(Nucleotide.A, Nucleotide.U); if(get(i)==Nucleotide.G) { Nucleotide.set(Nucleotide.G, Nucleotide.C); if(get(i)==Nucleotide.C){ }}}} return this;

---

Citation:

---

Envoyé par kylan

j'aimerai comparer les elements de mon arraylist index a index et les changer une a une par les valeurs correspondants.
mon code ne me montre pas d'erreur mais ne fait pas le remplacement.

---

Tu travailles sur une fonction statique alors que, visiblement, ce n'est pas ce que tu veux. Par ailleurs, ton imbrication de if semble tres suspecte.

Est ce que tu peux poster le code de Nucleotide.set ?

Code:

---

1 2 3 4 5 6 7 8

public enum Nucleotide { A, B, C, D, G, H, K, M, N, R, S, U, V, W, Y;   public static void set(Nucleotide nucleotide, Nucleotide nucleotide0) { nucleotide= nucleotide; nucleotide0=nucleotide0;   }

---

Bonjour,

Ce code ne sert à rien:

Citation:

---

Envoyé par kylan

Code:

---

1 2 3 4 5 6

public static void set(Nucleotide nucleotide, Nucleotide nucleotide0) { nucleotide= nucleotide; nucleotide0=nucleotide0;   }

---

---

Code:

---

1 2 3 4 5 6

public static void set(Nucleotide nucleotide, Nucleotide nucleotide0) { Nucleotide temp = nucleotide; nucleotide= nucleotide0; nucleotide0=temp;   }

---

A+.

Salut,

Hum, ce code ne sert pas plus que l'autre (il ne fait qu'échanger localement les valeurs des paramètres, qui ne sont pas utilisés ultérieurement, donc pour rien) :

Citation:

---

Envoyé par andry.aime

Code:

---

1 2 3 4 5 6

public static void set(Nucleotide nucleotide, Nucleotide nucleotide0) { Nucleotide temp = nucleotide; nucleotide= nucleotide0; nucleotide0=temp;   }

---

---

@kylan,

Code:

---

1 2 3 4 5 6 7 8 9 10

public AcideRibonucleique complement(){ for(int i=0;i<this.size();i++) { if(get(i)==Nucleotide.A) { Nucleotide.set(Nucleotide.A, Nucleotide.U); if(get(i)==Nucleotide.G) { Nucleotide.set(Nucleotide.G, Nucleotide.C); if(get(i)==Nucleotide.C){ }}}} return this;

---

Cette méthode est dans la classe AcideRibonucleique, qui est à priori une List. Il te suffit donc de faire des set() comme tu fais les get() :

Code:

---

1 2 3 4 5 6 7 8

public AcideRibonucleique complement(){ for(int i=0;i<this.size();i++) { if(get(i)==Nucleotide.A) { set(i, Nucleotide.U); } } return this; }

---

Voici un code qui remplace les A par des U.

Après, je rejoins @hwoarang sur ses doutes quant à l'imbrication de if, qui mérite quand même une indentation plus soignée (qui permet de voir tout de suite qu'il y a un problème) :

Code:

---

1 2 3 4 5 6 7 8

if ( get(i)==Nucleotide.A ) { set(i, Nucleotide.U); if ( get(i)==Nucleotide.G ) { set(i, Nucleotide.C); if(get(i)==Nucleotide.C) { } } }

---

Si on remplace A par U, il est impossible qu'il valle G : donc inutile de tester ce cas, et tous les autres imbriqués, qui ne seront jamais exécutés.
Au lieu de if, utilise un switch, ce serait bien plus clair.

Voire un switch avec replaceAll, dans une lambda :

Code:

---

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

public AcideRibonucleique complement(){ replaceAll(nucleotide-> { switch(nucleotide) { case A: return Nucleotide.U; case G: return Nucleotide.G; case C: /** ...return jesaispasquoi **/ /**.... etc ... **/ default: throw new IllegalStateException("Unknown nucleotide"); } }); return this;

---

Citation:

---

Envoyé par joel.drigo

Voire un switch avec replaceAll, dans une lambda

---

Les lambdas quand il y a des erreurs aussi basiques, je n'aurais pas tenté :aie:

Je partirais sur un code plus simple (mais le switch est quand meme clairement plus lisible).

Code:

---

1 2 3 4 5 6 7 8 9

for(int i=0;i<this.size();i++) { switch(get(i)) { case Nucleotide.A: set(i, Nucleotide.U); break;   ... } }

---

Citation:

---

Envoyé par joel.drigo

Salut,

Hum, ce code ne sert pas plus que l'autre (il ne fait qu'échanger localement les valeurs des paramètres, qui ne sont pas utilisés ultérieurement, donc pour rien) :

---

J'ai pas lu la discussion depuis le début :oops:. Maintenant j'ai compris

Sinon, un Map fera bien l'affaire:

Code:

---

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

public enum Nucleotide{ A, B, C, D, G, H, K, M, N, R, S, U, V, W, Y;   private final static Map<Nucleotide, Nucleotide> NUCLEOTIDE_LIST; static{ NUCLEOTIDE_LIST = new HashMap<Nucleotide, Nucleotide>(); NUCLEOTIDE_LIST.put(Nucleotide.A, Nucleotide.U); NUCLEOTIDE_LIST.put(Nucleotide.U, Nucleotide.A); ....; }   public Nucleotide getComplement(){ return NUCLEOTIDE_LIST.get(this); } }

---

Code:

---

1 2 3 4 5 6

List<Nucleotide> arn = new ArrayList<Nucleotide>(); arn.add(Nucleotide.A); arn.add(Nucleotide.U); for(Nucleotide n: arn){ System.out.println(n.getComplement()); }

---

A+.

Citation:

---

Envoyé par hwoarang

Les lambdas quand il y a des erreurs aussi basiques, je n'aurais pas tenté :aie:

---

C'était pour éviter 2 mettre deux fois le code du switch :P Me semblait que l'exemple avec la lambda était suffisant pour évoquer la forme du switch. Et encore, j'aurais pu parler de la solution avec une EnumMap :

Code:

---

1 2 3 4 5 6 7 8

private static final EnumMap<Nucleotide, Nucleotide> COMPLEMENTS = createComplementMap(); private static EnumMap<Nucleotide, Nucleotide> createComplementMap() { EnumMap<Nucleotide, Nucleotide> map = new EnumMap<>(Nucleotide.class); map.put(Nucleotide.A, Nucleotide.U); map.put(Nucleotide.G, Nucleotide.C); /** ... **/ return map; }

---

Code:

---

1 2 3 4 5 6 7

public AcideRibonucleique complement() { for (int i = 0; i < this.size(); i++) { set(i, COMPLEMENTS.get(get(i))); } return this; }

---

Citation:

---

Envoyé par andry.aime

[...]

---

Oui, la solution d'une méthode complement() dans l'enum est sans doute la meilleure. J'y avais pensé au début sans map, mais ça obligeait à écrire ce genre de truc, qui devient moins clair pour le coup :

Code:

---

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

public enum Nucleotide {   A,G,C(G),U(A);     private Nucleotide complement; private Nucleotide() { } private Nucleotide(Nucleotide complement) { this.complement=complement; this.complement.complement=this; }   public Nucleotide getComplement() { return complement; }   }

---