リファクタリングされたPHPコードで実行を停止しますか？

Question

私は、PHPのページをリファクタリングして、将来的に拡張と保守をやや簡単にし、かなり単純な問題に取り掛かりました。

私は（3

largemethodに一つの方法を分割しているが）このようなものになっています：

nowsmallmethod(){ 
    doSomeChecks(); 
    assignProduct(); 
    giveFeedbacktoUser(); 
} 

これは、すべて良いですが、よく、私が午前問題はdoSomeChecks（です） ;

doSomeChecks(){ 
if(something that shouldnt be true is true){ 
    return Controller->redirectBk(); 
} 
} 

問題の核心は、nowsmallmethod（）が完了したときにコントローラ-redirectBkが最初にリダイレクトすることです。つまり、テストに失敗した場合でもユーザーに製品が割り当てられます。私はSilverstripeと呼ばれるPHPフレームワークを使用しているので、Controller-> redirectBk（）の動作を実際に変更することはできません。私が自分のメソッドでチェックをしなかったなら、 "return Controller-> redirectBk（）;"実行を停止してリダイレクトします。 テストが失敗したときにnowsmallmethod（）で実行を停止する最良の方法は？私はエラーのステータスコードを返すことができ、実行を停止することができたが、それは醜い方法と思われる。よりエレガントな方法はありませんか？もう一つのオプションはdoSomeChecks（）、return（return $ controller-> redirectBk（））でこれを返すことができますが、これは有効なPHP構文ではなく、特に読みにくいものではありません。どんな助けでも大変に感謝します。

週末をお楽しみください！ 乾杯 ニック

Answer 1

nowsmallmethod() { 
    if (doSomeChecks()) { 
    assignProduct(); 
    giveFeedbacktoUser(); 
    } 
} 

そしてdoSomeChecksいずれかのリダイレクトが起こるかないでしょうかどうかに基づいて、trueまたはfalseを返します。

また、dieまたはthrowとすることもできますが、あなたの場合は通常の条件がより適切であると想定しています。そのような状況に対処するために

Answer 2

使用例外：

<?php 
try { 
    // some code 
    if (!$check) { 
     throw new Exception('Check has failed!'); 
    } 
    // code after 'throw' will not be executed 
} 
catch(Exception $e) { 
    //this is executed when exception is caught 
} 
?> 

例外が子法にスローされた場合、スタックは戻って非常に最初のtry/catchブロックにロールバックされます。 http://php.net/manual/en/language.exceptions.php

また、トランザクションのメカニズムと例外を組み合わせて、データベースの整合性を処理することもできます。