Discussion :

[CoDuSu]

Salut, je souhaite crée du code réutilisable dans plusieurs programme. Une sorte d'"API" perso a mon humble niveau.

Ce code doit generer une question aléatoire, verifier si la réponse est bonne et surtout laisser choisir le developpeur du programme en question ce qu'il veut faire en cas de bonne ou mauvaise reponse. Le tout en étant re-utiliser de la manière la plus simple et intuitive possible.

Pour l'instant voici ou j'en suis:

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
import java.util.Random;
 
public abstract class ProfDeMaths {
	private String question;
	private String bonneReponse;
 
	public ProfDeMaths() {
	}
 
	abstract void siBonneReponse();
 
	abstract void siMauvaiseReponse();
 
	public void ChoisirUneQuestion() {
		Random rand = new Random();
		int facteur1 = rand.nextInt(99) + 1;
		int facteur2 = rand.nextInt(99) + 1;
		this.question = "Combien font " + facteur1 + " + " + facteur2 + " ?";
		this.bonneReponse = "" + (facteur1 + facteur2);
	}
 
	public void Corriger(String reponse) {
		if (reponse.equals(this.bonneReponse))
			this.siBonneReponse();
		else
			this.siMauvaiseReponse();
	};
 
	public String getQuestion() {
		return question;
	}
 
}

Voila ce que ça donne une fois utilisé:

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
import java.util.Scanner;
 
public class Main extends ProfDeMaths {
 
	static ProfDeMaths prof = new Main();
	static Scanner sc = new Scanner(System.in);
 
	@Override
	void siBonneReponse() {
		System.out.println("BONNE REPONSE !");	
	}
	@Override
	void siMauvaiseReponse() {
		System.out.println("MAUVAISE REPONSE !");
		prof.Corriger(sc.nextLine());
	}
 
 
	public static void main(String[] args) {
		// TODO Auto-generated method stub
 
		prof.ChoisirUneQuestion();
		System.out.println(prof.getQuestion());	
		prof.Corriger(sc.nextLine());
 
	}
 
}

Ca marche bien sauf qu'il y a un gros problèmes:

_ On ne peut étendre qu'une seul classe abstraite donc cette classe ProfDeMaths devient inutile si celui qui veut s'en servir veut le faire dans une classe qui herite deja d'un autre objet.

Et un autre probleme un peu moins gros:

_ L'implementation des methodes siBonneReponse et siMauvaiseReponse n'est pas obligatoire. Du coup sans les implementer manuellement et sans aller fouiller dans le code de ProfDeMaths on peut croire que la methode Corriger() ne fonctionne pas.


Y'a-t-il une meilleure manière de faire qui m'éviterait ces problemes afin que ce code soit re-utilisable sans ces inconvenients ?

[Drowan]

L'idée qui me vient en lisant ton problème est de creer in Listener. Une interface qui contient les deux fonctions siBonneReponse() et siMauvaiseReponse(). Ta classe Main devrait alors implémenter cette interface et se déclarer comme listener auprès de prof.

Class prof de math

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
37
38
import java.util.Random;
 
public abstract class ProfDeMaths {
	private String question;
	private String bonneReponse;
        private ProfDeMathsListener listener; //listener pour réagir à la réponse
 
	public ProfDeMaths() {
	}
 
        // Doit être fourni par l'utilisateur
        public void setListener(ProfDeMathsListener listener) {
                  this.listener = listener;
        }
 
	public void ChoisirUneQuestion() {
		Random rand = new Random();
		int facteur1 = rand.nextInt(99) + 1;
		int facteur2 = rand.nextInt(99) + 1;
		this.question = "Combien font " + facteur1 + " + " + facteur2 + " ?";
		this.bonneReponse = "" + (facteur1 + facteur2);
	}
 
	public void Corriger(String reponse) {
            //Il faut vérifier que le listener n'est pas null (si l'utilisateur a oublié d'en donner un)
            if(listener != null) {
		if (reponse.equals(this.bonneReponse))
			listener.siBonneReponse(); //réagir à une bonne réponse
		else
			listener.siMauvaiseReponse(); //réagir à une mauvaise réponse
            }
	};
 
	public String getQuestion() {
		return question;
	}
 
}

Interface listener :

Code :

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

1
2
3
4
5
6
7
public interface ProfDeMathListener {
 
     void siBonneReponse();
 
     void siMauvaiseReponse();
 
}

Class Main :

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
import java.util.Scanner;
 
public class Main implements ProfDeMathListener {
 
	static ProfDeMaths prof = new Main();
	static Scanner sc = new Scanner(System.in);
 
	void siBonneReponse() {
		System.out.println("BONNE REPONSE !");	
	}
 
	void siMauvaiseReponse() {
		System.out.println("MAUVAISE REPONSE !");
		prof.Corriger(sc.nextLine());
	}
 
 
	public static void main(String[] args) {
		// TODO Auto-generated method stub
 
                prof.setListener(this);
		prof.ChoisirUneQuestion();
		System.out.println(prof.getQuestion());	
		prof.Corriger(sc.nextLine());
 
	}
 
}

[CoDuSu]

Merci c'est une bien meilleure solution en effet !

Sauf que ça marche pas Pour l'instant du moins. J'ai une erreur dans la méthode Main:

Code :

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

prof.setListener(this);

"Cannot use this in a static context"


J'ai éssayé mais je n'arrive pas a la résoudre ça :/

[bouye]

Tu es dans la méthode main qui est statique, il t'est donc impossible d'invoquer this puisque tu n'est pas dans une instance.

[Drowan]

il faut faire prof.setListener(prof);

[CoDuSu]

Merci Drowan mais prof.setListener(prof) ne marche pas non plus car il attend un ProfListener et pas un Prof. J'ai cette erreur: "The method setListener(ProfListener) in the type Prof is not applicable for the arguments (Prof)"

Je post le code de là ou j'en suis car j'ai modifié 2-3 trucs de ta réponse (tu avais parfois oublié le "s" a la fin de "ProfDeMaths" et surtout tu avais laissé la classe en abstraite. C'était bien un oubli ?)

Prof:

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
37
import java.util.Random;
 
public class Prof {
	private String question;
	private String bonneReponse;
	private ProfListener listener;
 
	public Prof() {
	}
 
	// Doit être fourni par l'utilisateur
	public void setListener(ProfListener listener) {
		this.listener = listener;
	}
 
	public void ChoisirUneQuestion() {
		Random rand = new Random();
		int facteur1 = rand.nextInt(99) + 1;
		int facteur2 = rand.nextInt(99) + 1;
		this.question = "Combien font " + facteur1 + " + " + facteur2 + " ?";
		this.bonneReponse = "" + (facteur1 + facteur2);
	}
 
	public void Corriger(String reponse) {
		if (listener != null) {
			if (reponse.equals(this.bonneReponse))
				listener.siBonneReponse();
			else
				listener.siMauvaiseReponse(); 
		}
	};
 
	public String getQuestion() {
		return question;
	}
 
}

ProfListener:

Code :

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

1
2
3
4
5
6
7
public interface ProfListener {
 
     void siBonneReponse();
 
     void siMauvaiseReponse();
 
}

Main:

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
import java.util.Scanner;
 
public class Main implements ProfListener {
 
	static Prof prof = new Prof();
	static Scanner sc = new Scanner(System.in);
 
	public void siBonneReponse() {
		System.out.println("BONNE REPONSE !");
	}
 
	public void siMauvaiseReponse() {
		System.out.println("MAUVAISE REPONSE !");
		prof.Corriger(sc.nextLine());
	}
 
	public static void main(String[] args) {
		prof.setListener(prof); //The method setListener(ProfListener) in the type Prof is not applicable for the arguments (Prof)
		prof.ChoisirUneQuestion();
		System.out.println(prof.getQuestion());
		prof.Corriger(sc.nextLine());
 
	}
 
}