1. JavaScript / Говнокод #12372

    +159

    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
    var fmCategoryClass = function(fmCategory) {
                switch (parseInt(fmCategory))
                {
                    case 0: return 'important';
                    case 1: return 'sell';
                    case 2: return 'buy';
                    case 3: return 'exchange';
                    case 4: return 'service';
                    case 5: return 'rent';
                    case 6: return 'close';
                    default: return '';
                }
            }

    http://onliner.by - один из самых посещаемых сайтов в Беларуси - содержит прям в вёрстке

    Запостил: daemon_master, 28 Декабря 2012

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

    • ava bormand 28.12.2012 05:36 # +6

      Массивы - для слабаков! Свитч - путь настоящего профессионала.
      Ответить

      • ava daemon_master 28.12.2012 07:57 # +2

        путь Джедая :D
        Ответить

      • ava Vindicar 29.12.2012 11:36 # 0

        а вдруг потребуется сделать константы fmCategory простыми числами?
        Ответить

        • ava bormand 29.12.2012 14:35 # +4

          > простыми числами
          2 3 5 7 11 ? Ну а в чем проблема, в js разреженные массивы вполне работают.
          Ответить

    • ava Eugene 29.12.2012 02:03 # +2

      Типо так?
      var fmCategoryClass = function(fmCategory) {
	return (['important',
		'sell',
		'buy',
		'exchange',
		'service',
		'rent',
		'close'])[fmCategory] || '';
};
      Ответить

      • ava anonimb84a2f6fd141 30.12.2012 20:22 # +1

        Хайлайтер что-то облажался, вроде нет опечатки...
        Ответить

      • ava guest 22.01.2013 13:08 # 0

        А как вы думаете что будет если вы в к примеру fmCategoryClass вызовете с "sort" и т.п. параметрами?
        2. Круглые скобки зачем?
        Ответить

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