Cześć,
jest to moja pierwsza strona, którą stworzyłem. Jest to prosty blog, który łączy się z bazą danych i przekazuje zawartość (treść i tytuł artykułów).
http://www.testww1994.cba.pl/
https://github.com/WojciechWiderski/blog-testowy.github.io
Niestety z racji reklam na cba strona na mobilnych się rozjeżdża.
Nie jest to jeszcze koniec, ale proszę o opinie czy idę w dobrym kierunku. Jeżeli nie to proszę o wskazanie błędów, by ich nie pogłębiać.
Ocena kodu
Wątek przeniesiony 2017-07-12 12:46 z Webmastering przez ŁF.
- Rejestracja: dni
- Ostatnio: dni
- Postów: 60
- Rejestracja: dni
- Ostatnio: dni
Taki protip na przyszłość: Wystaw sobie to na heroku, nikt nie będzie Ci się wtryniał z reklamami.
- Rejestracja: dni
- Ostatnio: dni
- Lokalizacja: Piwnica
- Postów: 7697
nie wiem czy nie lepiej jednak użyć bootstrapa czy material design, bo trochę surowo jest, ponadto przy php wzorzec mvc, bo nie ma drzewa plików obecnie,czyli wrzucone do jednego katalogu, co może mylić, szczególnie jak ich z czasem przybędzie
- Rejestracja: dni
- Ostatnio: dni
- Lokalizacja: Warszawa
- Postów: 2255
Przy przechodzeniu z portfolio na about.php masz złe linki #about.php. A to oznacza że te linki wstawiasz ręcznie. Wstawiasz tez spacje przy stronicowaniu http://www.testww1994.cba.pl/index.php%20?pn=3
Masz podatność na sql injection, no i z tego co widzę w kodzie - nie masz ani jeden funkcji
- Rejestracja: dni
- Ostatnio: dni
- Lokalizacja: Norwegia, Stavanger
Sorry, ale wg mnie kod koszmarny.
Żadnych funkcji, pomieszanie z poplątaniem, powtarzanie kodu w każdym pliku i co to za ?> na początku kilku plików?