 [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: 479 Pomógł: 97 Dołączył: 6.09.2011 Skąd: php.net :) Ostrzeżenie: (0%)
|
Na pierwszy rzut:
- połaczenie z DB przekazuje jako parametr w __construct itp - nie wyświetlaj tylko zwracaj dane - skoro $id = 0; to po co jeszcze zapytanie do bazy ? Zacznij od jakiejśc dobrej książki albo tutoriali, dopiero pisz kod, zaczniesz od dobrych nawyków. |
|
|
|
|
Post
#3
|
|
|
Grupa: Zarejestrowani Postów: 14 Pomógł: 0 Dołączył: 19.11.2014 Ostrzeżenie: (0%)
|
Cytat(IProSoft @ 1.05.2015, 13:10:17 )  - nie wyświetlaj tylko zwracaj dane - skoro $id = 0; to po co jeszcze zapytanie do bazy ? Czyli zamiast echo - return? Jeśli chodzi o to $id = 0 w lini 68 to jest sprawdzanie czy w bazie jest rekord. Jeśli zapytanie zwróci id i RowCount go policzy - oznacza że jest jakiś news z tym id. Jesli nie ma wyświet stronę główną |
|
|
|
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 ohm Jeśli już zaczynasz coś obiektowo pisać, to pisz t... 1.05.2015, 12:51:00 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: 6.10.2025 - 15:39 |
