Question

我试图实现一种方法来设置一个具有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|---|---|---|---|---|---|---|---|---|
|---|---|---|---|---|---|---|---|---|---|
|---|---|---|---|---|---|---|---|---|---|
|---|---|---|---|---|---|---|---|---|---|
|---|---|---|---|---|---|---|---|---|---|
|---|---|---|---|---|---|---|---|---|---|
|---|---|---|---|---|---|---|---|---|---|
|---|---|---|---|---|---|---|---|---|---|
|---|---|---|---|---|---|---|---|---|---|

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

Answer 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++; } }

Answer 2

我不确定您想要实现的目标，但如果您消除了内部if else循环中的setBoard，那么您应该随机放置C10。< / p>

改变这个：

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

Answer 3

我认为你的意思是!=。

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

随机发生器不起作用

3 个答案: