Laravelで削除機能を作成したのですが、１行目以外は削除出来るのですが１行目だけ削除出来ません。

Question

前提・実現したいこと

####リストの１行目を削除したい
Laravelで削除機能を作成したのですが、１行目以外は削除出来るのですが１行目だけ削除出来ません。

ページを変わってもリストの１行目の削除だけ削除出来ず、同じidでも１行目以外にあると削除出来るようです。
エラーは出ません。

該当のソースコード

@extends('layouts.layout')

@section('content')
  <header id="">
　　　<h1>Laravel Project</h1>
<div class="nav"><a href="" class="btn list-btn">ログアウト</a>
  <a href="" class="btn list-btn">管理者情報</a></div>
  </header>
  <section id="serch">
    <div class="head"><h2>参加者名簿一覧</h2></div>
      <form action="" method="get">
        <div class="serch-wrap">
          <div class="serch">
            <input type="text" name="keyword" class="key-form" value="" placeholder="検索キーワード">
          </div>
          <div class="key">
            <input type="submit" value="検索"  class="btn serch-btn">
          </div>
            <a href="{{ action('MemberController@new_Index') }}" class="btn create-btn">新規登録</a>
      </div>
  </section>
  <section id="list-table">
   <table class="list">
     <thead>
       <tr>
       <th>ID</th>
       <th>名前</th>
       <th>Email</th>
       <th>電話番号</th>
       <th colspan="3"></th>
      </tr>
     </thead>
     <tbody>
       @foreach($members as $member)
       <tr>
         <td>{{$member->id}}</td>
         <td>{{$member->name}}</td>
         <td>{{$member->email}}</td>
         <td>{{$member->tel}}</td>
         <td><a href="" class="btn detail-btn">詳細</a></td>
         <td><a href="{{ action('MemberController@edit_index',[$member->id]) }}"  class="btn edit-btn">編集</a></td>
         <td>
         <form action="/member/delete/{{$member->id}}" method="post">
         @csrf
          <input type="submit" value="削除" class="btn dell-btn"></td>
          </form>
       </tr>
       @endforeach
     </tbody>
   </table>
  </section>

  <!-- page controll -->
  <div class="pager">
  {{ $members->links() }}
  </div>
@endsection

<?php

/*
|--------------------------------------------------------------------------
| 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('member/list', 'MemberController@getIndex');

Route::group(['prefix' => 'member'], function() {
  Route::get('list', 'MemberController@getIndex');
  Route::get('new', 'MemberController@new_Index');
  Route::patch('new', 'MemberController@new_confirm');
  Route::post('new', 'MemberController@new_finish');
});

Route::group(['prefix' => 'member'], function() {
  Route::get('edit/{id}/', 'MemberController@edit_index');
  Route::patch('edit/{id}/', 'MemberController@edit_confirm');
  Route::post('edit/{id}/', 'MemberController@edit_finish');
});

Route::group(['prefix' => 'member'], function() {
  Route::post('delete/{id}/', 'MemberController@member_delete');
});

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;

class MemberController extends Controller
{

    public function getIndex()
    {
        $members = \App\Member::paginate(10);
        return view('layouts.member.list')->with('members',$members);
    }

    public function new_index()
    {
        return view('layouts.member.new_index');
    }

    public function new_confirm(\App\Http\Requests\CheckMemberRequest $request)
    {
        $data = $request->all();
        return view('layouts.member.new_confirm')->with($data);
    }

    public function new_finish(Request $request)
    {
        $member = new \App\Member;

        $member->name = $request->name;
        $member->email = $request->email;
        $member->tel = $request->tel;
        $member->addr = $request->addr;
        $member->birth = $request->birth;
        $member->save();

        return redirect()->to('member/list');
    }

    public function edit_index($id)
    {
        $member = \App\Member::findOrFail($id);
        return view('layouts.member.edit_index')->with('member',$member);
    }

    public function edit_confirm(\App\Http\Requests\CheckMemberRequest $request)
    {
        $data = $request->all();
        return view('layouts.member.edit_confirm')->with($data);
    }

    public function edit_finish(Request $request, $id)
    {
        $member = \App\Member::findOrFail($id);

        $member->name = $request->name;
        $member->email = $request->email;
        $member->tel = $request->tel;
        $member->addr = $request->addr;
        $member->birth = $request->birth;
        $member->save();

        return redirect()->to('member/list');
    }

    public function member_delete($id)
    {
        $member = \App\Member::find($id);
        $member->delete();

        return redirect()->to('member/list');
    }
}

補足情報

laravel5.8
laradock
Mysql
TablePlus
github
よろしくお願いします。

Comments

[m.ts10806]

$member = \App\Member::find($id);
↑この$memberは想定通りの情報がとれているのでしょうか。
指定のidがきていますか？

また
<form action="/member/delete/{{$member->id}}" method="post">
↑ここの$member->idは間違いなくその人のIDが入っているのでしょうか

Answer 1

formタグとtdタグが噛み合ってないのでそれが原因でformが二重と解釈されてしまっていたりしませんか？

Comments

[m.ts10806]

気づかなかった。その可能性は高そうですね。

[Shinji1129]

ありがとうございます。全く気がつきませんでした。
しかしタグを修正しても処理されませんでした。

[Shinji1129]

訂正します。
やはりformタグが噛み合っておりませんでした

Answer 2

自己解決しました。
検索機能を追加したところ正常に削除されるようになりました。
ページネーションに不備があったように考えますが、はっきりとは分かりません。
修正コードはgithubにて、
DockerLaravel
ありがとうございました

Comments

[mikkame]

https://github.com/Shinji1129/DockerLaravel/blob/121b7af2e68bc696ce0740be366703b7876980f5/laravel-practice/resources/views/layouts/member/list.blade.php

github見ましたが、やはり質問時点で<form>タグの数が一致しておりません。
おそらく、検索機能を実装した時に閉じタグが追加されたことにより、たまたまうまく解釈されるようになったのでしょう。
ページネーションは関係ないかと思います

[mikkame]

11行目の対になる閉じタグがない感じですね

[Shinji1129]

再度ご覧いただきありがとうございます。
はっきり確認しました！
全くその通りです!!
私の確認不足です。ありがとうございました。

Answer 3

idが正しく来てないように思いますので、idが意図したものかどうか確認してください。

あと下記も試してみてください。

find()ではなくwhere()

主キーなのであればdestroy()

Comments

[Shinji1129]

ありがとうございます。
where(), destroy()共に試したのですが１行目だけ処理されません。
２行目以降はエラーがでることもあり、
１行目は実行しても処理すらかけられていないようです。