 [Skrypt PHP] Początek prostego CMSa, Czyli "programowanie" dla wprawy |
| [Skrypt PHP] Początek prostego CMSa, Czyli "programowanie" dla wprawy |
Post
#1
|
|
|
Grupa: Zarejestrowani Postów: 14 Pomógł: 0 Dołączył: 19.11.2014 Ostrzeżenie: (0%) 
 |
Postanowiłem stworzyć w ramach ćwiczenia i nauki prostego CMSa. Na razie większość funkcji jest w budowie jednak mam już stworzonych parę funkcji (function.php). Chciałbym abyście ocenili jaka jest "amatorskość" tego kodu i jakie rozwiązania zmienić pisząc kolejne funkcje. Co poprawić, co zmienić żeby nie była to typowa "amatorszczyzna początkującego".
Załączam skrypt. Proponuje skupić się na pliku function.php bo jest to "serce" tego skryptu. https://sourceforge.net/projects/ncmsrekami.../?source=navbar Adres przykładowej strony: http://rekamil.zz.mu/ Ten post edytował rekamil97 1.05.2015, 22:55:59 |
 |
 
|
 |
|
Post
#2
|
|
|
Grupa: Zarejestrowani Postów: 623 Pomógł: 144 Dołączył: 22.12.2010 Ostrzeżenie: (0%)
|
Jeśli już zaczynasz coś obiektowo pisać, to pisz to chociaż wg jakichś standardów: http://www.php-fig.org/psr/psr-1/ http://www.php-fig.org/psr/psr-2/ (szczególnie 4.1 i 4.2)
global - tego już się nie używa Zamiast $id = 0 wyrzuć wyjątkiem czy 404 po prostu. Zainteresuj się jakimś systemem szablonów (twig np) bo będziesz miał sieczkę a nie CMS (IMG:style_emoticons/default/wink.gif) A i jeszcze jedna sprawa, albo utrzymujesz kod po angielsku, albo po polsku (chociaż wg mnie dużo lepiej po angielsku (IMG:style_emoticons/default/tongue.gif) ) Ten post edytował ohm 1.05.2015, 12:52:03 |
|
|
|
rekamil97 [Skrypt PHP] Początek prostego CMSa 1.05.2015, 00:46:18 
PrinceOfPersia CytatZałączam link do pobrania tych kliku plików w... 1.05.2015, 09:50:00 rekamil97 Dodałem na "sourceforge". Plik function.... 1.05.2015, 11:22:50 IProSoft Na pierwszy rzut:
- połaczenie z DB przekazuje jak... 1.05.2015, 12:10:17 
rekamil97 Cytat(IProSoft @ 1.05.2015, 13:10:17 ... 1.05.2015, 13:45:23 IProSoft W 65 przypisujesz pod $id 0 więc oczywiste, j... 1.05.2015, 13:52:31 rekamil97 Przypisuje po to pod $id 0 aby potem RowCount... 1.05.2015, 13:59:01 IProSoft Inaczej: skoro wiesz, że Id będzie równe zero, to ... 1.05.2015, 14:04:22 rekamil97 Cytat(IProSoft @ 1.05.2015, 15:04:22 ... 5.05.2015, 23:56:58 rekamil97 A jeszcze pytanie o return - gdzie zamiast echo. T... 1.05.2015, 21:46:37 IProSoft Nie używaj nawiasów wraz z echo, są zbedne.
Retur... 2.05.2015, 07:47:52 salfunglandyare ? wystarczy Ci jeden warunek w sql: id = $id,... 6.05.2015, 01:33:58 rekamil97 Pomyśle nad takim rozwiązaniem tej funkcji. Jeszcz... 6.05.2015, 15:24:40 IProSoft Nie zamieszczaj starego kodu, tylko popraw to o cz... 6.05.2015, 16:26:33 Pyton_000 [PHP] pobierz, plaintext $id = (int)$_GET... 7.05.2015, 08:30:14 Xelah Skoro się dopiero uczysz, to ja polecam zapoznać s... 18.05.2015, 08:11:58  |
|
Aktualny czas: 13.10.2025 - 18:43 |
