 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%)
|
Kurcze Pilsener dla mnie 2 wersja jest bardziej czytelna. Może dlatego, że w pierwszej zgubiłeś ! w pierwszym warunku i się zastanawiałem co ta funkcja robi. Ale nawet bez tego jakoś przyzwyczaiłem się do drugiego zapisu i tak bym to zrobił... Widać wszystko kwestią gustu i coś dla jednych jest bardziej czytelne niż dla innych.
BTW poruszyłeś ciekawy temat, który uciekł w toku dyskusji, czyli CQS - command-guery separation. Ostatnio staram się stosować (ostatnio w sumie całe swoje kodowe nawyki zmieniam) i zauważyłem, że czasami powoduje puchnięcie kodu (zwłaszcza jak funkcja na wyższym poziomie grupuje działania wykonywane przez funkcje na niższym poziomie, gdzie część jest typu query, a inne typu command). Stosujecie tą regułę bez odstępstw, czy Wam się jednak zdarza. Ja staram się bez odstępstw, bo uznałem, że czytelny kod czasami może być dłuższy niż ten nieczytelny. |
|
|
|
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 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 athabus Czy ja wiem czy zle zaprojektowany. Php ma w sobie... 24.09.2018, 19:22:12 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: 15.10.2025 - 16:49 |
