if和else語句

Question

我試圖執行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」。

有人能告訴我我做錯了什麼嗎？

我是新來這個網站，所以我很感激任何反饋，以備將來參考。謝謝。

Answer 1

你已經掉進語言 「陷阱」

這一個...

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 = "---" 

實際分配true到isChute。您應該使用==不=

更新

一個更好的辦法是......

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 type。

Answer 2

isChute被分配爲true。所以「C10」一直由toString（）返回。 改變它

if(isChute){ 
    ... 
}else if(isLadder){ 
    ... 
}else{ 
    .. 
}