Одні функції.php або розділені на багато невеликих файлів?


14

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

Тепер у моєму головному functions.phpфайлі у мене є лише require_onceдзвінки до цих файлів.

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

ЗАПИТАННЯ:

  1. Чи впливає це на продуктивність WP помітно?
  2. Чи краще тримати це все в 1 файлі (function.php)
  3. який найкращий шлях для цього?

Спасибі.

Відповіді:


12

1. Чи впливає це на видимість роботи WP?

ЯКЩО це матиме реальний вплив на деякі невеликі файли, то він матиме вплив, який має вплив менший, ніж WP: PHP та продуктивність сервера. Чи справді це впливає? Не зовсім. Але ви все одно можете просто почати робити тести на продуктивність самостійно.

2. Чи краще тримати це все в 1 файлі (function.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майже і тим же кодом.


круто. Дякую за пояснення :) перевірка файлів у певній папці, ймовірно, не допоможе в тому, що я роблю, хоча це гарна ідея. Я намагаюся зробити основу, яка є модульною. Таким чином, всі "модулі" будуть знаходитись в окремих файлах, які будуть вказані як окремі записи (Requir_once) у function.php. Таким чином, якщо хтось не хоче включати один з модулів (наприклад, тема-наладчик) - тоді можна просто прокоментувати це тощо. Такий план все одно :) Дякую ще раз.
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 );
}
Використовуючи наш веб-сайт, ви визнаєте, що прочитали та зрозуміли наші Політику щодо файлів cookie та Політику конфіденційності.
Licensed under cc by-sa 3.0 with attribution required.