[inny]Laravel problem z rolami użytkownika, nie bierze pod uwagę warunku |
[inny]Laravel problem z rolami użytkownika, nie bierze pod uwagę warunku |
27.09.2020, 13:28:05
Post
#1
|
|
Grupa: Zarejestrowani Postów: 680 Pomógł: 28 Dołączył: 14.08.2014 Ostrzeżenie: (0%) |
Panowie, przejdę może od razu do rzeczy, mam kod:
Role.php :
oraz RegisterController.php
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. |
|
|
27.09.2020, 16:45:22
Post
#2
|
|
Grupa: Zarejestrowani Postów: 1 707 Pomógł: 266 Dołączył: 3.07.2012 Skąd: Poznań Ostrzeżenie: (0%) |
Jakiś dziwny ten Twój kod. Sprawdzasz, czy istnieje rola z kolumną *name* o wartości *type*? Tak po prostu, bez względu na przypisania do użytkownika?
Po drugie nie pokazałeś tego $data i nie powiedziałeś o którym warunku mówisz. Napisz jaśniej i pokaż więcej kodu. Poza tym, firma to po angielsku company, a nie firm Ten post edytował SmokAnalog 27.09.2020, 16:46:32 |
|
|
27.09.2020, 20:03:24
Post
#3
|
|
Grupa: Zarejestrowani Postów: 680 Pomógł: 28 Dołączył: 14.08.2014 Ostrzeżenie: (0%) |
@SmokAnalog odnośnie twojej uwagi:
They are almost the same, but "firm" is often used for legal type businesses. "Company" is more common. co do kodu :
Mam 3 tabele : users , roles, role_user, users - wiadomo tabela roles: id, name 1, person 2, firm oraz trzecią tabele role_user user_id, role_id 1, 1 2, 1 3, 2 4, 1 5, 2 czyli zamierzeniem / zabezpieczeniem kodu:
powinno być jeśli nie ma danych w tabeli roles , uzupełnij je danymi :
Niestety każde dodanie nowego użytkownika powoduje, że w tabeli roles dane mi się dopisują tzn: 1, person 2, firm 3, peron 4, firm 5, person 6, firm itd. |
|
|
27.09.2020, 20:54:53
Post
#4
|
|
Grupa: Zarejestrowani Postów: 1 707 Pomógł: 266 Dołączył: 3.07.2012 Skąd: Poznań Ostrzeżenie: (0%) |
|
|
|
27.09.2020, 21:19:07
Post
#5
|
|
Grupa: Zarejestrowani Postów: 680 Pomógł: 28 Dołączył: 14.08.2014 Ostrzeżenie: (0%) |
|
|
|
27.09.2020, 21:21:45
Post
#6
|
|
Grupa: Zarejestrowani Postów: 1 707 Pomógł: 266 Dołączył: 3.07.2012 Skąd: Poznań Ostrzeżenie: (0%) |
A nie lepiej sprawdzić czy w ogóle są jakiekolwiek, zamiast zakładać, że ma być ta jedna? Moim zdaniem taki kod prosi się o trudne do znalezienia błędy w przyszłości.
|
|
|
27.09.2020, 21:23:09
Post
#7
|
|
Grupa: Zarejestrowani Postów: 680 Pomógł: 28 Dołączył: 14.08.2014 Ostrzeżenie: (0%) |
jakaś podpowiedź ?
|
|
|
27.09.2020, 21:27:03
Post
#8
|
|
Grupa: Zarejestrowani Postów: 1 707 Pomógł: 266 Dołączył: 3.07.2012 Skąd: Poznań Ostrzeżenie: (0%) |
Moim zdaniem lepiej byłoby po prostu tego nie sprawdzać, tylko mieć seedy do wstawiania tych podstawowych ról. Logika nie powinna się sypać, kiedy nie masz ról. Ewentualnie dałbym do panelu admina ostrzeżenie, gdy role są puste. Wiesz, warto unikać takich sprawdzajek, bo to niewiele wnosi, a dodaje warstwę logiki, o której się zapomina.
|
|
|
27.09.2020, 21:30:48
Post
#9
|
|
Grupa: Zarejestrowani Postów: 680 Pomógł: 28 Dołączył: 14.08.2014 Ostrzeżenie: (0%) |
oczywiście na dalszym planie jest możliwość dodawania / edytowania / usuwania ról z poziomu panelu administracyjnego. Nie mniej jednak dziękuje za sugestię
Ten post edytował casperii 27.09.2020, 21:47:15 |
|
|
27.09.2020, 21:37:03
Post
#10
|
|
Grupa: Zarejestrowani Postów: 1 707 Pomógł: 266 Dołączył: 3.07.2012 Skąd: Poznań Ostrzeżenie: (0%) |
No tak szczerze mówiąc to wstawianie ról przy wstawianiu użytkownika prosi się o problemy To nie jest miejsce w kodzie, w którym chcesz cichaczem dodawać rekordy do innego modelu.
Gdybyś pytał, to żeby sprawdzić czy tabela zawiera jakiekolwiek rekordy, możesz dać exists() od razu, czyli:
|
|
|
27.09.2020, 21:46:59
Post
#11
|
|
Grupa: Zarejestrowani Postów: 680 Pomógł: 28 Dołączył: 14.08.2014 Ostrzeżenie: (0%) |
@SmokAnalog jeszcze mam pytanie odnośnie dodatkowych danych usera (imie, nazwisko , płeć, data ur, adres ... ) te dane będzie można dopiero dopisać po weryfikacji maila ( nie chcę usera zanudzać wypełnieniem pól)
czy lepiej dodać powyższe pola do tabeli user, czy np utworzyć inną tabelę i tam wstawiać te pola ? |
|
|
27.09.2020, 21:55:20
Post
#12
|
|
Grupa: Zarejestrowani Postów: 1 707 Pomógł: 266 Dołączył: 3.07.2012 Skąd: Poznań Ostrzeżenie: (0%) |
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 |
|
|
Wersja Lo-Fi | Aktualny czas: 25.09.2024 - 15:19 |