イベントに受け入れられたオファーがある場合、機能をチェックしますか？

Question

は、私がイベントを提示申し出を持っているかどうかを確認する必要があると私は、この機能

public function hasAcceptedOffer() 
{ 
    foreach ($this->offers as $offer) { 

     if($offer->accepted == 1){ 
     return true; 

    } else { 

     return false; 

    } 
} 

を行っているが、私は、これは、より最適化され、良好なものとすることができると思います。オファーがたくさんある場合、私はそれらのすべてを通過したくないからです。私が望むのは、関数が受け入れられたオファーを見つけたらそれ以上の反復を止めるべきだということです。

Answer 1

ここでうまくいったと思います。

あなたはがどのオファーが受け入れられた場合にのみ気に仮定すると、この

public function hasAcceptedOffer() 
{ 
    foreach ($this->offers as $offer) { 

     if($offer->accepted == 1){ 
      return true; 

     } 
    } 
    return false; 
} 

Answer 2

を行うことによって、他の部分を削除することができ、あなたが、私はこのようにそれを行うのが好きどちらかを知る必要はありません：

public function hasAcceptedOffer() 
{ 
    foreach($this->offers as $offer) { 
     if($offer->accepted !== 1) 
      continue; 

     return true; 
    } 

    return false; 
} 

ループの各反復はすぐにそれはあなたの希望の条件に一致しない場合、それは最初の陽性結果を見つけたときにすぐに戻り、あなたドンので、（他の処理をスキップします次のいずれかにスキップしますとにかく気にしない）一致するものが見つからない場合はfalseを返します。

あなたが受け入れられたオファーを収集する場合、あなたはそのようにそれを修正することができます

public function getAcceptedOffers() 
{ 
    $results = array(); 

    foreach($this->offers as $offer) { 
     if($offer->accepted !== 1) 
      continue; 

     $results[] = $offer; 
    } 

    return $results; 
}