| Форумы
ЧАТ |
Орешник |
|
| Форумы -» | Основная («Начинающим»)/ Базы / WinAPI / Компоненты / Сети / Media / Игры / Corba и COM / KOL / FreePascal / .Net / Прочее / RSDN.ORG |
|
| ||
|
|
Новости |
Поиск |
FAQ |
Статьи |
Книги |
Новости VCL
| | Форумы ЧАТ |
Орешник |
|
| |||
| |||
| |||
« предыдущая ветвь | форум | следующая ветвь »
Прошу мастеров посмотреть [Delphi, Windows]
Аццкий_рыцарь © (2006-09-20 02:33:00)Я нюб в кодинге. Хотя начал кодить довольно давно, но времени на него мало посвящал. Я тут временами лазил и задавал вопросы. Вот снова решил "расконсервировать" :) мой нубский проэкт. Мне бы очень хотелось, чтоб вы посмотрели на мой код и рассказали об ошибках в нем. Написано на ДелфиХ.
вот собсна код:
http://file.cyfra.net/files.php/fed0151693f2b5b03a4e294441263abf/SuperGame.rar
плиз неполенитесь и опишите самые основные ошибки, чтоб впредь я и остальные нубы не повторяли их :)
з.ы. Cash привет, если помнишь :)
ANTPro (2006-11-15 16:07:00) [1]> [50] rts111 © (15.11.06 14:47)
> А что это такое, "эксперт для делфи"?
http://www.dow.wau.nl/aew/downloads/DelForEx.zip
rts111 (2006-11-15 14:47:00) [2]> И не забудь эксперта скачать для делфи...
А что это такое, "эксперт для делфи"?
ANTPro (2006-11-14 19:59:00) [3]> [48] Аццкий_рыцарь (14.11.06 18:21)
OpenGL конечно рулит ;)
И не забудь эксперта скачать для делфи...
Аццкий_рыцарь (2006-11-14 18:21:00) [4]о, наерное начну ковырять PowerDraw3. Типа аналогия ДелфиХ, но намного быстрее и круче. Или может даже ОпенГЛ попытаться потянуть? О_о
Аццкий_рыцарь (2006-11-14 18:12:00) [5]впринципе, ты прав насчет того что нет разницы чем, но просто я очень много маялся со всякими заливаниями в сорс названия анимация зомби и игрока, всякие пути к ним переписывать, лучше уже заново все написать :)
з.ы. а че ща для 2Д рулит кроме 3Д библиотек? Омега?
grouzd)ev (2006-11-14 15:26:00) [6]1) ну попробуй проверить столкновение снарядов летящих со скоростью света)
2) уходть с DelphiX?
5) емое) уже хочется посотреть. тем более на DelphiX
насчет переходить - какая разница чем выводить графику? основная часть кода все равно одинаковая
---
... we are walking on a thin line and you better avoid the risk ...
Аццкий_рыцарь (2006-11-14 02:54:00) [7]2 grouzd)ev
- снаряды это не проблема, могу сделать чтоб летали со скоростью света :) ведь это даже не демка. Это просто наработки, собранные в кучу.
- значь, с ФПС ниче не поделаешь?
- просто на одном сайте доброжелатель непонятно откуда наделал полно всяких анимаций монстров\людей\вещей, долго пытался такое нарыть.
- магия будет разная, а это просто (сорри за такие слова) какашка :)
- рождаться будут красиво. Типа какой стихии магия, такой гаммы частицы и будут разлетаться от места кастования патрона.
- да уж, курсор это тема :)
и ваще, вижу прийдется с ДелфиХ слазить ибо на нем можно написать только нормальный тетрис. А чтоб самому его модернизировать, нехватает знаний. Просто жалко что столько труда пропадет :( Больше всего бился над процедурой столкновения игрока с препятствиями :(
grouzd)ev (2006-11-13 22:36:00) [8]
> оставить скомпилированный ехе
> кто-то еще говорит удалить скомпилированный ехе, потому
> что сильно много трафика :)))
ну что ж поделать что у всех инет разный. а искать, качать и устанавливать DelphiX в час ночи ну правда очень не хочется. Нет, правда, я даже твой проект в делфи открыл, а там такая подстава)
> Ребята, а вы не можете сказать почему так мало ФПС выдает игра?
потому что DelphiX
по критике:
- снаряды летят ну очеень медленно. имхо ускорить раза в 2
- конечно, мало фпс
- очень понравилась анимация монстряков
- а вот анимация патрона нет
- да и рождаются они (патроны) где-то не там
- ps курсор очень забавный)
---
... we are walking on a thin line and you better avoid the risk ...
Аццкий_рыцарь (2006-11-13 16:27:00) [9]Спасибо, я попытаюсь учесть все.
Я уже ночью вчера немного код подправил, разгреб всякие неясности с десятками "левых" переменных.
Ребята, а вы не можете сказать почему так мало ФПС выдает игра?
Servelat (2006-11-13 15:48:00) [10]В добавление к [40]
-
if nepravilno=true then
а почему не
if ((nepravilno=true)=true)=true then
:)
-
if deathvector=1 then
begin
image:= form1.DXImageList3.Items.Items[17]
end else
if deathvector=2 then
begin
image:= form1.DXImageList3.Items.Items[20]
end else
...
case. А еще лучше через константный массив.
-
отсутствие комментариев; без ста грамм и не разобратся, что бы мог означать следующий кусок кода:
lasty:=lasty+6;
ticks:=240+random(30);
nepravilno:=false;
a:=a+1;
kozelangle:=kozelangle+128;
kozelangle:=192+64-random(128);;
kozelangle:=kozelangle-128;
Зачем прибавлять к чему-либо 128 если в следующей строке этой же переменной присваивается 192+64-random(128)? В общем шаманство какое-то, код должен быть кратким и ясным, с комментированием мест которые делаются нестандартными методами (через ж...). Имхо конечно. Также, по всей программе разсыпаны непонятные числовые константы, которые то прибавляются, то вычитаются, то умножаются. Если нужно изменить какой-нибудь параметр (ну скажем разрешение экрана решишь другое использовать), то видимо придется вначале читать весь код, потом пытаться понять в скольких и каких именно местах надо поправить числа. По-моему, это очень нерационально.
Продолжать не буду, ибо код страшный и читать его неприятно. Рассмотри вариант о переписывании его сначала.
Компилировать не стал, ибо лень ставить ДелфиИкс, да и по скрину понятно что я увижу.
ANTPro (2006-11-13 15:46:00) [11]> [40] KA_ © (13.11.06 12:26)
> извращенное форматирование, прочесть код очень сложно
Вот это может исправить:
http://www.dow.wau.nl/aew/downloads/DelForEx.zip
KA_ (2006-11-13 12:26:00) [12]По коду:
- извращенное форматирование, прочесть код очень сложно
- в паблик секции классов не размещают поля, т.к. при этом теряется смысл полей как таковых
- некрасиво сделано описание объектов мира - в глобальных переменных, сделай класс для этого. сейчас фактически код игры размазан по процедурам формы, если будешь развивать и усложнять игру, то код будет усложняться. новый класс поможет собрать весь игровой код в одном месте. фактически получишь класс движка игры, это будет удобно.
KA_ (2006-11-13 12:09:00) [13]Предложение не по коду а по скрину.
Местность смотрится однообразно.
Добавь еще пару спрайтов для земли с какой-нибудь зеленой травой или камнями и т.п.
Аццкий_рыцарь (2006-11-13 02:54:00) [14]лана, пойду спать... спок ночи. Посмотрим что завтра тут появится нового. :)
Ketmar (2006-11-13 02:48:00) [15]да не нервничай ты. ну лениво, лениво. к тому же ночь. подожди до пятницы/субботы, периодически поднимая ветку. народ на выходных иногда добреет (пиво там, прочее %-). тогда и посмотрят. а в будни -- это вряд ли.
Аццкий_рыцарь (2006-11-13 02:45:00) [16]хы, я понимаю что флуд - форевер и все такое, но ВАМ ЖЕ НЕСЛОЖНО ПОСМОТРЕТЬ! ВЫ Ж МАТЕРЫЕ ПРОГРАММЕРЫ! ЧТО ВАМ СТОИТ ЗАБРОСАТЬ ПОМИДОРАМИ МОЮ ПРОГУ И РАССКАЗАТЬ ИЗ-ЗА ЧЕГО ВЫ ЭТО СДЕЛАЛИ!?
Ketmar (2006-11-13 02:41:00) [17]зыж не нервничай. сообщению к двухсотому это тебя уже не будет волновать. %-)
Ketmar (2006-11-13 02:41:00) [18]>[33] Аццкий_рыцарь 13-Nov-2006, 02:39
>ТЕПЕРЬ ВЫ ПОСМОТРИТЕ НАКОНЕЦ?
вряд ли. ни один уважающий себя флудер никогда не станет смотреть материалы по теме. особенно если они напрямую касаются темы ветки. %-)
Аццкий_рыцарь (2006-11-13 02:39:00) [19]http://slil.ru/23396092 - исходники
http://slil.ru/23396102 - ехе файл
ТЕПЕРЬ ВЫ ПОСМОТРИТЕ НАКОНЕЦ?
Ketmar (2006-11-13 01:59:00) [20]поделить на два архива. в одном исходник, в другом -- exe. дел-то.
| |
Страницы: 