Почему Java возвращение статистики внутри блока catch не работает? - PullRequest
3 голосов
/ 17 февраля 2010

Почему следующий код всегда возвращает true, даже если выдается исключение?

public boolean write (ArrayList<String> inputText, String locationToSave){

    try {           
        File fileDir = new File(locationToSave);
        Writer out = new BufferedWriter(new OutputStreamWriter(
        new FileOutputStream(fileDir), "utf8"));

        int index = 0;
        int size = inputText.size();
        while (index < size) {
                    out.append(inputText.get(index));
                    out.append("\n");
                    index++;
                    }
        out.flush();
        out.close();

        return true;

   } catch (UnsupportedEncodingException e) {
        System.out.println("UnsupportedEncodingException is : \n" + e.getMessage());
        return false;
   } catch (IOException e) {
        System.out.println("IOException is : \n" + e.getMessage());
        return false;
   } catch (Exception e) {
        System.out.println("Exception is : \n" + e.getMessage());
        return false;
   }
}

Издание 01

Это код, который я использую для проверки предыдущего кода:

 if (fileReader.write(fileReader.read(selectedFile), selectedSaveLocation)) {
        System.out.println("The file : " + selectedFile + " as been successfully"
        + "converted to : " + selectedSaveLocation );
    } else {
        System.out.println("The file : " + selectedFile + " failed to convert!" );
    }

Ответы [ 4 ]

7 голосов
/ 17 февраля 2010

Я не думаю, что вы видите то, что, как вы думаете, вы видите. Другими словами, я почти уверен, что на самом деле он возвращает false, и вам следует проверить код вызова.

Например, я вставил ваш код в новое консольное приложение Java, сделал его статическим и написал основной метод с таким телом:

System.out.println(write(null, null)); 

Вывод был:

Exception is : 
null
false
3 голосов
/ 17 февраля 2010

Не всегда возвращает истину. Я создал тестовый проект, вызвал IOException ... и получил false! В ваших рассуждениях должна быть ошибка.

1 голос
/ 17 февраля 2010

Как все уже говорили, исключение не является тем, о котором вы думаете. Я бы угадал метод

fileReader.read(selectedFile)

регистрирует исключение, которое вы видите в своих журналах ...

Покажите нам код этого метода ... А также покажите нам исключение ...

1 голос
/ 17 февраля 2010

Если вы видите исключение в консоли, а возвращаемое значение по-прежнему равно true, проверьте тип исключения. Поскольку вы ловите Exception, я предполагаю, что это может быть непроверенный Throwable, который срабатывает. В этом случае вы бы никогда не установили флаг на false.

Я мог бы написать это так:

public boolean write (Collection<String> inputText, String locationToSave)
{

    boolean isSuccessful = false;
    Writer out;

    try
    {

        File fileDir = new File(locationToSave);
        out = new BufferedWriter(new OutputStreamWriter(
        new FileOutputStream(fileDir), "utf8"));

        for (String line : inputText)
        {
            out.append(inputText.get(index));
            out.append("\n");
        }

        isSuccessful = true;
    }
    catch (Exception e)
    {
        e.printStackTrace();
    }
    finally
    {
        cleanup(out);
    }    

    return isSuccessful;
}

private static void cleanup(Writer out)
{
    try
    {
        if (out != null)
        {
            out.flush();
            out.close();
        }
    }
    catch (Exception e)
    {
        e.printStackTrace();
    }
}
...