Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
 
Рейтинг 4.74/46: Рейтинг темы: голосов - 46, средняя оценка - 4.74
 Аватар для dimakozyr
162 / 29 / 3
Регистрация: 17.08.2013
Сообщений: 560

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

25.10.2013, 20:47. Показов 9875. Ответов 126
Метки нет (Все метки)

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

Скриншот


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

Цитата Сообщение от dimakozyr Посмотреть сообщение
Кидаю с dll, если у кого не запускается из-за того, что dll не найдены
Beaver.7z

Цитата Сообщение от dimakozyr Посмотреть сообщение
А так вроде упрощать уже некуда
Beaver.7z


Миниатюры
Покритикуйте мою игру  
5
Лучшие ответы (1)
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
25.10.2013, 20:47
Ответы с готовыми решениями:

Оцените мою 1 игру "21 card game"
Вот решил я сделать свою 1 игру и залить сюда для "конструктивной критики". Правила игры: Выигрывает тот кто наберет больше очков но не...

Покритикуйте код
Покритикуйте код, я точно знаю, что он нубовский но все же. Это моя первая программа на с++ которая делает , что то полезное и типо мой...

Покритикуйте код
Есть класс Студенты (реализован через односвязный список), хотел бы услышать критику по поводу его улучшения, если кому не лень разбираться...

126
Эксперт С++
4986 / 3093 / 456
Регистрация: 10.11.2010
Сообщений: 11,170
Записей в блоге: 10
27.10.2013, 17:11
Студворк — интернет-сервис помощи студентам
Цитата Сообщение от dimakozyr Посмотреть сообщение
История коммитов в гитхабе вам о чем-нибудь говорит?
Да, она говорит о том что вы вносили изменения. А слово fork вам о чем-нибудь говорит? Меня больше беспокоит вопрос: почему вы, написав эту игру, не можете найти элементарной ошибки?
0
 Аватар для dimakozyr
162 / 29 / 3
Регистрация: 17.08.2013
Сообщений: 560
27.10.2013, 17:14  [ТС]
Цитата Сообщение от castaway Посмотреть сообщение
Меня больше беспокоит вопрос: почему вы, написав эту игру, не можете найти элементарной ошибки?
Невнимательность к мелочам, которая свойственна каждому
0
Эксперт С++
4986 / 3093 / 456
Регистрация: 10.11.2010
Сообщений: 11,170
Записей в блоге: 10
27.10.2013, 17:15
Да не такая уж это и мелочь..
0
5499 / 4894 / 831
Регистрация: 04.06.2011
Сообщений: 13,587
28.10.2013, 03:42
dimakozyr, если не трудно, в коде из 84 поста, в DrawScene.cpp, допишите, временно:
C++
1
2
3
4
5
6
7
8
9
10
11
#include <iostream> 
void DrawScene::blittingImage(int xCoordinate,
             int yCoordinate,
             SDL_Surface* source,
             SDL_Surface* destination) const {
    std::cout << source << std::endl;
    SDL_Rect SDLRect_Blitting;
    SDLRect_Blitting.x = xCoordinate;
    SDLRect_Blitting.y = yCoordinate;
    SDL_BlitSurface(source, NULL, destination, &SDLRect_Blitting);
}
И запустите программу в консольном варианте. И, если можно, скрин того, что выведется на консоль при нажатии на play. Нужно мне эту загадку разрешить.
Миниатюры
Покритикуйте мою игру  
0
Диванный эксперт
Эксперт С++
 Аватар для Max Dark
2550 / 2064 / 971
Регистрация: 09.10.2013
Сообщений: 4,793
Записей в блоге: 4
28.10.2013, 04:23
alsav22, это происходит изза неинициализированых шрифтов
нужно раскоментировать строки с ttfInit() и ttfQuit() в функции GameCicle::callFunctionsForScene_2()
0
 Аватар для dimakozyr
162 / 29 / 3
Регистрация: 17.08.2013
Сообщений: 560
28.10.2013, 09:44  [ТС]
Цитата Сообщение от alsav22 Посмотреть сообщение
И запустите программу в консольном варианте
Дописал, но не пойму, как запустить в консольном варианте?
Или Cra3y уже ответил на вопрос?
0
Почетный модератор
Эксперт С++
 Аватар для SatanaXIII
5851 / 2862 / 392
Регистрация: 01.11.2011
Сообщений: 6,906
28.10.2013, 09:56
Цитата Сообщение от programina Посмотреть сообщение
Можно было скриншоты из игры приложить.
И фак.


dimakozyr, было бы неплохо, если б вы в первом посте еще б привели ссылки на темы с обсуждением тех или иных вопросов, возникавших в процессе написания игры.

Цитата Сообщение от MrGluck Посмотреть сообщение
Модеры, закрепите новый архив к шапке, иначе одни и те же сообщения о недостающих dll будут плодиться
Исполнено.
1
 Аватар для dimakozyr
162 / 29 / 3
Регистрация: 17.08.2013
Сообщений: 560
28.10.2013, 10:33  [ТС]
Цитата Сообщение от SatanaXIII Посмотреть сообщение
dimakozyr, было бы неплохо, если б вы в первом посте еще б привели ссылки на темы с обсуждением тех или иных вопросов, возникавших в процессе написания игры.
Вот, только не знаю, как в первом посте это закрепить
Ссылки на темы, которые создавались в процессе написания игры:

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

При нажатии на бобра не увеличивается счет
0
Почетный модератор
Эксперт С++
 Аватар для SatanaXIII
5851 / 2862 / 392
Регистрация: 01.11.2011
Сообщений: 6,906
28.10.2013, 10:42
Цитата Сообщение от dimakozyr Посмотреть сообщение
Вот, только не знаю, как в первом посте это закрепить
Сейчас сделаем.

И да, кстати,
Цитата Сообщение от dimakozyr Посмотреть сообщение
Покритикуйте мою игру
Ая-яй-яй. Какая недостойная игра. Ужас-ужас. Глаза бобра за мышью не следят, общий счетчик мировой популяции бобров нигде не отображается, бобры не обучаются и не проводят групповые контратаки и вообще больше шести часов к ряду играть невозможно. Х) На самом деле забавная вещь, вы молодец.
0
 Аватар для dimakozyr
162 / 29 / 3
Регистрация: 17.08.2013
Сообщений: 560
28.10.2013, 10:44  [ТС]
Цитата Сообщение от SatanaXIII Посмотреть сообщение
больше шести часов к ряду играть невозможно
Это, конечно, мое упущение
0
5499 / 4894 / 831
Регистрация: 04.06.2011
Сообщений: 13,587
28.10.2013, 14:24
Цитата Сообщение от Cra3y Посмотреть сообщение
alsav22, это происходит изза неинициализированых шрифтов
нужно раскоментировать строки с ttfInit() и ttfQuit() в функции GameCicle::callFunctionsForScene_2()
Я знаю из-за чего это происходит. ТС утверждает, что у него работает нормально код, который выложен (с закоментированной строкой), отчего и мой вопрос.

Добавлено через 3 минуты
Цитата Сообщение от dimakozyr Посмотреть сообщение
Или Cra3y уже ответил на вопрос?
Мне не ответ на вопрос нужен (я вопроса не задавал: почему у меня так происходит), а скрин работы у вас.
Цитата Сообщение от dimakozyr Посмотреть сообщение
Дописал, но не пойму, как запустить в консольном варианте?
В свойстах проекта, компоновщик - система - подсистема, выбрать консоль.
0
Диванный эксперт
Эксперт С++
 Аватар для Max Dark
2550 / 2064 / 971
Регистрация: 09.10.2013
Сообщений: 4,793
Записей в блоге: 4
28.10.2013, 14:49
У меня проскакивали нули в выводе
Но у меня происходит обнуление указателей в конструкторе и код с этими изменениями сюда кидал

1
5499 / 4894 / 831
Регистрация: 04.06.2011
Сообщений: 13,587
28.10.2013, 15:25
Да, если принудительно занулить передаваемые указатели, то и у меня начинает работать нормально. Компиляторы разные? Какие-то компиляторы пустые указатели обнуляют, какие-то нет?
0
Диванный эксперт
Эксперт С++
 Аватар для Max Dark
2550 / 2064 / 971
Регистрация: 09.10.2013
Сообщений: 4,793
Записей в блоге: 4
28.10.2013, 15:38
Тестовый код
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
#include <iostream>
#include <string>
using namespace std;
 
int  main() {
    void* v;
    std::string* s;
    char* c;
    std::cout<<v<<std::endl
        <<(void*)s<<std::endl
        <<(void*)c<<std::endl;
    return 0;
}
Code
1
2
3
4
5
6
7
max@notebook:~/src/tests$ g++ c.cpp
max@notebook:~/src/tests$ ./a.out
0x400740
0x7fffb6c01de0
0
max@notebook:~/src/tests$ g++ --version
g++ (Ubuntu/Linaro 4.8.1-10ubuntu8) 4.8.1
Вывод: указатели надо инициализировать )
1
 Аватар для dimakozyr
162 / 29 / 3
Регистрация: 17.08.2013
Сообщений: 560
28.10.2013, 15:42  [ТС]
Вот что у меня вывело.
Первые 5 строк в коноли - до нажатия на Play, остальные - после
Миниатюры
Покритикуйте мою игру  
1
5499 / 4894 / 831
Регистрация: 04.06.2011
Сообщений: 13,587
28.10.2013, 15:57
Понятно. У вас компилятор неинициализированные указатели обнуляет, у меня - нет (студии разные).
0
 Аватар для dimakozyr
162 / 29 / 3
Регистрация: 17.08.2013
Сообщений: 560
28.10.2013, 16:06  [ТС]
Цитата Сообщение от alsav22 Посмотреть сообщение
студии разные
У меня тринадцатая
0
5499 / 4894 / 831
Регистрация: 04.06.2011
Сообщений: 13,587
28.10.2013, 16:07
Цитата Сообщение от dimakozyr Посмотреть сообщение
У меня тринадцатая
Я помню, что не 10 (как у меня). Но:
Цитата Сообщение от Cra3y Посмотреть сообщение
Вывод: указатели надо инициализировать )
0
 Аватар для dimakozyr
162 / 29 / 3
Регистрация: 17.08.2013
Сообщений: 560
28.10.2013, 16:13  [ТС]
Цитата Сообщение от alsav22 Посмотреть сообщение
Я помню, что не 10 (как у меня). Но:
Сообщение от Cra3y
Вывод: указатели надо инициализировать )
Или переходить на 13-ю студию
0
5499 / 4894 / 831
Регистрация: 04.06.2011
Сообщений: 13,587
28.10.2013, 16:17
Цитата Сообщение от dimakozyr Посмотреть сообщение
Или переходить на 13-ю студию
Вы зря к этому несерьёзно относитесь. Это кривизна в коде, и хорошо, что 10 студия даёт возможнось это обнаружить.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
28.10.2013, 16:17
Помогаю со студенческими работами здесь

Пожалуйста, покритикуйте код
Стиль, логика, реализация, все что угодно. Заранее благодарен за любые замечания или советы. Делал в VS 2015 Задача 1, вывод даты...

Покритикуйте пожалуйста программу
Student.h#ifndef _STUDENT_H #define _STUDENT_H class Student { public: Student(); void del(); ...

Графы. Покритикуйте код
Нужно помощь тех кто работает и пишет хороший и красивый код. У меня построено три матрицы, подскажите как улучшить код. Где можно...

Шифратор пароля. Покритикуйте пожалуйста.
Это моя первая программка на С++, если кому не лень, натычте меня носом в ляпы. Чтобы не топтаться по граблям. :) #include...

Покритикуйте мою сборку комплектации на базе i5-2400 Sandy Bridge
Проц - Intel Core i5-2400 Sandy Bridge (3100MHz, LGA1155, L3 6144Kb) Охлад - Cooler Master V8 LGA1366/ 1156/ 1155/ 775 &amp; AM3/ AM2+/ AM2...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
120
Ответ Создать тему
Новые блоги и статьи
PhpStorm 2025.3: WSL Terminal всегда стартует в ~
and_y87 14.12.2025
PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .
Как объединить две одинаковые БД Access с разными данными
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha
Etyuhibosecyu 20.11.2025
Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
Мысли в слух
kumehtar 18.11.2025
Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .
Создание Single Page Application на фреймах
krapotkin 16.11.2025
Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru