WordPress PHP 编码规范-鲍成龙创作记录

鲍成龙

引号

正确的使用单引号和双引号，如果字符串中不包含变量的时候，则使用单引号，永远不要在字符串中转移引号，而是通过切换引号类型，比如：

echo '<a href="/static/link" title="Yeah yeah!">Link name</a>';
echo "<a href='$link' title='$linktitle'>$linkname</a>";

HTML 或 XML 属性中的文本应该进行转义，以便单引号或者双引号不会结束属性使得 HTML 标签无效，甚至引起安全问题，如何对属性进行转义，我们会在以后的文章中详细讲解。

缩进

首先使用制表符而不是空格进行缩进，并且使用空格把代码对齐，以便更易阅读：

$foo   = 'somevalue';
    $foo2  = 'somevalue2';
    $foo34 = 'somevalue3';
    $foo5  = 'somevalue4';

对于关联数组，如果数据含有多个元素的时候，每个元素都应该新起一行：

$query = new WP_Query( array( 'ID' => 123 ) );

$args = array(
    'post_type'   => 'page',
    'post_author' => 123,
    'post_status' => 'publish',
);
 
$query = new WP_Query( $args );

特别关注一下数组最后一个元素后面的逗号，推荐都加上，因为这样更容易调整数组的顺序，并且更容易添加新的元素，因为不用关注之前最后一个是否有逗号。

对于 switch 结构, case 语句应该比 switch 语句多缩进一个制表符， case 的内容也要比 case 条件语句缩进一个 tab。

switch ( $type ) {
    case 'foo':
        some_function();
        break;
    case 'bar':
        some_function();
        break;
}

经验法则：行首缩进使用制表符，行中对齐使用空格。

大括号

大括号的使用样式如下所示：

if ( condition ) {
    action1();
    action2();
} elseif ( condition2 && condition3 ) {
    action3();
    action4();
} else {
    defaultaction();
}

如果代码块非常长，那就要考虑将其分解成更多的较短的代码块，函数或者方法，以便降低复杂度，提高测试的便利性，当然也增加了代码的可读性。

不管是否必需，都要使用大括号：

if ( condition ) {
    action0();
}
 
if ( condition ) {
    action1();
} elseif ( condition2 ) {
    action2a();
    action2b();
}
 
foreach ( $items as $item ) {
    process_item( $item );
}

特别注意，强制使用大括号意味着禁止单语句内联控制结构，但是可以使用控制结构的替代语法（例如 if/endif, while/endwhile）——尤其是在 HTML 的模板中嵌入 PHP 代码的时候，例如：

<?php if ( have_posts() ) : ?>
    <div class="hfeed">
        <?php while ( have_posts() ) : the_post(); ?>
            <article id="post-<?php the_ID() ?>" class="<?php post_class() ?>">
                <!-- ... -->
            </article>
        <?php endwhile; ?>
    </div>
<?php endif; ?>

使用 `elseif` 而不是 `else if`

因为 else if 和 if|elseif 代码块的冒号语法不兼容，因此条件语句中使用 elseif。

数组声明

使用长数组语法 ( array( 1, 2, 3 ) ) 声明数组通常比短数组语法 ( [ 1, 2, 3 ] ) 更具有可读性，对于初学者，也更有描述性。

所以数组声明必须使用长数组语法。

闭包（匿名函数）

在一些的情况下（比如回调函数只需要用一次），可以使用闭包而非重写一个新函数来作为回调函数传递，比如：

$caption = preg_replace_callback(
    '/<[a-zA-Z0-9]+(?: [^<>]+>)*/',
    function ( $matches ) {
        return preg_replace( '/[\r\n\t]+/', ' ', $matches[0] );
    },
    $caption
);

但是不建议 filter 或 action 的回调函数使用闭包，因为通过 remove_action() / remove_filter() 移除的时候，就变得复杂了。

函数多行调用

将一个函数调用拆分为多行时，每个参数必须位于单独的行上，单行内联注释可以单独一行：每个参数不得超过一行，如果一个参数需要多行，那么可以先将其赋值给一个变量，然后再将该变量传递给函数调用。

$bar = array(
    'use_this' => true,
    'meta_key' => 'field_name',
);
$baz = sprintf(
    /* translators: %s: Friend's name */
    esc_html__( 'Hello, %s!', 'yourtextdomain' ),
    $friend_name
);
 
$a = foo(
    $bar,
    $baz,
    /* translators: %s: cat */
    sprintf( __( 'The best pet is a %s.' ), 'cat' )
);

正则表达式

正则表达式应该使用 Perl 兼容的正则表达式（PCRE, preg_函数），另外永远不要使用 /e 开关，而是使用 preg_replace_callback。在正则表达式中使用单引号字符串是最简便的，因为相比双引号，单引号字符串只有两个元序列需要转移：\' 和 \\。

PHP 开始和结束标记

在 HTML 模板中如果要嵌入多行 PHP 代码时，PHP 开始和结束标记都要自己单独一行。

正确（多行）：

function foo() {
    ?>
    <div>
        <?php
        echo bar(
            $baz,
            $bat
        );
        ?>
    </div>
    <?php
}

正确（单行）：

<input name="<?php echo esc_attr( $name ); ?>" />

错误：

if ( $a === $b ) { ?><some html><?php }

不要使用简写的 PHP 标记

重要：永远不要使用简写的 PHP 标记，使用完整版。

正确：

<?php ... ?>
<?php echo $var; ?>

错误：

<? ... ?>
<?= $var ?>

删除行尾的空格

删除每行末尾的空格，最好在文件末尾省略 PHP 结束标记，如果没有省略，那就确保删除 PHP 结束标记后面的空格。

空格的用法

始终在逗号后放置空格，并在逻辑运算符、比较运算符、字符串连接符和赋值运算符的两侧放置空格。

x === 23
foo && bar
! foo
array( 1, 2, 3 )
$baz . '-5'
$term .= 'X'

在控制语句中的左括号和右括号的两侧防止空格：

foreach ( $foo as $bar ) { ...

定义函数时，这样使用空格：

function my_function( $param1 = 'foo', $param2 = 'bar' ) { ...
function my_other_function() { ...

调用函数时：

my_function( $param1, func_param( $param2 ) );
my_other_function();

当执行逻辑运算时：

if ( ! $foo ) { ...

类型转换必须使用小写的，并且使用简短形式，(int) 而不是 (integer) ，(bool) 而不是 (boolean)，对于浮点类型转换，请使用 (float) 而不是 (real)，因为 (real) 在 PHP 7.4 已被弃用，并在 PHP 8 被移除。

foreach ( (array) $foo as $bar ) { ...
$foo = (bool) $bar;

当涉及到数组元素的时候，仅当元素的索引是变量的时候，在索引周围包含空格，例如：

$x = $foo['bar']; // 正确
$x = $foo[ 'bar' ]; // 错误
 
$x = $foo[0]; // 正确
$x = $foo[ 0 ]; // 错误
 
$x = $foo[ $bar ]; // 正确
$x = $foo[$bar]; // 错误

在 switch 代码中, case 提交和冒号之间不要有空格：

switch ( $foo ) {
    case 'bar': // 正确
    case 'ba' : // 错误
}

同样，返回的类型声明的冒号前不应有空格：

function sum( $a, $b ): float {
    return $a + $b;
}

除非另有说明，括号内应有空格。

if ( $foo && ( $bar || $baz ) ) { ...
my_function( ( $x - 1 ) * 5, $y );

格式化 SQL 语句

在格式化 SQL 语句时，如果 SQL 很复杂，可以将 SQL 语句分成几行并缩进。当然大部分 SQL 语句一行就可以了。然后将 SQL 语句中的关键字（比如 UPDATE 或者 WHERE）大写。需要更新数据库的函数的参数，传递来之前应该没有对数据进行 SQL 斜杠转义，转义应该尽可能接近查询的时候执行，并且最好使用 $wpdb->prepare() 进行。$wpdb->prepare() 是一种处理 SQL 查询的转义、引用和整数转换的方法。它使用 sprintf() 格式的子集。例子：$wpdb->prepare() 是用来对 SQL 查询进行转义、引用和整数转换等操作的方法，它 sprintf() 的一部分格式化方法，比如：

$var = "dangerous'"; // 可能未转义的原始数据
$id = some_foo_number(); // 期待是整形的数据，但是不能确定
 
$wpdb->query( $wpdb->prepare( "UPDATE $wpdb->posts SET post_title = %s WHERE ID = %d", $var, $id ) );

%s 用于字符串占位符，而 %d 用于整数占位符。注意他们没有被引用，$wpdb->prepare() 会执行转义和引用的工作。

数据库查询

避免直接操作数据库，如果有定义的函数可以获取你需要的数据，则使用它。数据库抽象（使用函数而不是查询）有助于保持代码向前兼容，并且在查询结果被缓存到内存中的时候，它可以快很多倍。

命名约定

在变量名，action/filter 的名称和函数名使用小写，不要使用驼峰式，通过下划线分割单词，如非必需不要使用缩写，让代码无歧义并能自我说明：

function some_name( $some_variable ) { [...] }

类名必须首字母大写，并用下划线分割，首字母缩写词都应全部大写：

class Walker_Category extends Walker { [...] }
class WP_HTTP { [...] }

常量必须全部大写，并用下划线分割

define( 'DOING_AJAX', true );

文件名应使用小写字母进行描述性命名，使用连字符应分隔：

my-plugin-name.php

类文件名应该基于类名，然后在前面加上 class-，然后类名中的下划线替换为连字符，例如 WP_Error 的文件名：

class-wp-error.php

在 wp-includes 目录中含有函数模板标签函数的文件，都会在文件名称末尾附加 -template 以便它们显而易见。

general-template.php

每个文件只有一个对象结构（类/接口/特征）

比如，有个名为 class-example-class.php 的文件，它只包含一个类：

// 错误： 文件 class-example-class.php
class Example_Class { [...] }
class Example_Class_Extended { [...] }

第二个类应该在自己的名字为 class-example-class-extended.php 的文件中：

// 正确：文件 class-example-class-extended.php.
class Example_Class_Extended { [...] }

// 正确：文件 class-example-class-extended.php.
class Example_Class_Extended { [...] }

函数参数的自解释标志值

调用函数时使用字符串值而不是 true 和 false：

// 错误
function eat( $what, $slowly = true ) {
...
}
eat( 'mushrooms' );
eat( 'mushrooms', true ); // true 是什么意思呢？
eat( 'dogfood', false ); // false 又是什么意思呢？false 的反面？

PHP 直到 8.0 起，才支持命名参数，但是，由于 WordPress 目前仍支持较旧的 PHP 版本，我们还不能使用这些版本。

没有命名参数，标志的值是没有意义的，每次遇到像上面例子这样的函数调用时，我们都必须搜索函数定义才明白什么意思。通过使用描述性字符串值而不是布尔值，可以使代码更具可读性。

// 正确
function eat( $what, $speed = 'slowly' ) {
...
}
eat( 'mushrooms' );
eat( 'mushrooms', 'slowly' );
eat( 'dogfood', 'quickly' );

当需要使用更多词汇来描述函数参数时，$args 数组参数可能是更好的模式：

// 更好
function eat( $what, $args ) {
...
}
eat ( 'noodles', array( 'speed' => 'moderate' ) );

动态 Hook 的插值命名

出于可读性和可发现性的目的，应使用插值而不是串联来命名动态 Hook。动态 Hook 是在其标签名中包含动态值的 Hook，例如 {$new_status}_{$post->post_type}（publish_post）。HooK 标签中使用的变量应该用大括号 { 和 } 括起来，完整的外部标签名称用双引号括起来。这是为了确保 PHP 可以正确解析内插字符串中给定的变量。

do_action( "{$new_status}_{$post->post_type}", $post->ID, $post );

可能的话，标签名称中的动态值也应尽可能简洁明了，比如 $user_id 就比 $this->id 更加清晰明了。

三元运算符

使用三元运算符很好，但是尽量让他们先测试为真，而不是假，否则容易混淆（一个例外是使用 ! empty(), 因为这里为假反而更直观）。

// (if statement is true) ? (do this) : (else, do this);
$musictype = ( 'jazz' === $music ) ? 'cool' : 'blah';
// (if field is not empty ) ? (do this) : (else, do this);

尤达表达式

if ( true === $the_force ) {
    $victorious = you_will( $be );
}

在涉及变量的逻辑比较时，始终将变量放在右侧，将常量、文字或函数调用放在左侧。如果双方都不是变量，则顺序并不重要。（在计算机科学术语中，在比较中总是尝试将 l 值放在右侧，将 r 值放在左侧。）

在上面的例子中，如果省略了一个等号（承认吧，即使是我们当中最有经验的人也会犯这种错误），你会得到一个解析错误，因为你不能赋值给一个像 true 这样的常量，如果该语句是相反的（ $the_force = true ），则赋值将完全有效，返回 1，导致 if 语句结果为 true，这种错误可能让你花费很长时间去 Debug。

尤达表达式适用于 ==, !=, ===, 和 !==。而 <, >, <= 或者 >= 情况不会出现赋值的可能，并且不易阅读，不建议使用。

“聪明”的代码

一般来说，代码的可读性比聪明和简洁更重要：

isset( $var) || $var= some_function();

虽然上面的代码很巧妙，但如果你不熟悉它，需要一段时间才能理解。所以还是这样写吧：

if ( ! isset( $var ) ) {
    $var = some_function();
}

除非绝对的必要，否则不应使用松散的比较，因为可能会产生误导。

正确：

if ( 0 === strpos( 'WordPress', 'foo' ) ) {
    echo __( 'Yay WordPress!' );
}

错误：

if ( 0 == strpos( 'WordPress', 'foo' ) ) {
    echo __( 'Yay WordPress!' );}

赋值最好不要在条件表达式中：

正确：

$data = $wpdb->get_var( '...' );
if ( $data ) {

    // Use $data
}

错误：

if ( $data = $wpdb->get_var( '...' ) ) {
    // Use $data
}

在 switch 语句中，可以将多个空的 case 放到一起。但是，如果一个 case 包含代码，然后直接进入下一个代码块，则必须明确注释。

switch ( $foo ) {
    case 'bar':       // 正确，空的 case 无需注释
    case 'baz':
        echo $foo;    // 错误, 含有代码的 case 必须 break，return 或者含有注释
    case 'cat':
        echo 'mouse';
        break;        // 正确，有 break 的 case 无需注释
    case 'dog':
        echo 'horse';
        // no break   // 正确，含有注释的 case 可以不要 break。
    case 'fish':
        echo 'bird';
        break;
}

goto 语句绝对不能用。

eval() 是非常危险并且无法确保安全。 create_function() 函数，相当于内部执行了内部执行 eval()，PHP 7.2 起已弃用，并已在 PHP 8.0 中删除，所以都不能使用。

错误控制符 @

引用 PHP 文档：

[alert class="info"]PHP 支持一种错误控制运算符：at 符号 (@)。当附加到 PHP 中的表达式时，该表达式可能生成的任何诊断错误都将被抑制。[/alert]

虽然在 WordPress 核心代码中确实存在此运算符，但它经常被懒惰地使用，而不是进行适当的错误检查。强烈建议不要使用它，甚至 PHP 文档也指出：

[alert class="info"]警告：在 PHP 8.0.0 之前，@ 运算符可以禁用将终止脚本执行的严重错误。例如，将 @ 附加到不存在的函数调用之前，由于不可用或输入错误，将导致脚本终止而没有说明原因。[/alert]

不要使用 `extract()`

extract() 是一个非常糟糕的函数，它使代码更难调试和更难理解，我们应该不应该使用它，并删除现有代码中的所有使用。

2年前

引号

缩进

大括号

使用 elseif 而不是 else if