Одиночный functions.php или разбитый на множество маленьких файлов?

14

Я создаю простую структуру с параметрами темы. Я разделил куски кода внутри functions.phpи поместил его в определенную структуру папок.

Теперь в моем основном functions.phpфайле у меня есть только require_onceобращения к этим файлам.

Но ради аргумента - скажем, у меня будет 20 файлов для включения.

ВОПРОСОВ:

  1. Влияет ли это на производительность WP видимым образом?
  2. Лучше ли держать все это в пределах одного файла (functions.php)
  3. Каков наилучший способ пойти по этому поводу?

Благодарю.

MegaMan
источник

Ответы:

12

1. Влияет ли это на производительность WP видимым образом?

Если бы это оказало реальное влияние на некоторые небольшие файлы, то это имело бы влияние, которое оказало бы влияние ниже, чем WP: PHP и производительность сервера. Это действительно влияет? На самом деле, нет. Но вы все равно можете просто начать делать тесты производительности самостоятельно.

2. Лучше ли хранить все в одном файле (functions.php)

Теперь вопрос «что лучше»? Из общего времени загрузки файлов? С точки зрения организации файлов? Во всяком случае, это не имеет никакого значения. Сделайте это так, чтобы не потерять обзор и сохранить результат так, как вам удобно.

3. как лучше всего это сделать?

То , что я обычно делаю это просто зацепив где - то в ( plugins_loaded, after_setup_themeи т.д. - в зависимости от того, что вам нужно) , а затем просто требовать их всех:

foreach ( glob( plugin_dir_path( __FILE__ ) ) as $file )
    require_once $file;

В любом случае, вы можете сделать его немного более сложным и гибким. Взгляните на этот пример:

<?php

namespace WCM;

defined( 'ABSPATH' ) OR exit;

class FilesLoader implements \IteratorAggregate
{
    private $path = '';

    private $files = array();

    public function __construct( $path )
    {
        $this->setPath( $path );
        $this->setFiles();
    }

    public function setPath( $path )
    {
        if ( empty( $this->path ) )
            $this->path = \plugin_dir_path( __FILE__ ).$path;
    }

    public function setFiles()
    {
        return $this->files = glob( "{$this->getPath()}/*.php" );
    }

    public function getPath()
    {
        return $this->path;
    }

    public function getFiles()
    {
        return $this->files;
    }

    public function getIterator()
    {
        $iterator = new \ArrayIterator( $this->getFiles() );
        return $iterator;
    }

    public function loadFile( $file )
    {
        include_once $file;
    }
}

Это класс, который делает в основном то же самое (требуется PHP 5.3+). Преимущество состоит в том, что он немного более детализирован, поэтому вы можете легко загружать файлы из папок, которые необходимы для выполнения определенной задачи:

$fileLoader = new WCM\FilesLoader( 'assets/php' );

foreach ( $fileLoader as $file )
    $fileLoader->loadFile( $file );

Обновить

Поскольку мы живем в новом мире PHP v5.2, мы можем использовать его \FilterIterator. Пример кратчайшего варианта:

$files = new \FilesystemIterator( __DIR__.'/src', \FilesystemIterator::SKIP_DOTS );
foreach ( $files as $file )
{
    /** @noinspection PhpIncludeInspection */
    ! $files->isDir() and include $files->getRealPath();
}

Если вам нужно придерживаться PHP v5.2, то вы все равно можете использовать \DirectoryIteratorи тот же код.

кайзер
источник
круто. Спасибо за объяснение :) Проверка файлов в определенной папке, вероятно, не поможет с тем, что я делаю, хотя это хорошая идея. Я пытаюсь сделать структуру, которая является модульной. Таким образом, все «модули» будут находиться в отдельных файлах, которые будут перечислены как отдельные записи (require_once) в functions.php. Таким образом, если кто-то не хочет включать один из модулей (например: theme-customizer) - тогда можете просто закомментировать это и так далее. Это план в любом случае :) Еще раз спасибо.
MegaMan
@MegaMan Вы также можете отфильтровать вывод перед вызовом loadFile()или require_once. Поэтому просто предложите что-то вроде поддержки тем, где пользователь может использовать add_theme_support()/remove_*()только те модули, которые ему нужны. Затем просто используйте результат для $loadFile()или glob(). Кстати, если это было ваше решение, пожалуйста, пометьте его как таковое. Благодарю.
Кайзер
0

Я переделал ответ @kaiser немного под свои нужды - думал, что поделюсь им. Я хотел больше вариантов, они объясняются внутри кода и в примере использования ниже.

Код:

<?php

defined( 'ABSPATH' ) OR exit;

/**
 * Functions_File_Loader
 * 
 * Makes it possible to clutter the functions.php into single files.
 * 
 * @author kaiser
 * @author ialocin
 * @link http://wordpress.stackexchange.com/q/111970/22534
 *
 */

class Functions_File_Loader implements IteratorAggregate {

    /**
     * @var array
     */
    private $parameter = array();

    /**
     * @var string
     */
    private $path;

    /**
     * @var string
     */
    private $pattern;

    /**
     * @var integer
     */
    private $flags;

    /**
     * @var array
     */
    private $files = array();

    /**
     * __construct
     *
     * @access public 
     * @param array $parameter
     */
    public function __construct( $parameter ) {
        $this->set_parameter( $parameter );
        $this->set_path( $this->parameter[ 'path' ] );
        $this->set_pattern( $this->parameter[ 'pattern' ] );
        $this->set_flags( $this->parameter[ 'flags' ] );
        $this->set_files();
    }

    /**
     * set_parameter
     *
     * @access public 
     * @param array $parameter
     */
    public function set_parameter( $parameter ) {
        if ( empty( $parameter ) )
            $this->parameter = array('','','');
        else
            $this->parameter = $parameter;
    }

    /**
     * get_parameter
     *
     * @access public 
     * @return array
     */
    public function get_parameter() {
        return $this->parameter;
    }

    /**
     * set_path
     *
     * defaults to get_stylesheet_directory()
     * 
     * @access public 
     * @param string $path
     */
    public function set_path( $path ) {
        if ( empty( $path ) )
            $this->path = get_stylesheet_directory().'/';
        else
            $this->path = get_stylesheet_directory().'/'.$path.'/';
    }

    /**
     * get_path
     *
     * @access public 
     * @return string
     */
    public function get_path() {
        return $this->path;
    }

    /**
     * set_pattern
     *
     * defaults to path plus asterisk »*«
     * 
     * @access public 
     * @param string $pattern
     */
    public function set_pattern( $pattern ) {
        if ( empty( $pattern ) )
            $this->pattern = $this->get_path() . '*';
        else
            $this->pattern = $this->get_path() . $pattern;
    }

    /**
     * get_pattern
     *
     * @access public 
     * @return string
     */
    public function get_pattern() {
        return $this->pattern;
    }

    /**
     * set_flags
     *
     * @access public 
     * @param integer $flags
     */
    public function set_flags( $flags ) {
        if ( empty( $flags ) )
            $this->flags = '0';
        else
            $this->flags = $flags;
    }

    /**
     * get_flags
     *
     * @access public 
     * @return integer
     */
    public function get_flags() {
        return $this->flags;
    }


    /**
     * set_files
     *
     * @access public 
     */
    public function set_files() {
        $pattern = $this->get_pattern();
        $flags = $this->get_flags();
        $files = glob( $pattern, $flags );
        $this->files = $files;
    }


    /**
     * get_files
     *
     * @access public 
     * @return array
     */
    public function get_files() {
        return $this->files;
    }

    /**
     * getIterator
     * 
     * This function name has to be kept
     * 
     * @access public 
     * @return void
     */
    public function getIterator() {
        $iterator = new ArrayIterator( $this->get_files() );
        return $iterator;
    }

    /**
     * load_file
     *
     * @access public 
     * @param string $file
     */
    public function load_file( $file ) {
        include_once $file;
    }
}


Пример использования:

$parameter = array(
        // define path relative to get_stylesheet_directory()
        // optional, defaults to get_stylesheet_directory()
        'path' => 'includes/plugins',
        // optional, defaults to asterisk »*«
        // matches all files ending with ».php« 
        // and not beginning with »_«, good for quickly deactivating 
        // directories searched are »path« and »subfolders«
        // Additional examples:
        // '{*/,}{[!_],}func-*.php' same as above but for files with a prefix
        // '[!_]*.php' php files in defined »path«, not beginning with »_« 
        'pattern' => '{*/,}[!_]*.php',
        // optional, defaults to 0
        // needed if for example brackets are used
        // more information: http://www.php.net/manual/en/function.glob.php
        'flags' => GLOB_BRACE
    );
// create object
$functionsfileloader = new Functions_File_Loader( $parameter );
// load the files
foreach ( $functionsfileloader as $file ) {
    $functionsfileloader->load_file( $file );
}
Nicolai
источник