1. PHP / Говнокод #5595

    +158

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    function changeType(what) {
                    if (what == "forward") {
                            if (document.forms[0].elements['forward_list'].disabled == true) {
                                    document.forms[0].elements['forward_list'].disabled = false;
                            } else {
                                    document.forms[0].elements['forward_list'].disabled = true;
                            }
                    }
            }

    Есть замечательная ispCP и форма для настройки форвардинга е-мэйлов. Тут описывается поведение чек-бокса для включения формы.

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

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

    • при чем тут PHP?
      Ответить
    • Mr_Alone, где здесь C++?
      Ответить
    • строчками на доширак зарабатываем?
      Ответить
    • Кому 2, тому 5. Кому 5, тому 2
      Ответить
    • А как строчки сократить?
      Либо заводить новую процедуру
      reverse(bool &b)
      {
      b := !b
      };
      либо писать
      Какая-то-длинная-непонятная-мутная-хреновина-на-много-букв := ! Какая-то-длинная-непонятная-мутная-хреновина-на-много-букв.

      Первый вариант строчки не экономит, второй - говнокод.
      Писать 1й вариант в 1 строчку можно, но тоже говнисто.
      Ответить
      • if (what == "forward") {
        var input = document.forms[0].forward_list;
        input.disabled = !input.disabled;
        }
        Ответить
      • например так
        document.forms[0].elements['forward_list'].disabled != document.forms[0].elements['forward_list'].disabled;

        хоть и говнокод, но длинных-непонятных-мутных-хреновин-на-много-букв уже меньше, и сразу же понятно что оно делает.

        Через
        reverse(bool &b)
        {
        b := !b
        };
        будет тоже самое )))
        document.forms[0].elements['forward_list'].disabled = reverse(document.forms[0].elements['forward_list'].disabled);
        Ответить
        • > document.forms[0].elements['forward_list'].disabled = reverse(document.forms[0].elements['forward_list'].disabled);

          Вообще-то, я имел в виду просто
          reverse(document.forms[0].elements['forward_list'].disabled);
          Ответить
      • Фишка в "if (disabled == true) { disabled = false; } else { disabled = true; }
        Ответить
    • Кстати, зачем такие отступы?
      Ответить

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