1. Java / Говнокод #14499

    +70

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 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;
    	}
    }

    изобретение типа bool

    Запостил: evg_ever, 04 Февраля 2014

    Комментарии (37) RSS

    • показать все, что скрытоТы нежно облизываешь шершавую головку моего пениса...
      Ответить
    • Надо на enum заменить для расширяемости, и будет ок.
      Ответить
    • criticality -> severity.
      Ну и да, @someone дело говорит: енум лучше. Убедись сам:
      throw new ImportException("Шеф, всё пропало!", true);
      
      throw new ImportException("Шеф, всё пропало!", Severity.CRITICAL);

      Ну и наследование от Exception вместо RuntimeException сомнительно.
      Ответить
      • Удалено администрацией.
        Ответить
        • Hello, evg_ever!

          Между прочим, у Вашего говнокода #14499 появился новый комментарий!

          Вот его содержание:
          А почему не throw new CriticalImportException?


          http://govnokod.ru/14499/#comment215600

          1. Почему удалено администрацией? Или сам тупо отредактировал?))
          2. Я с этим согласен, в общем-то
          Ответить
          • Модераторы не спят.
            Разве не очевидно, что этот комментарий разжигает межнациональную вражду, ненависть и призывы ко свержению существующего строя?
            Ответить
    • показать все, что скрытоЯ вообще заметил, что с моим появлением на этом сайте он стал скатываться в говно. Я обладаю роковым влиянием.
      Ответить
      • РАКовым влиянием
        Ответить
        • показать все, что скрытоДа. Я медленно умерщвляю любой сайт, на котором появляюсь.
          Ответить
          • А еще я ебу собак.
            Ответить
            • А свою маму не пробовал?
              Гость, на тебя вся надежда - поддержи троллинг.
              Ответить
              • Нож тебе в спину, тогда.
                Ответить
                • Stertor. Мне очень сложно сейчас говорить с тобой. Меня переполняют чувства. Я тебе честно признаюсь: мне сейчас очень стыдно за то что я раньше себя так неадекватно вел. Мне очень стыдно даже за своё сегодняшнее поведение.
                  Шутка.
                  Ответить
            • s/собак/собчак/
              Ответить
    • > public static final int NOT_CRITICAL = 1;
      > public static final int CRITICAL = 2;

      переизобретание була/энума как бы мелочи, по сравнению с тем что разраб считает от 1, а не от 0. тем более в жабе, где все культурно обнуляется по умолчанию, это просто абсурдно.

      разрабов которые от 1 считают, надо *censored* *censored* *censored* и *censored*.
      Ответить
      • > как бы мелочи
        А потом кто-то захочет кинуть особо критичную ошибку, и передаст в качестве аргумента 42... int же ;) Кто б знал в чем оно измеряется, может быть вообще по стобалльной шкале. Здесь всяко нужен энум...

        > разрабов которые от 1 считают, надо *censored* *censored* *censored* и *censored*.
        Ну зачем столько ненависти? :)
        Ответить
        • > и передаст в качестве аргумента 42...

          вот и я про то же.

          ноль/не ноль - оно как бы нормально. может в перспективе була не хватит.

          ноль/много не нулей - тоже тянет: главня проверка на ненольность, а потом детальная проверка значения.

          > Ну зачем столько ненависти? :)

          видел слишком много последствий вот таких решений. для полноты ГК не хватает только "public static final int CRITICALITY_UNKNOWN = 0;" и пусть все кому это надо проверять мучаются, городят проверки.

          у нас на системе есть несколько подобных мест, типа (char)'Y'/'N'/etc или даже (char)0/'0'/'1'. с последним вообще был настоящий феерверк, когда выяснилось что почти все аппликухи проверяли на 0, в то время как нужно было на '0' проверять. как оно до этого работало никто не знает. но так как теперь "пофиксили", менять уже никто ничего не будет.
          Ответить
    • показать все, что скрытоХуй.
      Ответить
    • показать все, что скрытоЯ ПРИЧИНЯЮ АНАЛЬНЕЙШИЕ СТРАДАНИЯ "bormand`у"!!!!!!!!!!!!!!!!!!!!!!!!!!!!! !!
      Ответить
      • Ты просто жалок. Натяни волосы от очка на батарею, и води по ним смычком.
        Ответить
        • За что минус? Жалок потому, что пиздит с анонимной учетки. Потусторонние пусть держат свое мнение при себе.
          Ответить
          • > За что минус?
            За спор с троллем.

            > Жалок потому, что пиздит с анонимной учетки.
            Ты так говоришь, как будто пиздежь с учетки horse3 чем-то отличается от анонимного.
            Ответить
      • О нет, что ты делаешь, прекрати, я не выдержу этих страданий!
        Ответить
        • Так и вижу как борманд лежит в котле связанный задницей наружу, а над ним дьявол с вилами, на которых вместо острия - фалосы...

          Спасибо, подрочил!
          Ответить

    Добавить комментарий