Прошу мастеров посмотреть

← →
Аццкий_рыцарь (2006-09-20 02:33) [0]

Я нюб в кодинге. Хотя начал кодить довольно давно, но времени на него мало посвящал. Я тут временами лазил и задавал вопросы. Вот снова решил "расконсервировать" :) мой нубский проэкт. Мне бы очень хотелось, чтоб вы посмотрели на мой код и рассказали об ошибках в нем. Написано на ДелфиХ.

вот собсна код:

http://file.cyfra.net/files.php/fed0151693f2b5b03a4e294441263abf/Super Game.rar

плиз неполенитесь и опишите самые основные ошибки, чтоб впредь я и остальные нубы не повторяли их :)

з.ы. Cash привет, если помнишь :)

← →
Аццкий_рыцарь (2006-09-20 02:38) [1]

ой сорри, вот правильная ссылка

http://file.cyfra.net/files.php/1659e36736313c48a9a59e04632bf3f9/Super_Game.rar

← →
Cash © (2006-09-20 08:27) [2]

Whatsup! я людей всегда помню! :)
Во прикол! Мне Download мастер сказал, что ссылка битая.

← →
Аццкий_рыцарь (2006-09-20 09:12) [3]

А так норм?

http://slil.ru/23143230

← →
Аццкий_рыцарь (2006-09-21 14:21) [4]

ЛЮДИ! НУ ПОСМОТРИТЕ ПЛИЗ!

← →
Cash © (2006-09-21 14:27) [5]

Аццкий_рыцарь (20.09.06 09:12) [3]:
Во, так норм! Тока у меня трафик дорогущий. (2.5 мега не подниму)
Можешь только исходники слить, без графики и звуков?

← →
Аццкий_рыцарь (2006-09-21 18:08) [6]

А как же ты проверишь тогда без графики? :) Звуков там нету. Я могу так сделать, а ты разберешся?

← →
Cash © (2006-09-21 18:56) [7]

Разберусь, для выявления лагов (в большинстве случаев) компиль не
нужен.

← →
Аццкий_рыцарь (2006-09-21 20:36) [8]

а если я без екзешника залью? а то графику геморно удалять

← →
Cash © (2006-09-22 12:05) [9]

Для первичного анализа проги необходим только ее код, и больше ничего!

ЗЫ: наверняк еще кто ни будь проявит интерес, если качать надо
будет поменьше.

← →
Наиль © (2006-09-22 15:28) [10]

> а если я без екзешника залью? а то графику геморно удалять

Если архивируешь WinRar"ом, то достаточно указать маску файлов которые не нужно добавлять в архив (закладка "Файлы"). И не какого гемороя!

← →
ANTPro © (2006-09-22 22:24) [11]

Наиль © (22.09.06 15:28) [10]
Он всю графику в одном .dfm хранит :)

Аццкий_рыцарь (20.09.06 2:33)
Ты бы хоть темпы удалил и dcu (*.~* и *.dcu)

Super Game1.7z [ 701 КБ] Тока сорцы
Super Game.7z [1,38 МБ] Сорцы + EXE - хлам

А подобное вроде XProger когда то делал
Анимация меня порадовала, но всего 25 фпс
(Конфиг компа на моем сайте)

Сорцы не плохо бы по стандартам оформлять (Читабельней как то особенно без коментариев)

← →
Аццкий_рыцарь (2006-09-22 23:33) [12]

Тоесть мне надо сделать чтоб графика хранилась в одном файле? И как это "по стандартам" ?

← →
ANTPro © (2006-09-23 01:06) [13]

Аццкий_рыцарь (22.09.06 23:33) [12]
Тоесть мне надо сделать чтоб графика хранилась в одном файле?
Только не так :) зачем?

Поправочка
ИМХО Сорцы не плохо бы по стандартам оформлять

Аццкий_рыцарь (22.09.06 23:33) [12]
И как это "по стандартам" ?
Где то http://rsdn.ru/ видел но там для сей
Качни любой сорец открытой проги (Inno Setup например)

У тебя все в куче:
... tick:=tick-1; if explode=false then begin if tick=0 then begin for kolchastic:=1 to trunc (20) do begin n:=random(100); particle[n].xloc:=trunc(x)-mirx+random(13); particle[n].yloc:=trunc(y)-miry+random(13); end; tick:=1; end; end; collision; end; procedure Tfireball.DoCollision(Sprite: TSprite; var Done: Boolean); begin pixelcheck:=false; if Sprite is TObjectdo then // see"s if what the bullet collided with was an enemy begin image:=form1.DXImageList2.Items.Items[2]; x:=x-20; y:=y-20; animlooped:=false ; explode:=true; end; if Sprite is Tkozel then begin if (y>sprite.y+15) and (y<sprite.y+70) and (x>sprite.x+30) and (x<sprite.x+65) or(y>sprite.y+15) and (y<sprite.y+80) and (x+15>sprite.x+30) and (x+15<sprite.x+65) or(y+15>sprite.y+15) and (y+15<sprite.y+80) and (x>sprite.x+30) and (x<sprite.x+65) or(y+15>sprite.y+15) and (y+15<sprite.y+80) and (x+15>sprite.x+30) and (x+15<sprite.x+65) then begin if TKozel(sprite).movego= true then begin frags:=frags+1; image:=form1.DXImageList2.Items.Items[2]; tkozel(sprite).DoDie; x:=x-20; y:=y-20; explode:=true; animlooped:=false ; end; end; end; end; ...

← →
ПЛОВ © (2006-09-25 18:23) [14]

Начинать нужно со скриншотов. Так делают культурные люди. :)

← →
Аццкий_рыцарь (2006-11-11 03:33) [15]

Всем снова хай:) Поднимаю топик, наконец-то снова добрался сюда.

вот сливаю вариант игры без темп файлов и экзешника. Он весит 746 кб, думаю это нормально :)

http://slil.ru/23386914

очень прошу посмотреть! укажите плиз на главные ошибки. И расскажите где и как можно модернизировать\ускорить\отладить код. И еще, почему игра показывает так мало ФПС?

← →
Аццкий_рыцарь (2006-11-11 03:37) [16]

упс, забыл! вот скрин http://slil.ru/23386921

← →
Аццкий_рыцарь (2006-11-11 14:40) [17]

Пацаны! Ну плиз, посмотрите!

← →
Ketmar © (2006-11-11 15:08) [18]

>[16] Аццкий_рыцарь 11-Nov-2006, 03:37
>упс, забыл! вот скрин http://slil.ru/23386921
какой это, в пень, скрин? это архив. в топку.

← →
Аццкий_рыцарь (2006-11-11 15:40) [19]

Какая разница? Уйдет столько же трафика как и на скрин...

А ты не знаешь куда можно скрины выкладывать? Всмысле российский сервак. А то я незнаю че у вас да как :)

← →
grisme © (2006-11-11 16:16) [20]

понимаешь, просто хочется ткнуть и сразу увидеть, что за скрин :0) а не тыкать, потом опять тыкать, выбирать где сохранить, тыкать, набирать в консоли, распаковывать, открывать :0) видишь какая разница в количестве действий? вот, а программисты - люди ленивые, в основном

← →
Аццкий_рыцарь (2006-11-11 16:19) [21]

о, я тоже очень ленивый, знач смогу быть программистом :)

люди вы мне только скажите сервак куда можно кинуть скрин чтоб сразу открывался! Я сразу залью!

← →
Ketmar © (2006-11-11 23:45) [22]

>[19] Аццкий_рыцарь 11-Nov-2006, 15:40
>Какая разница? Уйдет столько же трафика как и на скрин...
а такая, что в бровзере умеют показывать bmp, jpg, png, gif. а rar -- не умеют.

>российский сервак. А то я незнаю че у вас да как :)
я тоже не знаю, что в России да как.

>[21] Аццкий_рыцарь 11-Nov-2006, 16:19
>люди вы мне только скажите сервак куда можно кинуть скрин
>чтоб сразу открывался! Я сразу залью!
narod.ru

← →
Аццкий_рыцарь (2006-11-12 07:14) [23]

Скриншот? пжалста!

Megadich1.narod.ru/screenshot01.JPG

← →
Ketmar © (2006-11-12 12:49) [24]

а теперь ссылку поленился прописать полностью. ну что ж у тебя как не одно, так другое?! %-)

← →
Аццкий_рыцарь (2006-11-12 13:11) [25]

www.Megadich1.narod.ru/screenshot01.JPG

так лучше? :)

← →
Ketmar © (2006-11-12 13:13) [26]

ни разу. ты проверь, проверь свои посты после отправки.

хинт: форум распознает линки, если они начинаются с "http". мне лично лениво делать лишние мышкодвижения. это надо клаву бросать, крысу искать, целиться... %-)

← →
rts111 © (2006-11-12 17:59) [27]

Специально для ленивых программистов:

http://Megadich1.narod.ru/screenshot01.JPG

← →
Аццкий_рыцарь (2006-11-13 00:55) [28]

а! вот так надо было! :) люди, ну вы даете. Разве можно быть таким ленивым?

Лана! Флуд закончен! Сможет ли кто-то раскритиковать моё ваяние и обьяснить мои ошибки?

> [28] Аццкий_рыцарь (13.11.06 00:55)

Положи сорцы отдельно они около 5кб...
Я лично больше ЭТО качать не собираюсь...

Смогу. Почему нет скомпилированного exe? У меня нет DelphiX. Устанавливать что? Правильно, лень =)

---
... we are walking on a thin line and you better avoid the risk ...

← →
Аццкий_рыцарь (2006-11-13 01:55) [31]

ANTPro, ты говоришь оставить только сорцы.

grouzd)ev, ты говоришь оставить скомпилированный ехе

кто-то еще говорит удалить скомпилированный ехе, потому что сильно много трафика :)))

grouzd)ev, установи плиз, неленись! :) буду ОЧЕНЬ ПРИЗНАТЕЛЕН!

поделить на два архива. в одном исходник, в другом -- exe. дел-то.

← →
Аццкий_рыцарь (2006-11-13 02:39) [33]

http://slil.ru/23396092 - исходники

http://slil.ru/23396102 - ехе файл

ТЕПЕРЬ ВЫ ПОСМОТРИТЕ НАКОНЕЦ?

>[33] Аццкий_рыцарь 13-Nov-2006, 02:39
>ТЕПЕРЬ ВЫ ПОСМОТРИТЕ НАКОНЕЦ?
вряд ли. ни один уважающий себя флудер никогда не станет смотреть материалы по теме. особенно если они напрямую касаются темы ветки. %-)

зыж не нервничай. сообщению к двухсотому это тебя уже не будет волновать. %-)

← →
Аццкий_рыцарь (2006-11-13 02:45) [36]

хы, я понимаю что флуд - форевер и все такое, но ВАМ ЖЕ НЕСЛОЖНО ПОСМОТРЕТЬ! ВЫ Ж МАТЕРЫЕ ПРОГРАММЕРЫ! ЧТО ВАМ СТОИТ ЗАБРОСАТЬ ПОМИДОРАМИ МОЮ ПРОГУ И РАССКАЗАТЬ ИЗ-ЗА ЧЕГО ВЫ ЭТО СДЕЛАЛИ!?

да не нервничай ты. ну лениво, лениво. к тому же ночь. подожди до пятницы/субботы, периодически поднимая ветку. народ на выходных иногда добреет (пиво там, прочее %-). тогда и посмотрят. а в будни -- это вряд ли.

← →
Аццкий_рыцарь (2006-11-13 02:54) [38]

лана, пойду спать... спок ночи. Посмотрим что завтра тут появится нового. :)

Предложение не по коду а по скрину.
Местность смотрится однообразно.
Добавь еще пару спрайтов для земли с какой-нибудь зеленой травой или камнями и т.п.

По коду:
- извращенное форматирование, прочесть код очень сложно
- в паблик секции классов не размещают поля, т.к. при этом теряется смысл полей как таковых
- некрасиво сделано описание объектов мира - в глобальных переменных, сделай класс для этого. сейчас фактически код игры размазан по процедурам формы, если будешь развивать и усложнять игру, то код будет усложняться. новый класс поможет собрать весь игровой код в одном месте. фактически получишь класс движка игры, это будет удобно.

Прошу мастеров посмотреть Найти похожие ветки