Post
#1
|
|
|
Grupa: Zarejestrowani Postów: 114 Pomógł: 0 Dołączył: 3.08.2017 Ostrzeżenie: (0%)
|
Chciałbym prosić o ocenę kodu programowego (PHP + MySQL) mojej bardzo prostej aplikacji internetowej (sieć reklamowa). Całość napisana w dwóch wersjach (Symfony i własny szkielet aplikacji w PHP).
Aplikacja w Symfony + biblioteka do obsługi Api: https://github.com/webeeq/symfony.eeq https://github.com/webeeq/sieciq Własny szkielet tej samej aplikacji w PHP: https://github.com/webeeq/framework.eeq Obydwa projekty poddawałem już ocenie jakieś dwa lata temu, ale nie widzę nigdzie tamtego wątku, więc tworzę nowy. Pewnie został skasowany. Obecne wersje zawierają liczne poprawki. Prosiłbym zwrócić szczególną uwagę na zagadnienia: 1. Tworzenie klas encji tabel bazy danych w Symfony. Czy dobrze to robię? Chodzi mi głównie o brak deklaracji opcji "default". 2. Jakie wzorce projektowe poza Dependency Injection powinienem użyć w moich projektach. Jakie wzorce warto jeszcze znać? 3. Czy zapytania SQL (w cudzysłowie '...') mogą być w wielu liniach. Nie powoduje to błędu, ale czy to ładnie tak programować? 4. Czy z takim programowaniem, jak moje, jest sens w ogóle starć się o pracę? Bądź lepiej będzie, jeśli dam sobie z tym spokój? |
|
|
|
![]() |
Post
#2
|
|
|
Grupa: Zarejestrowani Postów: 6 381 Pomógł: 1116 Dołączył: 30.08.2006 Ostrzeżenie: (0%)
|
Mnie się jeszcze w oczy rzuciła sama struktura. Wszystko wrzucone do folderu / wliczając cssy i src. Ani to bezpieczne, ani uporządkowane. W src jest Config/cośtam. Dlaczego to zapisałeś w ten sposób zamiast stworzyć oddzielny folder poza strukturą src? Osobiście nie podoba mi się definiowane w każdej akcji layout itd (które oczywiście odnosi się do src).
PS. $code = ($_GET['code']) ? (int) $_GET['code'] : 404; Ja też tak piszę. Ma to zapobiec przypadkowemu złemu warunkowaniu w operatorze ternary. Ten post edytował viking 7.04.2020, 11:03:50 |
|
|
|
eerie [www] Prosta sieć reklamowa - Symfony i Framework w PHP 7.04.2020, 10:44:14
nospor $code = ($_GET['code']) ? (int) ... 7.04.2020, 11:01:06
nospor Klasa Param to cala prawie do wyrzucenia. Jaki jes... 7.04.2020, 11:13:32 
eerie Cytat(nospor @ 7.04.2020, 12:13:32 ) ... 7.04.2020, 15:22:17
viking Nawet z dokumentacji echo (true?'true':fal... 7.04.2020, 11:45:59
nospor Mimo wszystko podany przez ciebie przyklad z dokum... 7.04.2020, 11:48:52
viking CR są per firma. U mnie by nie przeszło bez nawias... 7.04.2020, 11:50:04
nospor Cytatcs fixer dodający spacje.Przynajmniej w jedny... 7.04.2020, 11:52:05
nospor Przestan tak bezgranicznie patrzec na dokumentacje... 7.04.2020, 15:50:08
eerie CytatMnie się jeszcze w oczy rzuciła sama struktur... 7.04.2020, 16:01:19
nospor CytatZabezpiecza przed atakami. Uniemożliwia wklej... 7.04.2020, 16:25:50
viking Jedna strona jeden kontroler. W różnych fw różnie ... 7.04.2020, 16:50:14
eerie CytatTak. I model nie powinieniec dziedziczyc po d... 8.04.2020, 10:21:48
viking Na przykładzie tego https://github.com/webeeq/fram... 8.04.2020, 11:02:09
nospor CytatNasuwa mi się pytanie... Gdzie i jak powinien... 8.04.2020, 11:09:40
eerie Dzięki serdeczne za trafne uwagi. Wyciągnę z nich ... 8.04.2020, 13:24:10 ![]() ![]() |
|
Aktualny czas: 14.01.2026 - 21:00 |