0

我试图实现一种方法来设置带有 Cell 对象数组的板。然后该方法在“---”字符串上随机放置一个新字符串“C10”。我的班级和主要课程如下

public class Cell {
    public int addSpaces;

    public Cell() {
        addSpaces = 0;
    }

    public Cell(int x) {
        addSpaces = x;
    }

    public String toString() {
        String print;
        if (addSpaces == -10)
            print = "C10";
        else
            print = "---";
        return print;
    }
}

import java.util.Random;
public class ChutesAndLadders {
    Cell[] board = new Cell[100]; // Set array of Cell object
    Random ran = new Random();
    Cell s = new Cell();
    public int Chut, Ladd;

    public ChutesAndLadders() {
    }

    public ChutesAndLadders(int numChutes, int numLadders) {
        Chut = numChutes;
        Ladd = numLadders;
    }

    public void setBoard() {
        for (int i = 0; i < board.length; i++)
            board[i] = new Cell(); // board now has 100 Cell with toString "---"
        for (int k = 1; k <= Chut; k++) {
            int RanNum = ran.nextInt(board.length); // Randomly replace the
                                                    // toString
            if (board[RanNum] == board[k])
                this.board[RanNum] = new Cell(-10);
            else
                k--;
        }
    }

    public void printBoard() { // method to print out board
        int count = 0;
        for (int i = 0; i < board.length; i++) {
            count++;
            System.out.print("|" + board[i]);
            if (count == 10) {
                System.out.print("|");
                System.out.println();
                count = 0;
            }
        }
    }

    public static void main(String[] args) {
        ChutesAndLadders cl = new ChutesAndLadders(10, 10);
        cl.setBoard();
        cl.printBoard();
    }
}

我没有在整个板上随机放置 C10,而是得到了这个输出;

|---|C10|C10|C10|C10|C10|C10|C10|C10|C10|
|C10|---|---|---|---|---|---|---|---|---|
|---|---|---|---|---|---|---|---|---|---|
|---|---|---|---|---|---|---|---|---|---|
|---|---|---|---|---|---|---|---|---|---|
|---|---|---|---|---|---|---|---|---|---|
|---|---|---|---|---|---|---|---|---|---|
|---|---|---|---|---|---|---|---|---|---|
|---|---|---|---|---|---|---|---|---|---|
|---|---|---|---|---|---|---|---|---|---|

有人可以告诉我我做错了什么吗?谢谢你。

4

3 回答 3

1

我不确定你在 for 循环中的意图是什么:

if (board[RanNum] == board[k])

但它会导致你的 for 循环为每个 k 生成随机数,直到它生成 k,然后设置该单元格。因此,对于 k == 1,它将始终设置单元格 1,将单元格 2 设置为 k == 2,将单元格 3 设置为 k == 3,等等。

我猜你想做更多类似的事情:

for (int k = 1; k <= Chut; k++) {
    int RanNum = ran.nextInt(board.length);
    if (board[RanNum].addSpaces == 0) // uninitialized
        this.board[RanNum] = new Cell(-10);
    else
        k--;
}

编辑:

正如您可能从评论 + 其他答案中意识到的那样,上面的代码不是特别可读。像这样的东西应该更好:

int chutCount = 0;
while (chutCount < Chut)
{
    int randomNum = ran.nextInt(board.length);
    if (board[randomNum].addSpaces == 0) // uninitialized
    {
        board[randomNum] = new Cell(-10);
        chutCount++;
    }
}
于 2013-03-15T22:55:41.463 回答
0

我不确定您想要实现什么,但是如果您消除了if else内部setBoard循环中的 ,那么您应该随机放置C10.

改变这个:

    for (int k = 1; k <= Chut; k++) {
        int RanNum = ran.nextInt(board.length); // Randomly replace the
                                                // toString
        if (board[RanNum] == board[k])
            this.board[RanNum] = new Cell(-10);
        else
            k--;
    }

对此:

    for (int k = 1; k <= Chut; k++) {
        int RanNum = ran.nextInt(board.length); // Randomly replace the
                                                // toString
        this.board[RanNum] = new Cell(-10);
    }
于 2013-03-15T22:55:26.440 回答
0

我想你的意思是!=

for (int k = 1; k <= Chut; k++) {
   int ranNum = (int)(Math.random()*board.length);
   if (board[ranNum] != board[k])
      this.board[ranNum] = new Cell(-10);
   else
      k--;
}
于 2013-03-15T22:58:27.793 回答