Algorithme "Appels de méthodes" Fermé

Question

Bonjour,  voilà j'ai fait un petit programme qui propose 10 exercices de multiplications , la première étant une multiplication  "Difficile" si l'utilisateur répond bien il a 2 point sinon le programme lui proposera de faire un exercice simple si l'utilisateur répond bien il obtiendra 1 point dans le cas contraire il en perds 2 à la fin du programme le score total que l'utilisateur a eu est affiché : 
Voici mon code: 
 
import java.util.Scanner;
public class MultiDificil{
  public static Scanner scanner = new Scanner(System.in);
  public static void main(String[]args){
   String multipli = ""; 
   int score = 0; 
   int point = 0;
  for(int i = 1; i <= 10; i++){
    System.out.println("Multiplication Difficile ");   
     int premierNombre , secondNombre, reponseCorrecte, reponseEleve;
     premierNombre =  unEntierAuHasardEntre(5,10);
     secondNombre =  unEntierAuHasardEntre(11,19);
     System.out.println("Calculer : " + premierNombre + " x " + secondNombre + " = " );
     reponseCorrecte = premierNombre * secondNombre;  
     reponseEleve = scanner.nextInt();
     
       if(reponseEleve == reponseCorrecte){
          multipli = "Difficile";
          point = score(premierNombre,secondNombre,reponseCorrecte,reponseEleve , multipli);
          score = score + point;
          System.out.println("Très bien vous gagnez : " + point + " point");
       }
       else{
        multipli = "Facile";
        System.out.println("Non C'etait : " + reponseCorrecte + " Voici une Multiplication facile");
         premierNombre = unEntierAuHasardEntre(1,6);
         secondNombre = unEntierAuHasardEntre(3,10);
         reponseCorrecte = premierNombre * secondNombre;
         System.out.println("Calculer : " + premierNombre + " x " + secondNombre + " = " );
         reponseEleve = scanner.nextInt();
         if(reponseEleve == reponseCorrecte){        
           point = score(premierNombre,secondNombre,reponseCorrecte,reponseEleve , multipli);
           score = score + point;
           System.out.println("Bien vous gagnez : " + point + " point");
         }
         else{
           point = score(premierNombre,secondNombre,reponseCorrecte,reponseEleve , multipli);
           score = score + point;
           System.out.println("Non C'était : " + reponseCorrecte + " Vous perdez : " + point + " points");
         }
       }
     } 
     System.out.println("Voici votre score : " + score);
    }

public static int score(int premierNombre , int secondNombre , int reponseCorrecte , int reponseEleve , String multipli){
  
   if(reponseCorrecte == reponseEleve && multipli == "Difficile" ){
     return 2;  
  }
  
  else{
     if(reponseCorrecte == reponseEleve && multipli == "Facile"){
       return 1;
     }
     
     return -2;
   }
 }

public static int unEntierAuHasardEntre (int valeurMinimale, int valeurMaximale){
		double nombreReel;
		int resultat;

nombreReel = Math.random();
		resultat = (int) (nombreReel * (valeurMaximale - valeurMinimale + 1))
								+ valeurMinimale;
		return resultat;
  }
 }

Cependant je ne suis pas du tout satisfait de mon code , je l'ai fait de façon à évité de mauvaises habitudes que j'ai tels que mélangés (les scanner.next et les System) dans une méthode ... Je voudrais donc savoirs qlqn peut m'aider a amélioré mon code de façon à ce qu'il soit plus lisible et concis Merci d'avance :)

KX · Answer

Bonjour,
Pour que ton code soit plus lisible il faut sortir toutes les "valeurs magiques" qui sont liées à tes niveaux (5, 10, 11, 19, +2, +1, -2...)
Je pense que la bonne idée est d'utiliser un enum où chaque valeur représente un niveau. Pour l'instant il n'y a que FACILE et DIFFICILE, mais si tu veux rajouter 1, 2 ou 15 niveaux intermédiaires avec ton code c'est impossible alors qu'avec un enum il suffirait de rajouter une valeur.
import java.util.Random;
import java.util.Scanner;

public class MultiDificil {

private static final Random random = new Random();
    private static final Scanner scanner = new Scanner(System.in);

public static int unEntierAuHasardEntre(int valeurMinimale, int valeurMaximale) {
        return random.nextInt(valeurMaximale - valeurMinimale + 1) + valeurMinimale;
    }

public enum Niveau {
        FACILE(1, 6, 3, 10, 1, 2), DIFFICILE(5, 10, 11, 19, 2, 2);

private final int minA, maxA, minB, maxB, scoreOk, scoreKo;

Niveau(int minA, int maxA, int minB, int maxB, int scoreOk, int scoreKo) {
            this.minA = minA;
            this.maxA = maxA;
            this.minB = minB;
            this.maxB = maxB;
            this.scoreOk = scoreOk;
            this.scoreKo = scoreKo;
        }

public int getPremierNombreAuHasard() {
            return unEntierAuHasardEntre(minA, maxA);
        }

public int getSecondNombreAuHasard() {
            return unEntierAuHasardEntre(minB, maxB);
        }

public int getNbPointsSucces() {
            return scoreOk;
        }

public int getNbPointsEchec() {
            return scoreKo;
        }

public Niveau suivant(int delta) {
            int pos = ordinal() + delta;
            if (pos < 0) {
                pos = 0;
            } else {
                int len = Niveau.values().length;
                if (pos >= len)
                    pos = len - 1;
            }
            return Niveau.values()[pos];
        }
    }

public static int saisirResultat(Niveau niveau, int premierNombre, int secondNombre) {
        System.out.println("MULTIPLICATION " + niveau);
        System.out.print(premierNombre + " x " + secondNombre + " = ");
        return scanner.nextInt();
    }

public static void afficherScore(boolean succes, int reponse, int point, int score) {
        if (succes) {
            System.out.print("Très bien. Vous gagnez ");
        } else {
            System.out.print("Non, c'était " + reponse + ". Vous perdez ");
        }
        System.out.println(point + " points. Votre score est de " + score + ".");
    }

public static void main(String[] args) {
        Niveau niveau = Niveau.DIFFICILE;
        int score = 0;

for (int i = 1; i <= 10; i++) {
            int premierNombre = niveau.getPremierNombreAuHasard();
            int secondNombre = niveau.getSecondNombreAuHasard();
            int reponseEleve = saisirResultat(niveau, premierNombre, secondNombre);
            int reponseCorrecte = premierNombre * secondNombre;
            boolean succes = reponseEleve == reponseCorrecte;

int point;
            if (succes) {
                point = niveau.getNbPointsSucces();
                score += point;
                niveau = niveau.suivant(+1);
            } else {
                point = niveau.getNbPointsEchec();
                score -= point;
                niveau = niveau.suivant(-1);
            }

afficherScore(succes, reponseCorrecte, point, score);
        }
    }
}
Je t'invites à ajouter des niveaux intermédiaires, avec par exemple :
public enum Niveau {
    FACILE(1, 6, 3, 10, 1, 2),
    NORMALE(3, 5, 2, 8, 1, 2),
    DIFFICILE(5, 10, 11, 19, 2, 2);

Algorithme "Appels de méthodes"

1 réponse

Discussions similaires