ループから呼び出されたメソッドを中断する

Question

非常に大きなメソッドを繰り返しリファクタリングしています。

方法では、多くのがありますが含まwhileループ：

if (count > maxResults){ 
     // Send error response 
     sendResponse(XMLHelper.buildErrorXMLString("Too many results found, Please refine your search"), out, session); 
     break; 

は、私はそれが現在、この一つの方法では3回起こるので、方法としてこれを抽出したいが、私はそうするとき、私は取得ループ内に存在しなくなったため、ブレークにエラーが発生しました。問題は、結果の最大数に達したときだけwhileループから抜け出す必要があることです。

提案がありますか？

Answer 1

メソッドがあると仮定：ループから

public boolean test(int count, int maXResult) { 
if (count > maxResults) { 
     // Send error response 
     sendResponse(XMLHelper.buildErrorXMLString("Too many results found, Please refine your search"), out, session); 
     return true; 
     } 
return false; 
} 

Callメソッドとして：

while(testCondition) { 
    if (test(count, maxResults)) { 
    break; 
    } 
} 

Answer 2

これは直接行うことはできません。

ソリューションを見つけて検索する必要がないため、ほとんどの場合、中断したいと思っています。それで、成功したかどうかを、例えば、結果を返すか、成功を示すブール値を返すなどして、呼び出された関数に示します。関数が成功を返した場合は、中断します。

Answer 3

returnを使用してメソッド内のループを解除してください。

Answer 4

Thrilerは、あなたはそれを直接行うことはできないと言います。もちろん、あなたのisTooManyResults方法は、それがメソッド内で、今の場合はあまりにも多くの結果がある場合trueとfalseそう

Answer 5

を返却する必要があるだろう

if(isTooManyResults(count)) { break; } 

：あなたはメソッドにその一部を抽出し、のようなものを行うことができますwhileループの代わりに値を返してからそれに基づいてブレークします。

すなわち

public bool refactoredMethod(parameters) 
{ 
    if (count > maxResults){ 
    // Send error response 
    sendResponse(XMLHelper.buildErrorXMLString("Too many results found, Please refine your  search"), out, session); 

    return true; 
    } 

    return false; 
}