unno 9 сен 2011 в 12:07

DOOM Classic и Wolfenstein 3D в исходниках

1 мин

11K

Разработка под iOS*

+34

Комментарии 74

shpaker 9 сен 2011 в 12:24

This source release does not contain any game data, the game data remains subject to the original EULA and applicable law.

Хм… Что впрочем и не удивительно

TheShock 9 сен 2011 в 12:27

wolf_actor_ai.c, 300 строк одинакового кода:

case en_mecha:
	Sound_StartSound( NULL, 1, CHAN_VOICE, Sound_RegisterSound( "sfx/084.wav" ), 1, ATTN_NORM, 0 );			
	break;

case en_hitler:
	Sound_StartSound( NULL, 1, CHAN_VOICE, Sound_RegisterSound( "sfx/044.wav" ), 1, ATTN_NORM, 0 );
	break;

// ...

case en_will:
	Sound_StartSound( NULL, 1, CHAN_VOICE, Sound_RegisterSound( "sfx/072.wav" ), 1, ATTN_NORM, 0 );
	break;

case en_death:
	Sound_StartSound( NULL, 1, CHAN_VOICE, Sound_RegisterSound( "sfx/090.wav" ), 1, ATTN_NORM, 0 );
	break;

Интересно, у них не было ни малейшего желания сделать хотя-бы так?

case en_mecha:
	Play_Sound( "sfx/084.wav" );
	break;

case en_hitler:
	Play_Sound( "sfx/044.wav" );
	break;

// ...

case en_will:
	Play_Sound( "sfx/072.wav" );
	break;

case en_death:
	Play_Sound( "sfx/090.wav" );
	break;

Или это была такая оптимизация? Тогда неужели звуки проигрываются так часто, что она имеет смысл? +1 вызов функции в секунду — это разве критично на фоне того, сколько ресурсов тратится на сообственно проигрывание этой функции? Я, конечно, в Си не особо силён, но всё-таки интересно услышать аргументы по этому поводу.

+11

BuriK666 9 сен 2011 в 12:45

Возможно этот код сгенерировал какой-нибудь софт, в котором они делали модели.

+31

НЛО прилетело и опубликовало эту надпись здесь

smind 9 сен 2011 в 12:46

иногда не стоит переделывать то что и так работает…

RetroGuy 9 сен 2011 в 13:39

А почему бы и нет? Я в исходники не заглядывал, но подозреваю что Sound_StartSound — какая-то универсальная функция для работы со звуком, которая в различных ситуациях может принимать различные аргументы. Видимо, так уж получилось что в wolf_actor_ai.c все эти аргументы были одинаковы. Чтож в этом плохого-то? Зачем городить какую-то промежуточную прослойку? А если они могли поменяться? Вдруг на en_death было бы решено поставить не ATTN_NORM а что-нибудь другое? Добавлять в промежуточный Play_Sound еще одино значение? Можно конечно, но зачем?..

Повторюсь, в исходники я не заглядывал, просто высказал свое предположение :)

lalaki 9 сен 2011 в 15:22

В этом как раз скрытый смысл уменьшения дублирования: если мы можем все завернуть в функцию PlaySound, значит, эти звуки примерно одинаковы, одинаково выполняются — это важная метаинформация разработчику, которая не всегда осознанно выполняется. И если для какого-то звука нужно будет поменять какое-то значение аргумента, это будет не просто так — условно, вместо CHAN_VOICE будет CHAN_MUSIC, и тогда мы сделаем функции PlayVoice / PlayMusic — опять же все гораздо понятнее.

А когда все делается через одну универсальную функцию с большим количеством аргументов (не говоря про «magic numbers»), вся эта логика скрывается за кодом, и приходится ее же вычислять в уме.

Rome 9 сен 2011 в 19:25

Это сейчас легко рассуждать. А в то время языки 90-х вели себя крайне странно, и порой впадали в метафизику.
Из вашего примера, вполне возможно, что оптимизированный вариант как раз переставал работать после третьего раза. Нечто с похожим я встречался в 90-х и никто из профи не мог мне дать внятный ответ.
В моем случае, был вариант подключения к графической карте. В учебниках рекомендовали оптимизированный вариант из одной строки. На деле он срабатывал через раз. Опытным путем я пришел к выводу, что код из 5 строк является надежным и всегда его использовал. Чем вызывал удивление у тех, кто видел мой код.

Lof 9 сен 2011 в 13:58

Оправданий нет. Просто Copy/Paste :)

Constantine 9 сен 2011 в 16:31

Предложите свой вариант реализации без copy/paste. Вам нужно сделать ф-ю, которая проигрывает нужный звук по его id (enum).

dixx 9 сен 2011 в 17:28

Табличку (массив) сделать не судьба?

NickLion 9 сен 2011 в 18:36

Компилятор сделает табличку. Но согласен, что выглядит некрасиво.

Constantine 10 сен 2011 в 00:34

У этого варианта тоже есть минусы. Если сделать массив, то нужно будет всегда следить, чтобы индексы не съехали (чтобы кто-то в центр значений enum или наоборот в массив не вставил новое значение например). Можно сделать и массив структур/хеш, но нужно будет реализовывать/использовать поиск, что может подойти для офисного приложения, но неоптимально для игры. Ну и кол-во строк остается практически то же самое. Учитывая остальные минусы, не ясно, чем это лучше. Хотя, конечно, это вариант без copy/paste :)

dixx 10 сен 2011 в 01:09

Всё там отлично с индексами enum'ов, надо просто знать язык, на котором пишешь, а не как обычно. Пример без всяких извращённых поисков и хэшей, не имеющий проблем с модификацией enum'а:

enum myenum {

 value_a,

 value_b

};



char *myvalues[] = {

 [value_a] = "123",

 [value_b] = "456"

}

Это тот же самый язык Си, если что.

Constantine 11 сен 2011 в 20:27

Хм, не знал что так можно. Тогда да, так лучше видимо…

dixx 10 сен 2011 в 01:19

Даже если использовать не C99, а старый добрый C89, то вопрос на корню решается функцией инициализации на три строчки:

typedef struct mypair mypair_t;



struct mypair {

int id;

char *path;

};



enum values {

value_a,

value_b

last_value

};



mypair_t source[] = {

 { value_a, "123" },

 { value_b, "456" }

 { -1, NULL }

};



char *cache[last_value];



void init() {

for (int i = 0; i < last_value; i++)

cache[source[i].id] = source[i].path;

}

И как бы всё. Дальше init преобразуется в init_array_cache(mypair_t *ptr, char *cache) и пишется:

init_array_cache(sound_spec, sound_cache);

init_array_cache(video_spec, video_cache);

dixx 10 сен 2011 в 01:20

Хотя корректнее в цикле там будет проверять не индекс, а source[i].path != NULL. Это я спросони ступил, прошу прощения.

НЛО прилетело и опубликовало эту надпись здесь

Constantine 10 сен 2011 в 01:34

Это решение хуже, имхо, т.к. названия звуков/файлов ничем не хуже переменных или методов — им тоже нужны осмысленные названия.

Lof 12 сен 2011 в 11:22

char* soundFile = dbSounds[id]; // en_mecha, en_hitler, etc

PlaySound( soundFile );

crjd 9 сен 2011 в 14:08

Да даже если оптимизировали, можно было хотя бы макросами заменить.

esc 9 сен 2011 в 18:54

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

micbsv 9 сен 2011 в 21:33

Капец!
Мэппинг туда так и просится.

fizz 9 сен 2011 в 12:30

на N900 это бесплатно =)

Lof 9 сен 2011 в 13:57

Не бесплатно. За звук и графику надо платить.

x8tty 9 сен 2011 в 12:32

интересно, что использованная gpl2+ не совместима с эппловскими правилами и продавать или выкладывать в магазине приложений имеют право только владельцы кода. в общем-то, довольно справедливо

Grayman 9 сен 2011 в 12:43

Почему у фашиста на обложке Wolfenstein 3D в руках M16 интересно)

+11

imbeat 9 сен 2011 в 12:49

Вы раскрыли всю подноготную американцев! :)

moooV 9 сен 2011 в 12:54

Это и так всем известно — дед Буша сотрудничал с Гитлером, и узники Аушвица работали на него.

TheShock 9 сен 2011 в 12:51

Судя по длине ствола — скорее таки М4 без ручки

Grayman 9 сен 2011 в 13:12

Я думаю художник вряд ли запаривался в детализацией оружия, в любом случае непонятно откуда оно у фашистов)

unno 9 сен 2011 в 13:13

С трупо подобрал, когда патроны закончились.

Grayman 9 сен 2011 в 13:53

в таком случае у художника очень хорошая фантазия, да и на вооружении ее не было, если конечно действие в игре происходит в 40-е)

НЛО прилетело и опубликовало эту надпись здесь

shpaker 9 сен 2011 в 12:52

Хах… id Software толсто тролит

chuma3 9 сен 2011 в 12:56

M16A2

TheShock 9 сен 2011 в 12:58

Ствол коротковат.

shpaker 9 сен 2011 в 12:59

Сверху вниз: M16A1, M16A2, M4, M16A4

Всё таки на М4 больше похоже

chuma3 9 сен 2011 в 13:09

Не, все-таки A2. См. телескопический приклад, подствольная рукоятка — это отличает A2 от M4.
Ствол только короткий нарисовали.

TheShock 9 сен 2011 в 13:28

Подствольная рукоятка — снимается. А вот приклад — да, другой.

НЛО прилетело и опубликовало эту надпись здесь

shpaker 9 сен 2011 в 16:21

Художник курит что-то забористое

chuma3 9 сен 2011 в 13:00

А вот граната (у фашиста на заднем плане) похожа на немецкую Stielhandgranate.
ЗЫ ну и куда же без американских джинсов

Colobock 9 сен 2011 в 13:10

Джинсы во время Второй Мировой уже были, чего не скажешь о винтовке.

НЛО прилетело и опубликовало эту надпись здесь

silvansky 9 сен 2011 в 13:03

Интересно, а WADы от десктопного дума пойдут?

ZimM 12 сен 2011 в 00:09

Как-то ковырял айфоновскую версию Дума, и ЕМНИП, там действительный обычный WAD с дополнительным отдельным архивом «high-res» звуков

imbeat 9 сен 2011 в 13:03

А кто там на заднем плане рукой машет на картинке «DOOM»?

TheShock 9 сен 2011 в 13:28

Второй игрок?

unno 9 сен 2011 в 13:29

Это Пахомыч, обедать зовет.

beeruser 9 сен 2011 в 13:48

Прохожий. Кричит «Чувак! ты там мобилу обронил»

НЛО прилетело и опубликовало эту надпись здесь

Igg 9 сен 2011 в 13:13

Тематический портал… срач по поводу версии винтовки. Fuck yeah.

+35

grokinn 9 сен 2011 в 13:16

Вот интересно, тут недавно писали что размашистые и подробные комментарии это признак новичка в программировании, а тут код откомментирован по самое нехочу чем то вроде этого:

// A proportional drop in lighting sounds like a better idea, but

// this linear drop seems to look nicer. It's not like Doom's

// lighting is realistic in any case...

Это что то говорит о качестве программистов в ID?

TheShock 9 сен 2011 в 13:30

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

grokinn 9 сен 2011 в 13:36

там по разному в разных местах, например, iphone_render.c (из doom) весь покрыт комментариями

evilbloodydemon 9 сен 2011 в 13:32

В комментарии написано не как сделано, а почему. Так что все ок

RetroGuy 9 сен 2011 в 13:46

Не надо обобщать :) Размашистые коментарии могут также говорить о том, что код специально готовили для ознакомления людьми разного профессионального уровня. Вполне возможно, что перед открытием они как раз и провели такую «комментизацию» и, возможно, рефакторинг.

beeruser 9 сен 2011 в 13:55

комментарий новичка
a += 4; // к переменной а добавляем 4

комментарий профессионала
i = 0x5f3759df — (i >> 1); //первое приближение 1/sqrt()

+12

skeeet 9 сен 2011 в 13:22

Господа, вы забыли, что с недавних пор на ftp.idsoftware.com/idstuff/doom3/source/ давно лежат исходники Doom3. и поверьте, там код не менее странен.

gibson_dev 9 сен 2011 в 13:41

Вообще то это SDK а не исходники игры.

SystemPanic 9 сен 2011 в 18:44

Исходники Doom 3 выложат после выхода RAGE. В октябре.

skeeet 10 сен 2011 в 05:35

Извиняюсь, немного поторопился.

graninas 9 сен 2011 в 14:18

У меня лежат исходники Doom'а под названием linuxdoom. Судя по внутренностям, это самые натуральные исходники самого натурального Doom'a, открытые когда-то давно. Копирайты соответствующие. Сейчас скачал предложенные исходники, пусть будет много версий, хороших и разных.

imbeat 9 сен 2011 в 15:01

Зачем они Вам? Внукам показывать? ;)

graninas 9 сен 2011 в 15:09

Да нет, что вы. :) Внукам я сам Doom покажу, а исходники меня интересуют исключительно с программистской точки зрения. Не только just for fun, но и самообразование. В общем, мне интересно их устройство. Понимаете, в ~~вакууме~~ Сети много ~~сферических~~ исходников самых разнообразных программ. Их можно изучать, а некоторые — даже нужно, только удовольствия бывает маловато. Другое дело, когда изучаешь исходники того, что давно живет в сердце (Doom), — чувство удивительное просто.