問題上，如果和else語句

Question

這是我在CodingBat看到了這樣的問題：

鑑於2陽性int類型，返回較大值的範圍是10..20包容性，或返回0，如果沒有在該範圍。

這是我寫的代碼：

public int max1020(int a, int b) { 
    if (a>=10 && a<=20 && b>=10 && b<=20 && a>b) 
    return a; 
    if (a>=10 && a<=20 && b>=10 && b<=20 && b>a) 
    return b; 
    if (a>=10 && a<=20 && b<=10 || b>=20) 
    return a; 
    if (a<=10 || a>=20 && b>=10 && b<=20) 
    return b; 

    else return 0; 
} 

我相當有信心，這是正確的，但仍然後我點擊運行，該網站說：max1020（9,21）→0，但我代碼返回9.有人可以幫助我檢查我的代碼有什麼問題嗎？ :)

Answer 1

public int max1020(int a, int b) { 
    if (a>=10 && a<=20 && b>=10 && b<=20 && a>b) 
    return a; 
    if (a>=10 && a<=20 && b>=10 && b<=20 && b>a) 
    return b; 
    if ((a>=10 && a<=20) && (b<=10 || b>=20)) 
    return a; 
    if ((a<=10 || a>=20) && (b>=10 && b<=20)) 
    return b; 

    else return 0; 
} 

在第3行和第4行添加括號可以解決問題。

我建議你改變if聲明和使用else if。只要有可能，使用else if語句而不是幾個if只是一種良好的編碼習慣。

Answer 2

你的代碼是在第三if條件，那就是你有破|| B> = 20。第三和第四個條件應該更具體，如下所示：

if (a>=10 && a<=20 && (b<=10 || b>=20)) 
     return a; 
    if ((a<=10 || a>=20) && b>=10 && b<=20) 
     return b; 

添加這些括號會起到一定作用。

Answer 3

我寧願到檢查分配給一個變量以具有更多的「可讀的」代碼。但這取決於個人喜好。

public int max1020(int a, int b) { 
    final boolean aInRange = a>=10 && a<=20; 
    final boolean bInRange = b>=10 && b<=20; 

    if (aInRange && bInRange) { 
     if (a > b) { 
      return a; 
     } else if (a < b) { 
      return b; 
     } else { 
      return 0; 
     } 
    } else if (aInRange) { 
     return a; 
    } else if (bInRange) { 
     return b; 
    } else { 
     return 0; 
    } 
} 

Answer 4

當然Varun的回答是正確的。另外，我想詳細說明一些意見，並展示解決問題的另一種方法，該方法更簡單，不太可能包含錯誤。

在閱讀問題的陳述，你可能會注意到，該方法返回一個值，如果滿足一定條件，或返回0其他。因此，您可以使用默認值0初始化結果，如果條件滿足，則更改結果，然後返回結果。這將減少代碼：

public int max1020(int a, int b) { 
    int result = 0; 

    if (a >= 10 && a <= 20) result = a; 
    if (b >= 10 && b <= 20 && b > result) result = b; 

    return result; 
} 

不能讓它更簡單，我想。 （但是，如果可以的話，請評論，我愛KISS！:)）

該解決方案產生略有不同的結果，如果兩個a和b在範圍和a=b，它將返回a。如果這種情況發生，問題陳述並不是很清楚，Varun的回答也表明了這一點。巧合（或不）Codingbat不檢查這種情況。在這種情況下，網站上提出的解決方案也會返回a。

如果你認爲它應該返回0時a=b，它很容易調整，

public int max1020(int a, int b) { 
    int result = 0; 

    if (a != b) { 
     if (a >= 10 && a <= 20) result = a; 
     if (b >= 10 && b <= 20 && b > result) result = b; 
    } 

    return result; 
} 

還是蠻簡單的:)

爲了解釋TJCrowder的約壓痕評論：如果你把身體的if聲明在下一行，你應該使用大括號和縮進行。否則，很容易誤讀它，或者在更改代碼時出錯。

// this can be error prone and harder to read, 
// especially if you have multiple if statements, 
// or add a statement to the body of the if statement in the future 
// (shouldn't do this) 
if (condition) 
statement; 

// Personally I think this is totally fine for a simple statement. 
// But I know not everybody will agree 
if (condition) statement; 

// Usually, you'll see this formatting. 
// Even without reading anything, the formatting makes it instantly clear 
// which statements belong to the body of the if 
if (condition) { 
    statement; 
} 

邊注：在代碼中else聲明屬於只最後if。你的格式可以通過在這些之間不加空格來表示清楚，例如

if (condition) statement; 
else statement; 

if (condition) { 
    statement; 
} else { 
    statement; 
} 

在代碼中else聲明實際上是過時的，你可以簡單地return 0;在最後一行。

Varun建議您可以使用else if s是正確的。理論上，如果第一個if聲明的條件爲真，則使用else if將使代碼跳過其他if的測試，並且效率更高一些。此外，它會顯示只有在前面的if語句錯誤時纔會達到後續的if語句。

然而，實際上，在你的代碼中，它並不重要，因爲如果條件爲true，並且後續的if語句永遠無法到達，則該方法將完成並返回一個值。