Laravel5.2　リレーションの書き方について

リレーションの実装に挑戦したいと考えております。1対1のリレーションでuserがPhoneを持つとした場合、どのようにコードを書くべきでしょうか。
そもそも現在のUser.php(Model)に追加して書けばいいのか、新しいModelを作成すればいいのかすらわかりません。大変恐縮ですが、ご教示お願いいたします。

※phoneテーブルにはuser_idとphone_numがあるとします。

php
1<?php
2
3namespace App;
4
5use Illuminate\Foundation\Auth\User as Authenticatable;
6
7class User extends Authenticatable
8{
9    /**
10     * The attributes that are mass assignable.
11     *
12     * @var array
13     */
14    protected $fillable = [
15        'name', 'email', 'password',
16    ];
17
18    /**
19     * The attributes that should be hidden for arrays.
20     *
21     * @var array
22     */
23    protected $hidden = [
24        'password', 'remember_token',
25    ];
26}

php
1<?php
2
3namespace App;
4
5use Illuminate\Foundation\Auth\User as Authenticatable;
6use App\Phone;
7
8class User extends Authenticatable
9{
10    /**
11     * The attributes that are mass assignable.
12     *
13     * @var array
14     */
15    protected $fillable = [
16        'name', 'email', 'password',
17    ];
18
19    /**
20     * The attributes that should be hidden for arrays.
21     *
22     * @var array
23     */
24    protected $hidden = [
25        'password', 'remember_token',
26    ];
27
28
29    public function phone()
30    {
31        return $this->hasOne(Phone::class, 'id', 'user_id');
32    }
33}
34

php
1    public function mypage( $name, $id )
2    {
3//Controller記載部分抜粋（※後にWhere（'id',$id）で個人の携帯を紐づける予定です）
4        $users = User::all();
5
6        foreach($users as $user) {
7            $phoneNumList[] = $user->phone->phone_num;  // Userモデルで定義したphoneメソッド
8        }
9        echo $phoneNumList;
10
11
12    }

php
1<?php
2
3namespace App;
4
5use Illuminate\Database\Eloquent\Model;
6
7class Phone extends Model
8{
9    protected $fillable = ['user_id','phone_num'];
10
11    protected $table = 'phones';
12
13}

行動規範の内容に同意します

回答1件

ベストアンサー

現在のUserモデルに追加してOKです。具体的にはUserモデルにメソッドを追加します。

php
1public function phone()
2{
3    return $this->hasOne(phoneモデルクラス名::class, users?テーブルのid?, user_id);
4}

これで準備はOKです。呼び出す際にはこのようにします。

php
1$users = User::all();
2foreach($users as $user) {
3    $phoneNumList[] = $user->phone->phone_num;  // Userモデルで定義したphoneメソッド
4}
5dd($phoneNumList); // 全ユーザの電話番号が表示される

詳しくはこちらを御覧ください。

投稿2016/09/13 02:20

natady

総合スコア606

AlwaysOreHungry

2016/09/13 02:40

早速のご回答ありがとうございます！参考サイトと私が質問欄で記載しているコードは下記の部分が違いますが、どう補完すべきでしょうか！？・参考サイト use Illuminate\Database\Eloquent\Model; class User extends Model ・私の質問欄コード use Illuminate\Foundation\Auth\User as Authenticatable; class User extends Authenticatable

natady

2016/09/13 02:46

https://github.com/laravel/framework/blob/5.2/src/Illuminate/Foundation/Auth/User.php#L13 ここでModelクラスを継承しているようなので、質問者様の方のコードで問題なさそうです。

AlwaysOreHungry

2016/09/15 15:02

Undefined property: Illuminate\Database\Eloquent\Collection::$phone_num 上記のような結果が得られました。すみませんが、原因が全くわからずお力添え頂けないでしょうか・・・。よろしくお願いいたします。

natady

2016/09/15 15:28

エラーメッセージ確認いたしました。 1. phoneテーブルのモデルは作っていますでしょうか？また、Userモデルに定義したリレーション用のphoneメソッド内のhasOneメソッドの第一引数にphoneモデルクラス名は間違いなく入っていますでしょうか？ 2. `$user->phone_num`ではなく`$user->phone->phone_num`で呼び出していますでしょうか？

AlwaysOreHungry

2016/09/16 03:30

コード追記致しました。Phoneモデルは作成しております。fillableとpromaryKeyのみ追記しただけのほぼ初期設定です。

natady

2016/09/16 04:06

追記されたコードを確認しました。 1. 使うテーブル名は`users`と`phones`で間違いないでしょうか？ 2. `phones`テーブルに間違いなく`phone_num`カラムがありますでしょうか？

AlwaysOreHungry

2016/09/16 04:41

何度もありがとうございます。１．２ともにnatady様の通りになっております。。。

natady

2016/09/16 04:50

長くなってしまい申し訳ありません・・・差し支えなければPhoneクラスのコードを追記していただきたいのですがよろしいでしょうか？