Comment bien faire du code re-utilisable ?

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:

---

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:

---

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 ?

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:

---

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:

---

1 2 3 4 5 6 7

public interface ProfDeMathListener {   void siBonneReponse();   void siMauvaiseReponse();   }

---

Class Main :

Code:

---

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

---

Merci c'est une bien meilleure solution en effet ! ;)

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

Code:

---

prof.setListener(this);

---

"Cannot use this in a static context"


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

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.

il faut faire prof.setListener(prof);

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:

---

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:

---

1 2 3 4 5 6 7

public interface ProfListener {   void siBonneReponse();   void siMauvaiseReponse();   }

---

Main:

Code:

---

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

---

Oui, c'était bien des fautes de frappes / oublis.
Pareil, je t'ai dis un truc faux en te disant prof.setListener(prof), j'ai pas été attentif. :aie:

J'ai commencé en te faisant implémenter l'interface profListener dans ta classe main ce qui n'est pas une bonne idée.

Du coup je pense qu'il faut repartir dans l'autre sens :mouarf: avec une classe profListener et des méthodes abtraites :

Code:

---

1 2 3 4 5 6 7

public class ProfListener{   abstract void siBonneReponse();   abstract void siMauvaiseReponse();   }

---

Et dans ton main, tu crée un objet de type ProfListener dans lequel tu vas définir les fonctions siBonneReponse() et siMauvaiseReponse(). Puis tu set cet objet comme listener pour prof.

Code:

---

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 39

import java.util.Scanner;   public class Main implements ProfListener {   static Prof prof = new Prof(); static Scanner sc = new Scanner(System.in); // Création d'un objet de type ProfListener et définition des fonctions static ProfListener profListener = new ProfListener(){ @Override siBonneReponse(){ System.out.println("BONNE REPONSE !"); }   @Override siMauvaiseReponse() { System.out.println("MAUVAISE REPONSE !"); prof.Corriger(sc.nextLine()); } };     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(profListener); prof.ChoisirUneQuestion(); System.out.println(prof.getQuestion()); prof.Corriger(sc.nextLine());   }   }

---

Ha Merci ça marche impécable j'aime bien cette solution :)

Dernière question, est-il possible ici de créer la classe ProfListener dans le même fichier que la classe Prof ou il faut necessairement 1 fichier pour chacunes d'entre elles ?

Oui tu peux en utilisant les classes internes

Mais en en faisant de ProfListener une classe interne la méthode Main n''y aura pas accès non ? Du coup le programme ne marchera plus.

Si tu suis le cours que je t'ai joins, tu verras que si.

Citation:

---

L'accessibilité à la classe interne respecte les règles de visibilité du langage.

---

Tu peux donc faire une classe interne publique qui sera accessible en dehors de la classe principale.

Code:

---

1 2 3 4

public class Principale { public class Interne { //Déclarée public donc accessible partout } }

---

Code:

---

1 2 3 4 5 6

public class Main { public static void main() { Pricipale cp = new Principale(); //création d'une instance de la classe Principale Principale.Interne ci = cp.new Interne(); // création d'une instance de la classe Interne à partir de l'instance de Principale } }

---