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

    +160

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    var fixGetMoreOrders = function(data){
    var i = 0;
    $.each(data.resultObject, function(k,v){
        i++;
    });
    if(i>=10){
        $('.btnGetOrders').css('display', 'block');
    }
    if(i<10){
        $('.btnGetOrders').css('display', 'none');
    }
};

    Предыдущему разработчику нужно было посчитать количество ордеров, при том, что поле data['count'] вертается с сервера =)

    Запостил: makzimko, 16 Мая 2014

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

    • ava guest 16.05.2014 17:35 # +2

      блть, я бы даже не догадался так сделать
      Ответить

    • ava DrDre 16.05.2014 19:10 # 0

      Я не понял, он че длину массива так считал? Или это не массив? В любом случае это просто эпический пиздец.
      Ответить

    • ava orion 17.05.2014 12:55 # +1

      жуквери головного мозга
      Ответить

      • ava 3.14159265 21.05.2014 17:37 # 0

        Ну через Array.forEach() тоже можно считать.
        жквери не виноват в том что лямбды предпочитают старому-доброму императиву.
        Ответить

    • ava kyzi007 20.05.2014 15:30 # 0

      Вот что случается если хотеть человека строго с жуквери но иметь деньги только на студента.
      Ответить

    • ava guest 20.05.2014 15:44 # 0

      var i = 0;
      $.each(data.resultObject, function(k,v){
      i++;
      });

      Это count?
      Ответить

      • ava DrDre 20.05.2014 19:42 # −1

        Нет. Это просто лютый говнокод. Если говорить об аналоге пхпшного count() то в js это метод length. То есть его код можно переписать так:
        function fixGetMoreOrders(data){
   var btn = $('.btnGetOrders');
   if(data.resultObject.length >= 10){
      btn.show();
   }
   if(data.resultObject.length < 10){
      btn.hide();
   }
};
        Ответить

        • ava guest 21.05.2014 15:58 # 0

          >if(data.resultObject.length < 10){
          else?
          Ответить

        • ava guest 23.05.2014 12:20 # +1 

          function fixGetMoreOrders(data){
    $('.btnGetOrders').toggle(data.resultObject.length >= 10);
};
          Ответить

    • ava 3.14159265 21.05.2014 17:35 # 0

      ФункциАнальненько.
      Ответить

    • ava sladkijBubaleh 30.05.2014 09:57 # 0

      зато не так как все :P
      Ответить

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