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

    +165

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    $fileSelect = array();
    if (is_dir($filefolder)) {
        if ($dh = @opendir($filefolder)) {
            while (($file = readdir($dh)) !== false) {
                if (filetype($filefolder . $file) == "file" and $file != "0.txt") {
                    $sql  = 'SELECT * FROM bd_file WHERE file_file = ?';
                    $stmt = db_prepare($sql, $file); $res  = db_query($stmt);
                    if (!db_num_rows($res)) {
                        $fileSelect[$file] = $file;
                    }
                }
            }
        }
        closedir($dh);
    }
    ...
    unset($fileSelect);

    Сервер медленно умирал, так как данная конструкция вызывалась при каждом удобном случае, будь то добавление нового файла или скачивание. Самое поразительное, что с полученным массивом нечего не делали, а тупо уничтожили

    P.S. минимальная стоимость сайта у этой фирмы 3.000.000 рублей

    Запостил: MoLe-X, 05 Апреля 2010

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

    • Имитация бурной деятельности.
      Ответить
    • во-во, при чем со временем одних картинок скопилось 50к, не говоря уж о документах.
      Ответить
    • многократная вложенность условий конечно всё упрощает.. )
      Ответить
    • Им наерное за объем платят, а на все вопросы по тому, что работает медленно говорят, что вас типа железо старое - обновите...
      Ответить
    • три лимона фегасе!11

      побежал заниматься тем же
      Ответить
    • ого... я тоже хочу так... за объем - 3 ляма... и давить ошибки собакой вместо проверки...
      Ответить
      • на самом деле 3 ляма - мелкая цена. это правительственные сайты
        Ответить
        • Фот плят, а потом удивляются, как это злобные хакеры порушили всё и вся.
          Ответить
    • ёптить... руки нужно поломать за такое!
      Ответить

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