Смена экранов игры (комментарии) (7 стр)

war_zes
> О, да здесь оОотец гаймдева... Ну давай, расскажи в чем там говнокод?
книгу прочитаешь, поймешь без подсказок.

> А то ведь за слова надо отвечать
Слышь, пацанчик, тываще с какова района?

war_zes
> надо модули зарегистрировать и выбрать стартовую.
Перегрузи AddScene на прием третьего булового параметра чтото типа isMainScene. Так ты на уровне обьявления будеш указывать какую сцену считать первой. Хуже не будет:)

war_zes
> Ты не поверишь, но я в своем движке (Sapphire3D) юзаю голый WinAPI (как минимум
> обработка ввода, сеть, потоки, окна), и данная система модулей идеально
> работает с ним
  Тогда бы уж писал на МFC, ведь система модулей MFC тоже идеально работает
с WinAPI)

RenGD
> Свитч лучше, чем менеджер.
> Есть много программ на WinAPI со свитчем,

Это где по 50 case на каждый switch ? :)

s.o.v.a
> > О, да здесь оОотец гаймдева... Ну давай, расскажи в чем там говнокод?
> книгу прочитаешь, поймешь без подсказок.

Всё же пример говнокода желательно привести ....

innuendo
> Всё же пример говнокода желательно привести ....
там везде быдлокод, я не могу выбрать что-то конкретное.

auto it = m_moduleList.find(name);
  if (it == m_moduleList.end())
    return;

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

void IModule::Init()
{
  if (m_isinit)
    Close();

тут если инициализируется уже инициализированный "модуль", то он сначала деинициализируется.
во первых зачем? во вторых - двойная инициализация это ошибка кодера, о чем нужно предупредить, а не выполнять какие-то бессмысленные действия.

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

все, я и так потратил слишком много времени.

s.o.v.a
> все, я и так потратил слишком много времени.

Предлагаю выложить твой код и пусть топикастер оторвётся по полной :) А так это игра в одни ворота...

innuendo
> Предлагаю выложить твой код и пусть топикастер оторвётся по полной :) А так это
> игра в одни ворота...
а больше для тебя ничего не сделать? заебал.

s.o.v.a
> а больше для тебя ничего не сделать?

Осталось указать свой возраст ... :)

s.o.v.a не вижу говнокода, вижу неосилятора не разобравшесося в логике кода. Завтра напишу подробней.

геймдев.ру, ты все такой же...

Админ - лох!

war_zes
> s.o.v.a не вижу говнокода
присмотрись повнимательнее, в этих кусках он полностью прав.

А стека экранов я так понимаю не предусмотрено конструкцией.

s.o.v.a
> геймдев.ру, ты все такой же...
  Всё столько же школоты, критикующей других, но которая не хочет показывать ничего своего. Из-за таких теперь лучше статьи вообще не писать.

zlos
> А стека экранов я так понимаю не предусмотрено конструкцией.
  А что это тогда такое если не стек?