我无法解决此代码异味:

public static boolean esStringVacio(final Object valor) {
        if (valor == null) {
            return true;
        }
        String valorTrim = valor.toString().trim();
        if ((valorTrim).equals("")) {
            return true;
        }
        if ((valorTrim).equals("null")) {
            return true;
        }
        return false;
    }


像这样尝试,但是代码气味仍然存在:

if (valor == null || valor.toString().trim().equals("") || valor.toString().trim().equals("null")) {
        return true;
    } else {
        return false;
    }

最佳答案

您可以将其缩短为:

return (valor == null || valor.toString().trim().equals("") || valor.toString().trim().equals("null"));


编辑:
您可以缩短更多时间以:

return ((String.valueOf(valor).trim().equals("null")) || (StringUtils.isBlank(valor)) ;


感谢Ernest提出的建议。

10-04 19:15