Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> [PHP] Nazewnictwo metod dla danych zwracanych z bazy
eerie
post 20.05.2020, 09:58:59
Post #1





Grupa: Zarejestrowani
Postów: 81
Pomógł: 0
Dołączył: 3.08.2017

Ostrzeżenie: (0%)
-----


Przy użyciu Doctrine np. zwracam z bazy obiekty:

https://github.com/webeeq/symfony.eeq/blob/...rRepository.php

I tu mam wątpliwość odnośnie nazewnictwa metod. Chcę wykorzystać same hasło, więc nazwałem metodę getIdPassword(). Jednak tak naprawdę zwracam cały obiekt user'a, który między innymi zawiera hasło. Tak samo jest z np. getApiUserPassword(). Jak powinienem nazywać tego typu metody, które zwracają podobne dane (np. obiekty user'a), aby nie mieć problemu z powtarzającym się nazewnictwem? Jakich nazw dla podanych metod powinienem użyć, aby było jak najbardziej poprawnie?

I jeszcze pytanie odnośnie np. isUserId(). Czy może zwracać obiekt lub null bądź powinna zwracać wartość logiczną typu bool? Po podstawieniu rezultatu do if'a tak i tak działa ok, ale jak jest poprawnie?
Go to the top of the page
+Quote Post
nospor
post 20.05.2020, 10:09:53
Post #2





Grupa: Moderatorzy
Postów: 36 455
Pomógł: 6292
Dołączył: 27.12.2004




Jakis balagan masz...

metody zaczynajace sie od "is" w domysle powinny zwracac true/false a nie null/object. Jak chcesz cos zwracac to masz zaczynac od "get"

zas co do pierwszego pytania: przeciez te metody zwracaja to samo. Zawsze obiekt usera z calymi danymi. Wiec albo pytasz o haslo i zwracaj zawsze haslo a nie caly obiekt, albo nazywaj je jak trzeba
getIdPassword -> getUserById
getApiUserPassword -> getUserByLogin


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer

Go to the top of the page
+Quote Post
eerie
post 20.05.2020, 11:08:52
Post #3





Grupa: Zarejestrowani
Postów: 81
Pomógł: 0
Dołączył: 3.08.2017

Ostrzeżenie: (0%)
-----


Dzięki serdeczne za odpowiedź. smile.gif
Go to the top of the page
+Quote Post

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Wersja Lo-Fi Aktualny czas: 25.04.2024 - 13:07