質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.03%

Laravelでのエラーの原因がわかりません

解決済

回答 5

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,715

前提・実現したいこと

Laravelで下のbladeからweb.phpにルートするというボタンを作ったところ、下のようなエラーがコントローラー内で出てきました。
しかしこのweb.phpのrouteはエラーが発生しているメソッドは愚か、コントローラーさえ経由していません。
このようなエラーがなぜ出てしまうのでしょうか。
問題点と改善点をお願いいたします。

発生している問題・エラーメッセージ

Call to a member function comments() on null

該当のソースコード

<a href="{{route('create_post')}}">
       <button type="button" class="btn btn-primary btn-lg">新規投稿</button>
</a>
//web.php
<?php

use Illuminate\Support\Facades\Route;

/*
|--------------------------------------------------------------------------
| Web Routes
|--------------------------------------------------------------------------
|
| Here is where you can register web routes for your application. These
| routes are loaded by the RouteServiceProvider within a group which
| contains the "web" middleware group. Now create something great!
|
*/

Route::get('/', function () {
    return view('welcome');
});

//ログイン確認不要
Route::get('/mypage/{post_id}', 'PostController@edit')->name('edit_post');
Route::post('/mypage/{post_id}', 'PostController@editing');

Route::get('/mypage/{post_id}/{comment_id}')->name('comment_delete');


//ログイン確認必要
//
Route::get('/mypage', 'MypageController@index')->name('mypage');
//新規投稿
//ここのルートを実行しました
Route::get('/mypage/post', function(){
    return view('create_post');
})->name('create_post');


Auth::routes();
Route::get('/home', 'HomeController@index')->name('home');
//controller
<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Post;
use Illuminate\Support\Facades\Auth;
use App\Comment;
use App\User;

class PostController extends Controller
{
    //
    public function index($id){
        $post = Post::find($id);
        $comment = $post->comments()->get();

        return view('edit_post', [
            'id' => $id,
            'post' => $post,
            'comment' => $comment,
        ]);
    }


    public function edit(Request $request ,$id){
        $post = Post::find($id);
        $post->url = $request->input('url');
        $post->save();

        return redirect()->route('mypage');
    }
}
//Post.php
<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Post extends Model
{
//
public function comments(){
return $this->hasMany('App\Comment');
}

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

追記

ターミナルでphp artisan route:list --name=create_postを実行した結果です。

  +--------+----------+-------------+-------------+---------+------------+
| Domain | Method   | URI         | Name        | Action  | Middleware |
+--------+----------+-------------+-------------+---------+------------+
|        | GET|HEAD | mypage/post | create_post | Closure | web        |
+--------+----------+-------------+-------------+---------+------------+
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • Himajin_akisyou

    2020/04/12 14:57

    ありがとうございます。
    create_post.blade.phpの中身を全て消して実行してみたのですが同じエラーが出てしまったのでないと考えています。

    キャンセル

  • Lulucom

    2020/04/12 15:04

    そうですよね…。
    create_postのルートを最初の方(welcomeのルートの直後あたり)に移動すると挙動が変わらないでしょうか。つまりルート定義順が絡んでいるのではないかと推測しています。

    キャンセル

  • Himajin_akisyou

    2020/04/12 15:10

    できました!
    何度も連絡いただいて本当にありがとうございます!
    ベストアンサーにさせていただきたいのでお手数をおかけしますが回答に書いていただいてもよろしいでしょうか?

    キャンセル

回答 5

checkベストアンサー

0

create_postのルートを最初の方(welcomeのルートの直後あたり)に移動すると挙動が変わらないでしょうか。ルート定義順が絡んでいるのではないかと推測しています。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/04/12 15:20

    何度も連絡いただき紳士に解決に尽力していただき本当にありがとうございます。
    解決することができました。

    キャンセル

0

findメソッドの返却値は「コレクション」です。

Laravel 7.x Eloquent:利用の開始
また、主キーの配列をfindメソッドに渡し、呼び出すこともできます。一致したレコードのコレクションが返されます。

        $post = Post::find($id);
        $comment = $post->comments()->get(); //$postはPostモデルではなくコレクション

コレクションなので独自で作られているであろうcomments()というメソッドは持っていません。

Eloquent:コレクション
の「使用できるメソッド」参照

つまり、「エラーの通り、指定したメソッドをオブジェクトが持っていない」が原因です。

タイトルからすると「エラーの原因」をお探しのようなので、以上が回答になります。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/04/11 17:22

    質問内容確認してますか?→確認して欲しいところを具体的に書け→具体的に書いた→「そのように長文で私の回答云々言ってる暇に、質問本文を充実させた方が幾らか問題解決に繋がるでしょう。」

    そりゃ誰だって怒りますよ。
    最初は回答していただけたと思いましたよ。でもそれが質問内容をしっかし読んでいないことがわかって「内容確認しましたか?」って言ったら、「どこ?詳しく言え」って言われたから詳しく書いたら、「そんなに長文で詳しく書いている暇があるんだったらその時間で解決するんじゃないですか」とか意味不明でしょう。

    <上文>
    私自身、問題解決にしか興味がないので、喧嘩腰でもなんでもないですよ。
    淡々としてます。
    それを喧嘩腰だと捉えるのはあなたが喧嘩吹っ掛けてるつもりだからだと思います。
    落ち着きましょう。
    <上文>

    あなたが私の時間を奪っているから問題が解決しないんですよ。
    あなたこそ自分の回答を読んでください。矛盾ばっかり言ってますから。
    解決にしか興味がないんだったらしっかり質問文を読んだ上で答えたほうがいいんじゃないですか?
    もうコメントしないでください。
    本当に迷惑です。

    キャンセル

  • 2020/04/11 17:24

    >あなたが私の時間を奪っているから

    そういうスタンスなら結構です。
    質問者が神様とでも思ってるんでしょうか。

    キャンセル

  • 2020/04/11 17:27

    コメントしないでください。
    何度も言いますが迷惑です。

    キャンセル

0

class PostController extends Controller
{
    //
    public function index($id){
        $post = Post::find($id);
        // $comment = $post->comments()->get();
        $comments = $post->comments; // hasManyなんだから、$comments のほうが適切。そもそもこの行は不要。別途説明する。

        return view('edit_post', [
            'id' => $id,
            'post' => $post,
            'comment' => $comments,
        ]);
    }
}

$post を bladeに渡せば、 blade内で comments を取得できるのでコントローラーに書く必要はない。
同じ理由で$id も話たす必要ない。

class PostController extends Controller
{
    //
    public function index($id){
        $post = Post::find($id);
        return view('edit_post', [
            'post' => $post
        ]);
    }
}

その他おかしいところ

index() は一覧を実装すべきメソッド名です。なぜに「edit_post」なのか??非常に紛らわしいし、ミスを誘発するダメな実装です。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/04/11 18:56

    この回答のどこが低評価なのかな?

    キャンセル

-2

button要素を使わずa要素だけにするとどうでしょうか。

<a href="{{ route('create_post')}}" class="btn btn-primary btn-lg">新規投稿</a>

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/04/11 15:48

    ご回答ありがとうございます。
    やってみましたが同じエラーが出てしまいました。

    キャンセル

  • 2020/04/11 17:19

    そうでしたか。ただ、このボタンはリンクですのでbuttonを使わずaのみで作成すべきだとは思います。

    キャンセル

  • 2020/04/11 17:25

    わかりました。
    貴重な助言ありがとうございます。

    キャンセル

-2

php artisan route:clearを実行してから再確認するとどうでしょうか。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/04/11 17:24

    ご回答ありがとうございます。
    やってみましたが解決しませんでした。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.03%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る