metaclass: (дзедline)
[personal profile] metaclass
А почему в java нету готовых методов, которые бы позволяли парсить числа, не кидая исключений?
Т.е. возвращали бы true+число, либо false, типа как int.TryParse в дотнете.

Date: 2013-03-21 09:53 am (UTC)
From: [identity profile] kzn.livejournal.com
Хмм. откуда такой постулат?
В исходном посте в C# - tryParse - попробовать распарсить. в Java - parseInt() - требует распарсить. Вполне логично по-моему.

Date: 2013-03-21 09:57 am (UTC)
From: [identity profile] vp.livejournal.com
Так мы дойдем, что проверка if() при не выполнении условия тоже будет бросать исключения вместо того, чтобы ветвиться.

Date: 2013-03-21 10:23 am (UTC)
From: [identity profile] kzn.livejournal.com
Не надо преувеличивать :) Скажем isInteger() исключений бросать не должна.

Опять же, что плохого в исключениях?

Date: 2013-03-21 10:24 am (UTC)
From: [identity profile] vp.livejournal.com
Есть мнение, что исключение - это исключительная ситуация. По определению здесь нет факта исключительной ситуации.

Date: 2013-03-21 10:26 am (UTC)
From: [identity profile] kzn.livejournal.com
Каков критерий исключительной ситуации?
Edited Date: 2013-03-21 10:27 am (UTC)

Date: 2013-03-21 10:30 am (UTC)
From: [identity profile] vp.livejournal.com
Ситуация, когда нормальная работа приложения либо модуля не может быть продолжена обычным образом, и вам необходимо вернуться, возможно, на несколько уровней вложенности в первоначальное состояние, передав наверх информацию о ситуации.

Date: 2013-03-21 10:36 am (UTC)
From: [identity profile] kzn.livejournal.com
Ну а чем это не подходит к ситуации, когда мы ожидаем распарсит число, а не получается? Тут тоже возможно нужно передать информацию на уровень выше?

Чем принципиально это отличается от IOException?

Date: 2013-03-21 10:38 am (UTC)
From: [identity profile] vp.livejournal.com
А не факт, что IOException - это правильное архитектурное решение. У нормальных людей эти операции возвращают результат операции, число там записанных байт и т.п. Там исключение - как корове седло.

Date: 2013-03-21 10:39 am (UTC)
From: [identity profile] kzn.livejournal.com
Ну ок. А какое труъ исключение?

Date: 2013-03-21 10:42 am (UTC)
From: [identity profile] vp.livejournal.com
Тру исключение то, которое спроектировано для выхода на определенный уровень обработки наверх.
То есть правильно было бы иметь по 2 версии функций, с исключениями и без них.
Просто использование исключений вместо кодов возврата приводит к тому, что 50 строчек линейного рабочего кода, например, для работы с файлом, превращаются в гору try блоков, которые могут быть не нужны.

Date: 2013-03-21 10:56 am (UTC)
From: [identity profile] kzn.livejournal.com
Ну почему же? Что-то пошло не так при обработке файла? IOException и обрабатывать его на нужном уровне. Код вполне линейным получается.

В жабе конечно есть Checked Exceptions, но при правильном использовании вполне полезны.

Date: 2013-03-21 10:58 am (UTC)
From: [identity profile] vp.livejournal.com
правильнее, мне кажется, было бы все-таки поиметь на месте код возврата и если нужно - бросить исключение. Потому что чаще это при таких операциях не сильно нужно.
Сделать из кода возврата исключение пишется лаконичнее, чем сделать код возврата из исключения :)

Date: 2013-03-22 07:50 am (UTC)
From: [identity profile] sil-e-nt.livejournal.com
Ребе. Вот смотрите, решила индустрия вас послушать и наклепала дублирующих функций с кодами возврата. Хомячки обрадовались (это ж не надо теперь эти долбаные исключения обрабатывать) и начали их использовать. А принуждения к верификации кода возврата -- нету! Нету, блять, ващще! Кроме вменяемого тимлида, который будет каждый день проверять выхлоп и все фиксы и тэпэ и тэдэ.

А теперь представьте во что превратятся либы через пару лет. Как вы будете материться на то, что "какого хера оно продолжает\дохнет\подставьте сами без какой либо инфы?".

Date: 2013-03-21 10:30 am (UTC)
From: [identity profile] metaclass.livejournal.com
Очень простой.
Если работу можно продолжать - исключение кидать не принято.
Исключения - для случаев, когда надо свалится в корень исполняемого потока (main loop или там обработчик выдающий 500 в веб-сервисе).
Делать логику на исключениях - очень нехорошо, но конкретно в данном случае жаба вынуждает это делать.

Date: 2013-03-21 10:54 am (UTC)
From: [identity profile] kzn.livejournal.com
Это крайность имхо.

Исключение кидаются когда произошло нечто неожиданное с точки зрения разработчика.

parseInt() ожидает увидеть число. А получило. - вполне неожиданное событие.

Ожидали прочитать из файла, а он закончился - тоже.

Но и то, и другое можно сделать на статусах, как C.

Date: 2013-03-21 10:29 am (UTC)
From: [identity profile] metaclass.livejournal.com
с if часть алгоритмов сильно короче и нагляднее. Ну, еще насчет производительности вопрос сложный.

Date: 2013-03-21 10:38 am (UTC)
From: [identity profile] kzn.livejournal.com
Да. С этим согласен. Сам переписывал парсера, который не проверял границы, а кидал Out Of Bounds. Стало короче и даже быстрее.

Но вот с числами.. можно конечно сделать Integer Integer.parseInt() и возвращать null если не удалось распарсить.

Date: 2013-03-21 11:36 am (UTC)
From: [identity profile] henu3detb.livejournal.com
Потом этот налл удетит по веб-сервису на N удаленных систем. Там сохранится в базах, потечет по бизнес-процессам и через пол года на другом континенте начнет вываливаться NPE. Нал он такой, да. Как и любители нала.

Date: 2013-03-21 11:39 am (UTC)
From: [identity profile] kzn.livejournal.com
Ну ногу-то практически всегда отстрелить можно. Особенно при остром желании.

Date: 2013-03-21 11:49 am (UTC)
From: [identity profile] henu3detb.livejournal.com
Про острое желание я ничего не говорил. А про случайные ситуации - говорил. И от таких ситуаций взрослые люди стараются себя страховать. Предохранители делают на приборах или эксепшны в коде. От всех несчастных случаев это не застрахует, но это не повод расслабляться и рассказывать про отстреленную ногу, случайность, карму, фазы луны и прочую ересь.

Date: 2013-03-21 11:59 am (UTC)
From: [identity profile] kzn.livejournal.com
Ну так и говорю, что то, что parseInt() кидает исключение - вполне нормально.

На грабли всегда наступить можно.

Profile

metaclass: (Default)
metaclass

April 2017

S M T W T F S
      1
2345678
9101112 131415
16171819202122
23242526272829
30      

Most Popular Tags

Style Credit

Expand Cut Tags

No cut tags
Page generated Aug. 25th, 2025 10:22 pm
Powered by Dreamwidth Studios