Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: Ocena gry na przeglądarkę. SkuteRace v 1.0
Forum PHP.pl > Inne > Oceny
kubek15
Witam.
Chciał bym abyście ocenili moją gre na przeglądarkę która obecnie znajduje się na stronie http://www.skuterrace.boo.pl i jest w fazie testów. Jest to dopiero pierwsza wersja v1.0 bo od niedawna ją piszę i chciałbym znac wasze zdanie. Grafika zrobiona w kilka minut tak aby to jakoś wyglądało smile.gif .

Konto możecie sobie utworzyć bo obecnie wyłączyłem akceptacje na mejla lub zalogowac się tymi passami:
login i hasło: test

Proszę o szczerą ocenę, ja sam wiem że to dopiero początki bo w php tak długo nie siedze, ale mam zamiar dodac jeszcze jave.

Będę wdzieczny za wszelkie opinie i sugestie z waszej strony, ponieważ będę wiedział co mam poprawić.
Pozdrawiam
Lion_87
heh zrób tak jak byś oddawał beta wersje czu cus.....z gotową grafiką i kodem.
Tak na szybko to....sprzedałem kilka razy ten sam skuter i nabiłem kase wstydnis.gif z 600 na 9800
Mefiuu
Nie oceniałem pod względem bezpieczeństwa ale co mi się nasuwa na myśl :
1. Główna strona wewnątrz : tabelki brzydko porozstawiane.
2. Trening -> fajnie by wyglądał efekt odliczania do końca wyścigu i wtedy dowiedzieć się o ewentualnej wygranej/przegranej.
3. Bank = nic tam nie ma, nie wiem czy to niedopracowanie czy jeszcze pracujesz nad tym.
4. Sklep - po kupnie danej rzeczy informacja mogłaby się pojawić na górze, a nie żeby trzeba było klikać wstecz lub znów "sklep"
5. Kasyno ubogie, ale zawsze urozmaicenie winksmiley.jpg
6. Grafika słaba ale zaznaczyłeś że na poczekaniu zrobiłeś.
7. Popraw uprawnienia testowego konta, żeby nie można było zmieniać hasła, bo można łatwo zmienić winksmiley.jpg


W adresie wpisać http://www.skuterrace.boo.pl/index.php?wys...amp;sell=100000 i już masz więcej kasy winksmiley.jpg zmień GET na POST może lepiej winksmiley.jpg


Ulepszam świecę = ulepsza mi łańcuch i odwrotnie winksmiley.jpg
kubek15
dzięki za wypowiedzi będe to poprawiał
zzeus
A gdzie można kupić paliwo ?
kubek15
Paliwo jest na cron'a ale moge mieć tylko jedną opcje na cron dlatego za kilka dni zmieniam serwer i paliwo samo sie będzie dodawało.

Poprawiłem łańcuch i świece, oraz to sprzedawanie skutera. A odliczanie dodam za chwilkę.
phpion
Wysłałem wiadomość do samego siebie. Jako odbiorcę wpisałem "test " (spacja na końcu).
kubek15
Dzięki za odpowiedz, potem to poprawie.
Cypherq
Dlaczego oddajesz do oceniania wersję pełną idiotycznych, prostych do wykrycia minimalnym nakładem sił i z 'byle jaką grafiką' wersję? Czy nie możesz samemu sprawdzić najpierw i przetestować swój skrypt a potem dopiero dawac go do oceny? Nie mów, ze oddajesz do testów, bo takiego szajsu nie ma sensu testować.

Po za tym co to za numeracja? Wczesna beta oznaczona 1.0? Żeby fajnie wygladało, dodałeś to 1.0?

http://pl.wikipedia.org/wiki/Numeracja_wersji_oprogramowania

kubek15
To specjalnie dla ciebie teraz dodam wersje 284.8.1 Pasuje ci? Nie masz się co pluć co do mojej gry bo ty pewnie jesteś super-master-pro . Ja nie prosiłem o jakieś idiotyczne komentarze tylko o pisaniu błędów.

Powiem ci jedno żal mi cię.
#luq
Skupmy się na bezpieczeństwie które teraz właściwie na stronie leży.

Od razu po wejściu na strone i zalogowaniu się na test widzimy linki w stylu
Kod
http://www.skuterrace.boo.pl/index.php?wys=sklep

?wys=sklep

Pierwsze co przychodzi do głowy czy "sklep" to nazwa pliku? Naturalnie i oczywiście tak, pierwszy test, którego się aż prosisz to test na Path Traversal
Kod
http://www.skuterrace.boo.pl/index.php?wys=./salon

i się okazuję że działa! W ten sposób można wczytać dowolny plik z tego serwera. No może nie taki dowolny bo niestety Null Byte Poison mi się nie udał...
Kod
http://www.skuterrace.boo.pl/index.php?wys=./salon.inc%00

i nawet nie wiem czy includowane pliki mają dodawane rozszerzenie .inc, .php, .html czy jakieś inne.

Co można zrobić? Zasypać Ci serwer żądaniami aż się zawiesi.
Kod
http://www.skuterrace.boo.pl/index.php?wys=index


Druga sprawa to błąd typu XSS.
W wiadomościach nic nie jest filtrowane (podejrzewam, że rejestracja tak samo), a więc bez problemu wpisując:
Kod
<script type="text/javascript">
alert(1);
</script>

kod zostanie wykonany.

Dzięki temu jestem w stanie ukraść ciasteczka, a więc i sesje a więc i konto każdemu który odczyta moją wiadomość.

Masz brak jakiegokolwiek filtrowania i sprawdzania danych wprowadzonych przez usera. Bez problemu kupiłem skuter którego nie ma:
Kod
http://www.skuterrace.boo.pl/index.php?wys=salon&kup=123456


SQL Injection nie udało mi się znaleźć.

Pozdrawiam,
luq
kubek15
Dzięki za twoje uwagi na pewno zaraz zajmę sie usuwaniem tych błędów. A jeżeli chodzi o te filtrowania to wystarczy jak zablokuje wszystkie znaki oprócz 0-9 A-Z a-z ?
phpion
Zapraszam do działu SMS dla konta "test". Zniknęło? Magia smile.gif
#luq
Cytat(kubek15 @ 25.02.2010, 14:00:25 ) *
A jeżeli chodzi o te filtrowania to wystarczy jak zablokuje wszystkie znaki oprócz 0-9 A-Z a-z ?


A niby dlaczego masz ograniczać sobie funkcjonalność. Masz do tego gotową funkcje.

Cytat(phpion @ 25.02.2010, 14:05:02 ) *
Zapraszam do działu SMS dla konta "test". Zniknęło? Magia smile.gif

Kod
#cala { display:none;}

Pewnie stara się zabezpieczyć i uniemożliwić redirecta na mojego bloga ;D
Cypherq
Cytat(kubek15 @ 25.02.2010, 13:32:14 ) *
To specjalnie dla ciebie teraz dodam wersje 284.8.1 Pasuje ci? Nie masz się co pluć co do mojej gry bo ty pewnie jesteś super-master-pro . Ja nie prosiłem o jakieś idiotyczne komentarze tylko o pisaniu błędów.

Powiem ci jedno żal mi cię.


Chłopcze, w prostych - że aż boli - słowach, wytłumaczyłem Ci, że dawanie do oceny czegoś (śmiecia), który ma więcej BŁĘDÓW, DZIUR niż FUNKCJONALNOŚCI, myli się z ideą tego działu. Czy to jest dział "ocena" czy "powiedz gdzie mam błąd"? Skoro błędów jest multum to może wróciłbyś do literatury fachowej i napisał coś porządnie a nie dawał do oceny skrypt beznadziejny?

Wystawiłem Ci również jak na tacy tekst o numeracji. Widocznie nie chcesz się uczyć, bo co to za tekst z podstawówki? Numer 284.1? Bo ośmieliłem się zwrócić dzieciakowi uwagę? Proszę Cię, przynajmniej staraj się nie zachowywać jak dziecko.

Jeszcze to "żal mi Cię". Naprawdę tylko tyle wyniosłeś z podstawówki? Może trochę więcej kultury i okrzesania?
Michaxxx
Witam to mój pierwszy post na forum. Zobacz jeśli wchodzisz na link http://www.skuterrace.boo.pl/index.php?wys=podglad to nawet jak nie jesteś zalogowany to Ci wyświetli
Cytat
Witaj !
Twoja przygoda z motorami własnie sie rozpoczeła. Kupuj najlepsze częśći i motory. Wybierz najlepszą strategie dla ciebie i bądz najlepszy.

Życzę powodzenia z twoją grą biggrin.gif

-Jeśli ulepszasz Turbo sprężarkę przez odświeżanie to sprzedaje skuter
emp
Cytat(Cypherq @ 25.02.2010, 11:53:31 ) *
Dlaczego oddajesz do oceniania wersję pełną idiotycznych, prostych do wykrycia minimalnym nakładem sił i z 'byle jaką grafiką' wersję? Czy nie możesz samemu sprawdzić najpierw i przetestować swój skrypt a potem dopiero dawac go do oceny? Nie mów, ze oddajesz do testów, bo takiego szajsu nie ma sensu testować.

Po za tym co to za numeracja? Wczesna beta oznaczona 1.0? Żeby fajnie wygladało, dodałeś to 1.0?

Chłopcze, w prostych - że aż boli - słowach, wytłumaczyłem Ci, że dawanie do oceny czegoś (śmiecia), który ma więcej BŁĘDÓW, DZIUR niż FUNKCJONALNOŚCI, myli się z ideą tego działu. Czy to jest dział "ocena" czy "powiedz gdzie mam błąd"? Skoro błędów jest multum to może wróciłbyś do literatury fachowej i napisał coś porządnie a nie dawał do oceny skrypt beznadziejny?

Wystawiłem Ci również jak na tacy tekst o numeracji. Widocznie nie chcesz się uczyć, bo co to za tekst z podstawówki? Numer 284.1? Bo ośmieliłem się zwrócić dzieciakowi uwagę? Proszę Cię, przynajmniej staraj się nie zachowywać jak dziecko.

Jeszcze to "żal mi Cię". Naprawdę tylko tyle wyniosłeś z podstawówki? Może trochę więcej kultury i okrzesania?


Cypherg obrażasz chłopaka, poniżasz, wyzywasz, prowokujesz i twierdzisz że jego praca to śmieć !. To wszystko co piszesz świadczy o tobie nie o nim. Mogłeś to napisać w inny sposób jak ci się gra nie podoba i uważasz że jest kiepska.

Gra jest bardzo przeciętna, amatorska, pełna błędów, ale pisze ją początkujący amator, a to jest istotna informacja którą się bierze pod uwagę podczas oceny czyjejś pracy. To nie jest zaawansowany programista dający do oceny jakiś banalny, prosty skrypt w celach autopromocji.

Kubek15 pracuj dalej nad swoją grą i nad swoim warsztatem z czasem będziesz coraz lepszy i twoja gra.

@down zapomnij że będę z tobą dyskutował i to skomentuje.
Cypherq
@up: byłoby Ci ciężko polemizować, ja zacytowałem i pogrubiłem regulamin. Będę pamiętał, że zamiast wysłać priva wolisz szukać wymówek.

Co to za gadanie z obrażaniem? Nie jestem kulturalny bo? Nazwałem jego pracę śmieciem? Tak, bo jest to śmieć. Dział przedszkole jest gdzie indziej. Tam można poprawiać przedszkolne skrypty, sam odpowiedziałeś do czego służy dział Oceny.

Chłopak wyskakuje z tekstem "żal mi Cię" a Ty zwracasz mi uwagę, że jestem niekulturalny? No halo?

Specjalnie dla Ciebie:
http://www.miejski.pl/slowo-Szajs - zwróć uwagę na słowo-klucz: "badziewie".

Po co podkreśliłeś słowa takie jak "idiotycznych"? Czy operowanie dostępem do modułów skryptu za pomocą zmiennych GET nie jest idiotyczne? Jest, ale wolisz owijać to w bawełnę i bawić się w wysublimowane tekściki.

"Tyle wyniosłeś z podstawówki" - to również pogrubione. Bo odnosiło się do tekstu z podstawówki, który nijak odnosił się do postu który wcześniej zaprezentowałem, na to też nie zwróciłeś uwagi?

Nie odkryłeś ameryki twierdząc, że "grę" napisał amator. Tak, amator napisał coś amatorskiego. Co zrobić by stać się kimś więcej niż amatorem? Czy do tego służy dział oceny? Czy nie można siąść i przeczytać jeden, drugi, trzeci, 50 artykuł, napisać kilkanaście skryptów dla SIEBIE i stać się lepszym?

Forum jest zalewane setkami gier nie wychodzących po za amatorszczyznę:
- pomysłem
- grafiką
- wykonaniem

Dla Ciebie również pobawię się w podkreślanie słów jednego z moderatorów:

Cytat
2. Należy podać przeznaczenie skryptu.
3. Skrypty nie mogą zawierać błędów.
4. Każdy skrypt winien posiadać komentarze w kodzie.


Ten nie zawiera, prawda? To tylko mój wymysł, że szwajcarskiego sera 0.1 nie daje sie do oceny, prawda?
Jasne, moze dostanę procenty za - może nazbyt agresywne wypowiedzenie moich racji, ale racja jest po mojej stronie, chyba, że moderatorzy nagle zmienią zasady panujące na tym forum. A w dziale oceny co raz trudniej znaleźć coś naprawdę DOBREGO do OCENY a nie wyciagania setek aż nazbyt widocznych błędów użytkownikowi, który niedawno zaczął przygodę z PHP i bierze się za tworzenie gry.
kubek15
@up ty kolego chyba złapałeś jakiegoś "filma" na tą grę. Nie musisz tak się pluć jeżeli ci się to nie podoba smile.gif
A to że są błędy to ja wiem bo jest to moja pierwsza gra.

Ale ok dzięki za wszystkie komentarze za jak i przeciw. Nie zniechęciłem się do pisania kolejnej wersji. Niedługo uwzględnię wasze uwagi i przeniosę gre na nowy serwer bo na boo.pl juz są reklamy to dam w tym temacie linka.

Pozdro
SHiP
Dziurawe jak ser szwajcarski. Kod źródłowy w 15 sekund wyciągnąłem ;]
http://wklej.to/CUkK (hasło: michalsrodek)

Polecam dobry kurs PHP 5

PS: Jest SQL Injection! winksmiley.jpg Już w 11 czy 12 linijce. Trzeba tylko dane POSTem wysyłać

Pozdrawiam
#luq
Cytat(SHiP @ 26.02.2010, 18:34:09 ) *
Dziurawe jak ser szwajcarski. Kod źródłowy w 15 sekund wyciągnąłem ;]
PS: Jest SQL Injection! winksmiley.jpg Już w 11 czy 12 linijce. Trzeba tylko dane POSTem wysyłać


Good job! winksmiley.jpg
Można się dowiedzieć w jaki sposób? Z czystej ciekawości pytam ;]

Niestety nie sprawdzałem wszystkich wektorów ataku bo nie miałem czasu sad.gif
SHiP
Wyślę Ci PW. Właśnie piszę książke na ten temat i nie chcę zdradzać wszystkich technik bo nikt jej nie kupi biggrin.gif.
darophp
Trochę błędów i dopracowałbym graficznie tongue.gif
kubek15
już wiem co mam poprawić zrobie filtrowanie dodam troche js i za jakies czas pokaże wam nową wersje. A jeżeli chodzi o grafe to ja reczej nie będe robił jak juz skończe gre to cos kupie.

Dzięki i pozdro.
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.