BomberMan на Delphi 7

[Odinok]

Сообщение от Ulex

Ну как бы не прикольная фича.


А слабо просто бомбермена но до конца доделать, со своей графикой адекватной, со звуками, с таблицей рекордов, бонусами, препятствиями различными, врагами разными, уровней хотя бы 30, сохранение и загрузку игры сделать?

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

Чем ближе я к завершению проекта, тем больше мне хочется, усложнить задачу

[Ulex]

Я глянул ваш исходник. Сразу скажу, я не бог весть какой программист, тем более на Delphi, поэтому можете не воспринимать мой гундёж всерьёз. Но, пардон. То, чего можно было бы назвать движком - не увидел. Ожидал увидеть крутой замес из объектов, классов, структур и т.д. А не, нема их тут. Всё очень жёстко, куча констант.

У вас всего один объект. Тип клетка. И в этом объекте всё. Мне кажется это неправильно. Например, вот это:

  TGPlayerTxtr: Byte;                   {номер текстуры моба}
  TGPlayerKadr: Byte;                   {кадр анимации моба}
  TGPprTime:    Integer;                   {время для смены кадров анимации}
  TGPVector:    Byte;                   {направление для моба куда движится}
  TGPLast:      Byte;                   {направление для моба где уже был}
  TGPmesto:     Byte;
  TGPl_X:       Integer;                {смещение моба относительно клетки}
  TGPl_Y:       Integer;                {смещение моба относительно клетки}

Зачем всё это в объекте карта? Это параметры не карты, это параметры моба. Тот же самый вопрос про бомбу. У вас карта размером 64*64 и каждый элемент массива MapMas содержит резерв под моба и под бомбу. А бомб на поле может быть всего две. Зачем такая избыточность?

Нет уж, извините. Мухи отдельно, котлеты отдельно. Есть бомбы, заводите класс "бомбы", заводите банк (массив) объектов класса "бомба" и работаете с этим банком. То же самое с мобами, то же самое с бонусами.

Про бонусы - их я вообще не увидел. Нету их там вообще ни в каком виде. Если не заметил, ткните носом.

с таблицей рекордов легко

Да ладно. Прям так и легко? Хорошо, допустим у вас пять типов мобов и каждый из них стоит по разному, а также очки плюсуются за разрушение препятствий (разные типы препятствий, разная стоимость) и за подъём бонуса.

Анимации... Анимации должны вообще быть независимы от типа объектов. Должен быть прям класс (структура, как хотите) под названием "анимация". И универсальные методы для работы с анимациями. А то вот такой фрагмент:

            If MapMas[x,y].TGPlayerKadr=5 then
                MapMas[x,y].TGPlayerKadr:=1;

А если анимация будет из десяти кадров? А если у одного типа мобов анимация из пяти кадров, а у другого из десяти? Что тогда будете делать, опять кучу case-ов городить?

[Odinok]

Сообщение от Ulex

Я глянул ваш исходник. Сразу скажу, я не бог весть какой программист, тем более на Delphi, поэтому можете не воспринимать мой гундёж всерьёз. Но, пардон. То, чего можно было бы назвать движком - не увидел. Ожидал увидеть крутой замес из объектов, классов, структур и т.д. А не, нема их тут. Всё очень жёстко, куча констант.

У вас всего один объект. Тип клетка. И в этом объекте всё. Мне кажется это неправильно. Например, вот это:

  TGPlayerTxtr: Byte;                   {номер текстуры моба}
  TGPlayerKadr: Byte;                   {кадр анимации моба}
  TGPprTime:    Integer;                   {время для смены кадров анимации}
  TGPVector:    Byte;                   {направление для моба куда движится}
  TGPLast:      Byte;                   {направление для моба где уже был}
  TGPmesto:     Byte;
  TGPl_X:       Integer;                {смещение моба относительно клетки}
  TGPl_Y:       Integer;                {смещение моба относительно клетки}

Зачем всё это в объекте карта? Это параметры не карты, это параметры моба. Тот же самый вопрос про бомбу. У вас карта размером 64*64 и каждый элемент массива MapMas содержит резерв под моба и под бомбу. А бомб на поле может быть всего две. Зачем такая избыточность?

Нет уж, извините. Мухи отдельно, котлеты отдельно. Есть бомбы, заводите класс "бомбы", заводите банк (массив) объектов класса "бомба" и работаете с этим банком. То же самое с мобами, то же самое с бонусами.

Про бонусы - их я вообще не увидел. Нету их там вообще ни в каком виде. Если не заметил, ткните носом.


Да ладно. Прям так и легко? Хорошо, допустим у вас пять типов мобов и каждый из них стоит по разному, а также очки плюсуются за разрушение препятствий (разные типы препятствий, разная стоимость) и за подъём бонуса.

Анимации... Анимации должны вообще быть независимы от типа объектов. Должен быть прям класс (структура, как хотите) под названием "анимация". И универсальные методы для работы с анимациями. А то вот такой фрагмент:

            If MapMas[x,y].TGPlayerKadr=5 then
                MapMas[x,y].TGPlayerKadr:=1;

А если анимация будет из десяти кадров? А если у одного типа мобов анимация из пяти кадров, а у другого из десяти? Что тогда будете делать, опять кучу case-ов городить?

про бомбы будет отельный массив из 100 элементов, ну при убийстве моба добовляется значение в переменную, не так уж сложно это организовать, с анимацией вы скорее правы, над этим мне стоит подумать

То, чего можно было бы назвать движком - не увидел.
МММмм а по вашу, что есть движок игры

[Ulex]

МММмм а по вашу, что есть движок игры

Если рассматривать движок как некую логическую структуру внутренней организации программы, то это есть полный и достаточный набор классов, структур, методов и процедур, необходимых для реализации такого типа программ.
Вот например - моб, враг, в случае убийства которого из него могут вываливаться всякие плюшки. У него есть набор свойств. И надо постараться описать его класс как можно универсальнее. Они могут быть разных типов, могут быть одинаковых типов, но разные по скорости, силе, уровню брони, могут таскать разные вкусности и т.д. Вот как вы сейчас сделаете двух одинаковых мобов, но с разными скоростями движения, или как вы сейчас сделаете моба, которого можно убить с двух/трёх попаданий? Опять будете прописывать это в виде case-ов и if-ов непосредственно в функциях? Тем самым вы всё больше будете усугублять неуниверсальность ваших процедур. Это плохо. Процедуры должны быть по возможности универсальны. Это достигается путём заложения свойств объекта в сам объект, а не в методы, его обрабатывающие. Например, есть процедура, обрабатывающая передвижения моба, она универсальная, она берёт структуру объекта и там есть параметр скорость, поэтому ей впринципе даже неинтересно кого она двигает, лучника, или дракончика. А вдруг это медленный танк, который наелся бонусов скорость и должен носиться как угорелый по карте. Как вы сейчас это сделаете? Вот это и называется структурой программы (и, наверное, может называться движком).

[Beermonza]

Добавлю пару слов, если позволите.

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

Для начала нужно разделить объекты на группы, что есть карта, что есть моб, что есть бомба, что есть бонус, что есть банк анимации, и пр. ...все это типы записей. К этим типам нужны элементарные процедуры, например, к мобу: появление, движение, атака (съедание героя), гибель. А что нужно пользователю, если это будет РПГ? ...подумай, там хуча та еще.

[Odinok]

Сообщение от Beermonza

Добавлю пару слов, если позволите.

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

Для начала нужно разделить объекты на группы, что есть карта, что есть моб, что есть бомба, что есть бонус, что есть банк анимации, и пр. ...все это типы записей. К этим типам нужны элементарные процедуры, например, к мобу: появление, движение, атака (съедание героя), гибель. А что нужно пользователю, если это будет РПГ? ...подумай, там хуча та еще.

Щас над этим думаем

[Odinok]

Переделал структуру программы, появились отдельные типы для игрока, бомб, карты
Кто может сказать как оптимизировать вывод графики на экран, так как при взрыве бомбы заметно падает производительность игры


PS Игра еще далеко не готова, нет интерфейса, анимации взрыва стен, не сделано нормальное меню, и прочие плюшки для комфортной игры

[Гром]

В код не залезал, но если поставить две бомбы подряд, то они взрываются одновременно. Стало быть, недостаточно хорошо вы выделили сущности. Каждая бомба должна иметь свой собственный таймер.

[Wicort]

В код не залезал, но если поставить две бомбы подряд, то они взрываются одновременно. Стало быть, недостаточно хорошо вы выделили сущности. Каждая бомба должна иметь свой собственный таймер.

Да нет, как раз это правильно. Вы в оригинал играли? Если бомба взрываясь задевает другую, то они взрываются вместе.

[Odinok]

МММмм, кто-нибудь может залезть в код и посмотреть, что можно оптимизировать