Говнокод #4035 — Java — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

Java / Говнокод #4035

+81

public class Constants {

    /* ... */

    public static final int FOUR = 4;
    public static final int THREE = 3;

    public static final int INTEGER_FOUR = 4;
    public static final int INTEGER_FIVE = 5;
    public static final int INTEGER_ONE = 1;
    public static final int LENGTH_FOUR = 4;
    public static final int LENGTH_FIVE = 5;
    public static final int LENGTH_SEVEN = 7;
    public static final int LENGTH_EIGHT = 8;
    public static final int LENGTH_NINE = 9;
    public static final int LENGTH_ELEVEN = 11;
    public static final int LENGTH_TWELVE = 12;
    public static final int LENGTH_EIGHTEEN = 18;
    public static final int LENGTH_FIFTEEN = 15;
    public static final int ONE = 1;
    public static final int INTEGER_FIVE = 5;
    public static final int INTEGER_ONE = 1;
    public static final int PLUS_ONE = 1;
    public static final int INTEGER_THREE = 3;

    public static final Long LONG_VALUE_TEN = Long.valueOf(10);
    public static final Long LONG_VALUE_ZEARO = Long.valueOf(0);

    public static final BigDecimal BIGDECIMAL_ZERO = new BigDecimal(0);
    public static final BigDecimal BIGDECIMAL_ONE = new BigDecimal(1);

    public static final BigDecimal NEGATIVE_NUMBER_ONE = new BigDecimal(-1);

    public static final int COMPARE_RESULT_ZERO = 0 ;
    public static final int COMPARE_RESULT_ONE = 1 ;
    public static final int COMPARE_RESULT_NEGATIVE = -1 ;

    /* ... */

    public static final String NINE_STRING = "9";
    public static final String ONE_STRING = "9";

}

Хардкодить волшебные числа - плохой стиль. @dailywtf

Запостил:

bugmenot, 20 Августа 2010

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

Lure Of Chaos 20.08.2010 11:04 # −1

казалось бы, тип говнокода не оригинален. но FOUR и тут же INTEGER_FOUR - эта плюс.
хотелось бы, что бы до автора донесли, что волшебные числа сами по себе не так плохи, как ничего не говорящие константы
Ответить

wecanstoptrain 20.08.2010 11:05 # +6

$ cat /usr/include/boost/math/constants/constants.hpp
...

BOOST_DEFINE_MATH_CONSTANT(pi, 3.141592653589793238462643383279502884197169399375105820974944, 
59230781640628620899862803482534211706798214808651328230664709384460955058223172535940812848111745028410270193852110555964462294895493038196, 0)
//...
  BOOST_DEFINE_MATH_CONSTANT(third, 0.3333333333333333333333333333333333333333333333333333333333333333333333, 
3333333333333333333333333333333333333333333333333333333333333333333333333, 0)
  BOOST_DEFINE_MATH_CONSTANT(twothirds, 0.66666666666666666666666666666666666666666666666666666666666666666666, 
66666666666666666666666666666666666666666666666666666666666666666666667, 0)
  BOOST_DEFINE_MATH_CONSTANT(pi_minus_three, 0.141592653589793238462643383279502884197169399375105820974944, 
59230781640628620899862803482534211706798214808651328230664709384460955058223172535940812848111745028410270193852110555964462294895493038196, 0)

...

Ответить

Oleg_quadro 30.10.2010 23:18 # 0

С этими даблами морока, их надо округлять.
К примеру в браузерной строке адреса наберите:
javascript:alert(0.1 + 0.2)
получите
0.30000000000000004

Так что тут это может оправдано.
Так что вот так.
Ответить

kainwinterheart 20.08.2010 11:20 # +1

А повторяются, видимо, для надежности.
Ответить
- Lure Of Chaos 20.08.2010 11:23 # −1
  
  видимо, для свободы выбора
  Ответить
burdakovd 20.08.2010 11:38 # 0

а чо
public static final String ONE_STRING = "9";
Ответить
- absolut 20.08.2010 12:49 # +1
  
  всё нормально. строка-то одна :)))
  Ответить
HyperGeek 20.08.2010 13:40 # +1

Автор умеет считать только до 18-ти? Слабовато для 5-го класса =)
Ответить
Lennis 20.08.2010 14:27 # +6

LONG_VALUE_ZEARO
Ответить
- Altravert 20.08.2010 14:43 # +1
  
  Глазастый)
  Ответить
A1mighty 21.08.2010 04:31 # 0

Волшебно.
Ответить
krushi 31.10.2010 10:47 # 0

Когда-то программисты на GWBasic-е вовсю использовали этот прием в целях оптимизации кода и еще чтобы сделать его менее читаемым. При этом имена констант старались максимально сокращать. Кто-то копировал куски чужого кода в свой, так что некоторые константы кочевали из листинга в листинг. Например, если "пи" называлось PI, то число 180 почему-то часто обозначали K.
Ответить
- gorsash 08.11.2010 12:17 # 0
  
  шикарно. хорошо, что мне не пришлось работать с такими творениями
  Ответить
- abatishchev 17.11.2010 14:13 # 0
  
  зачем делать код менее читаемым? сделать себя ещё более незаменимым?
  Ответить
  - krushi 17.11.2010 14:17 # 0
    
    с той же целью, с которой сейчас запускают обфускаторы
    Ответить
    - abatishchev 17.11.2010 14:56 # 0
      
      хм..
      ну обфускатор делает нечитабельным сорс, интерпритируемый из байт-кода, иначе ео не скроешь
      а как в нежелательные руки попадал сорс на гвбасике?
      Ответить
      - krushi 17.11.2010 15:10 # 0
        
        Да нет, не обязательно дизассемблированный байт-код. Вот, например, обфускатор для JavaScript: http://www.daftlogic.com/projects-online-javascript-obfuscator.htm.
        Ответить
        
        abatishchev 17.11.2010 15:14 # 0
        
        да, верно
        говорил про обфускацию с целью скрытия
        и забыл про обфускацию с целью компрессии
        Ответить
        
        bugmenot 17.11.2010 15:47 # 0
        
        Content-Encoding: gzip уделает любой супер-дупер лицензионный javascript compressor enterprise edition
        Ответить
        
        bugmenot 17.11.2010 15:17 # 0
        
        WTF? Как соотносятся GW-BASIC и байт-код?
        
        http://jsbeautifier.org/ если чо
        Ответить
        
        krushi 17.11.2010 15:26 # 0
        
        Я как раз и хотел сказать, что байт-код тут ни при чем.
        Ответить
    - bugmenot 17.11.2010 14:58 # 0
      
      это еще до gwbasic зародилось и причиной имело жуткую нехватку памяти на тогдашних писюках
      Ответить

Говнокод: по колено в коде.

Java / Говнокод #4035

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

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