Programme Jeu Yhatzee

Verification

a marqué ce sujet comme résolu.
Auteur du sujet

Bonjour,

Je suis débutante en langage informatique et j’ai du programmer un jeu Yhatzee à l’aide d’un TP et des diagrammes de classe qui sont dans le TP. J’aurais besoin d’une vérification, de recommandations, conseils. Il y aura quelque fois des oublis de méthodes c’est dut à une incapacité de le coder. Je ne sais pas comment le coder. En vous remerciant d’avance pour vos investissements.

Bien à vous, Chyne

public class Die {
  private int MAX = 6; // maximum face value
  private int faceValue; // current value showing on the die
  private static Random generator = new Random();
  private boolean Blocked;

  public Die() {
    faceValue = 1;
  }

  public boolean isBlocked() {

    return false;
  }

  public void unblock(int value) {
    int var = 1;

  }

  public void block(int value) {
    int var = 0;
  }

  // Rolls the die and returns the result.public
  int rollDie() {

    faceValue = (int) (Math.random() * MAX) + 1;
    return faceValue;
  }
  // Face value mutator.
  public void setFaceValue(int faceValue) {
    if( faceValue >0 && faceValue <= MAX)
      this.faceValue = faceValue;
  }
  // Face value accessor.
  public int getFaceValue() {
    return faceValue;
  }

  // Returns a string representation of this die.
  public String toString() {
    String result = Integer.toString(faceValue);
    return result;
  }
}
Classe Die
package model;

import java.util.ArrayList;
import java.util.stream.IntStream;

public class DiceHand {

  ArrayList<Die> savedDice = new ArrayList<Die>();

  // Constructor
  private DiceHand(){}


  public static void main(String[] args){
    int ROLLS = 100000;
    int count = 0;
    Die die1 = new Die();
    Die die2 = new Die();
    for (int i = 0; i < ROLLS; i++){
      if (die1.rollDie() == 1 && die2.rollDie() == 1) {
        count++;
      }

      System.out.println("”snake eyes probability:" +(float)count / ROLLS);
    }
    public String getDie() {
      return die;
    }

    int[] die = {1,2,3,4,5,6};
    int sum = IntStream.of(die).sum();
    System.out.println("The sum is" + sum);
  }

}
Classe DiceHand
package model;

import java.util.ArrayList;
import java.util.List;

public class Game {

  ArrayList<Die> dice = new ArrayList<Die>();
  for(
      int i = 0;
      i< 6;i++)

  {
    dice.add(new Die());
  }

  public int[] ReRollDice(int[] dice) {

    System.out.println("What dices do you want to reroll?");
    int diceToReRoll = keyboard.nextInt();

  }

  Die DieObject = new Die();

  public List<Die> getDice() {

    if (lost) return new ArrayList<Die>();
    return dice;
  }

  public List<ScoreRow> getRows() {
    return sheet.getRows(); // ou return list of

  }

  public boolean choose(ScoreRow scoreRow) {
    return false;
  }

  public void reRoll() {

    for (int i = 0; i < die.length; i++) {
      die[i] = (int) (Math.random() * 6) + 1;

    }

    public void initialRoll () {

      int initialRoll;
      initialRoll = die1.roll();
      System.out.print("Initial roll = " + initialRoll +
          "\nRolling to match:");
    }


    public boolean isOver () {
      return one.getHand().empty() || two.getHand().empty();

    }

    private int List<Die > getHand() {
      return hand;

    }

  }
}
Classe Game
package model;

public class Chance implements Chance {


  public interface Combination {

    public static int getScore(int roundNumber, int dice5, int dice2, int dice3) {
      int score = 21;

      //sommes des valeurs des dés(1+2+3+4+5+6)


      public String description (String Chance){
        return Chance;

      }


    }

  } 
} // @ache: J'ai ajouté cette accolade
Classe Chance
package model;

public class ScoreRow {
  private boolean filled;

  private static void main(String[] args) {
    int score = 0;
    System.out.println(score);

  }

  public boolean isFilled() {
    return filled;
  }


  public void reset() {
    // TODO
  }

  public String description(String Chance) {
    return "Chance";
  }

  public boolean isAvailable() {
    return false;
  }

  public int getScore() {
    return 0;

  }
}
Classe ScoreRow

Les diagrammes de classes images

Édité par ache

+0 -0

Staff

Bonjour et bienvenue ! ^^

Je me suis permis de formater le code afin qu’il soit lisible. La prochaine fois, n’oublie pas d’utiliser la fonction Aperçu afin de détecter les problèmes de formatage. ;)

Pour ton code, je n’ai pas vraiment eu le temps de regarder. Aurais-tu un dépôt Git que l’on puisse cloner ? Si non, ne t’embêtes pas avec ça c’était simplement une question.

Je regarderais ton code plus en détail quand (si >_<) j’aurais le temps.

ache.one                 🦹         👾                                🦊

+0 -0
Auteur du sujet

Bonjour ache,

Je vous remercie pour tout investissement mais j’ai fait une erreur, je l’ai déjà envoyé, fait et corrigé. J’aurais du le supprimer du forum. Mais de toute maniére je posterai d’autres programmes,j’espère te retrouver dans les commentaires.

Merci encore.

Cordialement,

Chyne

+0 -0

Classe Die:

  • private int MAX = 6; Généralement, on déclare les constantes private static final
  • private static Random generator = new Random(); pareil ici (final). Aussi, si je ne m’abuse, tu ne l’utilises nulle part
  • private boolean Blocked; La convention est que les noms d’attributs commencent par une minuscule
  • Les methodes isBlocked, unblock et block n’ont aucun effet
  • Je trouve ça bizarre (et pas très POO) d’autoriser le monde extérieur de modifier la valeur de ton dé. Je pense que faceValue ne devrait être modifié que par roll() et rien d’autre

Classe DiceHand:

  • ArrayList<Die> savedDice = new ArrayList<Die>(); Généralement, on utilise les interfaces autant que possible. Aussi, le type n’est pas nécessaire dans le deuxième opérateur diamant (depuis quelque chose comme Java 7). Ca donne List<Die> savedDice = new ArrayList<>();
  • Je doute que cette classe compile. Tu définis une méthode getDie() au milieu de ta méthode main

Classe Game:

  • Pareil ici pour List au lieu de ArrayList
  • Tu ne peux pas exécuter une boucle comme ça au début de la classe hors méthode. Une possibilité est d’utiliser un bloc d’initialisation.
  • La méthode ReRollDice devrait s’appeler rerollDice
  • int diceToReRoll = keyboard.nextInt(); keyboard n’est pas défini
  • if (lost) return new ArrayList<Die>(); lost n’est pas défini
  • return sheet.getRows(); sheet n’est pas défini

Bon, je vais en rester là. Je pense que tu pourrais déjà trouver beaucoup de choses en compilant et en voyant ce que le compilateur te dit.

+0 -0
Connectez-vous pour pouvoir poster un message.
Connexion

Pas encore membre ?

Créez un compte en une minute pour profiter pleinement de toutes les fonctionnalités de Zeste de Savoir. Ici, tout est gratuit et sans publicité.
Créer un compte