Discussion :

[Pierre8r]

Bonjour,

J'ai deux class presque identiques à une méthode près.
Je leur ai donné une interface commune, pour pouvoir les utiliser avec une ArrayList.
Néanmoins il reste du code commun dupliqué.
Peut-on éviter de dupliquer le code commun aux deux class ?


Projet exemple complet

Merci,

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
public class AAA01 implements Iaaa {
	private int zz;
 
	public AAA01(int zz) {
		this.zz = zz;
	}
 
	private void setZz(int zz) {
		this.zz = zz;
	}
 
	private int getZz() {
		return zz;
	}
 
	public  int methodeAA(int x) {
		setZz(getZz() + x);
		return getZz();
	}
 
	public  int methodeBB(int x) {
		setZz(getZz() * x);
		return getZz();
	}
 
	@Override
	public String toString() {
		return String.valueOf(getZz());
	}
 
}

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
public class AAA02 implements Iaaa {
	private int zz;
 
	public AAA02(int zz) {
		this.zz = zz;
	}
 
	private void setZz(int zz) {
		this.zz = zz;
	}
 
	private int getZz() {
		return zz;
	}
 
	public int methodeAA(int x) {
		setZz(getZz() + x + 10);
		return getZz();
	}
 
	public int methodeBB(int x) {
		setZz(getZz() * x);
		return getZz();
	}
 
	@Override
	public String toString() {
		return String.valueOf(getZz());
	}
 
}

Code :

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

1
2
3
4
5
6
7
public interface Iaaa {
 
	public abstract int methodeAA(int x);
 
	public abstract int methodeBB(int x);
 
}

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
import java.util.ArrayList;
import java.util.List;
 
public class MainTestJava {
 
	/**
         * @param args
         */
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		MainTestJava mainTestJava = new MainTestJava();
 
	}
 
	public MainTestJava() {
		List<Iaaa> aaa = new ArrayList<Iaaa>();
 
		aaa.add(new AAA01(100));
		aaa.add(new AAA01(200));
		aaa.add(new AAA02(100));
		aaa.add(new AAA02(200));
 
		for (Iaaa iaaa : aaa) {
			iaaa.methodeAA(2);
			iaaa.methodeBB(10);
			System.out.println(iaaa.toString());
		}
	}
 
}

[fr1man]

Tu peux passer par une classe abstraite intermédiaire.

[hibour]

ou tu peux hériter AAA02 de AAA01 en redéfinissant methodAA

Code :

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

1
2
3
4
5
6
7
public class AAA02 extends AAA01{
	//....
 
	public  int methodeAA(int x) {
		//....
		return getZz();
	}

[entreprise38]

+1 pour la classe abstraitre, c'est la vision la plus logique.

=> AAA02 de AAA01 ? Si l'héritage entre les deux a un sens, oui, mais ça n'a pas l'air d'être le cas ici.

=> AAA02 et AAA01 héritent d'une classe abstraite ? Les deux validant la même interface et ayant du code en commun : leur trouver un parent commun qui, par la même occasion, validera aussi l'interface, est sans doute la solution. Ce parent n'ayant aucune raison d'être instancié, il remplit les conditions pour être une classe abstraite.

[hibour]

c'est vrai que s'il n'y a pas de relation entre les 2 classes alors l'héritage n'a aucun sens mais bon, s'il y a autant de ressemblance dans le code alors je pense qu'une tel relation existe et un simple héritage ferai l'affaire

[Pierre8r]

J'ai bien compris ?

Code :

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

1
2
3
4
5
6
7
8
9
10
 
public interface Iaaa {
 
	public abstract int methodeAA(int x);
 
	public abstract int methodeBB(int x);
 
	public abstract String toString();
 
}

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
 
abstract class AbsAAA implements Iaaa {
	private int zz;
 
	public AbsAAA(int zz) {
		this.zz = zz;
	}
 
	protected void setZz(int zz) {
		this.zz = zz;
	}
 
	protected int getZz() {
		return zz;
	}
 
	public  int methodeBB(int x) {
		setZz(getZz() * x);
		return getZz();
	}
 
	@Override
	public String toString() {
		return String.valueOf(getZz());
	}
 
}

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 class AAA01 extends AbsAAA{
 
	public AAA01(int zz) {
		super(zz);
		// TODO Auto-generated constructor stub
	}
 
	@Override
	public int methodeAA(int x) {
		setZz(getZz() + x);
		return getZz();
	}
 
}

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 class AAA02 extends AbsAAA{
 
	public AAA02(int zz) {
		super(zz);
		// TODO Auto-generated constructor stub
	}
 
	@Override
	public int methodeAA(int x) {
		setZz(getZz() + x + 10);
		return getZz();
	}
 
}

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
 
import java.util.ArrayList;
import java.util.List;
 
public class MainTestJava {
 
	/**
         * @param args
         */
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		MainTestJava mainTestJava = new MainTestJava();
 
	}
 
	public MainTestJava() {
		List<Iaaa> aaa = new ArrayList<Iaaa>();
 
		aaa.add(new AAA01(100));
		aaa.add(new AAA01(200));
		aaa.add(new AAA02(100));
		aaa.add(new AAA02(200));
 
		for (Iaaa iaaa : aaa) {
			iaaa.methodeAA(2);
			iaaa.methodeBB(10);
			System.out.println(iaaa.toString());
		}
	}
 
}