HLFX.Ru Forum - S.T.A.L.K.E.R. Lost Alpha

Страницы (14): « 1 2 3 [4] 5 6 7 8 » ... Последняя »
Показать все 207 сообщений этой темы на одной странице

HLFX.Ru Forum (https://hlfx.ru/forum/index.php)
- Компьютерные игры (https://hlfx.ru/forum/forumdisplay.php?forumid=19)
-- S.T.A.L.K.E.R. Lost Alpha (https://hlfx.ru/forum/showthread.php?threadid=4379)

Отправлено Government-Man 01-05-2014 в 18:52:

Цитата:

XaeroX писал:
но по самому коду у тебя замечания есть? Если да, то какие?

Ну во-первых, почему-то между именем вызываемой функции и открывающей скобкой вставлены отступы - такое чувство, что авторы раньше писали на паскале. Во-вторых внутри вызова почему-то никаких пробелов вообще - даже вокруг операторов. В третьих в коде частенько встречаются маджик намберы типа x < 4096 - а чего 4096-то? Причем один и тот же меджик юзается в нескольких местах, так что правка его в случае чего чревата ошибками. Ну длиннющие вызовы типа Foo.Bar->Blah.MyFunction() читабельности не добавляют, к тому же что если изменится путь к объекту? А ну и идиотские_имена_переменных конечно же.

Отправлено XaeroX 01-05-2014 в 18:59:

Цитата:

Government-Man писал:
Ну во-первых, почему-то между именем вызываемой функции и открывающей скобкой вставлены отступы

Скажи спасибо, что не как в QC - сначала скобки, а потом название функции. %)

Цитата:

Government-Man писал:
В третьих в коде частенько встречаются маджик намберы типа x < 4096 - а чего 4096-то?

Они буквально поняли фразу "Macros are tools of satan" и просто везде натыкали константы.

Цитата:

Government-Man писал:
А ну и идиотские_имена_переменных конечно же.

int slava_Urkaine_heroyam_slava?

__________________

Отправлено Дядя Миша 01-05-2014 в 19:48:

Цитата:

XaeroX писал:
Скажи спасибо, что не как в QC - сначала скобки, а потом название функции. %)

К слову сказать все современные компиляторы куси успешно иммитирую синтаксис чистого Си, даже массивы позволяют, да и препроцессор на уровне шестой студии

А для клона первого дуума под названием vavoom был даже создан VirtualC с классами

Там прямо как в джаве - public static void, final class и так далее.
Очень забавная штука, рекомендую.

__________________
My Projects: download page

F.A.Q по XashNT
Блог разработчика в телеграме

Цитата:

C:\DOCUME~1\C4C5~1\LOCALS~1\Temp\a33328if(72) : see declaration of 'size_t'

Отправлено Government-Man 01-05-2014 в 19:50:

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

Цитата:

XaeroX писал:
Они буквально поняли фразу "Macros are tools of satan" и просто везде натыкали константы.

А enum и const - это я так понимаю инструменты москалей и их тоже использовать нельзя?

Цитата:

XaeroX писал:
int slava_Urkaine_heroyam_slava?

Отправлено Дядя Миша 02-05-2014 в 10:01:

Я еще не видел ни одной программы, у которой был бы бардак в коде и при этом она стабильно работала. Под бардаком я имею в виду выстраивание архитектуры или её отсутствие.
Вот у сталкера архитектура загадочная. Кто их так учил проектировать движки - для меня загадка. Вот смотрите немцы у Кармака учились и у них кризис вполне стабильный вышел. А эти...
Ну что вот это вот всё? В ядре - виндовый бакэнд. Какой-то xrCpuPipes - это вообще что такое? Какого он в отдельной дллке?
Партиклы в отдельной. Зато все три рендера - внутри экзешника.
Я могу напутать, но вроде как рендеры вынесли из него только в Lost Alpha.
Постоянные вот эти падения из-за ненайденной модели или шейдера.
Шейдер не скомпилился - вылет. Кость в модели не найдена - вылет.
И судя по тому, что это никто не исправил, это попросту невозможно исправить с текущей архитектурой. Т.е. без модели оно упадёт в любом случае, код не рассчитан.

__________________
My Projects: download page

F.A.Q по XashNT
Блог разработчика в телеграме

Цитата:

C:\DOCUME~1\C4C5~1\LOCALS~1\Temp\a33328if(72) : see declaration of 'size_t'

Отправлено XaeroX 02-05-2014 в 10:21:

Цитата:

Дядя Миша писал:
Я еще не видел ни одной программы, у которой был бы бардак в коде и при этом она стабильно работала

Windows XP

__________________

Отправлено ~ X ~ 02-05-2014 в 10:28:

Цитата:

Дядя Миша писал:
Я еще не видел ни одной программы, у которой был бы бардак в коде и при этом она стабильно работала.

Quake

Цитата:

Дядя Миша писал:
Зато все три рендера - внутри экзешника.

это попросту невозможно исправить с текущей архитектурой[/QUOTE]
потому у них майдаданы и происходят, наверное.

__________________
Минутка полезного:
Бесплатный UT-подобный Half-Life mod.
Бесплатный редактор для 32-битных текстур. Без дотнета.
Бесплатный IDE для любых компиляторов и ЯП.
Бесплатная Windows-подобная ОС.
Проверка грамматики русского языка.
Чат по hl[fx]: [email protected]

Отправлено Дядя Миша 02-05-2014 в 10:31:

Цитата:

XaeroX писал:
Windows XP

не обманывай, не видел ты её исходников. У них там в соглашении написано, что после ознакомления отрубают голову. А ты живой.

Цитата:

~ X ~ писал:
Quake

Я про архитектуру, а не про локальные загрязнения. Из первокваки мусор может вычистить любой нубокодер (что они уже давно сделали).
Arqlite проект называется. А вот архитектуру выправить - это вам не это.

__________________
My Projects: download page

F.A.Q по XashNT
Блог разработчика в телеграме

Цитата:

C:\DOCUME~1\C4C5~1\LOCALS~1\Temp\a33328if(72) : see declaration of 'size_t'

Отправлено XaeroX 02-05-2014 в 11:13:

Цитата:

Дядя Миша писал:
не обманывай, не видел ты её исходников.

Я не видел, но народ рассказывал. Там то ли винтукей утёк, то ли экспишка. Полагаю, особой разницы нет.

Добавлено 02-05-2014 в 18:13:

Цитата:

Дядя Миша писал:
Arqlite проект называется

Да уж, вполне достойный ченжлог:

Цитата:

aRQ-Lite version 0.9.2:

- Increased min memory to 16MB, max memory to 64MB
- Removed #ifdef Quake2 code
- Removed #ifdef IDGODS code
- Removed #ifdef PARANOID code
- Removed #ifdef __sun__

А грамматические ошибки в комментариях они случайно не исправляли?
Или они вообще комментарии снесли на этапе чистки?

__________________

Отправлено Дядя Миша 02-05-2014 в 11:16:

Цитата:

XaeroX писал:
Да уж, вполне достойный ченжлог:

Ну я же говорю, это даже Хвидеру вполне под силу.

__________________
My Projects: download page

F.A.Q по XashNT
Блог разработчика в телеграме

Цитата:

C:\DOCUME~1\C4C5~1\LOCALS~1\Temp\a33328if(72) : see declaration of 'size_t'

Отправлено Government-Man 02-05-2014 в 18:37:

Цитата:

Да сорс же - вполне стабильно работает.

ИЧСХ в хаммере например код на первый взгляд гораздо чище и организованнее, что не мешает хаммеру падать в разы чаще самого движка.

Цитата:

XaeroX писал:
Windows XP

В ядре винды код очень даже чистый, я бы даже сказал, что это самый чистый код, что я видел. Но вот шелл и всякие там ИЕ - там да, настоящий срач.

ЗЫ. Кстати без головы жить реально удобнее...

Отправлено Дядя Миша 02-05-2014 в 18:47:

Цитата:

Government-Man писал:
Да сорс же - вполне стабильно работает.

Цитата:

Дядя Миша писал:
Под бардаком я имею в виду выстраивание архитектуры или её отсутствие.

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

__________________
My Projects: download page

F.A.Q по XashNT
Блог разработчика в телеграме

Цитата:

C:\DOCUME~1\C4C5~1\LOCALS~1\Temp\a33328if(72) : see declaration of 'size_t'

Отправлено Government-Man 02-05-2014 в 19:15:

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

Отправлено Дядя Миша 02-05-2014 в 19:18:

Маленькая корявая программка, особенно консольная вполне может быть стабильной даже при наличии адского бардака в коде. Пример - ZHLT\VHLT.
Багов там очень много, но хотя бы не вылетает. Обрати внимание, что китаец боится её переписывать. Фиксы, фиксы, да еще и в ifdef, т.е. он даже боится взять на себя ответственность за свои же изменения.
Системщик бы переписал полностью архитектуру.

__________________
My Projects: download page

F.A.Q по XashNT
Блог разработчика в телеграме

Цитата:

C:\DOCUME~1\C4C5~1\LOCALS~1\Temp\a33328if(72) : see declaration of 'size_t'

Отправлено ~ X ~ 02-05-2014 в 20:11:

Там такой матан что сразу повеситься можно, а не переписывать. А ифдефы - это очень грамотный подход. У меня MOVEWITH - тоже везде забит ифдефами. Т.к. даже тестировать не с кем.

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

Временная зона GMT. Текущее время 23:27.

Страницы (14): « 1 2 3 [4] 5 6 7 8 » ... Последняя »
Показать все 207 сообщений этой темы на одной странице

На основе vBulletin версии 2.3.0
Авторское право © Jelsoft Enterprises Limited 2000 - 2002.
Дизайн и программирование: Crystice Softworks © 2005 - 2024