laravelでviewへの値の渡し方について

Question

laravelを学習中です。アドバイス頂けたらと思います。

employees
id
dept_id
name

deptsテーブル
id
dept_name

・従業員は、一つの部署に所属する。
・部署は、複数の従業員をもつ。

Employeeモデルクラスに

    public function dept()
    {
        return $this->belongsTo(Dept::class);
    }

を追加する。

web.phpでの検証

Route::get('/belongsto',function(){
 $employees = App\Employee::all();
 foreach($employees as $employee)
    {
      echo $employee->dept->dept_name." ".$employee->name."<br>";
    }
    return;
});

この場合、部署と名前の一覧が取得・表示できました。

その後、コントローラー(EmployeeController)とビューファイルを作成し、

    function index()
    {
        $employees = Employee::all();
        return view('employee.belongsto',compact('employees'));
    }

として、
belongsto.blade.phpにて

@foreach($employees as $employee)
    <p>{{ $employee->dept->dept_name }}</p>
    <p>{{ $employee->name }}</p>
@endforeach

とすると、
Trying to get property 'dept_name' of non-object
とエラーが出てしまいます。

route/web.phpと同じように
$employ->dept->dept_name
とアクセスするのは間違っているのでしょうか？

関連テーブルの値をビューに渡すのはどのようにするのがいいのでしょうか？

宜しくおねがいします。

Answer 1

すみません、最初の回答誤っていたので忘れて下さい。

まず、

function index()
{
    $employees = Employee::all();
    return view('employee.belongsto',compact('employees'));
}

こちらのControllerの段階で$employeesに本当に値が入っているか確認してみて下さい。
値が入っていたら、

@foreach($employees as $employee)
    <p>{{ $employee->dept->dept_name }}</p>
    <p>{{ $employee->name }}</p>
@endforeach

こちらのViewで$employeesに値が入っているか確認してみて下さい。

追記

Laravelのlogger機能を使ってデバッグしてみて下さい。
まず、storage/logs/laravel.logを削除して下さい(すでに大量のログがあるはずで、これからのデバッグ結果が見づらいため)。
続いて、controllerを次のような感じに書いてみて下さい。

function index()
{
    $employees = Employee::all();
    logger($employees);
    logger($employees->first());
    logger($employees->first()->dept);
    return view('employee.belongsto',compact('employees'));
}

書き換えたら、ブラウザで問題のページにアクセスし、プログラムを動かして下さい。
そうすると、storage/logs/下にlaravel.logが生成されているはずですので、中身を確認して下さい。
必要なデータが有り、ちゃんとリレーションが設定できていれば、$employees->first()->deptの結果がログに出力されているはずです。
私が示したソースのパターン以外にも、色んな値のとり方を試みれば、うまくいかない原因に近づけるかと思います。

追記2

https://github.com/dyoshikawa/teratail-131441

サンプルコード書いてみたのでご参考になればと思います。
動かす場合は、

php artisan migrate
php artisan db:seed

を実行した後、サーバー起動してトップページを見て下さい。

Comments

[hiromitsu3]

回答有り難うございます。
試してみましたが、Undefined variableとエラーが出ます。
見直してみます！

[退会済みユーザー]

すみません、私の回答がおかしかったので修正致しました。

[hiromitsu3]

ありがとうございます。

Controllerで$employeesを確認すると、中身が入っておりました。
またViewのほうで、$employeesを確認すると同じく値が入っておりました。

$employeesの一つのemployeeは、deptテーブルのidを保持しているのですが、
deptテーブルのdeptnameが参照できないようです。
($employee->dept->dept_nameがおかしい)

見直してみます。

[退会済みユーザー]

ありがとうございます。$emploee->deptは確かにとれているでしょうか？dept_idはとれていてもdeptを引っ張ってこれていないことが原因のように思います。

[hiromitsu3]

Viewにおいて
echo $employees->count();

foreach($employees as $employee)
{
echo $employee->dept;
}
このようにしてみました。
countは登録されたデータの数が返ってきているようなのですが、
foreachが動いてないのですが、何が悪いのでしょうか？
宜しくおねがいします。

[hiromitsu3]

すいません。

$employee->deptは取得できていませんでした。

そうなるとModelのほうの記述がおかしいという理解でいいのでしょうか？

[退会済みユーザー]

うーん、web.phpではできていたようですし、原因が思いつかないですね・・・ちょっとデバッグ方法を追記したので、試してみて下さい。

[hiromitsu3]

度々すいません。

logger($employees);が
{"id":1,"dept_id":1,"name":"\u7530\u4e2d"},{"id":2,"dept_id":2,"name":"\u7389\u6728"}

logger($employees->first());が
[2018-06-16 15:43:49] local.DEBUG: {"id":1,"dept_id":1,"name":"\u7530\u4e2d"}
となりますが、

logger($employees->first()->dept);
は何も表示されません。

ローカルですので一度ゼロからやり直してみようと思います。
ありがとうございました。

[退会済みユーザー]

解決に繋がるかは分かりませんが、一応、EmployeeとDeptのModelソースを貼って頂ければ拝見します。

[hiromitsu3]

すいません。

Employee.php
<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Employee extends Model
{
public $timestamps = false;
protected $fillable = ['dept_id','name'];

public function dept()
{
return $this->belongsTo('App\Dept');
}
}

Dept.php

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Dept extends Model
{
public $timestamps = false;
protected $fillable = ['dept_name'];
protected $primaryKey = "dept_id";

public function employees()
{
return $this->hasMany('App\Employee');
}
}

となります。

ありがとうございます。

[退会済みユーザー]

deptモデルの主キー設定がちょっと違う気がしますね。マイグレーションファイルも見てよろしいですか？

[hiromitsu3]

ありがとうございます。

migrationしたあと、phpmyadminで変更した気がしますので

+-----------+------------------+------+-----+---------+----------------+
| Field | Type | Null | Key | Default | Extra |
+-----------+------------------+------+-----+---------+----------------+
| dept_name | varchar(32) | NO | | NULL | |
| dept_id | int(10) unsigned | NO | PRI | NULL | auto_increment |
+-----------+------------------+------+-----+---------+----------------+

+---------+------------------+------+-----+---------+----------------+
| Field | Type | Null | Key | Default | Extra |
+---------+------------------+------+-----+---------+----------------+
| id | int(10) unsigned | NO | PRI | NULL | auto_increment |
| dept_id | int(11) | NO | | NULL | |
| name | varchar(32) | NO | | NULL | |
+---------+------------------+------+-----+---------+----------------+

これで大丈夫でしょうか？

[退会済みユーザー]

恐らく、deptの主キーをdept_idにしているのがおかしいですね。
追記しますのでご確認下さい。

[trewa-nek9585]

もしかするとEagerローディングされていないのが原因ではないでしょうか？
https://laravel.com/docs/5.6/eloquent-relationships#eager-loading

Controller内での処理を下記で試して頂けませんでしょうか？
$employees = with('depts')->get();

[退会済みユーザー]

私もそれは考えたのですが、手元で動かしたら大丈夫だったんですよね。eagerはあくまでパフォーマンスの観点で必要という理解です。

[trewa-nek9585]

そうなんですね。勉強になります。横から失礼いたしました。

[退会済みユーザー]

いえ、補足ありがとうございます。

[hiromitsu3]

ありがとうございました。
頂いたソースで試した所ちゃんと動きました。

対象フィールドのデータ型の違いと、migrationファイルの
$table->foreign('dept_id')->references('id')->on('depts');
が相違してると思いました。
ありがとうございました。

[退会済みユーザー]

Laravelは楽しいフレームワークですので、引き続きがんばって下さい。