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

    +155

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    $cnt_ = count($array_objects['rate']);
    
    for($i=0; $i<$cnt_; $i++) {
      //...
    }
    
    $DB->query('REPAIR TABLE `' . $rates_table . '` QUICK;');

    Кусок скрипта для импорта данных из XML...

    Запостил: Eisenhorn, 13 Августа 2009

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

    • И что?
      Ответить
    • И причём здесь говнокод? Вне контекста вообще непонятный кусок кода.
      Ответить
      • а по-моему, все понятно.
        фишка в том, что про foreach народ и не слышал.
        Ответить
        • for на самом деле быстрее за счет того что не создает копию объекта.
          А вот зачем лечить таблицу после вставки - не ясно.
          Ответить
          • у вас на практике где-нибудь была проблема с нагрузкой, которая решилась заменой форичей на форы ? вряд ли. когда кончатся деньги инвесторов на мем-кеш кластеры - тогда, и только тогда будет рефакторинг, и в последнюю очередь будет замна форичей на форы.
            разве нет? так что все эти мнимости с форичами и одинарными-двойными ковычками - булщит
            Ответить
            • Пока до такого не доходило. Но вообще при переборе объектов понадобиться может. А ковычки - это да, бред.
              Ответить
    • piar ne ponjatno chego...

      p.s. predpologaju, chto zapros dolzhen byt' v cikle, a $rates_table = $array_objects['rate'][$i];

      koroche avtor topika - ne vyspalsja... YURGEN
      Ответить
    • народ нынче тупой :)
      скажите мне, а кто из вас писал восстановление таблиц внутри какого-то левого скрипта, который "неизвестно что делает"?
      а вы оказывается сами говнокодеры :)
      Ответить
    • стыдно, граждане )
      репаир - это обслуживание бд, делается DBA при помощи утилиты mysql_repair и тогда, когда он посчитает нужным. делать его в скрипте - это даже не говнокодерство, это просто автору сего надо идти в манагеры, или в грузчики
      Ответить
    • Прикол в том, что он такого там делает, что repair приходится вызывать.
      Ответить

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