[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

[MWL]

strassssznie zmula

przynajmniej u mnie...

Ten post edytował MWL 7.03.2009, 19:43:26

[blooregard]

Wybranych technologii uczę się od ok. 3-4 miesięcy i jest to moja pierwsza tego typu aplikacja

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

[lwc]

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:

Kod

http://roma2009.ovh.org/index.php?co=oferta&strona=-1

Rezultat:

Kod

Fatal error:  Call to a member function fetch_assoc() on a non-object in /home/r/o/m/roma2009/www/funkcje.php on line 1047

Ten post edytował lwc 7.03.2009, 20:54:35

[blooregard]

Też kombinowałem (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
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 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[Kildyt]

Jak na 4 miesiące pracy to bardzo dobrze. (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Design jest twój? Średni (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) 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.

[erix]

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

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

Ocena: 6.5/10

[.radex]

gdybym ja robił takie layouty po czterech miesiącach (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

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)

[pyro]

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.

[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