Браузър игра

[bobonik]

Здравейте, от около година работя по играта (втората ми игра ). Всичко по играта (код и дизайн) е правено от мен, като изключим 1-2 нещица - дребни js гъзарийки, които ме домързя да напиша сам. Бих искал да дадете своето мнение за играта, няма значение дали ще е похвала или критика. Знам, че дизайнът не е от най-добрите, но все още съм на 19 и нямам кой излишни пари, които бих могъл да дам, за да може някой да ми направи качествен дизайн, а и не смятам да ставам уеб дизайнер.
Emagicus е нова браузър игра, която ви предлага битки в реално време, както с чудовища, така и с други потребители. Купувайте магии, тренирайте ги и ставайте по-силни. Търсете предмети от чудовища, или си направете сами с някои от професиите. Използвайте камъни и подобрявайте вашите предмети.
Героите се разделят на 3 раси - воин, стрелец и магьосник.
Играта е все още в бета и постоянно ще бъдат добавяни нови неща.
Линк към играта - emagicus

[jazzman]

За 19г пишеш много добре, аз станах на 42 и все още правя правописни грешки

За играта като игра няма да говоря, но има някои забележки / препоръки, които бих искал да споделя.

1. Регистрацията - паролата може да бъде много къса. Мисля, че няма ограничение в дължината

2. Бях способен да brute-force-на акаунта си, много лесно

3, Имаш XSS, cross-site scripting vulnerabilities.

4. Лесно се крадът сесии

5. Сайта зарежда много бавно.

6. Ползваш прекалено много server-site( php ) код, товариш излишно базата и сървъра. Ползвай повече JS, виж какво е и AJAX. 

7. Вероятно имаш и уязвимости към базата данни - SQL injections.

Ами общо взето тва е, дето успях да видя за около 10 мин. Ако искаш ефективна помощ, ти трябва да споделиш кода с нас и съм убеден, че някой от по активните потребители или администратори ще ти дадат препоръки и насоки.

[bobonik]

Благодаря за отговора, ще оправя каквото мога. Мисля, че единствено покривам 7-та точка, тъй като навсякъде използвам prepared statements, но за пръв път използвам PDO и може да не съм го направил както трябва.

[theSit]

И аз си мислех, че pdo по подразбиране ползва prepared statements, но след теста се оказа друго - http://nau4i.me/forum/index.php/topic,15924.0.html

[canon4o]

На началната страница, долу на последните 2 снимки си забравил да затвори a href тага. Иначе играта е яка Трябва ѝ само развитие