Return в finally

Вопрос по задаче

Java Multithreading, 2 уровень, 9 лекция

Решен

Драсте. Решил задачу, но интел начал светит, мол, finally не заканчивается нормально из-за return. Как я понял, всё же немного неправильно так делать? Если да, то как стоило?

try {
        builder.delete(builder.length() - 5, builder.length());
    }
    finally {
        return builder.toString();
    }
}

Смысл его: попытаться убрать строку и, если уходит в outOfBounds, в любом случае произвести возврат StringBuilder-а builder.toString(), не важно, делитнута ли часть или нет. Гугл пока молчит на мои попытки узнать что-либо по этому поводу. Это вообще считается ошибкой?

Комментарии (1)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

hidden #598481

Уровень 27

7 сентября 2021, 08:31

решение

Так а почему нельзя сначала проверить размер, и если влезает, то удалить? Неправильно вешать логику программы на работу через исключения, поэтому тебе и кидается ворнинг. Файнали нужен, чтобы подчищать работу при возникновении неожиданных исключений. Например, чтобы закрыть поток чтения из файла, если при чтении у тебя возникло исключение. А return должен быть в основном блоке кода, потому что относится к основному функционалу метода. Тебе надо было просто написать иф:

...
if (builder.length > 4) {
    builder.delete(builder.length() - 5, builder.length());
}
return builder.toString();

Чище, понятней, логичней. try-catch нужен не для отлова рантайм-исключений (NullPointer, IndexOutOfBound и прочих), а для отлова исключений, не зависящих от алгоритма программы, например различных IOException.