関数

悪い:

 // Global variable referenced by following function.
// If we had another function that used this name, now it'd be an array and it could break it.
$name = 'Ryan McDermott';
 
function splitIntoFirstAndLastName(): void
{
    global $name;
 
    $name = explode(' ', $name);
}
 
splitIntoFirstAndLastName();
 
var_dump($name); // ['Ryan', 'McDermott'];

良い:

 function splitIntoFirstAndLastName(string $name): array
{
    return explode(' ', $name);
}
 
$name = 'Ryan McDermott';
$newName = splitIntoFirstAndLastName($name);
 
var_dump($name); // 'Ryan McDermott';
var_dump($newName); // ['Ryan', 'McDermott'];

6. やめてくださいwrite global Function

config() を作成しますが、同じことを行おうとする他のライブラリと競合する可能性があります。

 function config(): array
{
    return  [
        'foo' => 'bar',
    ]
}

class Configuration
{
    private $configuration = [];
 
    public function __construct(array $configuration)
    {
        $this->configuration = $configuration;
    }
 
    public function get(string $key): ?string
    {
        return isset($this->configuration[$key]) ? $this->configuration[$key] : null;
    }
}
加载配置并创建 Configuration 类的实例
 
$configuration = new Configuration([
    'foo' => 'bar',
]);

Configuration のインスタンスを使用する必要があります

7. シングルトン パターンを使用しないでください

1. は常にグローバル インスタンスとして使用されます。これらは通常、グローバル インスタンスとして使用されますが、なぜそれがそれほど悪いのでしょうか? アプリケーションの依存関係をインターフェイスを通じて公開するのではなく、コード内に隠すためです。渡さないように何かをグローバルにするのは、コードの匂いです。

2. これらは、独自の作成とライフサイクルを制御するという事実により、単一責任の原則に違反します。

3. コードへの誘導強い結合それらは本質的に原因を引き起こしますこのため、多くの場合、テスト中にコードを偽装することがかなり困難になります。

4. プログラムの存続期間中、常に状態を保持します。アプリケーションの存続期間中、状態が保持されます。単体テストでは、テストを順序付ける必要があるという状況が発生する可能性があるため、テストにもう 1 つの問題が発生します。これは、単体テストにとって大きな問題です。なぜでしょうか。各単体テストは他の単体テストから独立している必要があるためです。 .

5. ここでは、[シングルトン パターンの基本的な問題 ((http://misko.hevery.com/2008/08/25/root-cause-of-singletons/ ) この記事は Misko Hevery によって書かれました。

悪い:

 class DBConnection
{
    private static $instance;
 
    private function __construct(string $dsn)
    {
        // ...
    }
 
    public static function getInstance(): DBConnection
    {
        if (self::$instance === null) {
            self::$instance = new self();
        }
 
        return self::$instance;
    }
 
    // ...
}
 $singleton = DBConnection::getInstance();

良い:

 class DBConnection
{
    public function __construct(string $dsn)
    {
        // ...
    }
 
     // ...
}

DBConnection クラスのインスタンスを作成し、DSN 経由で設定します。

$connection = new DBConnection($dsn);

これで、 DBConnection

8 のインスタンスを使用する必要があります。条件文をカプセル化します。

Bad:

 if ($article->state === 'published') {
    // ...
}

Good:

 if ($article->isPublished()) {
    // ...
}

9.

を判断するために反意語条件を使用することは避けてください。 Bad:

 function isDOMNodeNotPresent(\DOMNode $node): bool
{
    // ...
}
 
if (!isDOMNodeNotPresent($node))
{
    // ...
}

Good:

 function isDOMNodePresent(\DOMNode $node): bool
{
    // ...
}
 
if (isDOMNodePresent($node)) {
    // ...
}

10. 条件判断を避ける

これは不可能なタスクのように思えます。この文を最初に聞いたとき、人々はそう言います。「if ステートメントなしで何ができるでしょうか?」答えは、ポリモーフィズムを使用して、複数のシナリオで同じタスクを達成できるということです。2 番目の質問は次のとおりです。 「これを実行するのは問題ありませんが、なぜこれを実行する必要があるのですか?」 答えは、前に学んだクリーン コードの原則です: 関数は 1 つのことだけを実行する必要があります。 大量の if ステートメントがある場合 クラスと関数を使用する場合、関数は複数のことを行います。実行することは 1 つだけにしてください。

悪い例:

 class Airplane
{
    // ...
 
    public function getCruisingAltitude(): int
    {
        switch ($this->type) {
            case '777':
                return $this->getMaxAltitude() - $this->getPassengerCount();
            case 'Air Force One':
                return $this->getMaxAltitude();
            case 'Cessna':
                return $this->getMaxAltitude() - $this->getFuelExpenditure();
        }
    }
}

良い例:

interface Airplane
{
    // ...
 
    public function getCruisingAltitude(): int;
}
 
class Boeing777 implements Airplane
{
    // ...
 
    public function getCruisingAltitude(): int
    {
        return $this->getMaxAltitude() - $this->getPassengerCount();
    }
}
 
class AirForceOne implements Airplane
{
    // ...
 
    public function getCruisingAltitude(): int
    {
        return $this->getMaxAltitude();
    }
}
 
class Cessna implements Airplane
{
    // ...
 
    public function getCruisingAltitude(): int
    {
        return $this->getMaxAltitude() - $this->getFuelExpenditure();
    }
}

11. 型チェックの回避 (パート 1)

PHP は弱い型付けであるため、関数はあらゆる型の引数を受け取ることができます。この自由を利用して、関数での型チェックを徐々に試してください。これを回避する方法はたくさんあります。1 つ目は、統合 API です。

#悪い:

 function travelToTexas($vehicle): void
{
    if ($vehicle instanceof Bicycle) {
        $vehicle->pedalTo(new Location('texas'));
    } elseif ($vehicle instanceof Car) {
        $vehicle->driveTo(new Location('texas'));
    }
}

Good:

 function travelToTexas(Vehicle $vehicle): void
{
    $vehicle->travelTo(new Location('texas'));
}

##12. 型チェックの回避 (パート 2)

悪い:

 function combine($val1, $val2): int
{
    if (!is_numeric($val1) || !is_numeric($val2)) {
        throw new \Exception('Must be of type Number');
    }
 
    return $val1 + $val2;
}

良い:

 function combine(int $val1, int $val2): int
{
    return $val1 + $val2;
}

13. ゾンビ コードを削除する

悪い:

function oldRequestModule(string $url): void
{
    // ...
}
 
function newRequestModule(string $url): void
{
    // ...
}
 
$request = newRequestModule($requestUrl);
inventoryTracker('apples', $request, 'www.inventory-awesome.io');

良い:

 function requestModule(string $url): void
{
    // ...
}
 
$request = requestModule($requestUrl);
inventoryTracker('apples', $request, 'www.inventory-awesome.io');

提出する