セットに複製を追加しようとすると、どのような例外がスローされますか？

Question

Doctrineモデルで関連オブジェクトをコレクションに追加するメソッドを作成していますが、そのコレクションに重複オブジェクトが追加されたときに例外をスローしたいとします。ここで

はテスト（秒）です：まず

public function testFluentInterface() 
{ 
    $sport = new Sport(); 
    $this->assertSame($sport, $sport->addCode('ANY'), 
    'Expected Sport to implement fluent interface.' 
); 
} 

public function testCannotAddSameCodeMoreThanOnce() 
{ 
    $code = 'BAZ'; 

    $sport = new Sport(); 
    $sport->addCode($code); 

    try 
    { 
    $sport->addCode($code); 
    $this->fail(
     'Expected error when trying to add the same code to a sport more than once.' 
    ); 
    } 
    catch(/*SomeKindOf*/Exception $e) 
    { 
    } 
} 

、私はOverflowExceptionが、この場合にスローされることが適切かもしれないと思ったが、私は「この値が既に存在する」かどうかは確かではありませんよ

要素をフルコンテナに追加すると例外がスローされます。

ありUnexpectedValueExceptionだが、それは間違った種類の変数により適用可能であると思わ：値は、値のセットと一致しない場合

例外がスローされます。通常、これは、関数が別の関数を呼び出し、戻り値が算術またはバッファー関連のエラーを含まない特定のタイプまたは値であることを期待する場合に発生します。

私はいつもLogicExceptionを使用することができますが、それはこのユースケースのために少しの一般的なようだ：プログラム・ロジックのエラーを表し

例外。このような種類の例外は、コード内の修正に直接つながるはずです。

ここでより適切な選択肢がありますか？私の観察は正しいのですか？一意の値を含むコレクションに複製を追加しようとするときにスローする最も適切なSPL例外は何ですか？

Answer 1

特に、ToniとFrancisのそれぞれの答えに照らして、このシナリオでは例外をスローすると便利だろうかと思います。

トニーのポイントごとに、多くのクラスがすでにこの状況を別々に処理しています。この場合、「失敗」値を返すことはオプションではありません（Sportクラスは流暢なインターフェイスを実装しています）。オブジェクトをコレクションに2回追加しようとすることは、必ずしも「例外的」なケースではありません。

フランシスはまた、PHPの他の構成要素（配列など）がこの種のことに気をつけていないことを知っています。重複したアイテムがコレクションに追加されたときに例外をスローすると、保守性の問題が発生する可能性のある前例のない動作になります。

また、例外をスローすることは、流暢なインターフェースのコンセプトと相反すると考えることができます。とにかく条件付きチェックでチェーンを分割する必要がある場合、開発者がインスタンスにメソッドをチェーンできるようにする点は何でしょうか？言い換えれば

、目標はこれである場合：

$sport 
    ->addCode($codeA) 
    ->addCode($codeB) 
    ->setSomeOtherProperties(...) 
    ->save(); 

そして、それはこれをしなければならないために、開発者を強制的にも意味がありません：

if(! $sport->hasCode($codeA)) 
{ 
    $sport->addCode($codeA); 
} 
// etc. - Why bother having a fluent interface if it's unsafe to use it? 

上記のコードは特に問題となるので、 addCode()はとにかくhasCode()を呼び出す予定であり、実際には開発者に重複計算を強制します。

とのtry/catchを使用すると、はるかに良いではありません。上記のブロックにLogicExceptionから回復しようとすると

try 
{ 
    $sport 
    ->addCode($codeA) 
    ->addCode($codeB) 
    ->setSomeOtherProperties(...) 
    ->save(); 
} 
catch(LogicException $e) 
{ 
    // $sport is now in an unknown state. 
} 

は非現実的であり、それがここでも必要はありません。コードを2回追加しようとすると、Sportオブジェクトが永続化されなくなります。

Answer 2

例として、Java Collection Framework（つまりSet）の多くのクラスは例外をスローしませんが、falseを返します。

個人的に私はそのように行きます。

Answer 3

実際に適合するSPL例外はありません。あなたはあなた自身の例外を作成したほうが良いかもしれません：

/** 
* Raised when item is added to a collection multiple times. 
*/ 
class DuplicateException extends RuntimeException {}; 

が、この場合に例外を投げるあなたは、アレイ内の同じキーに同じ値を割り当てるたびに例外をスローするやや同等、ビット抜本的なようです。このケースを検出するために例外の代わりに戻り値を使用することを検討しましたか？