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

    +166

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    if(VisabilityOnOff==true){
	   document.SearchForm.searchoption[0].disabled=true;
	   document.SearchForm.searchoption[1].disabled=true;
}else if(VisabilityOnOff==false){
	   document.SearchForm.searchoption[0].disabled=false;
	   document.SearchForm.searchoption[1].disabled=false;
}

    Запостил: kuku, 09 Февраля 2011

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

    • ava Lure Of Chaos 09.02.2011 17:52 # +1

      Visability http://blogs.nsb.org/women/files/2010/08/visa-humor.jpg
      Ответить

    • ava ningun 09.02.2011 18:02 # 0

      И снова выхлоп рефака 2 строки.
      "Может, в консерватории что-то подправить?" (С) М.Жванецкий
      Ответить

    • ava wmmorgun 09.02.2011 18:45 # +4 

      } else { //Зависаем, от непонимания что делать
  var xxx = true
  while (true)
  {     
    xxx = !xxx;
    document.SearchForm.searchoption[0].disabled=xxx;
    document.SearchForm.searchoption[1].disabled=xxx;
  }
}
      Ответить

      • ava TarasB 09.02.2011 19:25 # +4

        Ник "моргун" очень подходит результату действия вашего кода.
        Ответить

    • ava Мистер Хэнки 09.02.2011 19:45 # 0

      так всё таки ... on или off ?
      Ответить

      • ava istem 09.02.2011 23:37 # 0

        судя по "VisabilityOnOff", автор и сам непонимает что происходит, и вообще - "кто здесь"...
        Ответить

    • ava C_Plus_Plus_Govno 10.02.2011 19:08 # +2

      Это наглядный пример того, что в JavaScript не хватает макросов.
      С макросами бы этот код выглядел намного лучше:

      #define Check(x) if(VisabilityOnOff==(x)){\
      document.SearchForm.searchoption[0].disabled=(x);\
      document.SearchForm.searchoption[1].disabled=(x);\
      }

      Check(True) else Check(False);

      Жаль, что многие не понимают всей мощи препроцессора.
      Ответить

      • ava istem 10.02.2011 19:29 # 0

        document.SearchForm.searchoption[0].disabled = VisabilityOnOff;
        document.SearchForm.searchoption[1].disabled = VisabilityOnOff;
        Ответить

        • ava C_Plus_Plus_Govno 10.02.2011 19:44 # 0

          У тебя слово VisabilityOnOff повторяется 2 раза, у меня - один.
          Ответить

          • ava istem 10.02.2011 19:47 # 0

            function check( a, n ) {
            while ( n-- ) {
            document.SearchForm.searchoption[n].disabled = a;
            }
            }
            /*...*/
            check( VisabilityOnOff,1 );
            --
            не?
            Ответить

            • ava C_Plus_Plus_Govno 10.02.2011 19:51 # 0

              Ну и? Ты сделал то же самое, что и я, только ещё и загнал всё в функцию, и убрал else.
              Ответить

              • ava istem 10.02.2011 19:52 # 0

                зачем там else если это функция. при том см. исходный код наверху.
                Ответить

                • ava C_Plus_Plus_Govno 10.02.2011 20:12 # 0

                  Затем, что это уже не получится дословный перевод исходного кода.
                  Ответить

                  • ava istem 10.02.2011 20:42 # 0

                    ай, ну его в баню...
                    Ответить

                    • ava C_Plus_Plus_Govno 10.02.2011 20:58 # +1

                      Вот ты зачем-то рефакторишь, а раньше и слова такого не знали, и нормально всё было.
                      Ответить

                      • ava istem 10.02.2011 21:03 # +1

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

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