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

    • ava ling 03.08.2011 17:09 # +4

      Воистину, по двум яйцам ударишь - по обоим попадешь).
      Ответить

      • ava Lure Of Chaos 03.08.2011 19:11 # +5

        за двумя заяйцами погонишься - ни одного заяйца не поймаешь
        Ответить

    • ava MAGnit 03.08.2011 17:22 # 0

      Вот это говнокод высочайшего качества!
      Ответить

    • ava sayidandrtfm 03.08.2011 18:07 # +4

      Атмосферный такой получился...
      Ответить

    • ava riactr 04.08.2011 09:35 # 0

      женская логика детектед
      Ответить

    • ava kipar 04.08.2011 11:54 # +1

      Обычный код "на будущее". Потом строчку с присвоением перенести куда-нибудь в инициализацию и добавить пару-тройку других вариантов транспорта.
      Ответить

    • ava Suor 05.08.2011 08:52 # +1

      Да просто отладочный код с типичной заглушкой, какого хреа он в репозитории - это другой вопрос
      Ответить

    • ava Serabas 04.05.2012 06:35 # 0

      true/false/pol_litra
      1/0/0.5
      наверное
      Ответить

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