死碼從哪裏來？

Question

我遇到了一個問題，我在Eclipse中遇到了「死代碼」警告，我真的不知道爲什麼。代碼來自我的Connect Four項目，更確切地說，它是從Class中檢查是否有人獲勝。該方法檢查所有水平獲勝可能性爲紅色。代碼如下：

/** 
* Method to check the horizontal winning possibilities for red 
* @return true if red won or false if not 
*/ 
public boolean checkHorRed(){ 
    for(int line = 0; line < 6; line++) { 
     for(int column = 0; column < 4; column++) { //column++ is underlined and causes the "dead Code" warning 
      if(gw.buttons[line][column].getIcon().equals(gw.red)); 
       if(gw.buttons[line][column+1].getIcon().equals(gw.red)); 
        if(gw.buttons[line][column+2].getIcon().equals(gw.red)); 
         if(gw.buttons[line][column+3].getIcon().equals(gw.red)); 
          return true; 
     } 
    } 
    return false; 
}  

這個遊戲甚至因爲這個方法而立即獲勝。奇怪的是，類中所有其他方法看起來幾乎一樣，不會導致任何問題。以下是檢查黃色垂直獲勝可能性的方法，以進行比較：

/** 
* Method to check the vertical winning possibilities for yellow 
* @return true or false 
*/ 
public boolean checkVertYel(){ 
    for(int line = 3; line < 6; line++) { 
     for(int column = 0; column < 7; column++) { 
      if(gw.buttons[line][column].getIcon().equals(gw.yellow)) 
       if(gw.buttons[line-1][column].getIcon().equals(gw.yellow)) 
        if(gw.buttons[line-2][column].getIcon().equals(gw.yellow)) 
         if(gw.buttons[line-3][column].getIcon().equals(gw.yellow)) 
          return true; 
     } 
    } 
    return false; 
}  

這一個不會引起任何問題。有人可以告訴我警告來自哪裏嗎？如果您需要更多信息，請告訴我。

Answer 1

你函數中的死代碼是你內循環的the increment statement（column++）。總是會執行return true語句（如果循環被執行），所以循環增量永遠不會發生。

那是你的代碼，而是格式正確的：

// ... 

for(int column = 0; column < 4; column++) { 
    //column++ is underlined and causes the "dead Code" warning 
    if(gw.buttons[line][column].getIcon().equals(gw.red)); 

    if(gw.buttons[line][column+1].getIcon().equals(gw.red)); 

    if(gw.buttons[line][column+2].getIcon().equals(gw.red)); 

    if(gw.buttons[line][column+3].getIcon().equals(gw.red)); 

    return true; 
} 

// ... 

你可以很容易地發現錯誤：return true就一定會執行，所以內循環的增量語句將不會被執行。

這是你的代碼應該如何看起來像：

public boolean checkHorRed() { 
    for(int line = 0; line < 6; line++) { 
     for(int column = 0; column < 4; column++) { 
      //column++ is underlined and causes the "dead Code" warning 
      if(gw.buttons[line][column].getIcon().equals(gw.red) 
        && gw.buttons[line][column+1].getIcon().equals(gw.red) 
        && gw.buttons[line][column+2].getIcon().equals(gw.red) 
        && gw.buttons[line][column+3].getIcon().equals(gw.red) { 
       return true; 
      } 
     } 
    } 

    return false; 
} 

Answer 2

在上面的方法中，在每個if語句之後，作爲你的第二個方法是正確的，哪個是正確的方法。

if(gw.buttons[line][column].getIcon().equals(gw.red)); <-- 

終止，如果沒有它的自我。您的代碼行相當於

if(condition) 
    { 

    } 

這意味着if條件死後的代碼。

Answer 3

這是重新格式化後您的代碼：

public boolean checkHorRed() { 
    for (int line = 0; line < 6; line++) { 
     for (int column = 0; column < 4; column++) { //column++ is underlined and causes the "dead Code" warning 
      if (gw.buttons[line][column].getIcon().equals(gw.red)) { 
       ; 
      } 
      if (gw.buttons[line][column + 1].getIcon().equals(gw.red)) { 
       ; 
      } 
      if (gw.buttons[line][column + 2].getIcon().equals(gw.red)) { 
       ; 
      } 
      if (gw.buttons[line][column + 3].getIcon().equals(gw.red)) { 
       ; 
      } 
      return true; //this will always happen 
     } 
    } 
    return false; 
} 

而這是其他：

public boolean checkVertYel() { 
    for (int line = 3; line < 6; line++) { 
     for (int column = 0; column < 7; column++) { 
      if (gw.buttons[line][column].getIcon().equals(gw.yellow)) { 
       if (gw.buttons[line - 1][column].getIcon().equals(gw.yellow)) { 
        if (gw.buttons[line - 2][column].getIcon().equals(gw.yellow)) { 
         if (gw.buttons[line - 3][column].getIcon().equals(gw.yellow)) { 
          return true; 
         } 
        } 
       } 
      } 
     } 
    } 
    return false; 
} 

基本上，你真的應該沒有結束你的if語句用分號。