 URL shortener - Symfony 4. |
| URL shortener - Symfony 4. |
Post
#1
|
|
|
Grupa: Zarejestrowani Postów: 8 Pomógł: 0 Dołączył: 15.06.2018 Ostrzeżenie: (0%) 
 |
Witam.
Prosiłbym Was o code review aplikacji do skracania adresów (IMG:style_emoticons/default/smile.gif) Czym więcej feedbacku, tym lepiej! https://github.com/selfmadeking/URL-shortener-Symfony-4 |
 |
 
|
 |
|
Post
#2
|
|
|
Grupa: Zarejestrowani Postów: 898 Pomógł: 48 Dołączył: 2.11.2005 Skąd: Poznań Ostrzeżenie: (0%)
|
Czy ja wiem czy zle zaprojektowany. Php ma w sobie dynamike i czasem po prostu nie da sie twardo typowac (albo nie ma to sensu). Tak jak pisalem przyklad z dzisiaj - na wejsciu do kontenera DI mialem konfiguracje w postaci tablicy. Tablica zawierala parametry, ktore wykorzyatywalo reflection api do inicjalizacji klas. Co by nie zrobic to getter do takiej tablicy bedzie zwracal praktycznie dowolny typ danych. Object tu tematu nie rozwiaze, bo w takiej tablicy moga byc skalary, tablice, obiekty/kolejcje obiektow - praktycznie wszystko.
Od czasu jak narzucilem sobie typowanie udalo mi sie 90% sytuacji tak zorganizowac aby funkcja zwracala 1 i tylko 1 typ danych, co z perspektywy uwazam za super rozwiazanie, bo api metod i czytelnosc kodu na tym zyskaly. Ale w swiecie php zawsze znajdzie sie taki przypadek, ktorego nie da sie przerobic albo nie ma to sensu. Getter do rozbudowanej konfiguracji jest IMO takim przypadkiem. Skoro jest void to i mixed moim zdaniem tez powinno byc aby zachowac jednolity interfejs. W php mixed to tez typ danych w pewnyn sensie (IMG:style_emoticons/default/smile.gif) Ten post edytował athabus 24.09.2018, 19:25:16 |
|
|
|
smk URL shortener - Symfony 4. 17.08.2018, 16:34:55 
athabus Kurcze, aż szkoda żeby taki fajny temat się zmarno... 23.08.2018, 21:25:44 
Lion Cytat(athabus @ 23.08.2018, 22:25:44 ... 23.09.2018, 09:30:48 Pyton_000 Jedna uwaga do Form->isValid. Owszem sprawdza a... 24.08.2018, 08:07:15 athabus No proszę, człowiek uczy się całe życie. Zawsze dl... 26.08.2018, 17:15:12 Pilsener Jak powyżej, np. taka metoda:
[PHP] pobierz, plai... 1.09.2018, 22:14:03 Pyton_000 Odniosę się do pkt.6 z którym się nie zgodzę. Nie ... 2.09.2018, 09:49:50 viking Swoją drogą tam wzorzec na url jest niepoprawny. N... 2.09.2018, 09:54:35 athabus Z if'ami każdy ma swoją teorię. Ja akurat też ... 2.09.2018, 15:22:32 Pilsener Cytatdruga mnie trochę przyprawia o ból głowy - bo... 2.09.2018, 20:07:19 nospor Cytat- jest taka zasada w programowaniu "don... 3.09.2018, 10:53:23 athabus Kurcze Pilsener dla mnie 2 wersja jest bardziej cz... 3.09.2018, 15:07:55 Pilsener Cytatodnosisz sie do wypowiedzi Pytona wyrwanej z ... 4.09.2018, 09:45:20 athabus Zgadzam się, że nie można na zasady patrzeć ślepo ... 4.09.2018, 19:29:26 athabus Ciekawe, że każdy ma swoje standardy w kwestii kod... 24.09.2018, 14:45:59 nospor Na szczescie w nowym wydaniu php bedzie mozna typo... 24.09.2018, 14:49:29 athabus Chyba czytasz w moich myślach, bo właśnie miałem n... 24.09.2018, 14:54:18 nospor CytatNo to jeszcze poproszę return type mixed przy... 24.09.2018, 15:03:38 athabus Czasami sa sytuacje, ze nie da sie w php przewidzi... 24.09.2018, 16:29:57 nospor Jesli nie wiesz, co bedzie zwracac twoja funkcja, ... 24.09.2018, 18:41:44 nospor CytatObject tu tematu nie rozwiaze, bo w takiej ta... 25.09.2018, 10:36:37 Pyton_000 Nie ma takiej sytuacji że nie da się tego opisać j... 25.09.2018, 16:58:11 borabora Cytat(Pyton_000 @ 25.09.2018, 17:58:1... 27.09.2018, 07:21:27 athabus Cytat(Pyton_000 @ 25.09.2018, 17:58:1... 7.10.2018, 14:55:56 Pyton_000 Null jest dość specyficzny. Ale taki prototyp fun... 27.09.2018, 07:34:34  |
|
Aktualny czas: 30.12.2025 - 23:33 |
