鉴于以下片段，指定失败的原因或抛出IOException是明智的吗？

Question

我正在创建一个“文件经理”来练习我的IO技能。 如果要创建一个文件，如果文件＃createNewfile（）返回false？现在，我将如何处理错误？

@Override
    public void run() {
        File file = new File(src + "\\" + srcName);
        try {
            if (file.createNewFile()) {
                MessageHandler.createSuccess(comp);
            } else {
                throw new IOException();
            }
        } catch (IOException e) {
            MessageHandler.error500(comp);
        }
    }

现在，我只是扔了一个ioexception，以捕获下一个块，但是我'我不确定这是否是最好的主意。在客户端应该很好，但是当调试时，我不知道内部出了什么问题。

Answer 1

这取决于我想没有一个明确的答案，因此社区的其他成员可能对此有所不同。

根据 javadoc ，createNewfile仅在文件已经存在时返回false。

因此，如果不应该存在文件的情况，我也只扔一个ioexception。
如果我有理由假设文件可能存在，则应以某种方式将其与程序进行交互的用户相互作用。您的工作方式确切取决于您程序的UI。

Answer 2

您向我们展示的代码片段没有多大意义。抛出一个例外，只是为了立即抓住它，不是很好的表现（除非聪明的Java优化器消除了昂贵的操作）。

您的当前运行（）方法确实：

1. MessageHandler.CreateSuccess（）如果文件不存在并且可以创建（file.createnewfile（）返回/代码>）。
2. MessageHandler.Error500（）如果文件不存在但不能创建，例如，因为没有必要的权限（file.createnewfile（）投掷and <代码> ioexception ）。
3. MessageHandler.Error500（）如果文件已经存在（file> file.createnewfile（）返回true）。 [您确定已经有文件是错误的吗

？ >捕获子句）。

我只是简单地将代码重写为类似的内容，

@Override
public void run() {
    File file = new File(src + "\\" + srcName);
    try {
        if (file.createNewFile()) {
            MessageHandler.createSuccess(comp);
        } else {
            MessageHandler.error500(comp);
        }
    } catch (IOException e) {
        MessageHandler.error500(comp);
    }
}

甚至可能找到与500不同的结果代码，更好地表示失败的原因。

例外是一种很好的方法，可以使一种方法与其他方法交流其呼叫者层次结构，无法完成工作。在一种方法中投掷并捕捉例外很少是一个好主意。