Tasks 1-4 #29
Tasks 1-4 #29
Conversation
| @@ -16,7 +18,12 @@ | |||
| * @param value - объект, привязанный к вершине | |||
| */ | |||
| public BeanVertex addVertex(Bean value) { | |||
| return null; | |||
| if (value == null) { | |||
| throw new NullPointerException("null value"); | |||
There was a problem hiding this comment.
Мы не бросаем NullPointerException руками, говорили об этом на лекции про исключения. Это ошибка. Вы может бросить IllegalArgumentException, если хотите валидировать входящие параметры
There was a problem hiding this comment.
Хорошо. Это я исправлю, мой косяк. Просто если бы лекции не пересекались с основной учебой и собраниями команды (опять же по основной учебе), я бы знала об этом. Это просто по привычке.
There was a problem hiding this comment.
Мы говорили, что NPE - это в 99% случаев ошибка разработчика, аналог segfault в C. Если вылетает NPE - то значит мы где-то недоглядели. Поэтому избегаем использовать NPE в таком виде, как у вас. Иначе будет сложно отличить, руками это бросили или же действительно имеет место ошибка.
|
Сложно оценить. Крупной ошибкой является throw NPE (NullPointerException) - об этом говорили на лекции. Также в случае cyclic reference непрозрачно сигнализируется эта ситуация. В остально код выглядит неплохо. Смущает ситуация с кусочком кода про выставление значений поля, но я думаю, главное, что Вы разобрались, как это работает. 14/18 |
No description provided.