[Laravel]指定したidの内容を複数モデルから呼び出したい。

Question

選択したidのページにてid個別の情報を出そうと思っています。
ふたつのモデルにはいっているデータを呼び出したいのですがうまくできません。。
hasMany/belongsToでモデルにて結合しています。

Route::get('book/select/{id}', 'BookController@select');

<?php

namespace App\Http\Controllers;

use App\Book;
use App\Author;
use Illuminate\Http\Request;

class BookController extends Controller {

{省略}

    public function select(Request $request, $id) {
        
        $author = \App\Author::find($id);
        $name = $author->name;
        $books = $author->books()->orderBy('created_at', 'DESC');

        return view('book.select')->with(array('author' => $author, 'name' => $name, 'books' => $books));
    }

}

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Author extends Model {

    public $timestamps = false;
    protected $fillable = [
        'name'
    ];

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

}

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Book extends Model {

    protected $fillable = [
        'title',
        'author_name',
        'company'
    ];

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

}

Comments

[m.ts10806]

>うまくできません。。
それなりのコードは組まれているようですが「何が」「どう」うまくいかないのでしょうか？
想定する動作と現在のコードにおける問題を具体的に記載してください。
https://teratail.com/help/question-tips#questionTips3-4
提示内容だけでは他者がコピペですぐ再現できるわけではないので、その内容が回答のヒントとなります。

[noricoonco]

ご指摘ありがとうございます。
モデルのAuthorとBookをhasMany/belongsToで結合しているので、
$author = \App\Author::find($id);
$books = $author->books()->orderBy('created_at', 'DESC');
これでBookのデータが呼び出せると思ったのですが、画面には表示されませんでした。エラーも出ずに表示されない現象が起きたので、データ処理で正しいものを呼び出せていないものと思われます。
しかしエラーがでないということはどういうことなのか・・・と自分で解決できず質問させていただきました。

Answer 1

get() が抜けているため、Collectionではなくクエリビルダのオブジェクトがセットされてしまっているのではないでしょうか？

$books = $author->books()->orderBy('created_at', 'DESC');

を

$books = $author->books()->orderBy('created_at', 'DESC')->get();

と修正してみてください。

7/3追記

コメントに書いていただいたmigrationファイルの内容を見ました。
テーブル同士の参照関係を作るため、下記を追加してください。

$table->foreign('author_id')->references('id')->on('authors');

下記ドキュメントの[外部キー制約]項をご参考ください。
https://readouble.com/laravel/5.8/ja/migrations.html

Comments

[noricoonco]

ありがとうございます。空白だったものがちゃんとエラー表示されました！
ちなみに、SQLSTATE[HY000]: General error: 1 no such column: books.author_id (SQL: select * from "books" where "books"."author_id" = 2 and "books"."author_id" is not null)　と表示されました。
データはシーダーで入力済みなのでNullが出るのは入力されていないということですよね・・・
お手すきの間にご教授いただけたら嬉しいです・・・。

[tomoeine]

おお、一歩前進ですね????
エラーはbooksテーブルににauthor_idがないよ、と怒られていますね。
カラムの指定なくhasManyを書いた場合、参照先テーブル名_idのカラムが外部キーとして扱われますので、この場合booksテーブルにauthor_idカラムが必要です。
booksテーブルのmigrationファイルの内容を記載いただければ回答できるかと思います！

[noricoonco]

ありがとうございます！
hasManyの知識が浅かったですね。。

class CreateBooksTable extends Migration
{
/**
* Run the migrations.
*
* @return void
*/
public function up()
{
Schema::create('books', function (Blueprint $table) {
$table->bigIncrements('id');
$table->string('title');
$table->string('author_name');
$table->string('company');
$table->date('date');
$table->timestamps();
});
}
上記に、
$table = bigIncrement('author_id')を追加するということでしょうか？

[tomoeine]

migrationファイルありがとうございます！
回答に追記しましたのでご確認ください。

質問とは直接関係ないのですが、 `author_name` は 本来authorsテーブルにあると思われるので、booksテーブルには不要ではないでしょうか？

[noricoonco]

ご指摘、参考サイト諸々ご丁寧にありがとうございます！

>>`author_name` は 本来authorsテーブルにあると思われるので、booksテーブルには不要ではないでしょうか？
まさにそうでした。
外部キーの制約とあわせてmigrationを今一度学習します。
長々ありがとうございました！

Answer 2

とりあえずこ以下でデータは呼び出せるので一旦解決とします。
DBやEroquentについては勉強不足なのでやり直そうと思います。
回答いただきありがとうございました。

public function select(Request $request, $id) {

    $author = \App\Author::find($id);
    $books = \App\Book::find($id);
    $name = $author->name;
    $title = $books->title;

    return view('book.select')->with(compact('name', 'title'));

Answer 3

とりあえずこ以下でデータは呼び出せるので一旦解決とします。
DBやEroquentについては勉強不足なのでやり直そうと思います。
回答いただきありがとうございました。

public function select(Request $request, $id) {

    $author = \App\Author::find($id);
    $books = \App\Book::find($id);
    $name = $author->name;
    $title = $books->title;

    return view('book.select')->with(compact('name', 'title'));