Question

我有这个方法：

private Boolean compare(String property, String relationOperator,
        String operand) {
    Integer propertyValue = NumberUtils.toInt(property);
    Integer operandValue = NumberUtils.toInt(operand);

    switch (relationOperator)
    {
        case "<":  return propertyValue.compareTo(operandValue) < 0;
        case "<=": return propertyValue.compareTo(operandValue) <= 0;
/*WARN*/case "=":  return propertyValue.compareTo(operandValue) == 0;
        case ">=": return propertyValue.compareTo(operandValue) >= 0;
        case ">":  return propertyValue.compareTo(operandValue) > 0;
        case "!=": return propertyValue.compareTo(operandValue) != 0;
    }
    return Boolean.FALSE;
}

对于标记为/*WARN*/的行，FindBugs 3.0.0告诉我：

可疑的整数引用比较 com.foo.MyClass.compare（String，String，String）[最可怕的（1），高信心]

我认为代码没问题，因为我正在比较int而不是Integer s，所以我可以安全地@SuppressWarnings在这一行吗？

Answer 1

由于compareTo会返回原始int，所以您是正确的，这段代码很好。我建议将此作为针对FindBugs的错误提交。

作为一个注释，您还会为变量导致不必要的自动装箱。您可以将它们存储在int中并使用Integer.compare(propertyValue, operandValue)。

Answer 2

你的代码是可怕的，因为它使用包装类并且可以使用基元时进行比较。此外，您的代码过于聪明。你应该尝试write dumb code。像，

private boolean compare(String property, String operator, String operand) {
    int pv = Integer.parseInt(property);
    int ov = Integer.parseInt(operand);
    if (operator.equals("<")) {
        return pv < ov;
    } else if (operator.equals("<=")) {
        return pv <= ov;
    } else if (operator.equals(">")) {
        return pv > ov;
    } else if (operator.equals(">=")) {
        return pv >= ov;
    } else if (operator.equals("!=")) {
        return pv != ov;
    } else if (operator.equals("=") || operator.equals("==")) {
        return pv == ov;
    }
    return false;
}

Answer 3

我认为代码没问题，因为我比较的是整数而不是整数，所以我可以安全地@SuppressWarnings在这条线上吗？

我认为你可以。（我没有看到你在做什么本质上不安全。）

但是，您认为自己正在比较int s。

，这是不正确的

您已将propertyValue和operandValue声明为Integer，因此compareTo调用正在比较Integer个对象。（虽然方式安全......）我很确定是FindBugs所抱怨的。

请参阅Elliot Frisch的答案，了解更好的编写代码的方法......这很可能更快，而且不会激怒FindBugs。

这是真正的警告还是过于敏感的皮棉？

3 个答案: