The following looks like messy code, but I can't think how to make it neater. Any ideas? I want to call doSearch for values of 10, 20 and 30. If no results are returned for a value, then I want to try for the following value. Otherwise, just exit out. I know this would work, but is it the most readable way?
SearchResult result = doSearch("10");
if (result.getResults() == null) {
result = doSearch("20");
if (result.getResults() == null) {
result = doSearch("30");
if (result.getResults() == null) {
// put code to deal with lack of results here
}
}
}
解决方案
Here's a suggestion:
SearchResult result = null;
for (String attempt : "10,20,30".split(","))
if ((result = doSearch(attempt)) != null)
break;
if (result == null) {
// put code to deal with lack of results here
}
(as suggested by Marko Topolnik in the comments.)