Закрывающая скобка многострочного вызова функции должна быть на отдельной строке - PullRequest
Новая
       23

Закрывающая скобка многострочного вызова функции должна быть на отдельной строке

0 голосов
/ 28 января 2019

Я получаю это сообщение от phpcs.мой код: 

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

1 Ответ

0 голосов
/ 29 января 2019

Ваш 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")
    )
);

Это действительно зависит от того, какой блок кода вы считаете более читаемым, а какойкодовый блок лучше всего соответствует существующему стандарту кодирования.

...