捕捉NumberFormatException

Question

下面是别人写的类。

我面临的问题是，当它进入null as the rawString的parse method时，它抛出了NumberFormatException。

所以我想要做的是，我应该抓住NumberFormatException和set the value itself as null。所以我这样做是对的吗？

public class ByteAttr {

    @JExType(sequence = 1)
    private Byte value;

    public static ByteAttr parse(String rawString) {
        ByteAttr attr = new ByteAttr();
        try {
            attr.setValue(Byte.valueOf(rawString));
        } catch (NumberFormatException nfEx) {
            attr.setValue(null);
        }
        return attr;
    }

    public Byte getValue() {
        return this.value;
    }

    public void setValue(Byte value) {
        this.value = value;
    }
}

Answer 1

你需要做四件事：

1. 决定无法解析的数字字符串在您将使用该方法的上下文中的含义。这是否意味着程序中存在内部问题？一个损坏的文件？用户输入错误？没有错，但是这个字符串需要被处理differently?
2. Decide，这是最好的处理方式，考虑到这一点。几乎所有情况下，如果错误是由外部输入触发的，您需要将其报告回来。替换null 可能是处理它的好方法。
3. 记录您决定要做的事情。如果一个方法要返回具有特定含义的null，则需要将其写为注释，最好是Javadoc commments.
4. Implement您的决定。

我得到的印象可能是不公平的，您直接跳到了步骤4，而没有考虑可能的原因和适当的问题报告。

Answer 2

您可以添加一个带有如下条件的提前退出：

if (rawString != null) {
    return attr; // or other value you prefer
}

您还可以确保parse方法的调用方测试null值，并避免在空值时调用parse。

Answer 3

这取决于应用程序中对空值的容忍度。如果您希望用户不将空字符串传递给parse()方法，那么您应该进行防御性的null检查并抛出异常。

if (null == rawString) {
    throw new CustomException("rawString cannot be null");
}

这同样适用于NumberFormatException的catch块，您应该抛出一个带有适当消息的异常，而不是默默地将Byte属性的值设置为null。

但是，如果null是完全可接受的，那么您应该执行防御性的null检查，并将Byte属性设置为null。NumberFormatException当然不应该被抑制，IMHO。