Покритикуйте мою игру

dimakozyr · Регистрация: 17.08.2013

Студворк — интернет-сервис помощи студентам

Выкладываю код своей первой игры. Она готова процентов на 90, но уже работает.
Интересно узнать мнение людей, что в ней можно улучшить, упростить и т.п,
т.к. на данный момент в моих глазах это идеал и верх совершенства и мне трудно судить

Скриншот

Cсылки на архивы для скачивания

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

Кидаю с dll, если у кого не запускается из-за того, что dll не найдены
Beaver.7z

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

А так вроде упрощать уже некуда
Beaver.7z

Ссылки на темы, которые создавались в процессе написания игры

Настройка SDL в Visual Studio 2012

SDL - первый урок

Не исчезают сердца при потере жизни в игре

SDL - удаление загруженного изображения

Как изменить изображение для ярлыка?

Завершение бесконечного цикла через определенное время

Ошибка экземпляра класса: отсутствие ";" перед идентификатором "o_scenes_1"

Не определен базовый класс

SDL + C++ = Тормозит событие нажатия кнопки

Как сделать, чтобы пр нажатии на кнопку открывался сайт?

Подключение сторонних библиотек

Функция get() не возвращает значение

SDL ошибка на событии обработки нажатия кнопки мыши

Изменение цвета кнопки при многократных нажатиях

SDL Отображение текста на экране

При нажатии на бобра не увеличивается счет

alsav22 · 26.10.2013, 14:42

А o_DrawScene, в class SceneElements, вы так и не убрали. В методы scene_*_events(), этого класса, у вас предаются o_DrawScene из class GameCicle. Зачем тогда в нём этот объект?

dimakozyr · 26.10.2013, 18:14 **[ТС]**

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

А o_DrawScene, в class SceneElements, вы так и не убрали

Уже убрал. И убрал почти все магические числа

alsav22 · 26.10.2013, 18:44

class InitSurfaces тоже вызывает большие сомнения в нужности своего существования. Класс хранит screen главного окна. Инициализация главного окна происходит один раз. В дальнейшем, из этого класса, используется только метод getScreen(). Всё это, раз относится к главному окну, логично было бы и хранить, и делать в главном классе: class GameCicle. А так, это всё, делается через объект InitSurfaces из class GameCicle. Что даёт это лишнее усложнение? По-моему, ничего.

dimakozyr · 27.10.2013, 13:40 **[ТС]**

Упростил все, что только мог:
- Убрал все магические числа
- Удалил ненужные классы
- Переименовал переменные, чтобы они были более информативными

Еще можно убрать windows-зависимые элементы (exit(1)), чтобы сделать код кроссплатформенным, но тогда не знаю, как сделать, чтобы открывался фейсбук и твиттер, т.к. для этого использую Windows.h

А так вроде упрощать уже некуда

@Max Dark · 27.10.2013, 15:44

C++
1
2
3
4
5
6
7
void SceneElements::stopGame() {
    if(getEvent() -> type == SDL_QUIT)
        exit(1);
    if(getEvent() -> type = SDL_KEYDOWN && // присвоение вместо сравнения
       getEvent() -> key.keysym.sym == SDLK_ESCAPE)
        exit(1);
}

Добавлено через 1 час 25 минут
Изврат для открытия ссылки

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
#if defined(_WIN32) || defined (_WIN64)
#include <windows.h>
#else
#include <unistd.h>
#endif
void openURL(const char* url) {
#if defined(_WIN32) || defined (_WIN64)
    ShellExecute(NULL, "open", url,
                           NULL, NULL, SW_SHOWNORMAL);
#else
    pid_t pid;
    if((pid=fork())==0) {
        execlp("xdg-open","",url,NULL);
        std::cerr<<"can't launch"<<std::endl;
    }
    if(pid<0)
        std::cerr<<"can't fork"<<std::endl;
#endif
}

Работает под убунту

dimakozyr · 27.10.2013, 15:48 **[ТС]**

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

Изврат для открытия ссылки

И правда изврат

Сейчас попробуем

@Max Dark · 27.10.2013, 16:09

Подправил проект для сборки под линуксом в Eclipse
Собирается, но совершенно нет реакции на события
Ubuntu 13.10 amd64
gcc 4.8.1
Eclipse Kepler с оф сайта
game.7z

dimakozyr · 27.10.2013, 16:18 **[ТС]**

Если нажать на фейсбук, то почему-то запускается и фейсбук и твиттер

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
case 2:
#if defined(_WIN32) || defined (_WIN64)
    ShellExecute(NULL, "open", "http://facebook.com", NULL, NULL, SW_SHOWNORMAL);
#else
    pid_t pid;
    if ((pid = fork()) == 0)
    execlp("xdg-open", "open", "http://facebook.com", NULL);
#endif
 case 3:
    #if defined(_WIN32) || defined (_WIN64)
    ShellExecute(NULL, "open", "http://twitter.com", NULL, NULL, SW_SHOWNORMAL);
#else
    pid_t pid;
    if ((pid = fork()) == 0)
        execlp("xdg-open", "open", "http://twitter.com", NULL);
#endif

А еще код, который после #else тускло серый, что это означает?

@castaway · 27.10.2013, 16:21

У меня есть огромные сомнения в том, что этот код написал ты..
Ведь ты даже не можешь понять что тут не хватает банального break; между case-ами.

dimakozyr · 27.10.2013, 16:24 **[ТС]**

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

У меня есть огромные сомнения в том, что этот код написал ты..
Ведь ты даже не можешь понять что тут не хватает банального break; между case-ами.

Конечно, это доказательство того, что я его украл

@castaway · 27.10.2013, 16:25

А что же тогда говоришь что твоей?

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

Выкладываю код своей первой игры.

dimakozyr · 27.10.2013, 16:26 **[ТС]**

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

А что же тогда говоришь что твоей?

В предыдущем посте был сарказм....

@Max Dark · 27.10.2013, 16:26

dimakozyr, скопируйте код из моего сообщения в начало файла и используйте функцию

C++
1
openURL("http://some.site");

@castaway · 27.10.2013, 16:27

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

В предыдущем посте был сарказм....

Скорее и сарказм и правда одновременно..

dimakozyr · 27.10.2013, 16:42 **[ТС]**

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

dimakozyr, скопируйте код из моего сообщения в начало файла и используйте функцию
Код C++
openURL("http://some.site");

Заработало

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

Собирается, но совершенно нет реакции на события

Странно, в обработке событий нет ничего такого, что зависит от платформы

@Max Dark · 27.10.2013, 16:43

Рабочая сборка под Убунту 64 битаgame.7z
Я пофиксил все предупреждения, в предыдущей версии немного напутал

dimakozyr · 27.10.2013, 16:56 **[ТС]**

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

Рабочая сборка под Убунту 64 битаgame.7z
Я пофиксил все предупреждения, в предыдущей версии немного напутал

Так уже работало под убунту?
Нашел только данные изменения:

//DrawScene.h
В конструкторе добавлено screen(0)

//Events.cpp
В конце метода добавлено return 1;

dimakozyr · 27.10.2013, 17:06 **[ТС]**

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

А что же тогда говоришь что твоей?

История коммитов в гитхабе вам о чем-нибудь говорит?

@Max Dark · 27.10.2013, 17:08

Изменения по сравнению с вашей версией:
исправлены пути в #include на "SDL/SDL..."
исправлены предупреждения про неинициализированые поля классов
Добавлена функция открытия ссылок
добавлены return-ы в конец некоторых фунций(тут я намудрил в Events::buttonEvents, сначала поставил 0, а над было 1)
исправлено условие в функции stopGame();

вроде все

Сейчас работает полностью

dimakozyr · 27.10.2013, 17:10 **[ТС]**

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

добавлены return-ы в конец некоторых фунций(тут я намудрил в Events::buttonEvents, сначала поставил 0, а над было 1)

А, ну тогда понятно. Ну круто, кроссплатформенность - это супер

Новые блоги и статьи Все статьи Все блоги /
В чем ценность человеческого опыта в глобальном смысле? kumehtar 03.07.2026 Возможно, ценность человека не в том, что он однажды достигает мудрости, а в том, что он становится носителем карты пути. Он знает не только истину, но и последовательность внутренних изменений,. . .	интеграция AnyLogic с самописным REST API и переход на Odoo anaschu 03.07.2026 Успешная интеграция AnyLogic с самописным REST API и переход на промышленную Odoo WMS Сегодня проделал огромный путь от простой симуляции физических процессов до построения полноценной. . .	Поиск всех путей на ориентированном графе. Linux dcc0 02.07.2026 Переработка старого кода из моей статьи. Через несколько переработок от PHP кода к C89 (надеюсь, 89). Но довольно запутанно получилось. Код для Linux. Но если убрать time и то, что с ним. . .	Сам себя обучал rest api anaschu 02.07.2026 Педагогический лайфхак: Почему чистый REST API для ученика намного круче, чем готовые библиотеки Когда мы отказались от капризного JAR-файла AnyLogic и переписали код на стандартный HttpClient,. . .
rest api anylogic - выполнение модели на своём русском сайте anaschu 02.07.2026 Как подружиться с AnyLogic Cloud API, победить провайдеров и развернуться Java-бэкенд в Docker на бесплатном хостинге: Двухдневный лог борьбы Всем привет! Хочу поделиться свежим (и довольно. . .	Где деньги лежат kumehtar 02.07.2026 Это - японская подводная лодка I-52 (тип C2, кодовое имя Momi) вышла из Японии в марте 1944 года с миссией в оккупированную немцами Францию (Лорьян). Это была одна из «Янаги»-миссий по обмену. . .	Krabik для WoW 3.3.5a, многоязычный AmbA 02.07.2026 Допилил бота, думаю что окончательно. Изменения: - добавлена многоязычность - добавлено снятие скриншотов - добавлено поддержание бафов хождения по воде (для жреца, дк и шамана) - и так, по. . .	Алиса нашла кучу ошибок компиляции и запуска в проекте, который без проблем компилировался и запускался))) anaschu 30.06.2026 Я пока посмеюся, но завтра проверю. А вообще интерсно. Дал алисе файл, в котором точно нет ошибок компиляции и запуска, и попросил их найти. Нашла кучу))) Критические ошибки, мешающие компиляции и. . .

alsav22 5500 / 4895 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	26.10.2013, 14:42
	А o_DrawScene, в class SceneElements, вы так и не убрали. В методы scene_*_events(), этого класса, у вас предаются o_DrawScene из class GameCicle. Зачем тогда в нём этот объект? 1

alsav22 5500 / 4895 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	26.10.2013, 18:44
	class InitSurfaces тоже вызывает большие сомнения в нужности своего существования. Класс хранит screen главного окна. Инициализация главного окна происходит один раз. В дальнейшем, из этого класса, используется только метод getScreen(). Всё это, раз относится к главному окну, логично было бы и хранить, и делать в главном классе: class GameCicle. А так, это всё, делается через объект InitSurfaces из class GameCicle. Что даёт это лишнее усложнение? По-моему, ничего. 1

@Max Dark Диванный эксперт 2550 / 2064 / 971 Регистрация: 09.10.2013 Сообщений: 4,844 Записей в блоге: 4
	27.10.2013, 16:09
	Подправил проект для сборки под линуксом в Eclipse Собирается, но совершенно нет реакции на события Ubuntu 13.10 amd64 gcc 4.8.1 Eclipse Kepler с оф сайта game.7z 1

@castaway 4986 / 3093 / 456 Регистрация: 10.11.2010 Сообщений: 11,170 Записей в блоге: 10
	27.10.2013, 16:21
	У меня есть огромные сомнения в том, что этот код написал ты.. Ведь ты даже не можешь понять что тут не хватает банального break; между case-ами. 0

@Max Dark Диванный эксперт 2550 / 2064 / 971 Регистрация: 09.10.2013 Сообщений: 4,844 Записей в блоге: 4
	27.10.2013, 16:43
	Рабочая сборка под Убунту 64 битаgame.7z Я пофиксил все предупреждения, в предыдущей версии немного напутал 0

@Max Dark Диванный эксперт 2550 / 2064 / 971 Регистрация: 09.10.2013 Сообщений: 4,844 Записей в блоге: 4
	27.10.2013, 17:08
	Изменения по сравнению с вашей версией: исправлены пути в #include на "SDL/SDL..." исправлены предупреждения про неинициализированые поля классов Добавлена функция открытия ссылок добавлены return-ы в конец некоторых фунций(тут я намудрил в Events::buttonEvents, сначала поставил 0, а над было 1) исправлено условие в функции stopGame(); вроде все Сейчас работает полностью 1

Покритикуйте мою игру

Решение