Aide à l'expression régulière dans le programme de dés Java

Question

J'ai été à cette toute la journée et je ne peux pas pour la vie de moi comprendre ce que je fais mal. Le code plante mon émulateur Android. Ce que j'essaye de faire est d'écrire un programme de dés, et je l'ai en train de travailler avec des dés lancés comme des entités uniques, mais j'essaie de traiter des jets de dés comme 3d6 ou 5d4. L'appel à singleRoll lance un seul dé à la fois, et j'essaie de décomposer une équation plus longue en bits plus simples ... "3d4 + 5-13d6 + 7d8 + 9"

bbs.randInt renvoie [0 , diceSize).

public int multiPartRoll(String roll) { 
     String[] parts = roll.split("(?=[+-])"); //split by +-, keeping them 
     int total = 0; 

     // TODO: Replace 5d4 with d4+d4+d4+d4+d4 
     for (String partOfRoll : parts) { //roll each dice specified 


      if (partOfRoll.matches("\\d+d\\d+")) { 
       String[] splitString = (partOfRoll.split("d")); 
       int times = Integer.getInteger(splitString[0]); 
       int die = Integer.getInteger(splitString[1]); 
       int i; 
       for (i = 0; i < times; i++) { 
        String rollStr = "d" + die; 
        total += singleRoll(rollStr); 
       } 
      } 
      else { 
       total += singleRoll(partOfRoll); 
      } 

     } 
     return total; 
    } 

public int singleRoll(String roll) { 
     int di = roll.indexOf('d'); 
     if (di == -1) //case where has no 'd' 
      return Integer.parseInt(roll); 
     int diceSize = Integer.parseInt(roll.substring(di + 1)); //value of string after 'd' 
     int result = bbs.randInt(diceSize) + 1; //roll the dice 
     if (roll.startsWith("-")) //negate if nessasary 
      result = -result; 
     return result; 
    } 

Answer 1

Le problème réside dans l'utilisation de la mauvaise méthode d'analyse Integer.

Essayez de remplacer toutes les occurrences de

Integer.getInteger

avec

Integer.parseInt

la méthode utilisée ne convertit pas une chaîne en un entier: http://docs.oracle.com/javase/6/docs/api/java/lang/Integer.html#getInteger%28java.lang.String%29

Meilleures salutations.

Answer 2

introduisons Dice Interface:

interface Dice { 

    int roll(); 
} 

Et deux classes NDice qui est 'dés normal':

class NDice implements Dice { 

    private final int nb; 
    private final int sides; 
    private Random r = new Random(); 

    NDice(String[] desc) { 
     this.nb = Integer.parseInt(desc[0]); 
     this.sides = Integer.parseInt(desc[1]); 
    } 

    @Override 
    public int roll() { 
     return nb < 0 ? -1 : 1 * IntStream.generate(() -> r.nextInt(sides) + 1).limit(Math.abs(nb)).sum(); 
    } 
} 

Et CDice qui est 'dés constant':

class CDice implements Dice { 
    private int constant; 

    public CDice(int constant) { 
     this.constant = constant; 
    } 

    @Override 
    public int roll() { 
     return constant; 
    } 
} 

Ensuite, nous pourrions introduire une méthode pour analyser rollDescription dans la collection de dés et rouler ce Dices:

static int roll(String rollDescription) { 
    String[] parts = rollDescription.split("(?=[+-])"); 
    return Arrays.stream(parts) 
      .map(s -> { 
       if (s.contains("d")) { 
        return new NDice(s.split("d")); 
       } 
       return new CDice(Integer.parseInt(s)); 
      }).map(Dice::roll).reduce(0, (a, b) -> a + b); 
} 

Maintenant brève explication:

• CDice nous avons besoin seulement de consolider l'interface
• dans l'expression nb < 0 ? -1 : 1 * ... Math.abs(nb) nous devons soutenir - avant nombre de rouleaux (nb).
• J'ai omis des méta-informations sur les rouleaux. Si vous avez besoin, vous pouvez introduire la classe RollResult qui contient le résultat et quelques informations supplémentaires sur les dés.
• Naming pourrait être mieux
• méthode statique roll besoin de 'classe de la maison'. Dice interface est agréable, mais pas parfait endroit pour cela.