Witam, chciałbym przedstawić skrypt turystyczny, który wykonałem w PHP i MySQL. Wybranych technologii uczę się od ok. 3-4 miesięcy i jest to moja pierwsza tego typu aplikacja. Oczywiście zdaję sobie sprawę, że w porównaniu z większymi systemami nie ma co się równać. Skrypt był pisany w pośpiechu i jeśli chodzi o całość(funkcjonalność, działanie, kod php itp.), to jestem średnio zadowolony z mojej pracy. Aplikację będę chciał ulepszać ale póki co w ramach dalszej nauki planuje napisać skrypt do sprzedaży ubrań online... uważam, że jest to dziedzina troszkę mniej skomplikowana od turystyki. Zapraszam do testowania pod adresem: http://roma2009.ovh.org/ - oczywiście jeśli macie cierpliwość... troszkę to wolno chodzi... ale już sam nie wiem czy to moja wina czy serwera Chciałbym jeszcze dodać, że skrypt był pisany w ramach nauki a nie z myślą, że mógłbym go sprzedać.

Do administracji możemy się dostać klikając link "Administracja" w topie.. login i hasło: admin

Pracownicy mają swój dział pod linkiem "Pracownicy" w topie.. login i hasło: pracownik

Oraz klienci... Link "Logowanie" w menu.. login i hasło: klient

P.s. Do skryptu dodałem jeszcze listę uczestników dla wybranej wycieczki oraz do faktur cenę netto bo nie było... niestety jeszcze nie umieściłem aktualizacji na serwerze... Postaram się to zrobić w najbliższym czasie.

P.s.s. Zdaję sobie sprawę, że nie wszystko jest zrobione tak jak powinno być. Dlatego proszę Was o ocenę.

Podrawiam

strassssznie zmula

przynajmniej u mnie...

No to całkiem przyzwoita, jak na 4 miesiące nauki.

Mnie się podoba. Panel niefunkcjonalny, ale z czasem sam dojdziesz do pewnych rzeczy.

Co do mulenia: stawiam na źle skonstruowane zapytania (aplikacja próbuje pobrać z bazy dane z kilku tabel, używając złączenia wewnetrznego i dla któregoś rekodu gdzieś brakuje danych).

Rzeczywiście jak na tak krótki czas nauki strona bardzo ładna. Podoba mi się. Jedyne do czego mógłbym się przyczepić to ten błąd:

URL:

Rezultat:

Też kombinowałem
Ale na SQLi odporne. Brawo za filtrację parametrów z GET-a. Ja po 4 miesiącach nauki PHP nie miałem o tym pojęcia.
Ale to było w czasach, kiedy o SQL Injection słyszała garstka, a register_globals były na porządku dziennym

Jak na 4 miesiące pracy to bardzo dobrze.
Design jest twój? Średni Jeżeli chodzi o kod html to mógłby być lepszy.
Admin nieintuicyjny, jak już wspomniał poprzednik. Popracuj nad bezpieczeństwem. Formularz (patrzyłem na rejestracji) nie jest filtrowany.

1. obrazek w topie jest za ciężki
2. panel administracyjny wymaga nieustannego kopania (menu rozwijane by się przydało, czy coś takiego)
3. teksty w topiku sklejone z grafiką
4. guzik administracja potrzebny jest tylko Tobie
5. niesemantyczny kod (brak oddzielenia np. stylów od HTML)
6. z tytułu strony nie mogę wywnioskować, gdzie jestem
7. zdałyby się przyjazne URL-e

Ale jak po 4 miesiącach nauki, bardzo dobrze.

Ocena: 6.5/10

gdybym ja robił takie layouty po czterech miesiącach

a od kodu:
- menu nie na listach
- divitis
- zbędne tabelki
- css inline
- cssy i js w tym samym pliku, zamiast w osobnych (warto to zrobić, bo zwiększa szybkość strony - css i js nie powinien być wtedy drugi raz pobierany z serwera, tylko z cache'u)

Duży minus: brak stopki
Generalnie nie powinno się dawać na stronie glownej linku do logowania pracownikow i administratorow, poprostu powinni wiedziec zeby sie tam dostac przez /admin.php

Wygląd strony: Jak na tak krotki czas nauki świetny.

Witam wszystkich i dziękuje za wszystkie komentarze Szczerze to myślałem, że będzie gorzej ale i tak trzymam się swojego i nadal uważam, że nie jest to nic specjalnego. Skrypt był pisany w pośpiechu, ponieważ musiałem go oddać na zaliczenie w szkole. Jeśli chodzi o PHP i MySQL to rzeczywiście jest to 4 miesiące nauki przy dobrej książce a do tego dochodzi przecież nauka w szkole, programowanie w Pascalu, C, C++, ASM, Javie i bazy danych, tak wiec pewne rzeczy przychodzą łatwiej ale dostalibyście zawału na widok kodu PHP Pic na wodę można powiedzieć i dlatego po przeczytaniu wszystkich uwag będę ten skrypt pisał od początku jeśli chodzi o kodowanie. Niektóre rzeczy lepiej napisać od początku niż bawić się w modyfikację. Myślę, że baza danych i grafika jest w porządku i nie trzeba tego zmieniać. Ewentualnie podzielić top na 2 czy 3 części bo faktycznie bardzo powoli się wczytuje Po prostu niektóre elementy zostały bardzo uproszczone, żeby zdążyć w terminie. Skrypt mógłby być lepszy, ale pośpiech to nic dobrego w programowaniu.


Przeglądając swój kod i czytając forum dowiedziałem się że lepiej by było użyć LEFT JOIN... ja to zrobiłem w inny sposób i to na pewno jest mój błąd. Wstyd, bo powinienem takie rzeczy wiedzieć...


Nawet mnie denerwuje, że trzeba tak klikać...


Nie wiem dokładnie na czym polega jeszcze filtracja danych w formularzach... no ale pewnie jest już taki wątek na forum to poszukam.


Nie wiedziałem, że w ogóle coś takiego istnieje ale już przeczytałem o co chodzi i nadal nie rozumiem dlaczego za często używam divów odnośnie menu to wiedziałem, że się robi na listach... ale tak było po prostu szybciej dla mnie niż ustawianie marginów paddingów w css


Stopka w kolejnej wersji już będzie. Natomiast jeśli chodzi o panel administracyjny to wiem, że się tak nie robi... Zrobiłem po prostu menu na skróty żeby nie trzeba było szukać gdzie jest np. admin

Jeśli chodzi o style CSS inline, nawigacja itd. to ze wszystkimi uwagami się zgodzę. W kolejnej wersji postaram się poprawić. Bez Waszych opinii na pewno do wszystkich błędów bym sam nie dotarł Dziękuje raz jeszcze i zapraszam do dalszego komentowania jeśli jeszcze komuś jakaś uwaga się nasunie. Pozdrawiam