與學說ORM遞歸 - 邏輯錯誤

Question

你好，

我有種每日隨機圖像系統建立與Codeigniter和Doctrine。

有一個關鍵功能，它返回每日圖像或設置一個新的。我正在使用遞歸調用，但有時是兩個圖像選擇一天。

class Daily extends Controller { 

    function find_daily(){ 
     $connection = Doctrine_Manager::connection(); 
     $connection->setCharset('utf8'); 

     $q = Doctrine_Query::create() 
      ->select('COUNT(id) as dailycount') 
      ->from('Image') 
      ->where('daily_date = ?',date("Y-m-d")); 



     $set = $q->execute(); 

     if($set[0]->dailycount != 0){ //theres one or more for today 



      $q = Doctrine_Query::create() 
      ->select('id') 
      ->from('Image') 
      ->where('daily_date = ?',date("Y-m-d")) 
      ->limit(1); 


      $set = $q->execute(); 
      $i = Doctrine::getTable('Image')->findOneById($set[0]["id"]); 

      return $i; 

      }else { // none image selected with today date 


       $q = Doctrine_Query::create() 
       ->select('id') 
       ->from('Image') 
       ->where('daily_date = ?', "0000-00-00") 
       ->andWhere("dir =?","queue") 
       ->orderBy("rand()") 
       ->limit(1); 


       $set = $q->execute(); 


       $i = Doctrine::getTable('Image')->findOneById($set[0]["id"]); 

       $i->daily_date = date("Y-m-d"); 
       $i->dir = "archive"; 
       $i->save(); 

       $this -> find_daily(); 
       }; 

     } 

... 

我覺得那裏有一些愚蠢的錯誤..也許叫什麼「主義:: doAllAndClear」 $this -> find_daily();之前或我忘了什麼東西......

還是最好不要使用遞歸調用？

謝謝。

Answer 1

嗯，我還沒有測試過這個，但我可能更有可能像這樣寫你的代碼;希望這會給你一些想法：

function find_daily() { 
    $connection = Doctrine_Manager::connection(); 
    $connection->setCharset('utf8'); 

    // Make sure "today" doesn't change while the script runs 
    $today = date("Y-m-d"); 

    // Find an image with today's date. 
    $q = Doctrine_Query::create() 
     ->from('Image') 
     ->where('daily_date = ?', $today) 
     ->limit(1); 

    $image = $q->fetchOne(); 

    if (!$image) { 
     // There was no image with today's date, so pick one from the queue at random instead. 
     $q = Doctrine_Query::create() 
     ->from('Image') 
     ->where('daily_date = ?', "0000-00-00") 
     ->andWhere("dir =?","queue") 
     ->orderBy("rand()") 
     ->limit(1); 

     $image = $q->fetchOne(); 
     // ToDo: Cope with there being no image in the queue 

     $image->setDailyDate($today); 
     $image->setDir("archive"); 
     $image->save(); 
    } 
    return $image; 
    } 

但是，你需要考慮一下會發生什麼，如果兩個人打你的腳本在同一時間。在這種情況下，他們可能會都取一個新的形象的一天（因爲他們可以各自在同一時間運行第一SELECT，都得到一個結果，說有沒有日常照片呢。）

爲了避免我猜你會想把它包裝在一個事務中，並設置一個事務隔離級別，以使該函數的第一個調用者「贏得」競爭條件，並且隨後的調用者阻塞，直到更新安全地保存。不完全確定你會如何做到這一點，但它不應該很難。

檢查手冊中，beginTransaction()在開始時，commit()末，一切之前，設置其他涉及$tx = $conn->transaction; $tx->setIsolation('SERIALIZABLE');（另有兩個用戶很可能仍運行一次選擇），很可能是你所需要的，但你可能要等待真正知道學說的人對此發表評論...

我還在腳本的開頭保存了「今天」的日期，否則在SELECT和期間會有不同的危險稍後更新，如果腳本跨午夜運行。