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

    +153

    1. 1
    $("#basketName").val($(this).parent().parent().find("span").text());

    Достался мне проект хз кем написанный, и я решил поменять кое-где в верстке <span> на другой элемент (из чисто эстетических побуждений). В результате появился баг в js, собственно, вот причина.

    Запостил: mixalich7b, 10 Апреля 2014

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

    • ava Lokich 10.04.2014 17:38 # 0

      какой например?
      Ответить

      • ava bormand 10.04.2014 17:59 # +1

        > какой например?
        Ну find() не нашел элемент... Какой тут еще может возникнуть баг? Типичный жкверокод прибитый гвоздями к верстке.
        Ответить

        • ava roman-kashitsyn 10.04.2014 18:00 # 0

          быть может, имелось в виду "какой тег?"
          Ответить

          • ava bormand 10.04.2014 18:01 # +3

            Ну раз из чисто эстетических побуждений - значит <blink>.
            Ответить

            • ava TauSigma 10.04.2014 18:08 # 0

              Или <label>
              Ответить

            • ava Lokich 10.04.2014 18:23 # 0

              его кстати не удалили еще? :)
              в действительности логика автора непонятно, зачем менять span, который по умолчанию не несет в себе никаких стилей, на другой элемент?
              а так задачу можно решить примерно следующим образом
              $('span'). each(function(){ $(this).replaceWith('<blink>'+$(this).text()+'</blink>')});
              Ответить

              • ava roman-kashitsyn 10.04.2014 18:32 # +1

                > а так задачу можно решить примерно следующим образом

                Какую задачу? Задачу изменения тега в шаблоне, по которому рендерится страничка?
                Ответить

                • ava Lokich 10.04.2014 19:31 # 0

                  ну, если делать по правильному - конечно в разметке страницы менять. если по тупому - писать скрипты.
                  Ответить

                  • ava bormand 10.04.2014 19:34 # +2

                    СПГС детектед ;)

                    Автор кода просто поменял в шаблоне один <span> на какой-нибудь <div>, т.к. это показалось ему более логичным. После чего перестал работать скрипт, который он и выложил сюда ;)
                    Ответить

                    • ava Lokich 10.04.2014 19:53 # 0

                      а, ну да :) я обычно в таких случаях делаю какой-нибудь левый класс
                      Ответить

              • ava mixalich7b 10.04.2014 18:36 # +1

                У меня не было задачи поменять все span'ы на что-то другое :-)
                Проблема кода, который я выложил, в том, что он привязан к вёрстке (как уже отметили выше).
                Ответить

                • ava TauSigma 11.04.2014 15:44 # 0

                  О, и я похожее нашёл:
                  this.i.elem.find("#SomeElementId").show()
			.find("a").bind("click", this.selectPoint.bind(this));

                  И это вместо того, чтобы в одном месте прописать onclick :/
                  Ответить

              • ava TauSigma 11.04.2014 15:54 # 0

                Я вот как раз вчера в вёрстке менял пару <span'ов на <label for="textBox" accesskey="k">Привет</label>
                Ну и, обычно, на форме, label и span делают с одинаковыми стилями по умолчанию.
                Ну а далее уже классами:
                .required {
    color: #990000;
    content: " *";/*На старых браузерах вылетит валидатором*/
    font-size: 11px;
    font-weight: bold;
}
<dl class="someForm">
    <dt><label for="t" accesskey="n" class="required">Обязательно</label></dt>
    <dd><input id="t" type="text" /></dd>
    <dt><span>Кастом</span></span></dt>
    <dd><div class="custmControl">...</div></dd>
</dl>
                Ответить

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