Discussion :

[fredericbdr]

Bonjour,

J'ai un petit soucis avec mon code, j'ai crée une interface puis une classe TestSelection avec une méthode de signature static Collection selection(...).
J'ai crée une LikedList et extrait les nombres entiers de 0 à 100, puis j'en extraits la liste des multiples de 3 avec l'appel de la méthode selection.

Mais j'ai pas mal d'erreurs dans mon code qui ne marche pas, je n'arrive pas à le corriger.

Voici le 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
27
28
29
30
31
32
33
34
35
36
import java.util.Collection;
import java.util.Iterator;
import java.util.LinkedList;
import java.util.Vector;
 
public class TestSelection {
	static Collection selection(Collection source, CritereSelection critere){
		Collection result = new Vector();
		Iterator iter = source.iterator();
		while(iter.hasNext()){
			Object aux = iter.next();
			if(critere.ok(aux))
				result.add(aux);
		}
		return result;
	}
}
 interface CritereSelection{
	boolean ok (Object x);
}
 
static class EtreMultipleDe3 implements CritereSelection{
	public boolean ok(Object x){
		return((Integer)x).IntValue()/3 == 0;
	}
}
 
	public static void main(String[] args) {
		LinkedList l = new LinkedList();
		for(int i=0;i<100;i++)
			l.add(i);
		System.out.println(l);
		Collection l2 = selection(l,new EtreMultipleDe3());
		System.out.println(l2);
	}
}

Merci.

[Deaf]

Bonjour,

Pourrais-tu préciser quelles sont les erreurs que tu as?

Déjà au premier coup d'oeil, j'ai quelques remarques:
- Pour savoir si un entier positif est multiple de N, il faut regarder si le modulo est nul, et non pas le résultat de la division euclidienne
- Si ta LinkedList ne sert qu'à stocker les nombres de 1 à 100, tu peux à priori utiliser un ArrayList que tu initialises avec une capacité de 100. Ce n'est pas une erreur en soi, c'est plus une bonne habitude à prendre. De plus, tu peux la déclarer en List (voire même en Collection), c'est toujours mieux de travailler avec les interfaces.
- Pour ta collection result, tu peux aussi utiliser une ArrayList avec une capacité de source.size(), car tu sais qu'elle ne dépassera pas cette taille.
- A moins que tu sois en Java 1.4 (ou antérieur), tu peux utiliser des generics (Collection<Integer>, List<Integer>, etc.), cela t'éviteras de caster et de détecter des erreurs dès la compilation.

[polymorphisme]

Bonjour,

Pour savoir si un entier positif est multiple de N, il faut regarder si le modulo est nul, et non pas le résultat de la division euclidienne

Quelques rappels :

Deux entiers a et b sont dits congruents modulo n, où n >= 2,
s'il existe un entier k tel que a − b = kn,
autrement dit s'il existe une division euclidienne telle que a = kn + b.

Deux entiers a et b sont multiples l'un de l'autre,
s'il existe un entier k tel que a = kb
autrement dit si le reste de la division euclidienne de a par b est nul.

Le symbole % désigne l'opération modulo qui est le reste de la division euclidienne de a par b.

287 % 7 retourne 0 car la division euclidienne de 287 par 7 a pour reste 0. Finalement 287 est un multiple de 7.