web-dev-qa-db-ja.com

一つのfunctions.phpまたは多くの小さなファイルに分割?

テーマオプション付きのシンプルなフレームワークを作成しています。私はコードの塊をfunctions.php内に分割し、それを特定のフォルダー構造内に配置しました。

私のメインのfunctions.phpファイルには、これらのファイルへのrequire_once呼び出ししかありません。

しかし、議論のために - インクルードするファイルが20個になってしまうとしましょう。

質問:

  1. これはWPのパフォーマンスに目に見える形で影響しますか?
  2. すべて1つのファイル(functions.php)に収めることをお勧めします。
  3. これについて最善の方法は何ですか?

ありがとう。

14
MegaMan

1. これはWPのパフォーマンスに明らかな影響を与えますか?

_ if _ いくつかの小さなファイルには実際に影響がありますが、WP:PHPとサーバーのパフォーマンスよりも影響が小さい影響があります。それは本当に影響がありますか?あんまり。しかし、それでも自分でパフォーマンステストを始めることができます。

2. 1つのファイル(functions.php)内にすべて保持する方が良いですか

さて、問題は「より良いもの」ですか?全体的なファイルのロード時間から?ファイル編成の観点から?とにかく、違いはありません。あなたが概要を失うことがなく、あなたにとって楽しい方法で結果を維持することができるような方法でそれをしてください。

3. これについて最も良い方法は何ですか?

私が通常していることは、単にどこかにフックすること(plugins_loadedafter_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 );

更新

私たちは新しいpost 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とほとんど同じコードで行くことができます。

11
kaiser

私は@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 );
}
1
Nicolai