Discussion :

[kylan]

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 :

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

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;

[hwoarang]

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 ?

[kylan]

Code :

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

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;
 
    }

[andry.aime]

Bonjour,

Ce code ne sert à rien:

Code :

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

1
2
3
4
5
6
 
    public static void set(Nucleotide nucleotide, Nucleotide nucleotide0) {
        nucleotide= nucleotide;
        nucleotide0=nucleotide0;
 
    }

Code :

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

1
2
3
4
5
6
    public static void set(Nucleotide nucleotide, Nucleotide nucleotide0) {
        Nucleotide temp = nucleotide;
        nucleotide= nucleotide0;
        nucleotide0=temp;
 
    }

A+.

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

Code :

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

1
2
3
4
5
6
    public static void set(Nucleotide nucleotide, Nucleotide nucleotide0) {
        Nucleotide temp = nucleotide;
        nucleotide= nucleotide0;
        nucleotide0=temp;
 
    }

@kylan,

Code :

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

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 :

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

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 :

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

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 :

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

[hwoarang]

Voire un switch avec replaceAll, dans une lambda

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

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

Code :

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

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;
 
   ...
}
}

[andry.aime]

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 . Maintenant j'ai compris

Sinon, un Map fera bien l'affaire:

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

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

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+.

[joel.drigo]

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

C'était pour éviter 2 mettre deux fois le code du switch 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 :

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

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 :

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

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;
}

[joel.drigo]

[...]

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 :

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
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;
	}
 
}