3項目入れずれかに値があればエラーにする方法

Question

前提・実現したいこと

CakePHP2を使っています。

バリデーションチェックの話ですが、
入力項目がA,B,Cと３つあり、3項目のうち1つでも入力値が入っていれば残りの2項目も必須チェックを行いたいのですが、
良い方法が思いつきません。

例えばですが、
・Aに値が入っており、BとCに値が入っていない場合はエラーにする。
・A,B,Cに値が入っていない場合はエラーなし。
・A,B,Cに値が入っている場合もエラーなし。

試したこと

以下のように３つの項目をチェックする関数を作りましたが、
複雑な見た目になってしまいました。

function checkThreeItems($item1, $item2, $item3) {
    if (!empty($item1)) {
        if (!empty($item2)) {
            if (empty($item3)) {
                // エラー
            }
        } else {
            // エラー
        }
    } else if (!empty($item2)) {
        if (!empty($item3)) {
            if (empty($item1)) {
                // エラー
            }
        } else {
            // エラー
        }
    } else if (!empty($item3)) {
        // エラー
    }
}

Comments

[azuapricot]

３つ入力されてない場合はチェックなし、
それ以外はABC全て必須チェックする、ではだめなのでしょうか。

質問文からはどうしたいのか読み取れません。

[m.ts10806]

↑このほうがシンプルですね。

[m.ts10806]

ただ、なぜ手動でバリデートしてるのかなと

[m.ts10806]

https://teratail.com/help/question-tips#questionTips4-2
全て検証したうえで「解決済み」としてください。
私は前提条件無視で単に「今のコードをシンプルに書く」だけに特化しています。
本来でしたらそもそも手動でチェックしているのがおかしいのです。

Answer 1

シンプルに考えると「どれかが空じゃないときにどれかが空ならエラー」ですよね。

こんな感じではどうでしょう

if (
  (!empty($item1) || 
   !empty($item2) ||
   !empty($item3))
   &&
  (empty($item1) || 
   empty($item2) ||
   empty($item3))
  ) {
  //エラー
}

Comments

[退会済みユーザー]

https://ideone.com/E7vjaD

[m.ts10806]

前提条件がないのでそこまで考慮してないですね。
あくまでコードをシンプルに書いただけです。
CakePHPでなぜ手動チェックしてるのかというところもあるし、
そもそも質問者さんが無言で終わらせてるので、議論の余地もないという。

[退会済みユーザー]

②だから

[m.ts10806]

組み合わせればできるような気がしたんですが、そうでもないんですかね。そもそもemptyが悪手なのは分かります。
https://book.cakephp.org/2/ja/models/data-validation.html

Answer 2

要するに、$item1,$item2,$item3で値が入っている数をカウントし、
1～2であればエラーということですよね？

$count = 0;
$count += (!empty($item1) ? 1 : 0);
$count += (!empty($item2) ? 1 : 0);
$count += (!empty($item3) ? 1 : 0);
if ($count >= 1 && $count <= 2  ) {
        // エラー
}