- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
- 29
public class ImportException extends Exception {
public static final int NOT_CRITICAL = 1;
public static final int CRITICAL = 2;
private int criticality = NOT_CRITICAL;
public ImportException(String message) {
this.message = message;
}
/**
* С критичностью
* @param message
* @param criticality
*/
public ImportException(String message, int criticality) {
this.message = message;
this.criticality = criticality;
}
public int getCriticality() {
return criticality;
}
public void setCriticality(int criticality) {
this.criticality = criticality;
}
}
guest 04.02.2014 15:17 # −9
guest 04.02.2014 15:28 # −7
guest 04.02.2014 15:52 # −7
guest 04.02.2014 21:55 # −1
someone 04.02.2014 16:20 # +4
roman-kashitsyn 04.02.2014 16:26 # +5
Ну и да, @someone дело говорит: енум лучше. Убедись сам:
Ну и наследование от Exception вместо RuntimeException сомнительно.
anonimb84a2f6fd141 06.02.2014 00:34 # +1
evg_ever 06.02.2014 17:02 # +1
Между прочим, у Вашего говнокода #14499 появился новый комментарий!
Вот его содержание:
А почему не throw new CriticalImportException?
http://govnokod.ru/14499/#comment215600
1. Почему удалено администрацией? Или сам тупо отредактировал?))
2. Я с этим согласен, в общем-то
eth0 06.02.2014 18:37 # +2
Разве не очевидно, что этот комментарий разжигает межнациональную вражду, ненависть и призывы ко свержению существующего строя?
guest 04.02.2014 16:31 # −7
guest 04.02.2014 16:32 # +5
guest 04.02.2014 16:54 # −5
guest 04.02.2014 21:12 # −1
Stertor 04.02.2014 21:14 # −2
Гость, на тебя вся надежда - поддержи троллинг.
Stertor 04.02.2014 21:50 # −2
LispGovno 04.02.2014 22:13 # 0
Шутка.
Stertor 04.02.2014 22:22 # 0
давай флудить и троллить! ;)
bormand 04.02.2014 22:24 # +2
kegdan 07.02.2014 08:12 # 0
Dummy00001 04.02.2014 17:17 # +3
> public static final int CRITICAL = 2;
переизобретание була/энума как бы мелочи, по сравнению с тем что разраб считает от 1, а не от 0. тем более в жабе, где все культурно обнуляется по умолчанию, это просто абсурдно.
разрабов которые от 1 считают, надо *censored* *censored* *censored* и *censored*.
bormand 04.02.2014 17:55 # 0
А потом кто-то захочет кинуть особо критичную ошибку, и передаст в качестве аргумента 42... int же ;) Кто б знал в чем оно измеряется, может быть вообще по стобалльной шкале. Здесь всяко нужен энум...
> разрабов которые от 1 считают, надо *censored* *censored* *censored* и *censored*.
Ну зачем столько ненависти? :)
Dummy00001 04.02.2014 18:13 # +1
вот и я про то же.
ноль/не ноль - оно как бы нормально. может в перспективе була не хватит.
ноль/много не нулей - тоже тянет: главня проверка на ненольность, а потом детальная проверка значения.
> Ну зачем столько ненависти? :)
видел слишком много последствий вот таких решений. для полноты ГК не хватает только "public static final int CRITICALITY_UNKNOWN = 0;" и пусть все кому это надо проверять мучаются, городят проверки.
у нас на системе есть несколько подобных мест, типа (char)'Y'/'N'/etc или даже (char)0/'0'/'1'. с последним вообще был настоящий феерверк, когда выяснилось что почти все аппликухи проверяли на 0, в то время как нужно было на '0' проверять. как оно до этого работало никто не знает. но так как теперь "пофиксили", менять уже никто ничего не будет.
guest 04.02.2014 18:15 # +1
bormand 04.02.2014 18:22 # +1
anonimb84a2f6fd141 06.02.2014 00:35 # 0
guest 04.02.2014 17:41 # −8
guest 04.02.2014 20:15 # −7
Stertor 04.02.2014 20:18 # 0
Stertor 04.02.2014 20:26 # 0
bormand 04.02.2014 20:35 # +7
За спор с троллем.
> Жалок потому, что пиздит с анонимной учетки.
Ты так говоришь, как будто пиздежь с учетки horse3 чем-то отличается от анонимного.
bormand 04.02.2014 20:34 # +8
kegdan 07.02.2014 08:20 # −3
Спасибо, подрочил!
bormand 07.02.2014 09:31 # 0
kegdan 07.02.2014 10:25 # 0
bormand 07.02.2014 12:44 # 0
kegdan 07.02.2014 12:50 # 0
Stertor 07.02.2014 10:31 # 0