HLFX.Ru Forum
профиль •  правила •  регистрация •  календарь •  народ •  FAQ •  поиск •  новое •  сутки •  главная •  выход  
HLFX.Ru Forum HLFX.Ru Forum > Наш форум > Компьютерные игры > S.T.A.L.K.E.R. Lost Alpha
впечатления от прохождения
Страницы (14): « 1 2 3 [4] 5 6 7 8 » ... Последняя »   Предыдущая тема   Следующая тема
Автор
Тема Новая тема    Ответить
Government-Man
Призрак

Дата регистрации: Apr 2006
Проживает: N/A
Сообщений: 3507

Рейтинг



Награды
 
[1 награда]


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


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

Сообщить модератору | | IP: Записан
Сообщение: 137133

Старое сообщение 01-05-2014 18:52
- За что?
 XaeroX
Crystice Softworks

Дата регистрации: Oct 2005
Проживает: Торонто
Сообщений: 34498
Нанёс повреждений: 514 ед.
Возраст: 37

Рейтинг



Награды
 
[1 награда]


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

Скажи спасибо, что не как в QC - сначала скобки, а потом название функции. %)
Цитата:
Government-Man писал:
В третьих в коде частенько встречаются маджик намберы типа x < 4096 - а чего 4096-то?

Они буквально поняли фразу "Macros are tools of satan" и просто везде натыкали константы.
Цитата:
Government-Man писал:
А ну и идиотские_имена_переменных конечно же.

int slava_Urkaine_heroyam_slava?

__________________
Правдой дорожить, лжи не потакать,
Дальних не судить, ближним помогать,
С тишиной сойтись на исходе дня
Научи меня, Родина моя!

Сообщить модератору | | IP: Записан
Сообщение: 137134

Старое сообщение 01-05-2014 18:59
-
 Дядя Миша
racing for fish

Дата регистрации: Oct 2005
Проживает: Кубань
Сообщений: 32188
Нанёс повреждений: 392 ед.

Рейтинг



Цитата:
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'

Сообщить модератору | | IP: Записан
Сообщение: 137137

Старое сообщение 01-05-2014 19:48
-
Government-Man
Призрак

Дата регистрации: Apr 2006
Проживает: N/A
Сообщений: 3507

Рейтинг



Награды
 
[1 награда]


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

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


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

Цитата:
XaeroX писал:
int slava_Urkaine_heroyam_slava?


Сообщить модератору | | IP: Записан
Сообщение: 137138

Старое сообщение 01-05-2014 19:50
- За что?
 Дядя Миша
racing for fish

Дата регистрации: Oct 2005
Проживает: Кубань
Сообщений: 32188
Нанёс повреждений: 392 ед.

Рейтинг



Я еще не видел ни одной программы, у которой был бы бардак в коде и при этом она стабильно работала. Под бардаком я имею в виду выстраивание архитектуры или её отсутствие.
Вот у сталкера архитектура загадочная. Кто их так учил проектировать движки - для меня загадка. Вот смотрите немцы у Кармака учились и у них кризис вполне стабильный вышел. А эти...
Ну что вот это вот всё? В ядре - виндовый бакэнд. Какой-то 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'

Сообщить модератору | | IP: Записан
Сообщение: 137152

Старое сообщение 02-05-2014 10:01
-
 XaeroX
Crystice Softworks

Дата регистрации: Oct 2005
Проживает: Торонто
Сообщений: 34498
Нанёс повреждений: 514 ед.
Возраст: 37

Рейтинг



Награды
 
[1 награда]


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

Windows XP

__________________
Правдой дорожить, лжи не потакать,
Дальних не судить, ближним помогать,
С тишиной сойтись на исходе дня
Научи меня, Родина моя!

Сообщить модератору | | IP: Записан
Сообщение: 137156

Старое сообщение 02-05-2014 10:21
-
~ X ~
offline

Дата регистрации: Oct 2005
Проживает: Архангельск
Сообщений: 3619

Рейтинг



Награды
 
[1 награда]


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

Quake

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


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

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

Сообщить модератору | | IP: Записан
Сообщение: 137160

Старое сообщение 02-05-2014 10:28
- За что?
 Дядя Миша
racing for fish

Дата регистрации: Oct 2005
Проживает: Кубань
Сообщений: 32188
Нанёс повреждений: 392 ед.

Рейтинг



Цитата:
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'

Сообщить модератору | | IP: Записан
Сообщение: 137163

Старое сообщение 02-05-2014 10:31
-
 XaeroX
Crystice Softworks

Дата регистрации: Oct 2005
Проживает: Торонто
Сообщений: 34498
Нанёс повреждений: 514 ед.
Возраст: 37

Рейтинг



Награды
 
[1 награда]


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

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

Добавлено 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__

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

__________________
Правдой дорожить, лжи не потакать,
Дальних не судить, ближним помогать,
С тишиной сойтись на исходе дня
Научи меня, Родина моя!

Сообщить модератору | | IP: Записан
Сообщение: 137168

Старое сообщение 02-05-2014 11:13
-
 Дядя Миша
racing for fish

Дата регистрации: Oct 2005
Проживает: Кубань
Сообщений: 32188
Нанёс повреждений: 392 ед.

Рейтинг



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

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

__________________
My Projects: download page

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

Цитата:

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

Сообщить модератору | | IP: Записан
Сообщение: 137169

Старое сообщение 02-05-2014 11:16
-
Government-Man
Призрак

Дата регистрации: Apr 2006
Проживает: N/A
Сообщений: 3507

Рейтинг



Награды
 
[1 награда]


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


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

Цитата:
XaeroX писал:
Windows XP


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

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

Сообщить модератору | | IP: Записан
Сообщение: 137177

Старое сообщение 02-05-2014 18:37
- За что?
 Дядя Миша
racing for fish

Дата регистрации: Oct 2005
Проживает: Кубань
Сообщений: 32188
Нанёс повреждений: 392 ед.

Рейтинг



Цитата:
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'

Сообщить модератору | | IP: Записан
Сообщение: 137178

Старое сообщение 02-05-2014 18:47
-
Government-Man
Призрак

Дата регистрации: Apr 2006
Проживает: N/A
Сообщений: 3507

Рейтинг



Награды
 
[1 награда]


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

Сообщить модератору | | IP: Записан
Сообщение: 137180

Старое сообщение 02-05-2014 19:15
- За что?
 Дядя Миша
racing for fish

Дата регистрации: Oct 2005
Проживает: Кубань
Сообщений: 32188
Нанёс повреждений: 392 ед.

Рейтинг



Маленькая корявая программка, особенно консольная вполне может быть стабильной даже при наличии адского бардака в коде. Пример - 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'

Сообщить модератору | | IP: Записан
Сообщение: 137181

Старое сообщение 02-05-2014 19:18
-
~ X ~
offline

Дата регистрации: Oct 2005
Проживает: Архангельск
Сообщений: 3619

Рейтинг



Награды
 
[1 награда]


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

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

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

Сообщить модератору | | IP: Записан
Сообщение: 137183

Старое сообщение 02-05-2014 20:11
- За что?
Тема: (Опционально)
Ваш ответ:



Переводчик транслита


[проверить длину сообщения]
Опции: Автоматическое формирование ссылок: автоматически добавлять [url] и [/url] вокруг интернет адресов.
Уведомление по E-Mail: отправить вам уведомление, если кто-то ответил в тему (только для зарегистрированных пользователей).
Отключить смайлики в сообщении: не преобразовывать текстовые смайлики в картинки.
Показать подпись: добавить вашу подпись в конец сообщения (только зарегистрированные пользователи могут иметь подписи).

Временная зона GMT. Текущее время 10:19. Новая тема    Ответить
Страницы (14): « 1 2 3 [4] 5 6 7 8 » ... Последняя »   Предыдущая тема   Следующая тема
HLFX.Ru Forum HLFX.Ru Forum > Наш форум > Компьютерные игры > S.T.A.L.K.E.R. Lost Alpha
впечатления от прохождения
Версия для печати | Отправить тему по E-Mail | Подписаться на эту тему

Быстрый переход:
Оцените эту тему:

Правила Форума:
Вы not можете создавать новые темы
Вы not можете отвечать в темы
Вы not можете прикреплять вложения
Вы not можете редактировать ваши сообщения
HTML Код ВЫКЛ
vB Код ВКЛ
Смайлики ВКЛ
[IMG] Код ВКЛ
 

< Обратная связь - HLFX.ru >

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