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

    +179

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    jQuery.atmosphere.request.transport = 'polling';
    if (jQuery.atmosphere.request.transport != 'websocket') {
    	jQuery.atmosphere.executeRequest();
    } else if (jQuery.atmosphere.request.transport == 'websocket') {
    	// ..
    }

    Сразу удар по двум яйцам.
    Сначала говорим, что transport = 'polling', а потом проверяем, не равен ли он случайно 'websocket'
    Второй удар - обычный else использовать, видимо, было не прикольно. Или у транспорта кроме "равен websocket" и "не равен websocket" может быть третье состояние? ;)

    Код актуальный и на данный момент присутствует в официальном репозитории, пруф: https://github.com/jfarcand/atmosphere/blob/master/modules/jquery/src/main/webapp/jquery/jquery.atmosphere.js#L493-503

    пс. А мы на работе удивляемся, почему 'websocket' у них не работает)

    Запостил: Shock, 03 Августа 2011

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

    • Воистину, по двум яйцам ударишь - по обоим попадешь).
      Ответить
      • за двумя заяйцами погонишься - ни одного заяйца не поймаешь
        Ответить
    • Вот это говнокод высочайшего качества!
      Ответить
    • Атмосферный такой получился...
      Ответить
    • женская логика детектед
      Ответить
    • Обычный код "на будущее". Потом строчку с присвоением перенести куда-нибудь в инициализацию и добавить пару-тройку других вариантов транспорта.
      Ответить
    • Да просто отладочный код с типичной заглушкой, какого хреа он в репозитории - это другой вопрос
      Ответить
    • true/false/pol_litra
      1/0/0.5
      наверное
      Ответить

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