Laravel Eloquent - 関係のフィールドを更新する

Question

私はこれを書いているより良い方法を探しています。私はより良い方法が必要であるように感じる？

public static function logClick($lid) 
{ 
    $clickId = link::find($lid); 

    $clicks = click::find($clickId->click_id); 

    $c = $clicks->clicks; 
    $c++; 

    $clicks->clicks = $c; 
    $clicks->save(); 

    return $clicks; 
} 

モデル：ところで

class link extends Model 
{ 
    public function click() 
    { 
     return $this->hasOne('App\click', 'id' ,'click_id'); 
    } 

} 

class click extends Model 
{ 
    public function link() 
    { 
     return $this->hasOne('App\link'); 
    } 
} 

。すべてがちょうど私のコードの書き込みを改善するために探して、働く:)

Answer 1

public static function logClick($lId){ 
    click::whereHas('link', function($q)use($lId){ 
     $q->where('id', $lId); 
    }) 
    ->increment('clicks'); 
} 

これは

EDITを実行する必要があります。あなたは数

public static function logClick($lId){ 
     $click = click::whereHas('link', function($q)use($lId){ 
      $q->where('id', $lId); 
     }) 
     ->increment('clicks'); 
     return $click->clicks; 
    } 

クリックを返したい場合は、この1つのライナーは利点があります単一のクエリであるということです。 $q->where('id', $lId);場合には動作しません、$q->where('links_table_name.id', $lId);

Answer 2

public static function logClick($lid) 
{ 
    $clicks = link::find($lid)->click; 
    $clicks->increment('clicks'); 
    return $clicks; 
}