在Java骰子滾動程序中的正則表達式幫助

Question

我一直在這一整天，我不能爲我的生活弄清楚我做錯了什麼。該代碼崩潰我的Android模擬器。我試圖做的是編寫一個骰子滾動程序，我把它作爲單個實體滾動，但我試圖處理像3d6或5d4的骰子滾動。對SingleRoll的調用一次擲出一個模子，我試圖將一個更長的方程分解成更簡單的位......「3d4 + 5-13d6 + 7d8 + 9」

bbs.randInt返回[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

問題出在使用錯誤的整數解析方法。

嘗試用

Integer.parseInt

您使用的字符串不轉換爲整數的方法替代的

Integer.getInteger

所有出現的：http://docs.oracle.com/javase/6/docs/api/java/lang/Integer.html#getInteger%28java.lang.String%29

最好的問候。

Answer 2

我們介紹Dice接口：

interface Dice { 

    int roll(); 
} 

和兩個班NDice這是 '正常骰子'：

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

而且CDice那就是 '恆骰子'：

class CDice implements Dice { 
    private int constant; 

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

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

然後我們可以引入方法來解析rollDescription到骰子和滾動的收集這個切成小方塊：

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

現在簡短說明：

• CDice我們只需要鞏固界面表達nb < 0 ? -1 : 1 * ... Math.abs(nb)
• 我們需要（輥號碼前支持-nb）。
• 我省略了關於卷的元信息。如果你需要，你可以引入類RollResult，其中包含結果和一些關於骰子的額外信息。
• 命名可能會更好
• 靜態方法roll需要'家庭班'。 Dice界面很好，但不是完美的地方。