phpcs - 多行函数调用的右括号必须单独在一行

Question

我从 phpcs 收到这条消息。我的代码是：

$userdata["expirydate"] = date("Y-m-d H:i:s", mktime(date("H"), date(
        "i"), date("s"), date("m"), date("d") - 1, date("y")));

Answer 1

您的date("i")函数调用跨越多行。发生这种情况时，该规则强制右括号本身在一行上。

如果您想遵守该规则，您有一些重新格式化代码的选项。

您可以使用 PHPCS 差异报告来查看 PHPCS 希望您如何格式化它。在这种情况下，使用--report=diff显示：

--- temp.php
+++ PHP_CodeSniffer
@@ -1,3 +1,4 @@
 <?php
 $userdata["expirydate"] = date("Y-m-d H:i:s", mktime(date("H"), date(
-    "i"), date("s"), date("m"), date("d") - 1, date("y")));
+    "i"
+), date("s"), date("m"), date("d") - 1, date("y")));

这意味着 PHPCS 认为您可以做出的最小更改是编写如下代码：

$userdata["expirydate"] = date("Y-m-d H:i:s", mktime(date("H"), date(
    "i"
), date("s"), date("m"), date("d") - 1, date("y")));

这是有效的，但不是很好。

您可以将所有内容放在一条长线上，这仍然有效：

$userdata["expirydate"] = date("Y-m-d H:i:s", mktime(date("H"), date("i"), date("s"), date("m"), date("d") - 1, date("y")));

您可以拆分主调date()用以使其有效并保持行长更短：

$userdata["expirydate"] = date(
    "Y-m-d H:i:s",
    mktime(date("H"), date("i"), date("s"), date("m"), date("d") - 1, date("y"))
);

或者您甚至可以将每个参数放在一个新行上：

$userdata["expirydate"] = date(
    "Y-m-d H:i:s",
    mktime(
        date("H"),
        date("i"),
        date("s"),
        date("m"),
        date("d") - 1,
        date("y")
    )
);

这实际上取决于您发现哪个代码块更具可读性，以及哪个代码块最适合您现有的编码标准。