[Помогите] Помогите понять, что же накодили создатели WordPress

Тема в разделе "Wordpress", создана пользователем PHPCod3r, 24 ноя 2011.

Статус темы:
Закрыта.
Модераторы: Sorcus
  1. PHPCod3r

    PHPCod3r

    Регистр.:
    7 июн 2010
    Сообщения:
    261
    Симпатии:
    25
    Вообщем решил мальца доработать свои блог, он у меня на вордпресс, полез в код движка и по находил много интересного.
    Вообщем я этой теме я бы хотел бы обсудить не ясные моменты в коде ну и то от чего можно отказаться без потери производительности блога.
    Итак поехали, смотрю я функцию wp_unique_filename
    и вижу в ней
    PHP:
    while ( file_exists$dir "/$filename) ) {
                if ( 
    '' == "$number$ext)
                    
    $filename $filename . ++$number $ext;
                else
                    
    $filename str_replace"$number$ext", ++$number $ext$filename );
            }
    Зачем while использовать если нам всего то нужно условие проверить
    file_exists?
    Далее "порадовал" класс File_Upload_Upgrader
    PHP:
    if ( empty($_FILES[$form]['name']) && empty($_GET[$urlholder]) )
                
    wp_die(__('Please select a file'));
            if ( !empty(
    $_FILES) )
                
    $this->filename $_FILES[$form]['name'];
            else if ( isset(
    $_GET[$urlholder]) )
                
    $this->filename $_GET[$urlholder];
    Зачем нам проверять
    PHP:
    if ( empty($_FILES[$form]['name']) && empty($_GET[$urlholder]) )
    если ниже проверяются более строгии условия?
    Вывести предупреждения можно и если не срастется с условиями ниже, а лишнии код убрать....
    Кстати лишнего кода в вордпрессе дохрена, так что если кому нужно могу выкладывать в эту тему очищенные файлы WordPress
    без идиотизмов описанных выше.
     
  2. sinusoida

    sinusoida Создатель

    Регистр.:
    12 сен 2007
    Сообщения:
    23
    Симпатии:
    3
    Строка вроде "$number$ext" тоже не блещет, прямо скажем.:))
    Удалять лишний код в WP, как мне кажется, - борьба с ветряными мельницами. После автоматического обновления версии изменения затрутся:( А вот пройтись с профайлером и посмотреть какие места кода отжерают больше всего ресурсов и чего-то исправить - тут сообщество WP, уверен, скажет огромное спасибо.
    Сам стараюсь в код самого движка не заглядывать по принципу "работает? не трожь", но ведь там есть места где такое нагорожено... Подобные перлы в коде часто появляются после того как находят какую-то уязвимость, а потом разработчики вместе с самой уязвимостью добавляют дополнительных проверок в других частях кода, где потенциально может что-то вылезти. "На всякий случай" :)
     
Статус темы:
Закрыта.