为什么我的数据不存储在数据库使用雄辩？

Question

我的表单已提交，但未存储在表中。如果我对请求执行dd（），则数据在中，但当我执行save（）时，它不会按预期工作。我想在backoffice表单上添加用户，仅包含姓名、电子邮件、用户类型和密码。

编辑：我将问题图像更改为代码，以便您更容易理解，很抱歉第一次尝试。编辑2：现在出现了更多的两件事，密码验证确认总是错误的，如果我跳过验证，则会出现以下错误：

对未定义方法App\User:：forget（）的BadMethodCallException调用

控制器上的功能存储：

public function store(Request $request)
    {
        $dados = $request->validate(
            [
                'name'  =>'required|min:2|max:20',
                'email' =>'required|email|unique:App\User,email',
                'password' =>'required|min:8',
                'password_confirmation' =>'confirmed'
            ],
            [
                'required' =>'Os campos têm de ser todos preenchidos!',
                'email' =>'O email tem de ser válido!',
                'unique'=>'O email inserido já está registado!',
                'min.name' =>'O nome tem de ter pelo menos 2 caracteres!',
                'min.password' =>'A password tem de ter no minimo 8 caracteres!',
                'confirmed' =>'As passwords deverão coincidir!'
            ]
        );

        $user = new User;
        $user->forget('password_confirmation');
        $user->fill($request->all());
        $user->save();

        return redirect()->route('utilizadores.index')->with('Success','Utilizador registado com sucesso!');
    }

路线：

Route::resource('/admin/utilizadores', 'UserController');

型号：

class User extends Authenticatable
{
    use Notifiable;

    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $guarded = [

    ];

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

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

表格：

<div class="modal-body">
            <form action="{{route('utilizadores.store')}}" method="POST" id="adicionarid">
                @csrf
                <input type="email" name="email" class="inputtext data" placeholder="Email">
                <div class="customselect data top1">
                    <select name="tipo">
                        <option value="revisor" selected disabled>Revisor</option>
                        <option value="editor">Editor</option>
                        <option value="revisor">Revisor</option>
                    </select>
                </div>
                <input type="text" name="name" class="inputtext" placeholder="Nome">
                <input type="password" name="password" class="inputtext" placeholder="Palavra-passe">
                <input type="password" name="password_confirmation" class="inputtext" placeholder="Confirmar palavra-passe">
            </form>
        </div>
        <div class="modal-footer">
            <button type="button" class="btn btn-secondary" data-dismiss="modal">Fechar</button>
            <button type="submit" class="btn btn-success" form="adicionarid">Guardar</button>
        </div>

Answer 1

我在你的代码中看到了很多问题。让我们先把它们列出来，这样你就知道你做错了什么

'password_confirmation' =>'confirmed'

我很确定这不会达到你期望的效果。确认验证规则必须用于密码，然后laravel将自动检查是否有名为密码确认（字段名确认）的字段，并检查其是否等于密码字段。因此，您的验证规则可以是

'password' =>'required|min:8|confirmed',

而且不需要关于密码确认的规则

其次是

$user->forget('password_confirmation');

在eloquent或laravel的查询生成器中没有“忘记”方法，因此

对未定义方法App\User:：forget（）的BadMethodCallException调用

在集合和Arr::forget（）上有忘记方法，但这不是您想要的。但是由于您已经有了$dados，它将是一个数组，只包含您在验证器中定义了规则的输入字段，因此您可以在创建用户时使用$dados（将在后面的答案中显示如何）

其次是

$user->fill($request->all());

永远不要这样做，因为恶意用户可以在表单中添加额外的字段，例如id，从而允许他们修改现有用户。

相反，你可以这样做

$user->fill($dados);

由于$dados只包含验证器返回的字段，因此使用起来更安全。

我注意到的另一个问题是表单中有一个名为tipo的字段，但它不在验证器中，因此需要将其添加到验证器中。

$dados = $request->validate(
    [
        'name'  =>'required|min:2|max:20',
        'email' =>'required|email|unique:App\User,email',
        'password' =>'required|min:8|confirmed',
        'tipo' =>'required' // Add whatever other validations you need
    ],
    [
        'required' =>'Os campos têm de ser todos preenchidos!',
        'email' =>'O email tem de ser válido!',
        'unique'=>'O email inserido já está registado!',
        'min.name' =>'O nome tem de ter pelo menos 2 caracteres!',
        'min.password' =>'A password tem de ter no minimo 8 caracteres!',
        'confirmed' =>'As passwords deverão coincidir!'
    ]
);

那你就做吧

$user = new User;
$user->fill($dados);
$user->save();

下一个问题是，您没有散列密码。除非您想直接将用户密码保存在数据库中，否则您应该散列密码；Laravel通过提供一个可以使用的Hash类使哈希变得简单。

$dados['password'] = Hash::make($request->password);

确保添加使用light\Support\Facades\Hash 位于文件的顶部

所以总的来说，通过一些调整，你的控制器应该

public function store(Request $request)
{
    $dados = $request->validate(
        [
            'name'  =>'required|min:2|max:20',
            'email' =>'required|email|unique:App\User,email',
            'password' =>'required|min:8|confirmed',
            'tipo' =>'required' // Add whatever other validations you need
        ],
        [
            'required' =>'Os campos têm de ser todos preenchidos!',
            'email' =>'O email tem de ser válido!',
            'unique'=>'O email inserido já está registado!',
            'name.min' =>'O nome tem de ter pelo menos 2 caracteres!',
            'password.min' =>'A password tem de ter no minimo 8 caracteres!',
        ]
    );

    $dados['password'] = Hash::make($request->password);

    $user = new User;
    $user->fill($dados);
    $user->save();

    return redirect()->route('utilizadores.index')->with('Success','Utilizador registado com sucesso!');
}

它应该能达到你的期望

现在，让我们给你一些额外的提示。

始终避免使用空的$guarded

protected $guarded = [

];

它可能会留下安全漏洞（即使你没有暴露任何这样的漏洞，粗心的人也可能会暴露，所以总是尝试使用$fillable，安全总比抱歉好）

protected $fillable = [
    'name', 'email', 'password', 'tipo'
];

然后在你的控制器里，你可以安全地使用

$user = User::create($dados);

这是一种比fill（）更广泛使用的方法。$filleble数组还确保在使用create（）时，只有$filleble数组中的属性才能保存在数据库中