0

我正在尝试实现一种toString方法,并且输出toString取决于boolean变量。以下是我的课程和主要课程。

public class Cell {

    public int addSpaces;
    boolean isEmpty;
    boolean isChute;
    boolean isLadder;

    public Cell() {
        addSpaces = 10; //I initialized addSpaces to 10 for testing purpose 
    }

    public boolean isChute() { //first boolean method
        if (addSpaces == -10) {
            return true;
        } else {
            return false;
        }
    }

    public boolean isLadder() {//second boolean method
        if (addSpaces == 10) {
            return true;
        } else {
            return false;
        }
    }

    public boolean isEmpty() { //third boolean method
        if (addSpaces == 0) {
            return true;
        } else {
            return false;
        }
    }

    public String toString() {
        String print;
        if (isChute = true) //if isChute is true return true.
        {
            print = "C10";       // toString output = "C10"
        } else if (isLadder = true) // if isLadder is true return true
        {
            print = "L10";          // toString output == "L10"
        } else {
            print = "---"; // else toString print output = "---"
        }
        return print;
    }

    public static void main(String[] arg) {
        Cell s = new Cell();

        System.out.println(s.addSpaces);
        System.out.println(s);
    }
}

不管 的输入状态如何toString,我基本上得到相同的输出“C10”。

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

我是这个网站的新手,所以我感谢任何反馈以供将来参考。谢谢你。

4

2 回答 2

8

您已陷入其中一种语言“陷阱”

这...

if(isChute = true) //if isChute is true return true.
    print = "C10";       // toString output = "C10"
else if (isLadder = true) // if isLadder is true return true
    print = "L10";          // toString output == "L10"
else   
    print = "---"

实际上是分配trueisChute. 你应该不==使用=

更新

更好的方法是......

if(isChute) //if isChute is true return true.
    print = "C10";       // toString output = "C10"
else if (isLadder) // if isLadder is true return true
    print = "L10";          // toString output == "L10"
else   
    print = "---"

如果对象只有两种状态(溜槽或梯子),您可以简单地使用

if(isChute) //if isChute is true return true.
    print = "C10";       // toString output = "C10"
else print = "L10";          // toString output == "L10"

如果它可以有超过 2 个状态,那么我会使用一个enum类型

于 2013-03-15T05:19:33.083 回答
0

isChute 被赋值为 true。所以“C10”一直被 toString() 返回。更改

if(isChute){
    ...
}else if(isLadder){
    ...
}else{
    ..
}
于 2013-03-15T05:31:54.847 回答