編集履歴

回答編集履歴

追記

2020/11/12 14:15

投稿

スコア3923

test CHANGED Viewed

@@ -1,466 +1,6 @@
-HasMany にしているのは間違いですね。
+＝＝＝＝　消した　＝＝＝＝
-likes テーブルの役割は、Post と User を結ぶ多対多のピポットテーブルですね。
-定義を見る限り。
-HasMany でもできなくはないけど。
-手元の環境は BelongsToMany で作ったのでそれを参考にしてください。
-User は全てデフォルトのままなので省略します。
-手元の環境では、これで正常に動作している。
-# DB
-```php
-<?php
-use Illuminate\Database\Migrations\Migration;
-use Illuminate\Database\Schema\Blueprint;
-use Illuminate\Support\Facades\Schema;
-class CreatePostsTable extends Migration
-{
-    /**
-     * Run the migrations.
-     *
-     * @return void
-     */
-    public function up()
-    {
-        Schema::create('posts', function (Blueprint $table) {
-            $table->bigIncrements('id');
-            $table->string('title');
-            $table->string('content');
-            $table->timestamps();
-        });
-    }
-    /**
-     * Reverse the migrations.
-     *
-     * @return void
-     */
-    public function down()
-    {
-        Schema::dropIfExists('posts');
-    }
-}
-```
-```php
-<?php
-use Illuminate\Database\Migrations\Migration;
-use Illuminate\Database\Schema\Blueprint;
-use Illuminate\Support\Facades\Schema;
-class CreateLikesTable extends Migration
-{
-    /**
-     * Run the migrations.
-     *
-     * @return void
-     */
-    public function up()
-    {
-        Schema::create('likes', function (Blueprint $table) {
-            $table->bigIncrements('id');
-            $table->unsignedBigInteger('post_id');
-            $table->unsignedBigInteger('user_id');
-            $table->timestamps();
-            $table->foreign('post_id')->references('id')->on('posts');
-            $table->foreign('user_id')->references('id')->on('users');
-        });
-    }
-    /**
-     * Reverse the migrations.
-     *
-     * @return void
-     */
-    public function down()
-    {
-        Schema::dropIfExists('likes');
-    }
-}
-```
-# Model
-```php
-<?php
-namespace App;
-use Carbon\Traits\Timestamp;
-use Illuminate\Database\Eloquent\Model;
-use Illuminate\Database\Eloquent\Relations\BelongsToMany;
-class Post extends Model
-{
-    use Timestamp;
-    /**
-     * @return BelongsToMany|User[]
-     */
-    public function likes(): BelongsToMany
-    {
-        return $this->belongsToMany(User::class, 'likes')
-            ->withTimestamps();
-    }
-}
-```
-# Factory
-```php
-<?php
-/** @var \Illuminate\Database\Eloquent\Factory $factory */
-use App\Post;
-use Faker\Generator as Faker;
-$factory->define(Post::class, function (Faker $faker) {
-    return [
-        'title' => $faker->word,
-        'content' => $faker->sentence
-    ];
-});
-```
-# Seeder
-```php
-<?php
-use App\Post;
-use App\User;
-use Illuminate\Database\Seeder;
-class PostSeeder extends Seeder
-{
-    /**
-     * Run the database seeds.
-     *
-     * @return void
-     */
-    public function run()
-    {
-        /** @var User[] users */
-        $users = factory(User::class, 20)
-            ->create();
-        $this->posts = factory(Post::class, 100)
-            ->create()
-            ->each(function (Post $post) use ($users) {
-                $post->likes()->attach(
-                    $users->random(random_int(1, 3))->pluck('id')->toArray()
-                );
-            });
-    }
-}
-```
-# Controller
-```php
-<?php
-namespace App\Http\Controllers;
-use App\Post;
-use Illuminate\Http\Request;
-use Illuminate\View\View;
-class PostController extends Controller
-{
-    /**
-     * Display a listing of the resource.
-     *
-     * @return View
-     */
-    public function index(): View
-    {
-        $posts = Post::query()
-            ->withCount(['likes'])
-            ->paginate();
-        return view('posts.index', compact('posts'));
-    }
-}
-```
-# View
-```php
-<table>
-    <tbody>
-    @foreach($posts as $post)
-    <tr>
-        <td>{{ $post->id }}</td>
-        <td>{{ $post->likes_count }}</td>
-    </tr>
-    @endforeach()
-    </tbody>
-</table>
-```
-# SQLエラーの確認方法
-```php
-        $posts = Post::query()
-            ->withCount(['likes'])
-            ->paginate();
-```
-このコードで仮に、エラーが出たとしよう。
-```diff
-        $posts = Post::query()
-            ->withCount(['likes'])
--            ->paginate();
-+            ->toSql();
-+       dd($posts);
-```
-こんなふうにすれば、組み立てられたSQLを確認できる
-# 余談

追記

2020/11/12 14:15

投稿

phper.k

スコア3923

test CHANGED Viewed

@@ -420,6 +420,46 @@
+# SQLエラーの確認方法
+```php
+        $posts = Post::query()
+            ->withCount(['likes'])
+            ->paginate();
+```
+このコードで仮に、エラーが出たとしよう。
+```diff
+        $posts = Post::query()
+            ->withCount(['likes'])
+-            ->paginate();
++            ->toSql();
++       dd($posts);
+```
+こんなふうにすれば、組み立てられたSQLを確認できる
 # 余談

修正

2020/11/12 09:43

投稿

phper.k

スコア3923

test CHANGED Viewed

@@ -424,8 +424,24 @@
-Laravel の開発にあたっては、以下のプッラグインは必ずインストすべし。
+Laravel の開発にあたっては、以下のプラグインは必ずインストすべし。
 [https://github.com/barryvdh/laravel-debugbar](https://github.com/barryvdh/laravel-debugbar)
+基本的に、LaravelのDB周りの処理は、 Eloquent で書くべき。
+`DB::select()`なんて滅多に使う場面がない。よほど、Eloquent では十分なパフォーマンスが出ない場合などに限られる。DBに用意されているメソッドはEloquentでも使えるのだから、Eloquentでかけないはずがない。
+Laravel のベストプラクティスがまとめられているので、参考にすると良いでしょう。
+[https://github.com/alexeymezenin/laravel-best-practices](https://github.com/alexeymezenin/laravel-best-practices/blob/master/japanese.md)

追記

2020/11/12 09:38

投稿

phper.k

スコア3923

test CHANGED Viewed

@@ -417,3 +417,15 @@
 </table>
 ```
+# 余談
+Laravel の開発にあたっては、以下のプッラグインは必ずインストすべし。
+[https://github.com/barryvdh/laravel-debugbar](https://github.com/barryvdh/laravel-debugbar)