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

    +153

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    /* Грязный хак для слайдера */
switch (id){
    case '1':
        num = 2;
        break;
    case '2':
        num = 1;
        break;
    default:
        num = 0;
        break;
}

    Запостил: lynx1983, 01 Августа 2013

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

    • ava Eugene 01.08.2013 16:52 # +8

      num=([null,2,1])[id]||0
      fixed.
      Ответить

      • ava 3Doomer 02.08.2013 07:00 # 0

        в этом варианте массив создаётся, нафиг надо?
        Ответить

        • ava bormand 02.08.2013 07:28 # 0

          Тогда еще более грязный хак: 
          num = id == 1 && 2 || id == 2 && 1 || 0
          Ответить

          • ava lynx1983 02.08.2013 07:34 # 0

            Самое интересное, необходимости в грязном хаке не было никакой. Банальное непонимание документации. Сие было выпилино полностью
            Ответить

            • ava bormand 02.08.2013 07:37 # 0

              > Банальное непонимание документации.
              А зачем эта штука юзалась, если не секрет?
              Ответить

              • ava lynx1983 02.08.2013 07:41 # +1

                Чувак не в том месте callback указал и получал не правильный индекс текущего слайда, этим он его менял на нужный, видимо после долгих проб и ошибок
                Ответить

          • ava Yuuri 02.08.2013 15:26 # +1

            num = max(0, 3 - min(3, id));
            Ответить

        • ava lynx1983 02.08.2013 07:36 # +1

          А чем массив не угодил?
          Ответить

          • ava 3Doomer 02.08.2013 07:37 # −1

            память жрёт
            Ответить

            • ava Eugene 02.08.2013 09:16 # 0

              Собственно, а на чем мы экономим? 8 байт на 2 числа в массиве - это называется "жрет"? Предложите свой вариант.
              Ответить

            • ava bormand 02.08.2013 09:26 # +4

              Это вызывается поди пару раз в минуту, или по клику, или вообще при инициализации... По сравнению с объемом картинок, которые переключает слайдер, этот массив ничтожен... Поэтому, имхо, экономия на спичках ;)
              Ответить

            • ava TauSigma 02.08.2013 14:05 # +1

              Я думаю, тут памяти больше отожрёт парсер свича, по сравнению с массивом. Ну и на передачу данных тоже буфер больше потребуется.
              Ответить

        • ava Eugene 28.08.2013 16:47 # 0

          Специально померял производительность говнокода-VS-кода с массивом.
          На 50К вызовов выигрыш switch составляет ~15%, что в натуральных величинах = switch(47ms), array(55ms).
          Имхо этим можно пожертвовать для улучшения читаемости и уменьшения кода.
          Ответить

          • ava 3.14159265 28.08.2013 20:01 # 0

            ложи на jsperf
            Ответить

            • ava Eugene 29.08.2013 10:11 # +1 

              http://jsperf.com/govnokod1


              Во всех браузерах код с switch работает быстрее:
              FF 23: 20%
              Chrome 29: 1%
              IE9: 26%
              Опять же, ИМХО, оно того не стоит, чтобы заморачиваться...
              Ответить

              • ava 3.14159265 29.08.2013 12:15 # 0

                Похоже что так. Добавил джве ревизии, проверил на 10-ой лисе - разница около 10%. Маловато, да.
                Но в остальных (старом хроме, и новых браузерах) - несколько раз.
                Ответить

              • ava 3.14159265 29.08.2013 12:30 # 0

                Но если вынести массив в константу, то иногда он оказывается быстрее
                Ответить

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