Checks for overly complicated boolean return statements.

This commit is contained in:
wenshao
2012-08-29 15:53:57 +08:00
parent fbbee2c651
commit eae1a7910c
2 changed files with 2 additions and 10 deletions
@@ -73,11 +73,7 @@ public class CharsetConvert {
* @return boolean
*/
public boolean isEmpty(String s) {
if (s == null || "".equals(s)) {
return true;
} else {
return false;
}
return s == null || "".equals(s);
}
}
@@ -56,11 +56,7 @@ public class CobarFailureDetecter extends DefaultDataSourceFailureDetecter {
String status = rs.getString(1);
if ("on".equalsIgnoreCase(status)) {
return true;
} else {
return false;
}
return "on".equalsIgnoreCase(status);
} catch (Exception ex) {
LOG.error("check datasource valid errror", ex);
return false;