laravelのデータベース更新について

Question

larevel初心者です。
laravelでの入力内容がデータベースに反映されず困っています。

データベースから名前を取得して、
inputのvalue属性で表示することはできています。
データベースから名前を取得している画面

そのため、データベースとは繋がっていると思いますが、
更新作業を色々と試してみましたが、名前が変わりません。

MAMPを使っているので、
phpMyAdminも見てみましたが、
やはり、こちらも名前の変更がされていません。

初心者ゆえの初歩的なミスの見落としでしたら、
申し訳ありません。

お手隙の際にお教えいただけるとありがたいです。
もし、他に必要なコードがあれば、再度あげるようにします。

よろしくお願いします。

edit.blade.php

@extends('layouts.top')
@section('title', '先生用ページ編集')
@section('content')
    <div>
     	<h2>先生登録</h2>
    </div>
    <div>
     	<form action="{{ action('Teacher\TeacherController@create') }}" method="post" enctype="multipart/form-data">
     		 {{ csrf_field() }}
     		 <div>
                <label for="name">名前</label>
                <input type="text"　name="name" value="{{ $teacher->name }}">
            </div>
            <input type="submit" value="送信">
     	</form>	
    </div> 	
    <div>
        <a href="{{ action('Teacher\TeacherController@index') }}">戻る</a>
    </div>   
@endsection

TeacherContoller.php

<?php

namespace App\Http\Controllers\Teacher;

use App\Http\Controllers\Controller;
use app\Http\Middleware\Authenticate;
use Illuminate\Http\Request;

use App\User;

class TeacherController extends Controller
{
    public function login()
  {
      return view('teacher.login');
  }

  public function index()
  {
      return view('teacher.index');
  }

    public function edit()
  { 
      $teacher =User::find(1);
      return view('teacher.edit',['teacher' => $teacher]);
  }


  public function create(Request $request){
        
        $this->validate($request, User::$rules);
        $teacher =User::find(1);
        $teacher_form = $request->all();
        $teacher->fill($teacher_form)->save();
        unset($teacher_form['_token']);

        return redirect ('teacher/index');
    }
}

Userモデル

<?php

namespace App;

use Illuminate\Contracts\Auth\MustVerifyEmail;
use Illuminate\Foundation\Auth\User as Authenticatable;
use Illuminate\Notifications\Notifiable;

class User extends Authenticatable
{
    use Notifiable;

    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'name', 'email', 'password',
    ];

    /**
     * The attributes that should be hidden for arrays.
     *
     * @var array
     */
    protected $hidden = [
        'password', 'remember_token',
    ];

    /**
     * The attributes that should be cast to native types.
     *
     * @var array
     */
    protected $casts = [
        'email_verified_at' => 'datetime',
    ];

    public static $rules = array(
            'name' => 'required',
        );
}

テーブル定義

<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

class CreateUsersTable extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::create('users', function (Blueprint $table) {
            $table->bigIncrements('id');
            $table->string('name');
            $table->string('email')->unique();
            $table->timestamp('email_verified_at')->nullable();
            $table->string('password');
            $table->rememberToken();
            $table->timestamps();
        });
    }

    /**
     * Reverse the migrations.
     *
     * @return void
     */
    public function down()
    {
        Schema::dropIfExists('users');
    }
}

ルーティング

<?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('/', function () {
    return view('welcome');
});

Route::get('', 'Top\TopController@top');

Route::get('top', 'Top\TopController@top');

Route::get('search', 'Top\TopController@search');

Route::get('logout', 'Auth\LoginController@logout', function () {
    return abort(404);
});

Route::group(['prefix' => 'teacher'], function() {
	Route::get('login', 'Teacher\TeacherController@login');
	Route::get('index', 'Teacher\TeacherController@index')->middleware('auth');
    Route::get('edit', 'Teacher\TeacherController@edit')->middleware('auth');
    Route::post('edit', 'Teacher\TeacherController@create')->middleware('auth');
});

Route::group(['prefix' => 'question'], function() {
	Route::get('index', 'Question\QuestionController@index');
});

Auth::routes();

Route::get('/home', 'HomeController@index')->name('home');

Comments

[hentaiman]

DBに間違いなく繋がっているとして、モデルも載せた方がいいですね
載せるついでにfillableかguardedが適切に設定されているか確認してみてください

[退会済みユーザー]

>hentaimanさん
ありがとうございます!

[Orlofsky]

URLは https://teratail.com/help/question-tips#questionTips3-7 の [リンク] で
[リンク先の説明](http...) に修正してください。

[退会済みユーザー]

>Orlofsky
ありがとうございます、修正させていただきました。

[m.ts10806]

ルーティングもご提示ください

[m.ts10806]

あとテーブル定義も必要ですね

[hentaiman]

あと更新対象のidが1である前提でコメントしてしまったけどそれは正しいのか？という事と、
laravel以外は慣れているのならDB側のクエリログを出力するようにしてどんなSQLが流れているか確認しましょう

[退会済みユーザー]

>m.ts10806
ありがとうございます。修正いたしました。

>hentaiman
ありがとうございます。idは1で正しいです。
クエリログ、確認してみます。

Answer 1

コントローラーの保存の書き方を変えて、
バリデーションをなくし、
サーバーを再起動したところ、
データベースにも名前の変更が更新されるようになりました。

TeacherContoller.php

public function create(Request $request){
        
        $teacher =User::find(1);
        $teacher->name = $request->name;
        $teacher->save();
        

        return redirect ('teacher/index');
    }

Userモデル

<?php

namespace App;

use Illuminate\Contracts\Auth\MustVerifyEmail;
use Illuminate\Foundation\Auth\User as Authenticatable;
use Illuminate\Notifications\Notifiable;


class User extends Authenticatable
{
    use Notifiable;

    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'name', 'email', 'password',
    ];

    /**
     * The attributes that should be hidden for arrays.
     *
     * @var array
     */
    protected $hidden = [
        'password', 'remember_token',
    ];

    /**
     * The attributes that should be cast to native types.
     *
     * @var array
     */
    protected $casts = [
        'email_verified_at' => 'datetime',
    ];
}