web-dev-qa-db-ja.com

モデルhasMany Relationにwhere条件にアクセスする方法は?

次のように、リレーションの条件/制約を使用してモデルゲームを作成しました。

class Game extends Eloquent {
    // many more stuff here

    // relation without any constraints ...works fine 
    public function videos() {
        return $this->hasMany('Video');
    }

    // results in a "problem", se examples below
    public function available_videos() {
        return $this->hasMany('Video')->where('available','=', 1);
    }
}

なんとかこのように使用する場合:

$game = Game::with('available_videos')->find(1);
$game->available_videos->count();

ロールが結果のコレクションであるため、すべてが正常に機能します。

私の問題:

熱心にロードせずにアクセスしようとすると

$game = Game::find(1);
$game->available_videos->count();

非オブジェクトのメンバー関数count()への呼び出し」と示されているため、例外がスローされます。

を使用して

$game = Game::find(1);
$game->load('available_videos');
$game->available_videos->count();

正常に動作しますが、リレーション内で条件を使用しない場合、関連するモデルをロードする必要がないため、非常に複雑に思えます。

私は何かを見逃しましたか?積極的な読み込みを使用せずにavailable_videosにアクセスできるようにするにはどうすればよいですか?

興味のある方は、この問題を http://forums.laravel.io/viewtopic.php?id=1047 に投稿しました。

40
Remluben

他の誰かが同じ問題に遭遇した場合に備えて。

リレーションはキャメルケースである必要があることに注意してください。したがって、私の場合、available_videos()はavailableVideos()でなければなりませんでした。

Laravelソースの調査を簡単に見つけることができます:

// Illuminate\Database\Eloquent\Model.php
...
/**
 * Get an attribute from the model.
 *
 * @param  string  $key
 * @return mixed
 */
public function getAttribute($key)
{
    $inAttributes = array_key_exists($key, $this->attributes);

    // If the key references an attribute, we can just go ahead and return the
    // plain attribute value from the model. This allows every attribute to
    // be dynamically accessed through the _get method without accessors.
    if ($inAttributes || $this->hasGetMutator($key))
    {
        return $this->getAttributeValue($key);
    }

    // If the key already exists in the relationships array, it just means the
    // relationship has already been loaded, so we'll just return it out of
    // here because there is no need to query within the relations twice.
    if (array_key_exists($key, $this->relations))
    {
        return $this->relations[$key];
    }

    // If the "attribute" exists as a method on the model, we will just assume
    // it is a relationship and will load and return results from the query
    // and hydrate the relationship's value on the "relationships" array.
    $camelKey = camel_case($key);

    if (method_exists($this, $camelKey))
    {
        return $this->getRelationshipFromMethod($key, $camelKey);
    }
}

これは、以前にload()メソッドを使用してデータをロードしたときに、コードが機能した理由も説明しています。

とにかく、私の例は今では完璧に機能し、$ model-> availableVideosは常にCollectionを返します。

10
Remluben

私はこれが正しい方法だと思う:

class Game extends Eloquent {
    // many more stuff here

    // relation without any constraints ...works fine 
    public function videos() {
        return $this->hasMany('Video');
    }

    // results in a "problem", se examples below
    public function available_videos() {
        return $this->videos()->where('available','=', 1);
    }
}

そして、あなたはする必要があります

$game = Game::find(1);
var_dump( $game->available_videos()->get() );

これがあなたが探しているものだと思います(Laravel 4、 http://laravel.com/docs/eloquent#querying-relations を参照)

$games = Game::whereHas('video', function($q)
{
    $q->where('available','=', 1);

})->get();
20
Sabrina Leggett

// v4の一部のバージョンでは低い

public function videos() {
    $instance =$this->hasMany('Video');
    $instance->getQuery()->where('available','=', 1);
    return $instance
}

// v5

public function videos() {
    return $this->hasMany('Video')->where('available','=', 1);
}
14
xsilen T

リレーショナルテーブルに条件を適用する場合は、他のソリューションも使用できます。このソリューションは、私の終わりから機能しています。

public static function getAllAvailableVideos() {
        $result = self::with(['videos' => function($q) {
                        $q->select('id', 'name');
                        $q->where('available', '=', 1);
                    }])                    
                ->get();
        return $result;
    }
6
Nikunj K.

モデル(App\Post.php):

/**
 * Get all comments for this post.
 */
public function comments($published = false)
{
    $comments = $this->hasMany('App\Comment');
    if($published) $comments->where('published', 1);

    return $comments;
}

コントローラー(App\Http\Controllers\PostController.php):

/**
 * Display the specified resource.
 *
 * @param int $id
 * @return \Illuminate\Http\Response
 */
public function post($id)
{
    $post = Post::with('comments')
        ->find($id);

    return view('posts')->with('post', $post);
}

ブレードテンプレート(posts.blade.php):

{{-- Get all comments--}}
@foreach ($post->comments as $comment)
    code...
@endforeach

{{-- Get only published comments--}}
@foreach ($post->comments(true)->get() as $comment)
    code...
@endforeach
0
Santo Boldižar

Builder::withメソッド内の最初の引数として連想配列を渡すことで、同様の問題を修正しました。

いくつかの動的パラメーターによる子関係を含めたいが、親結果をフィルターしたくないと想像してください。

Model.php

public function child ()
{
    return $this->hasMany(ChildModel::class);
}

次に、別の場所で、ロジックが配置されると、HasManyクラスによるリレーションのフィルタリングなどを行うことができます。例(私の場合と非常によく似ています):

$search = 'Some search string';
$result = Model::query()->with(
    ['child' => function (HasMany $query) use ($search) {
        $query->where('name', 'like', "%$name%");
    }]
);

次に、すべての子結果をフィルタリングしますが、親モデルはフィルタリングしません。清聴ありがとうございました。

0
Vitalii

「ロール」のモデルを作成しましたか。ロールのモデルを作成した後でも問題が存在するかどうかを確認します。

0
Sreedhar M B