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

    +86.5

    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
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    /**
     * константы платежей
     *
     * @author eav
     */
    public interface PaymentsConstants {
    	// какие-то цифры %)
    	static final String _25 = "25";
    	static final String _21 = "21";
    	static final String _27 = "27";
    	static final String _26 = "26";
    	static final String _93 = "93";
    	static final String _78 = "78";
    	static final String _80 = "80";
    	static final String _98 = "98";
    	static final String _81 = "81";
    	static final String _13 = "13";
    	static final String _12 = "12";
    	static final String _95 = "95";
    	static final String _20 = "20";
    	static final String _23 = "23";
    	static final String _53 = "53";
    	static final String _112 = "112";
    
    	static final String _270900 = "270900";
    	static final String _2710 = "2710";
    	static final String _2711210000 = "2711210000";
    	static final String _4907001010 = "4907001010";
    	static final String _4820409000 = "4820409000";
    	static final String _4907009000 = "4907009000";
    
    	// страны
    	static final String TM = "TM";
    	static final String AM = "AM";
    	static final String TJ = "TJ";
    	static final String UZ = "UZ";
    	static final String KG = "KG";
    	static final String AZ = "AZ";
    	static final String GE = "GE";
    	static final String UA = "UA";
    	static final String MD = "MD";
    	static final String BY = "BY";
    	static final String KZ = "KZ";
    
    	static final String ВРЕМ = "Врем";
    	static final String КОМП = "Комп";
    	static final String ДЕМП = "Демп";
    	static final String АКЦИЗ = "Акциз";
    
    	static final String УН = "УН";
    	static final String ИП = "ИП";
    
    	// импорт/экспорт
    	static final String ЭК = "ЭК";
    	static final String ИМ = "ИМ";
    }

    Тихий ужас...

    Запостил: ctepx, 11 Ноября 2009

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

    • еще бы отдельные классы сделал))))))))
      Ответить
    • зато никаких жестко вшитых переменных :) Лепота
      Ответить
    • Просто защита от опечаток. Хотя названия можно и поудобоваримей сделать.
      Ответить
      • Если уж всё так критично к безопасности, то для защиты от опечаток легче сделать метод validate(String s), который бы проверял литерал на вхождение в коллекцию строк -- и всё...
        Ответить
    • Да. А о существовании массивов он не слышал?
      Я уж не говорю о том, что всё это можно хранить в файле.
      Ответить
    • если размеры платежей действительно только такие могут быть и их ограниченное количество - то это лучше ибо:
      1. проверка, что не опечатаешься другим значением
      2. облегчение поиска, отслеживание где какая величина используется
      Ответить
    • ой, сколько же надо было выкурить
      Ответить
    • very 'flexible'
      Ответить
    • Для меня вот основное недоразумение - использование нелатиницы. Запариться же можно раскладку переключать, особенно когда горячие клавиши в русской раскладке перестают работать.
      Ответить
    • А вот и никуя, здес всё пучком, ибо не исключено что это юзается для Репортов, исходя из последних 10 строк.
      Поэтому и всё константами, и масивы сдеся накуй не упали, тока хуже бы сделали код генерации репорта.
      А для тех кто не знает -> константы юзаются вместо хардкода, чтобы меньше забивать память. Возможно этот репорт с этими константами генерится по 1000 раз в день или в час.

      Поэтому не видя всего приложения, я бы не стал называт нифкое случае гавнокодом сие тварение :) Ы

      Исчо рас пофтарюс, некоторые не видели, вернее на практике не сталкивалис с реальным ганокодом :) Ы

      У нас счас прожект. пост индуской мысли, дык вот тут пистпец на пистице.. И то, говорят тут ранше в jsp даже дёргали DB сразу.. Ы пистец проста.

      А вы какой-то класс с константами называете гавнакодом, да ещё и оценту до +10 подняли :) ппц.. ЫЫЫ
      Ответить
      • "А для тех кто не знает -> константы юзаются вместо хардкода, чтобы меньше забивать память."
        Полный бред ) Потому как константные строки в Java хранятся в пуле строк (см. String.intern()), и одинаковые строки в коде на самом деле есть одна и та же строка в памяти.
        Ответить
      • У Вас дисграфия, или Вы так шутите?
        Ответить

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