php - array_shift(array_keys) - (array_pop) - 旧代码问题

Question

我的代码可以追溯到 2011 年，而且我在代码布局方式的严格标准方面遇到了很多错误。

        if (!isset($End)) {
                    $Info['Card'] = array_shift(array_keys($Info['Cards']));
                }

            if (!isset($Game['Pack']) || !in_array($Game['Pack'], $Packs)) {
                $Game['PackName'] = array_shift(array_values($Packs));
                $Game['Pack'] = $this->GetPackInfo($Game['PackName']);
            }

                $Params['Password'] = array_pop(explode("\t", $Attr));

布置此代码的正确方法是什么？

Answer 1

您没有在问题中明确说明，但我将您的短语“有关我的代码布局方式的严格标准的错误”解释为您正在运行 PHP Code Sniffer或类似的工具，这是您从中获得这些错误的地方。

PHP Code Sniffer是分析代码是否符合特定准则的出色工具。

为此有许多指导方针——几十年来，人们一直在争论是否使用制表符或空格进行缩进，是否使用 CamelCase 或 snake_case 作为变量名，以及许多其他问题。

PHP Code Sniffer 足够灵活，能够处理这些准则中的任何一个，并且可以配置为检查您的代码是否符合通用标准之一或您自己的自定义集。

近年来，PHP 世界已经确定了一组由 PHP-FIG (Framework Interop Group) 定义的通用标准。 .这些标准称为 PSR1、PSR2 等。对于 PHP Code Sniffer，您只需指定 --standard=psr2 即可根据 PSR 标准检查您的代码。

正如我所说，您可以使用其他标准，但现在大多数 PHP 代码都是按照 PSR 标准编写的，因此这是一个很好的起点。

与所有其他编码标准一样，PSR 标准纯粹是意见一致；不符合任何标准的代码不是“错误的”；只是更难阅读。

您引用的代码在几个方面不符合 PSR 标准，主要是您引用的位中变量名称的缩进和大写。

如果您想更改代码以满足标准，应该相当容易。 PHP 代码嗅探器还附带一个名为 PHPCBF(PHP 代码美化器和修复器)的配套工具，它可以自动更正至少一些 PHP 代码嗅探器检测到的错误。

另一种选择就是保持原样；如果这是一大块要合并到新项目中的旧代码，那么将其分开并从测试中简单地排除旧代码可能是有意义的。