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")
    )
);

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

多行函数调用的右括号必须位于一行上

1 个答案: