[inny]Laravel problem z rolami użytkownika, nie bierze pod uwagę warunku Opcje

[casperii]

Panowie, przejdę może od razu do rzeczy, mam kod:

Role.php :

[PHP] pobierz, plaintext 
<?php
 
namespace App;
 
use Illuminate\Database\Eloquent\Model;
 
class Role extends Model
{
 
	public $guarded = []; 
    public $timestamps = false;
 
    public function users()
    {
        return $this->belongsToMany('App\User');
    }
}
[PHP] pobierz, plaintext 

oraz RegisterController.php

[PHP] pobierz, plaintext 
if(!Role::where('name','type')->exists())
        {
            Role::create(['name'=>'person']);
            Role::create(['name'=>'firm']);
        }
 
        //sprawdzamy czy typ = 0 czyli równy 1 - jeżeli tak dodaje Role firm w innym przypadku dodaje person w tabeli roles
 
        if($data['type'] (IMG:style_emoticons/default/questionmark.gif)  0)
            $user->roles()->attach( Role::where('name','firm')->first()->id );
        else
            $user->roles()->attach( Role::where('name','person')->first()->id );
 
[PHP] pobierz, plaintext 

czyli przy każdej rejestracji użytkownika dodaje także do tabeli role_user typ konta (firma / os. fiz) , oraz dodatkowo mam stworzony warunek , który nie działa zgodnie z jego logiką, tzn. sprawdzam czy istnieje w tabeli roles jakieś pola , jeżeli nie to dopisuje 2 rekordy person oraz firm. Problem w tym, że warunek ten działa zawsze bez znaczenia czy ów wartości znajdują się już w tabeli czy też nie.

[SmokAnalog]

Wiesz co, tak naprawdę to i tak, i tak będzie w porządku. Osobna tabela będzie miała taką zaletę, że możesz z poziomu bazy zadbać o integralność danych.

Konkretniej, to jest dobry przykład na użycie relacji 1:1. Dane byłyby połączone z użytkownikiem i user_id byłby unikalny. Kolumny w tej tabeli nie musiałyby być nullable. Mógłbyś też w jednoznaczny sposób się dowiedzieć, czy dany użytkownik wypełnił już swoje dane, czy nie. Świadczy o tym istnienie (lub brak) rekordu w tej dodatkowej tabeli.

Wadą takiego rozwiązania jest to, że przy domyślnym użyciu Eloquent, pojawiłby się oczywiście obiekt pośredni, czyli zamiast np. $user->name miałbyś np. $user->data->name. Trochę upierdliwe.

Ja bym chyba poszedł na skróty i dodał po prostu te kilka pól bezpośrednio do Usera w formie nullable. Integralność nie jest tak dobra, ale jest łatwiej ogarnąć dostęp do tych danych. Wtedy możesz wybrać jedną kolumnę jako dowód, że dane zostały już wypełnione, lub też wiele kolumn. Ja bym wstawił np. pole z datą wypełnienia danych (nullable) i tym się posługiwał przy ustalaniu czy ktoś wypełnił dane.

Ten post edytował SmokAnalog 27.09.2020, 21:56:27