[inny]Własny framework z auto wstrzykiwaniem |
[inny]Własny framework z auto wstrzykiwaniem |
4.08.2017, 07:28:51
Post
#1
|
|
Grupa: Zarejestrowani Postów: 17 Pomógł: 2 Dołączył: 26.10.2013 Ostrzeżenie: (0%) |
Witam,
Hobbistycznie napisałem własny framework. Czy ktoś mógłby się wypowiedzieć co o nim sądzi. https://github.com/primosz67/tahona-spark-mvc |
|
|
4.08.2017, 20:28:12
Post
#2
|
|
Grupa: Moderatorzy Postów: 36 457 Pomógł: 6296 Dołączył: 27.12.2004 |
Cytat Nie wiem czy znasz taki termin : syntactic sugar biggrin.gif. Myślę że na tym możemy zakończyć. Co ma slodzenie do bezsensownej duplikacji funkcji? Stworzyles dwie funkcje, obie robia dokladnie to samo i teraz user ma zagwostke, ktora funkcje ma uzyc i sie zastnawaia czy moze raz ma uzyc tej a raz innej. Na dodatek nazwy funkcji sa bzdurne bo jedna to isExist a druga isDirExist co sugeruje, ze druga sprawdza katalog a pierwsza sprawdza cos innego podczas gdy tez sprawdza katalog. No i te nieszczesne isExists.... Po polsku tez mowisz "jest istnieje"? Bo uwierz, po angielsku tez tak nie mowia. Wiec skoro juz poruszasz kwestie slodzenia to chociac zastanow sie czy wszyscy tez uwazaja ze sol jest slodka Cytat Właśnie zainstalowałem PHP Inspection i dostałem taką odpowiedź - "'empty(...)' counts too many values as empty, consider refactoring with type sensitive checks". Mi ani razu nie podkresla empty a uzywam tego pluginu dluzej od ciebie.PS:PHP Inspection właśnie polizał moje eg I jesli uwazasz, ze dzieki temu podkresleniu ten kod if (empty($a) || $a == 0) jest teraz ok to nadal jestes w duzym bledzie Cytat Z tego co pamiętam to chciałem rozwiązać jakiś przypadek biznesowy, w którym metoda zwraca 0 w przypadku błędu lub braku jakiejś wartości. No wlasnie, to jest kolejny problem w twoim kodzie. Juz nie raz napisales, ze cos zrobiles bo ma to sluzyc tobie. Czyli z jednej strony piszesz kod, ktory ma sluzyc tobie, a z drugiej sie pytasz nas, czy sie nadaje dla innych. Nie, z faktu, ze wiekszosc piszesz pod siebie to sie nie nadaje dla innych.
-------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
Wersja Lo-Fi | Aktualny czas: 26.04.2024 - 16:35 |