PHPのバリデーションの最適な方法を教えて下さい

Question

質問の概要

今PHPでバリデーションする処理を書いています。
バリデーションの内容としては、コンストラクタで受け取った配列の中身をチェックするような処理です。
そこで、バリデーション処理をリファクタリングしていてふと疑問に思いました。

PHPでバリデーションを書く際の最適な方法などはあるのでしょうか。

こんな感じの処理

既存のコードでは、このように1つでもエラーになったらすぐに return false してました。
が、これだと複数のバリデーションに該当してても、どこでエラーになったかわかりづらく、テストも書きづらいという課題を感じました。

class Hoge
{
    private $hoge = array();
    private $logger = null;

    public function __construct($hoge, $logger)
    {
        $this->hoge = $hoge;
        $this->loger = $logger;
    }

    /**
     * hogeのバリデーション
     *
     * @return boolean
     */
    public function isValid()
    {
        if (!isset($this->hoge['id']) || !is_numeric($this->hoge['id'])) {
            $this->logger->error('invalid_id');
            return false;
        }

        if (!isset($this->hoge['address']) || (strlen($this->hoge['address']) > 10)) {
            $this->logger->error(sprintf('invalid_id id=%s', $this->hoge['id']));
            return false;
        }
    }
}

こんな感じに修正してみた・・

そこで、クラスのプロパティに、 errors というものを用意して、バリデーションに引っかかったらこのプロパティに内容を追加していこうと考えました。
そうすれば、ログも全部吐かれるし、テストも、 getErrors のようなゲッターを用意して、エラーの配列が意図したものと一致してるかも確認できます。

ただ、1つ問題に遭遇しました。
これまで「この $hoge の一意なキーとなる id が存在していない」というエラー以外では、この id をログに吐く事でログチェックをやりやすくしていました。

しかし、今回からは id が存在しなくても他のバリデーションはするようにしたため、
最初のバリデーションでエラーになると、それ以降のバリデーションでエラーが起きた時にはログに吐く際の id が定義されていないため、 undefined id となってしまいます。

こういう問題ってどう解決したら良いのでしょうか。
最初の id キーの存在チェックだけ return false をしても良いかと思ったのですが、
ほかの部分との処理が異なっていてあまりすっきりしません。

class Hoge
{
    private $hoge = array();
    private $errors = array();
    private $logger = null;

    public function __construct($hoge, $logger)
    {
        $this->hoge = $hoge;
        $this->logger = $logger;
    }

    /**
     * hogeのバリデーション
     *
     * @return boolean
     */
    public function isValid()
    {
        if (!isset($this->hoge['id']) || !is_numeric($this->hoge['id'])) {
            $this->errors[] = 'invalid_id';
            $this->logger->error('invalid_id');
        }

        if (!isset($this->hoge['address']) || (strlen($this->hoge['address']) > 10)) {
            $this->errors[] = 'invalid_address';
            $this->logger->error(sprintf('invalid_id id=%s', $this->hoge['id']));
        }

        return count($this->errors) < 1;
    }
}

Answer 1

[PHP] まとめて例外をスローする小技

以前読んだ記事を紹介します。

Answer 2

例外を利用するのはどうでしょうか。

また、各入力チェックをそれぞれクラス、あるいはメソッドに分けて
テストしやすいようにするとか。

Answer 3

こんなんでは、駄目ですかね？？？
'id’の存在チェックをして？なければ、ダミーを定義しておくとか？

 public function isValid()
    {
        //idが$this->hogeにアルかな？
        if(array_key_exists('id', $this->hoge)){
            if (!isset($this->hoge['id']) || !is_numeric($this->hoge['id'])) {
                $this->errors[] = 'invalid_id';
                $this->logger->error('invalid_id');
            }
        }else{
            //idが定義されてないを一応エラーで報告
            $this->errors[] = 'no_array_key_id';
            $this->logger->error('no_array_key_id');
            //ダミーの['id]を定義しておくとかで、以降のエラーを抑止？するとか？
            $this->hoge['id'] = "dummy_id";
        }

        if (!isset($this->hoge['address']) || (strlen($this->hoge['address']) > 10)) {
            $this->errors[] = 'invalid_address';
            $this->logger->error(sprintf('invalid_id id=%s', $this->hoge['id']));
        }

        return count($this->errors) < 1;
    }
}