BGforgeNet / FO2tweaks

Fallout 2 tweaks
https://forums.bgforge.net/viewforum.php?f=26
119 stars 10 forks source link

Рекомендации. #48

Closed FakelsHub closed 4 years ago

FakelsHub commented 4 years ago

https://github.com/BGforgeNet/FO2tweaks/blob/d7900e3c880f94761bd4925ecc822b75e3473b1f/source/gl_g_healing_revision.ssl#L272 Тут просто достаточно очистить массивы, а не создать новые и делать их сохраняемыми.

И save_array() достаточно применить один раз к массиву, когда он создается, а не каждый раз при измении его значений. save_array() функция сохраняет не в буквальном смысле, а просто указывает, что нужно будет сохранить этот массив.

burner1024 commented 4 years ago

Ну так создать новые просто быстрее и понятнее, нет?

FakelsHub commented 4 years ago

Нет. В совокупности это получается медленее.

burner1024 commented 4 years ago

resize_array(array, 0) пойдет?

FakelsHub commented 4 years ago

Ну да.

// remove all elements from array
#define clear_array(array)          resize_array(array, 0)
FakelsHub commented 4 years ago

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

burner1024 commented 4 years ago

Все что есть было добавлено по надобности.

FakelsHub commented 4 years ago

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

burner1024 commented 4 years ago

Напротив, такой формат только и позволяет следить в реалтайме. И дополнительный перевод строки обязателен, чтобы пропускать вот такой ужас. Делаем tail -f debug.log | grep healing_revision и все четко выдается.