Покритикуйте и помогите улучшить код моей игры

Здравствуйте, недавно начал писать игру и собственно пишу потихоньку, но не в этом суть, просто я только сегодня понял, что мой код может быть не понятен другим, и вот, хочу задать вам вопрос, в прилагающемся архиве все файлы, которые я создал в процессе, исходники в общем, ну и файлы для игры требующиеся, все в архиве, основная моя идея в том, чтобы управлять игрой только с помощью файлов, оригинально, не правда ли? Но в этой оригинальности и кроются самые главные проблемы, так вот, я прошу у вас помощи, посмотрите мои исходники( хоть некоторые и кому не лень ) и скажите мне, какие функции можно укоротить и как повысить читаемость кода, а-то я просто понимаю, как все устроено, если что спрашивайте по функциям, отвечу что они делают. Я прошу тех, кому действительно просто делать нечего, потому что там, мне кажется, вообще не читается.

Вложения

Game.rar (288.7 Кб, 35 просмотров)

0

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

Разрисуйте на листике схему взаимосвязей хотя бы базовых (имеется в виду ключевых в плане полиморфизма) классов.

0

Сообщение от Fulcrum_013 При завершении процесса она освободится вся и сразу, так как ос просто грохнет область выделенную процессу под кучу.

Плохой тон. Выделил память - освободи. От вашего подхода и начинаются утечки и уязвимости в безопасности.

Сообщение от makfak Ну а как предложите реализовать выход из программы экстренный?

А "экстренный" не надо. Реализуй систему исключений, в конечной точке которой все нормально выгружается.

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

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

0

Сообщение от PavelPol Плохой тон. Выделил память - освободи

плохой тон - задавать компу лишнюю работу. Естественно что в деструкторах НАДО прописывать удаление всего динамически распределенного относящегося к объекту. И в середине работы проги их тоже удалять надо. Апо выхожу из проги пусть ось всю кучу грохнет - юыстрее будет. А то в деструкторах такие рекурсии бывают (ну если объекты шибко грамотные) что может и очень конкретная протормозка быть. То вы с COMMON - областями никогда не работали. Там думать надо что удалять а что нет даже по завершении жизненного цикла объекта/процесса.

0

PavelPol,

Плохой тон. Выделил память - освободи. От вашего подхода и начинаются утечки и уязвимости в безопасности.

Вам ниже ответили, я солидарен.

А "экстренный" не надо. Реализуй систему исключений, в конечной точке которой все нормально выгружается.

Совсем забыл о них, спасибо, что напомнили.

0

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

Что делать — следует из обозначенной проблемы. Беспорядочные связи между классами — берем и выносим из классов все, что к ним не относится. Из игровых объектов убираем пользовательский интерфейс, ввод-вывод, продумываем доменную модель. Почему характеристики персонажа — тупо строки? Почему герой что-то пишет в консоль? Вынести это все во внешние функции/классы/подсистемы, а там видно будет.

0

0x10, Хорошо, займусь сегодня же.

0

Сообщение от 0x10 Почему герой что-то пишет в консоль? Вынести это все во внешние функции/классы/подсистемы, а там видно будет.

Тут немного функционал карты засунут класс херо. Предлагаю немнонго более другую структуру: Класс Scenario хранит список персонажей (абстрактно персонажей). персонаж () умеет базовые действия - типа переместится, так или иначе что то отправить на вывод (желательно через объект-драйвер, либо функцию-драйвер), а так же имеет указатель на класс AIController который полиморфно реализует поведение как NPC так и ввод пользователя. В результате появится возможность менять поведение персонажей не трогая базовую структуру загрузчика и основной цикл управления, в том числе на лету, подменой прикрепленного объекта AIController

Добавлено через 11 минут

Сообщение от 0x10 Почему герой что-то пишет в консоль?

Потому что когда дойдет до визуализации вызывать отрисовку прикрепленной модели или прикреплять ее к сцене придется точно так же из цикла отрисовки/сборки сцены, причем полиморфно. И это дожно быть методом базового (абстрактного) отображаемого объекта. Сам герой должен быть глубже по иерархии, поражден от персонажа. т.е абстрактная иерархия примерно такая "отображаемый объект"-"динамический объект"-"персонаж". Каждая из приведенных сущностей - абстрактная база для полиморфной иерархии.

0

В общем, много воды утекло с последнего сообщения в этой теме, но я всё-таки взялся за проект, которому уже, как я думал, конец. Я решил не заморачиваться со взаимодействием классов и подобными вещами и создал класс Core, от которого наследовать буду нужное( прикреплён ). Я решил спросить совета, как мне реализовать загрузку героя? Загружать нужное когда требуется или всё-таки загружать всё в начале игры? Может ещё варианты есть, буду рад, если вы предложите.

Вложения

Core.zip (2.4 Кб, 6 просмотров)

0

Сообщение от makfak создал класс Core, от которого наследовать буду нужное

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

2

C++ 1 2 3         Core();                                             // Default constructor         ~Core();                                            // Destructor //и т.д.

Улыбнуло.
Вы считаете, что эти комментарии имеют смысл?

1

0x10, Нууу, разбираться со взаимодействием мне ещё больше не хочется.
Croessmah, А что?

0

Сообщение от makfak А что?

отсутствующий комментарий намного лучше ненужного комментария.

0

Croessmah, Хмм, ну хорошо, удалю.

Добавлено через 33 секунды
Так как насчёт моего вопроса?

Загружать нужное когда требуется или всё-таки загружать всё в начале игры?

0

Сообщение от makfak Загружать нужное когда требуется или всё-таки загружать всё в начале игры?

Если У Вас там данных десять гигов, то, наверное, лучше подгружать по необходимости, а если там 2 Кб, то какой смысл загружать частями? Разве что если очень старая машина (примерно из компьютерного мезозоя), на такой случай можете реализовать оба варианта загрузки.

0

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

Добавлено через 10 минут
Просто меня смущает то, что во время сражения мне придётся примерно каждую секунду загружать и сохранять статы.

0

Сообщение от makfak Просто меня смущает то, что во время сражения мне придётся примерно каждую секунду загружать и сохранять статы.

Загружай заранее то, что может пригодиться, к примеру появился на локации какой-то персонаж, загрузи его данные. Ушел - выгрузи. Не сохраняй статы пока они могут изменяться. К примеру - персонаж выходит из локации или погиб - можно сохранить.

В общем, должен быть кэш определенного размера. В нем все нужные данные и те данные, которые в скором времени могут стать нужными.

0

PavelPol, Да я бы и не против, только как этот кэш реализовать?

0

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

1