Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [inny]Własny framework z auto wstrzykiwaniem
primosz67
post 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



Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
nospor
post 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 tongue.gif

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".

PS:PHP Inspection właśnie polizał moje eg
Mi ani razu nie podkresla empty a uzywam tego pluginu dluzej od ciebie.
I jesli uwazasz, ze dzieki temu podkresleniu ten kod
if (empty($a) || $a == 0)
jest teraz ok to nadal jestes w duzym bledzie wink.gif

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

Go to the top of the page
+Quote Post

Posty w temacie
- primosz67   [inny]Własny framework z auto wstrzykiwaniem   4.08.2017, 07:28:51
- - nospor   Skoro piszesz teraz cos od podstaw, to wypadaloby ...   4.08.2017, 10:21:34
|- - primosz67   Cytat(nospor @ 4.08.2017, 11:21:34 ) ...   4.08.2017, 16:00:39
|- - emp   Cytat(nospor @ 4.08.2017, 09:21:34 ) ...   25.08.2017, 08:32:32
- - nospor   CytatO dzięki. korzystam z PHP storm,ale on nie wy...   4.08.2017, 16:07:42
|- - primosz67   Cytat(nospor @ 4.08.2017, 17:07:42 ) ...   4.08.2017, 17:46:30
- - viking   Czemu te klasy nie mają normalnych nazw Spark...   4.08.2017, 17:56:37
|- - primosz67   Cytat(viking @ 4.08.2017, 18:56:37 ) ...   4.08.2017, 18:55:30
- - nospor   CytatNie wiem czy znasz taki termin : syntactic su...   4.08.2017, 20:28:12
|- - primosz67   Cytat(nospor @ 4.08.2017, 21:28:12 ) ...   4.08.2017, 21:20:37
- - nospor   CytatNie. Podkreślam tylko, że w ogóle używanie me...   4.08.2017, 22:36:58
- - com   Sonar jest tak samo do PHP jak do Javy, tylko dorz...   20.08.2017, 14:44:04
|- - primosz67   Cytat(com @ 20.08.2017, 15:44:04 ) So...   23.08.2017, 16:09:46
- - nospor   Cytatcomposer.lock wrzuć do gitignora, bo one w re...   20.08.2017, 15:02:16
- - com   Wersje zależności określone są i tak w właściwym p...   20.08.2017, 16:14:25
- - nospor   Cytati raczej jeśli nie piszemy w Laravelu, to nie...   20.08.2017, 16:16:23
- - com   nospor Ok, w tym wypadku się z Tobą w 100% zgadza...   20.08.2017, 16:42:21
- - nospor   CytatOsobiście locka bym widział tu: https://githu...   20.08.2017, 18:57:45
- - Pyton_000   Tu jest dobrze wyjaśnione: https://stackoverflow.c...   21.08.2017, 06:48:58
- - nospor   Haha, dzieki emp za zwrocenie uwagi. Kopiowalem na...   25.08.2017, 09:24:45
- - emp   Cytat(nospor @ 25.08.2017, 08:24:45 )...   29.08.2017, 11:26:41


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: 26.04.2024 - 16:35