Wątek przeniesiony 2017-07-12 12:46 z Webmastering przez ŁF.

Ocena kodu

0

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

0

Taki protip na przyszłość: Wystaw sobie to na heroku, nikt nie będzie Ci się wtryniał z reklamami.

0

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

0

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

0

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?

0

Dzięki za opinie. Czas sie wziąć do jeszcze intensywniejszej nauki!

1 użytkowników online, w tym zalogowanych: 0, gości: 1