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

    +155

    1. 1
    2. 2
    $(element).width($(element).width());
$(element).height($(element).height());

    Просто нет слов.

    Запостил: Diwms, 10 Января 2014

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

    • ava kegdan 10.01.2014 15:18 # 0

      А вдруг?
      Ответить

    • ava guest 10.01.2014 15:45 # 0

      Подёргивание. Уж не помню, файрятся ли какие-то события на width() в jquery, но с ExtJS подобное видел. Плохая практика, конечно.
      Ответить

    • ava guest 10.01.2014 17:54 # 0

      Не файрятся. Это наверное фиксирование размеров элемента с помощью inline стилей. Нормально.
      Ответить

    • ava wvxvw 10.01.2014 21:25 # +4 

      var f = $(element).width; f(f());

      fix *
      Ответить

      • ava bormand 10.01.2014 21:26 # +2

        Почти Y комбинатор.
        Ответить

      • ava bormand 10.01.2014 21:54 # +7 

        var fix = function(f) { f(f()); }
fix($(element).width);
fix($(element).height);
        Чтобы подчеркнуть семантику и скрыть детали реализации.
        Ответить

      • ava zloirock 11.01.2014 00:06 # +2

        Контекст.
        Ответить

        • ava wvxvw 11.01.2014 00:20 # +2

          А ж.квери не привязывает методы автоматически к объектам? Хотя, да, кому такое может прийти в голову, это ж ж.скрипт...
          Ну, тогда:
          var f = function (element) {
    return function (f) { f.call(element, f.call(element)); } }($(element));
f($.prototype.width);
f($.prototype.height);

          (Скорее всего не заработает, т.как нельзя же так просто взять и объявить методы у прототипа, почти наверняка они добавляются каждому объекту отдельно.
          Ответить

          • ava Anderson 11.01.2014 03:13 # 0

            Даже так:

            (function($){
    $.fn.trueFix = function() {
        this.each(function() {
            $(this).width($(this).width());
            $(this).height($(this).height());
        }
    };
})(jQuery);

$(element).trueFix();
            Ответить

            • ava wvxvw 11.01.2014 03:23 # 0

              Ну так обычный говнокод с повторениями, повторным созданием одинаковых объектов и т.д.
              Ответить

            • ava zloirock 11.01.2014 05:08 # +8

              Пфф...
              jQuery.fn.fap = function(key){
  return this[key](this[key]());
}

$(element).fap('width').fap('height');
              Ответить

    • ava someone 10.01.2014 22:42 # +2

      Это особая уличная магия.
      Ответить

    • ava anonimb84a2f6fd141 11.01.2014 05:01 # 0

      Это типа document.location.href = document.location.href? А так да, подергивание.
      Ответить

      • ava Lure Of Chaos 12.01.2014 18:05 # +2

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

        • ava anonimb84a2f6fd141 13.01.2014 08:32 # 0

          Это height=height, я правильно понял?
          Ответить

          • ava inkanus-gray 13.01.2014 15:35 # 0

            Всё правильно. И я даже догадываюсь, для чего это. В вёрстке может быть задано max-height (и|или) min-height, но не задано явно height либо задано в процентах. Тогда этот код жёстко установит height.
            Ответить

            • ava bormand 13.01.2014 15:50 # +1

              > Тогда этот код жёстко установит height.
              А когда юзер сделает браузер поуже - весь сайт распидорасит ;)
              Ответить

              • ava Lure Of Chaos 13.01.2014 16:41 # 0

                скорее всего нет, догадываюсь, что этот код задан внутри обработчика ресайза, как раз чтобы сайт не пидорасило )
                Ответить

        • ava inkanus-gray 13.01.2014 15:49 # 0

          Да, вот полигон для демонстрации: 
          <div style="width:100%">kokoko</div>
<div style="max-width:300px">kokoko</div>
<div style="min-width:300px">kokoko</div>

<script type="text/javascript" src="http://yandex.st/jquery/2.0.3/jquery.min.js"></script>
<script type="text/javascript">
  $('div').each(function(){
    $(this).append(' style='+$(this).attr('style')+';');
    $(this).append(' width='+$(this).width());
  });
</script>
          Как бы мы ни задавали width/height, $.width() и $.height() вернут вычисленное значение.
          Ответить

    • ava guest 28.01.2014 22:20 # 0

      Такая контрукция может использоваться для очистки canvas'а (самый быстрый способ). А вообще - да, не надо так делать)
      Ответить

    • ava qixKopY 25.08.2021 09:32 # 0

      vanished
      Ответить

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