Przerwa w metodzie wywołanej z pętli

Question

Jestem refactoring bardzo dużej metody z dużą ilością powtórzeń w nim.

W metodzie istnieje wiele natomiast pętle, które obejmują:

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

chcę wyodrębnić tę metodę, ponieważ zdarza się 3 razy w tej jednej metody obecnie, ale kiedy to zrobić otrzymuję błąd w przerwie, ponieważ nie jest już w pętli. Problem polega na tym, że nadal konieczne jest wyjście z pętli while, ale tylko wtedy, gdy osiągnięta zostanie maksymalna liczba wyników.

Wszelkie sugestie?

Answer 1

Załóżmy, że metoda jest: Metoda

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; 
} 

połączenia z pętli jako:

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

Answer 2

Nie można tego zrobić bezpośrednio.

Najczęściej chcesz przerwać, ponieważ znalazłeś rozwiązanie i nie musisz już szukać. Zatem w wywołanej funkcji wskaż, że nastąpił/był sukces, na przykład zwracając wynik lub wartość logiczną, aby wskazać powodzenie. A jeśli funkcja zwraca sukces, a następnie przerwać.

Answer 3

Spróbuj przerwać pętlę w metodzie za pomocą return;

Answer 4

Jak Thriler mówi, że nie możesz tego zrobić bezpośrednio. Można wyodrębnić część, metody i zrobić coś takiego:

if(isTooManyResults(count)) { break; } 

Oczywiście metoda isTooManyResults musiałby powrócić true jeśli istnieje zbyt wiele wyników i false inaczej

Answer 5

Jeśli to jest teraz w metodzie zamiast pętli while zwracają wartość, a następnie łamią ją na podstawie tego.

tj

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; 
}