ЙажаСценарий / Говнокод #3245 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
function detectSourceindex(aobject){
    for (i=0; i<tabobjlinks.length; i++){
    if (aobject==tabobjlinks[i]){
    tabsourceindex=i //source index of tab bar relative to other tabs
    break
    }
    }
    }

Челы позиционируют себя как супер пупер команда разработчиков

Запостил: Vasiliy Vasiliy, (Updated )

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

    • ava turdman # Ссылка на оригинал 0 ⇈

      ГЛОБАЛЬНЫЕ ПЕРЕМЕННЫЕ (большыми буквами, чтобы передать их чудовищную вонь).
      i - глобальная (var не осилили)
      tabsourceindex - глобальная (return не осилили)

      Отступы и semicolon'ы (во всем коде нет ни одного, большинство однострочных блоков {})
      Ответить

      • ava Stormherz # Ссылка на оригинал 0 ⇈

        Ну с i да, тут конечно и не поспорить. Но tabsourceindex, допустим, необходим был глобально - да и функция своим названием не говорит нам что обязана вернуть индекс таба. Задетектить - да, должна, а вернуть - не обязательно =) Как-то не ясно, говнокод или нет.
        Ответить

        • ava turdman # Ссылка на оригинал 0 ⇈

          > Но tabsourceindex, допустим, необходим был глобально
          tabsourceindex = detectSourceindex(aobject);

          > Как-то не ясно, говнокод или нет.
          Функция, возвращающая свой результат (о она по смыслу именно возвращает результат, а не изменяет состояние системы) через глобальную переменную - говно в элементарном виде.
          Выигрыш по обьему - одно присваивание
          По скорости - 0
          Reusablility - 0
          Читаемость кода - 0
          Ответить

  • ava Vasiliy # Ссылка на оригинал 0

    Всегда делал - index=array.join().search("Строка поиска") ;
    Ответить

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

Переведи на "PHP", guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓