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

    +129

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    private static BigDecimal ZERO;
private static BigDecimal ONE;
private static BigDecimal TWO;
private static BigDecimal FIVE;
private static BigDecimal FOUR;
private static BigDecimal SEVEN;
public static BigDecimal SMALLDECIMAL;
private static BigDecimal THREEHUNDRED;

    http://multilevellayout.googlecode.com/svn-history/r13/trunk/MultilevelLayoutPlugin/src/multilevelLayoutPlugin/MultilevelLayoutAlgorithm.java

    Запостил: someone, 15 Июля 2012

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

    • ava absolut 16.07.2012 08:35 # −2

      и где по ссылке указанный ГК?
      Ответить

      • ava bormand 16.07.2012 08:40 # +2

        На строке 66 же.
        Ответить

        • ava absolut 16.07.2012 09:02 # +1

          ха. В поиске стояла галка учесть регистр. Вот и не находило при ручном поиске :)
          Ответить

    • ava tir 16.07.2012 12:51 # 0

      а в чем ГК?
      Ответить

      • ava absolut 16.07.2012 12:56 # +11

        не все числа перебрали
        Ответить

        • ava tir 16.07.2012 12:58 # 0

          я так понимаю они все используются внутри. или нет?
          Ответить

          • ava absolut 16.07.2012 16:26 # +2

            SMALLDECIMAL ещё и снаружи
            Ответить

            • ava bormand 16.07.2012 16:43 # 0

              Причем или я туплю, или для целых чисел указывать MathContext необязательно...
              Ответить

            • ava tir 16.07.2012 17:56 # 0

              так в чем ГК?
              Ответить

              • ava roman-kashitsyn 16.07.2012 18:32 # 0

                Вас не смущает, что статические поля обновляются из нестатичесткого метода updateSettings?
                Ответить

                • ava tir 16.07.2012 22:16 # 0

                  есть немного. ну так и надо было это постить.

                  а в том чтобы иметь константы для специальных значений, тем более для BigDecimal не вижу ничго плохого
                  Ответить

                  • ava roman-kashitsyn 16.07.2012 22:19 # 0

                    меня, например, плющит от того, что они не final и не проинициализированы. FOUR вдруг изменит значение при другой конфигурации, ага... ONE и ZERO есть в BigDecimal.
                    Ответить

                    • ava bormand 17.07.2012 05:28 # +1

                      В BigDecimal не такие ONE и ZERO, у них округление через MathContext не настроено.
                      Ответить

                    • ava tir 17.07.2012 07:42 # 0

                      к этому куску то, что они не финальные и не инициализированы - соглашусь.

                      Если же рассматривать весь класс - их нельзя сделать финальными и проинициализировать, т. к. инициализация зависит от конфига.

                      А вот публичное поле, которое прикидывается константой - тут я ничего не имею против :) (помнится, был большой холивар, когда выкладывал свой код с подобной штукой)
                      Ответить

                      • ava bormand 17.07.2012 12:21 # 0

                        > их нельзя сделать финальными и проинициализировать, т.к. инициализация зависит от конфига
                        MathContext может повлиять на округление маленьких целых чисел? Ну их тогда нафиг, эти BigDecimal'ы...
                        Ответить

      • ava guest 07.08.2012 13:08 # 0

        Константы должны быть описательными, ваш КО
        Ответить

    • ava roman-kashitsyn 16.07.2012 13:05 # +1

      > BigDecimal SMALLDECIMAL
      http://fuuu.us/179.png
      Ответить

      • ava tir 16.07.2012 13:17 # +1

        действительно странноватое название
        Ответить

    • ava konsoletyper 17.07.2012 11:33 # 0 

      private static BigDecimal THREEHUNDRED;

      Забыли ещё
      private static BigDecimal OVER9000;
      Ответить

      • ava absolut 17.07.2012 12:55 # 0

        OVER9000 вносит неоднозначность, так же как и SMALLDECIMAL. Значит должен быть public.
        Ответить

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