Модераторы: LSD, AntonSaburov
  

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Написание методов только для выбросов исключений, а нормально ли так? 
V
    Опции темы
Platon
Дата 24.10.2008, 11:58 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1801
Регистрация: 25.4.2006

Репутация: 16
Всего: 40
Здравствуйте, уважаемые.

Недавно, при написании очередного проекта, заметил за собой новый бич, обработку исключительных ситуаций выделяю в отдельный метод:

Код

private void checkAccessToCompetition(long competitionId, long userId) throws AccessDeniedException {
    if (!competitionRegistrationManager.isRegistered(userId, competitionId))
        throw new AccessDeniedException("exception.notAllowed");
    CompetitionItem item = competitionManager.get(competitionId);
    if (item.getCodingStart().getTime() > System.currentTimeMillis())
        throw new AccessDeniedException("exception.tooEarly");
}


метод-потребитель также выбрасывает AccessDeniedException.
Код

public List<Problem> getProblemSet(long competitionId, long userId) throws AccessDeniedException {
    checkAccessToCompetition(competitionId, userId);
    // <...>
}

Можно ли так безнаказанно писать?


--------------------
user posted image
Блог на винграде
Мой Блог о Java
For Java developers... only
PM MAIL ICQ   Вверх
powerOn
Дата 24.10.2008, 12:54 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


software saboteur
****


Профиль
Группа: Участник
Сообщений: 4367
Регистрация: 7.10.2005

Репутация: 47
Всего: 159
Цитата(Platon @  24.10.2008,  12:58 Найти цитируемый пост)
Недавно, при написании очередного проекта, заметил за собой новый бич, обработку исключительных ситуаций выделяю в отдельный метод:


Ну если это делает твой код более структурированным, то думаю что все ОК.


--------------------
user posted image нет времени думать - нужно писать КОД!
PM MAIL   Вверх
LSD
Дата 24.10.2008, 16:09 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Leprechaun Software Developer
****


Профиль
Группа: Модератор
Сообщений: 15718
Регистрация: 24.3.2004
Где: Dublin

Репутация: 210
Всего: 538
Sun-овцы тоже так делают.
Код

    private void RangeCheck(int index) {
    if (index >= size)
        throw new IndexOutOfBoundsException(
        "Index: "+index+", Size: "+size);
    }

(хотя при этом сами нарушают свои соглашения об именах smile )


--------------------
Disclaimer: this post contains explicit depictions of personal opinion. So, if it sounds sarcastic, don't take it seriously. If it sounds dangerous, do not try this at home or at all. And if it offends you, just don't read it.
PM MAIL WWW   Вверх
garbuz
Дата 24.10.2008, 17:00 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 677
Регистрация: 22.1.2008

Репутация: 3
Всего: 11
Цитата(LSD @  24.10.2008,  16:09 Найти цитируемый пост)
(хотя при этом сами нарушают свои соглашения об именах smile )

А где тут нарушение? Название метода с большой буквы? smile
PM MAIL   Вверх
Samotnik
Дата 24.10.2008, 18:33 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Super star !
****


Профиль
Группа: Awaiting Authorisation
Сообщений: 7192
Регистрация: 4.11.2006
Где: Минск City

Репутация: 8
Всего: 191
LSD, я не понял, смысл применения  ексепшенов в этом примере, можеш обьяснить почему нельзя просто заюзать конструкцию  if else ?
 smile   smile 
PM MAIL   Вверх
LSD
Дата 24.10.2008, 19:13 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Leprechaun Software Developer
****


Профиль
Группа: Модератор
Сообщений: 15718
Регистрация: 24.3.2004
Где: Dublin

Репутация: 210
Всего: 538
Цитата(garbuz @  24.10.2008,  18:00 Найти цитируемый пост)
Название метода с большой буквы?

Да.


Цитата(Samotnik @  24.10.2008,  19:33 Найти цитируемый пост)
я не понял, смысл применения  ексепшенов в этом примере, можеш обьяснить почему нельзя просто заюзать конструкцию  if else ?

Этот код из класса ArrayList. Он вызывается всегда когда идёт обращение к данным по индексу (get/remove/set). Чтобы не писать один и тот же код, по проверке границ его просто вынесли в отдельный метод и вызывают везде где требуется.


--------------------
Disclaimer: this post contains explicit depictions of personal opinion. So, if it sounds sarcastic, don't take it seriously. If it sounds dangerous, do not try this at home or at all. And if it offends you, just don't read it.
PM MAIL WWW   Вверх
  
 Ответ в темуСоздание новой темы Создание опроса
Правила форума "Java"
LSD   AntonSaburov
powerOn   tux
javastic
  • Прежде, чем задать вопрос, прочтите это!
  • Книги по Java собираются здесь.
  • Документация и ресурсы по Java находятся здесь.
  • Используйте теги [code=java][/code] для подсветки кода. Используйтe чекбокс "транслит", если у Вас нет русских шрифтов.
  • Помечайте свой вопрос как решённый, если на него получен ответ. Ссылка "Пометить как решённый" находится над первым постом.
  • Действия модераторов можно обсудить здесь.
  • FAQ раздела лежит здесь.

Если Вам помогли, и атмосфера форума Вам понравилась, то заходите к нам чаще! С уважением, LSD, AntonSaburov, powerOn, tux, javastic.
 
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | Java: Общие вопросы | Следующая тема »

Подписаться на тему | Подписка на этот форум | Скачать/Распечатать тему


 




[ Время генерации скрипта: 0.0868 ]   [ Использовано запросов: 22 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
 По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.