[www] Bardzo prosty skrypt turystyczny Opcje

[alejandro]

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 (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) 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

[alejandro]

Witam wszystkich i dziękuje za wszystkie komentarze (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) 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 (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) ale dostalibyście zawału na widok kodu PHP (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) 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 (IMG:http://forum.php.pl/style_emoticons/default/dry.gif) 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.

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

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

Panel niefunkcjonalny, ale z czasem sam dojdziesz do pewnych rzeczy.

Nawet mnie denerwuje, że trzeba tak klikać... (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)

Popracuj nad bezpieczeństwem. Formularz (patrzyłem na rejestracji) nie jest filtrowany.

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

- divitis
- menu nie na listach

Nie wiedziałem, że w ogóle coś takiego istnieje (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) ale już przeczytałem o co chodzi i nadal nie rozumiem dlaczego za często używam divów (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) 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

Duży minus: brak stopki
Generalnie nie powinno się dawać na stronie głównej linku do logowania pracowników i administratorów, po prostu powinni wiedzieć żeby się tam dostać przez /admin.php

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ł (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Dziękuje raz jeszcze i zapraszam do dalszego komentowania jeśli jeszcze komuś jakaś uwaga się nasunie. Pozdrawiam