Скриптование

Kirgudu · 7 Октября 2015

@Romz, почему только Артосу? Он дал необходимый инструмент и пояснения, дальше тебе всё доступно, надо только понять принцип. Приведу пример из своей практики.

Данные модуля проводников

guiders.script, как и многих других, в ЧН сохраняются в нет-пакет актора, раздувая его без необходимости:

function actor_binder:save(packet)
  ...
  guiders.save(packet)
  ...
end

Данных немного и в оригинале они представляют из себя простой набор переменных, одна из которых является плоской таблицей:

function save(package)
  set_save_marker(package, "save", false, "guiders")
  package:w_stringZ(selected_point)
  package:w_bool(teleported_to_another_level)
  if temp_points_table[selected_point] then
    package:w_stringZ(temp_points_table[selected_point].level)
    package:w_stringZ(temp_points_table[selected_point].point)
    package:w_stringZ(temp_points_table[selected_point].look)
  else
    package:w_stringZ("nil")
  end
  set_save_marker(package, "save", true, "guiders")
end

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

Для этого переписываем метод save в модуле guiders примерно так:

function save()
  local tSav = { teleported_to_another_level = teleported_to_another_level }
  if selected_point and selected_point:len() > 0 then
    tSav.selected_point = selected_point
  end
  if temp_points_table[selected_point] then
    tSav.temp_points_table = temp_points_table[selected_point]
  end
  SetVar("Guiders",tSav)
end

Осталось поправить вызов метода в биндере актора, удалив передачу в качестве аргумента нет-пакета актора:

function actor_binder:save(packet)
  ...
  guiders.save()
  ...
end

Аналогичным образом делаем правку чтения данных при загрузке.

guiders.script было:

function load()
  set_save_marker(package, "load", false, "guiders")
  selected_point = package:r_stringZ()
  teleported_to_another_level = package:r_bool()
  temp_points_table = {}
  local str = package:r_stringZ()
  if str~="nil" then
    temp_points_table[selected_point] = {}
    temp_points_table[selected_point].level = str
    temp_points_table[selected_point].point = package:r_stringZ()
    temp_points_table[selected_point].look = package:r_stringZ()
  end
  set_save_marker(package, "load", true, "guiders")
end

guiders.script стало:

function load()
  local tSav = GetVar("Guiders",{})
  if next(tSav) then DelVar("Guiders") else return end
  selected_point = tSav.selected_point or ""
  teleported_to_another_level = tSav.teleported_to_another_level
  temp_points_table = {}
  if tSav.temp_points_table then
    temp_points_table[selected_point] = table.clone(tSav.temp_points_table)
    --/ здесь table.clone() - аналог соотв. функции из lua_extension, полное копирование таблицы
  end
end

bind_stalker.script было:

function actor_binder:load(packet)
  ...
  guiders.load(packet)
  ...
end

bind_stalker.script стало:

function actor_binder:load(packet)
  ...
  guiders.load()
  ...
end

Всё это при условии, что методы SetVar, GetVar и DelVar добавлены в глобальном _g.script при внедрении комплекта se_stor. Если нет - можно воспользоваться методами se_stor.set() и se_stor.get().

Профит! Теперь все данные модуля guiders.script записываются в универсальном хранилище и читаются оттуда же. При этом нет необходимости записывать эти данные через чанки, а пстор актора освобождается от лишней информации.

Изменено 8 Октября 2015 пользователем Kirgudu

Romz · 7 Октября 2015

@Kirgudu,А как с этим быть?

Кроме того, в se_stor присутствует для записи local function save_storehouse() и local function save_chunks() но вот для загрузки имеется только local function read_chunks()

И вот этот момент непонятен

Этот вылет ловлю, когда пытаюсь закоментить вот этот кусок кода в lua_helper.script
local log = function(fmt,...)
 _G.log(sModule..":"..tostring(fmt),...)
end
if to_log and type(string.exformat) == 'function' then
 log = function(fmt,...) to_log(string.exformat(fmt,...)) end
end 
local printf = function(...) log(...) end --< конкретно вот эту строчку 
Сразу уточню - printf и log у меня и так работают. Поэтому хотел отрубить самопальные логи на основе get_console():execute

Kirgudu · 8 Октября 2015

@Romz, я сейчас с мобильного вошёл, посмотреть не могу. ЕМНИП, при загрузке данные (кроме чанков) читаются в другом месте. Глянь функцию read_tail_packet (как-то так) и откуда она вызывается.
Если надо будет, уточню завтра.
Что же касается вылета, видать твоя функция printf вовсе не так уж везде доступна. И при удалении, как ты говоришь, "самопальной" скрипту становится нечего вызывать.

Посмотрел. Да, так и есть, я правильно запомнил. Во время загрузки данные в глобальную таблицу хранилища читаются функцией read_tail_packet, которая вызывается при регистрации каждого кастомного объекта, используемого для сохранения данных в файле.

Изменено 8 Октября 2015 пользователем Kirgudu

Romz · 8 Октября 2015

Что же касается вылета, видать твоя функция printf вовсе не так уж везде доступна. И при удалении, как ты говоришь, "самопальной" скрипту становится нечего вызывать.

Вот это-то и непонятно... Она срабатывает во всех скриптах, кроме этого. Она в движке починена, вообще-то. И по всем скриптам Артоса я заменил его to_log на printf, и в остальных она нормально работает... Скорее всего, где-то в этом скрипте она неявно переопределяется... Поэкспериментирую (или забью).

Kirgudu · 8 Октября 2015

Честно говоря, я тоже других переопределений prinf (кроме как в комментируемой тобой строчке) не нашёл, не могу без предметного теста сказать, откуда берётся эта ошибка.

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

Romz · 8 Октября 2015

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

Вылет

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[error]ABORT CALLED:
se_stor:save_chunks:=[weathers],file=[level_weathers]~not_exist:<Error!>
ATTENTION! Game is interrupted! The information in lines above.
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
* Log file has been saved successfully!
! [SCRIPT ERROR]: c:\gam\s-cop\gamedata\scripts\se_stor.script:113: attempt to call global 'exit' (a nil value)
 
FATAL ERROR
 
[error]Expression    : fatal error
[error]Function      : CScriptEngine::lua_error
[error]File          : ..\xrServerEntities\script_engine.cpp
[error]Line          : 193
[error]Description   : <no expression>
[error]Arguments     : LUA error: c:\gam\s-cop\gamedata\scripts\se_stor.script:113: attempt to call global 'exit' (a nil value)

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

	tPkChunks["weathers"    ] = { file = "level_weathers"  } --/ погода
	tPkChunks["psy_antenna" ] = { file = "sr_psy_antenna"  } --/ зоны пси-излучения
	tPkChunks["treasure_mgr"] = { file = "treasure_manager"} --/ тайники-нычки
	tPkChunks["task_mgr"    ] = { file = "task_manager"    } --/ квесты-задания
	tPkChunks["bind_actor"  ] = { file = "bind_stalker"    } --/ ...

Все файлы присутствуют, именно так называются, методы save и load в них присутствуют.

Файл bind_stalker

Внешний доступ перенесён из примера без изменений.

Методы save и load изменены

----------------------------------------------------------------------------------------------------------------------
function actor_binder:save(packet)
	set_save_marker(...)
	local size = packet:w_tell() --/#+# for debug
--/ ... все последующие строки
--/ ... все предыдущие строки
--	self.weather_manager:save(packet)
--/ ... все последующие строки
--/ ... все предыдущие строки
	
	--/#+# вызов сохранений во внешних модулях ------------------------
	if se_stor then
		if to_log then to_log(script_name()..":save:size=["..(packet:w_tell() - size).."]") end --/#~#
		if type(se_stor.save) == 'function' then --/ 'прямой' вызов
			se_stor.save()
		end
	end
	--/<---------------------------------------------------------------
	set_save_marker(...)
end
--/#+# сохранения данных из внешних модулей: ------------------------
function actor_binder:ext_save(pk)
	self.weather_manager:save(pk)
	self.actor_detector:save(pk) --/#?# а нужно?
end
----------------------------------------------------------------------------------------------------------------------
function actor_binder:load(reader)
	set_save_marker(...)
	local size = reader:r_tell() --/#+# for debug
--/ ... все последующие строки
--/ ... все предыдущие строки
--	self.weather_manager:load(packet)
--/ ... все последующие строки
--/ ... все предыдущие строки
	
	--/#+# вызов загрузки во внешних модулях --------------------------
	if se_stor then
		if to_log then to_log(script_name()..":load:size=["..(reader:r_tell() - size).."]") end --/#~#
		if type(se_stor.save) == 'function' then --/ 'прямой' вызов
			se_stor.load()
		end
	end
	--/<---------------------------------------------------------------
	set_save_marker(...)
end
--/#+# загрузка данных во внешние модули: ---------------------------
function actor_binder:ext_load(pk)
	self.weather_manager:load(pk)
--	self.actor_detector:load(pk) --/#?# а нужно?
end
--/ ... все последующие строки

actor_detector закомментирован, потому что его нет в bind_stalker ЗП, в отличие от ТЧ

В se_stor обращение к файлам, которое вызывает эту ошибку, выглядит

	for name,v in pairs(tPkChunks) do
		local func = _G[v.file] and _G[v.file].save

Что я сделал не так применительно к ЗП?

Кстати, собственно se_stor.save и se_stor.load, которые и вызываются из методов save и load bind_stalker выглядят

--/ -----------------------------------------------------------------
--/ SAVE: вызов сохранения хранилища (from 'actor_binder:save')
--/ -----------------------------------------------------------------
function save(e) --/< from 'actor_saved'
	if event then --/#+# опционально: для сохранения локальных данных внешних модулей в общую таблицу db.storehouse
		event("storage_save"):trigger() --/#!#> генерируем сигнал события: 'запись в хранилище'
	end
	save_storehouse() --/ saving db.storehouse
end
--/ -----------------------------------------------------------------
--/ LOAD: вызов чтения хранилища внешними модулями (from 'actor_binder:load')
--/ -----------------------------------------------------------------
function load(e) --/< from 'actor_loaded'
	read_chunks()--/ чтение данных внешними модулями
	if event then --/#+# опционально: для чтения локальных данных внешних модулей из общей таблицы db.storehouse
		event("storage_load"):trigger() --/#!#> генерируем сигнал события: 'чтение из хранилища'
	end
end

То есть, сохраняем в таблицу, а читаем - из чанков. В read_chunks эта самая storehouse никак не упоминается.

Не, можно забить, конечно, оставить ext_save и ext_load пустой и раскомментировать self.weather_manager, но охота же понять, как пользоваться данным инструментом, хотя бы на самом элементарном уровне.

Изменено 8 Октября 2015 пользователем Romz

abramcumner · 8 Октября 2015

Все файлы присутствуют, именно так называются, методы save и load в них присутствуют.

Точно присутствуют? В оригинальном ЗПшном level_weathers.script нет функции save.

Kirgudu · 8 Октября 2015

@abramcumner, об этом я уже писал здесь.

@Romz, если подождёшь несколько дней, ориентировочно со вторника по пятницу на следующей неделе я постараюсь подготовить рабочий пример встраивания в чистый ЗП.

Все файлы присутствуют, именно так называются, методы save и load в них присутствуют.

В модуле level_weathers нет публичных функций save и load. Ну нету! Есть save и load, описанные только как методы класса WeatherManager. se_stor же ни о каких классах в подключаемых для сохранения модулях не знает. Поэтому тупо пользоваться примером из se_stor нельзя.

Подробнее смотри по ссылке выше.

Изменено 8 Октября 2015 пользователем Kirgudu

Romz · 8 Октября 2015

@Kirgudu,

То есть, это только для внутреннего, так сказать, пользования? Дык se_stor же, вроде, пытается методы тупо напрямую вызывать? Типа level_weathers.load?

Изменено 8 Октября 2015 пользователем Romz

Kirgudu · 8 Октября 2015

@Romz, а мы с @abramcumner тебе о чём? Похоже, ты просто не понимаешь разницы между публичной функцией модуля и методом класса в этом модуле.

Допустим, есть у нас некий модуль module.script. В нём может быть это:

function func()
  -- do something
end

и это:

class "MyClass"
function MyClass:__init()
 -- do something
end
function MyClass:func()
 -- do something
end

Чтобы вызвать функцию func из другого модуля в первом варианте, достаточно написать:

module.func()

Чтобы сделать то же самое во втором варианте, потребуется больше действий:

local my_class = module.MyClass() --/ тут получили экземпляр класса
my_class:func() --/ и только тут дёрнули метод

se_stor умеет вызывать только публичные функции, как описано в варианте 1.

Дык se_stor же, вроде, пытается методы тупо напрямую вызывать? Типа level_weathers.load?

Да! Но их там нет, так как save и load в этом модуле существуют только в контексте класса. Изменено 8 Октября 2015 пользователем Kirgudu

Romz · 8 Октября 2015

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

Ну есть такое... Я думал, они равнозначно вызываются... Теперь понятно.

В treasure_manager и task_manager тогда тоже нет публичных save и load.

Но зато в них присутствуют такие функции, как get_чтототам_manager, через которые их save и load в bind_stalker и вызываются...

И в level_weathers тоже такая есть

Поправка насчёт weather_manager. Его сохранение/загрузку Artos прописал и в таблицу чанков, и в доп. методы ext_save и ext_load. Это, понятное дело, избыточно (разве только в ЗП в этом модуле есть публичные save и load). Поэтому делаем что-то одно - либо оставляем в ext_save/load и убираем из таблицы чанков, либо убираем и из таблицы и из ext_save/load, но возвращаем в save/load.

То есть, всё, что не имеет публичных вызовов save и load убираем из чанков.

А если есть необходимость хранить их отдельно, переносим их вызовы save и load из actor_binder:save и actor_binder:load в ext_save и ext_load.

Я правильно понял?

Изменено 8 Октября 2015 пользователем Romz

Kirgudu · 8 Октября 2015

То есть, всё, что не имеет публичных вызовов save и load убираем из чанков.

А что имеет - добавляем (так как список в примере se_stor неточный).

А если есть необходимость хранить их отдельно, переносим их вызовы save и load из actor_binder:save и actor_binder:load в ext_save и ext_load.

Примерно так. В примере из se_stor такие модули уже указаны в ext_save и ext_load. Надо только актуализировать список согласно реалиям ЗП.

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

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

Romz · 8 Октября 2015

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

Подожду, конечно. А пока попробую сделать, как выше написал.

Собственно, похоже, что save и load в явном виде осталось только в sr_psy_antenna, xr_sound и xr_statistic, остальное - исключительно через классы...

Тогда имеет смысл, наверное, через se_stor сохранять только то, что нужно сохранять в модах. А от оригинала - оставить, как было, раз оно и так нормально сохраняется. Ну или наоборот, повыносить часть секций сохранений в se_stor, чтоб они уже с гарантией не загрузились на чистой игре.

Проверил, пока только с sr_psy_antenna в чанках и self.weather_manager в ext_save/load. НИ началась нормально. Игра сохранилась и загрузилась без проблем, вроде. Именные сохранения тоже. Единственное, set load marker: перевалил за 8000, это размер сохранения он считает, как я понял, по поводу чего он и начал ругаться, что, мол проблемное сохранение. Хотя, собственно размер сохранения и не увеличился.

Изменено 8 Октября 2015 пользователем Romz

Kirgudu · 9 Октября 2015

@Romz, скорее всего, даные self.weather_manager у тебя сейчас не сохраняются. И вот почему.

Методы ext_save/load вызываются из публичных (добавляемых комплектом se_stor) bind_stalker.save/load, которые, в свою очередь, должны вызываться при сохранении тех самых чанков. Но сейчас ты оставил в таблице tPkChunks, судя по всему, только sr_psy_antenna.

ИМХО, для описанного тобой выше частного случая всё должно выглядеть так:

bind_stalker.script

function actor_binder:ext_save(pk)
  self.weather_manager:save(pk)
end

function actor_binder:ext_load(pk)
  self.weather_manager:load(pk)
end

там же обязательно должно быть сделано (как и сказано в инструкции) это:

--[[ --/#x# отключено!
function actor_init    (npc)
  --npc:bind_object(actor_binder(npc))
end --]]
--/ -----------------------------------------------------------------
--/#+# организуем внешний доступ к биндеру актора:
--/ -----------------------------------------------------------------
function actor_init(npc)
  _G.g_bind_actor = this.actor_binder(npc)
  npc:bind_object(g_bind_actor)
end
function save(...)
  _G.g_bind_actor:ext_save(...)
end
function load(...)
  _G.g_bind_actor:ext_load(...)
end

se_stor.script

local tPkChunks     = {} --/ таблица нет-пакетов с чанками внешних модулей (option for SIMBION mod)
if iSSVer < 8 then --/ for SHoC
  -- ... для ЗП можно оставить без изменений
elseif iSSVer < 12 then --/ for CS
  -- ... для ЗП можно оставить без изменений
else --/ for CoP
  tPkChunks["bind_actor"  ] = { file = "bind_stalker"    } --/ ext_save/load в bind_stalker.script
  tPkChunks["psy_antenna" ] = { file = "sr_psy_antenna" } --/ зоны пси-излучения
end

Собственно, ext_save/load были сделаны как раз для того, чтобы данные внешних модулей, у которых отсутствуют публичные save/load, но есть класс с этими методами, инициализируемый в биндере актора, с минимальными изменениями перенести из нет-пакета актора в чанки.

Почему у тебя вырос объём данных, записываемых в нет-пакет актора, сказать не могу, тут надо смотреть, что ты пишешь. Если беспокоит или просто интересно, для анализа организуй сам дополнительный вывод в лог соответствующей информации, это не сложно. Но, в принципе, в случае ЗП объем 8000 байт не страшен, в ЗП и ЧН безопасно записывать в нет-пакет до 16000-16200 байт. Это, кстати, учитывается в механизме se_stor.

Изменено 9 Октября 2015 пользователем Kirgudu

Romz · 9 Октября 2015

@Kirgudu, Так и сделал. Только в таблицу чанков добавил

	tPkChunks["xr_statistic"] = { file = "xr_statistic"    } --/ статистика

Не знаю зачем. Чтоб было и для эксперимента. Плюс, переношу вот эти вызовы.

--[[	release_body_manager.get_release_body_manager():save(packet)
	-- AtmosFear
        if atmosfear then
		self.atmosfear:save(packet)
		self.psi_storm_manager:save(packet)
		self.fallout_manager:save(packet)
        end
	-- End AtmosFear
	self.surge_manager:save(packet)
--	treasure_manager.get_treasure_manager():save(packet)
--	task_manager.get_task_manager():save(packet)
-----------------------------makdm_start-----------------------------
	if art_sp_mgr then
		art_sp_mgr.get_art_spawner_manager():save( packet )
	end	]]
-----------------------------makdm_end-------------------------------

Пока, с целью эксперимента, чтобы проверить, что можно безболезненно вынести во внешние сохранения.

Почему у тебя вырос объём данных, записываемых в нет-пакет актора, сказать не могу, тут надо смотреть, что ты пишешь. Если беспокоит или просто интересно, для анализа организуй сам дополнительный вывод в лог соответствующей информации, это не сложно. Но, в принципе, в случае ЗП объем 8000 байт не страшен, в ЗП и ЧН безопасно записывать в нет-пакет до 16000-16200 байт. Это, кстати, учитывается в механизме se_stor.

Сам объём не вырос, судя по всему. Там, похоже, просто номер метки сохранения увеличывается, скачком и единомоментно. Вот пример лога

...
c:\gam\s-cop\se_stor:(298):read_tail_packet:chunk_key=[xr_statistic],chunk_size=[715]:(717)
c:\gam\s-cop\se_stor:(298):read_tail_packet:chunk_key=[psy_antenna],chunk_size=[3]:(5)
c:\gam\s-cop\se_stor:(298):read_tail_packet:chunk_key=[bind_actor],chunk_size=[80]:(82)
c:\gam\s-cop\se_stor:(315):~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
c:\gam\s-cop\se_stor:(316):Load Storehouse: size(1)=[856], keys=[1] :BuildSav=[-1]
c:\gam\s-cop\se_stor:(317):~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
* Game 0 is successfully loaded from file 'c:\users\public\documents\stalker-cop\savedgames\0.scop' (3.088s)
* phase time: 3041 ms
* phase cmem: 448435 K
_se_actor: set save marker: 8795
_se_actor: SAVE DIF: 1
_se_level_changer: set save marker: 268
_se_level_changer: SAVE DIF: 26
_se_level_changer: set save marker: 273
_se_level_changer: SAVE DIF: 26
...

...
* [Loading IB] 760575 indices, 1485 Kb
* phase time: 1282 ms
* phase cmem: 552458 K
* phase time: 170 ms
* phase cmem: 561488 K
* [DETAILS] VertexConsts(256), Batch(61)
* [DETAILS] 10126 v(20), 5368 p
* [DETAILS] Batch(61), VB(197K), IB(31K)
* phase time: 71 ms
* phase cmem: 561549 K
* Loading HOM: c:\gam\s-cop\gamedata\levels\zaton\level.hom
* phase time: 1763 ms
* phase cmem: 562359 K
* phase time: 24 ms
* phase cmem: 562359 K
* phase time: 21 ms
* phase cmem: 562359 K
* t-report - base: 2125, 1050208 K
* t-report - lmap: 16, 16386 K
_se_actor: set save marker: 8795
_se_actor: SAVE DIF: 1
_se_level_changer: set save marker: 268
_se_level_changer: SAVE DIF: 26
_se_level_changer: set save marker: 273
_se_level_changer: SAVE DIF: 26
...

...
* [win32]: free[1977620 K], reserved[84592 K], committed[2132028 K]
* [ D3D ]: textures[1141354 K]
* [x-ray]: process heap[628826 K], game lua[25658 K], render[7655 K]
* [x-ray]: economy: strings[28744 K], smem[319117 K]
* [win32]: free[1978452 K], reserved[84920 K], committed[2130868 K]
* [ D3D ]: textures[1142378 K]
* [x-ray]: process heap[628585 K], game lua[25766 K], render[7656 K]
* [x-ray]: economy: strings[28744 K], smem[319117 K]
_actor_binder: set save marker: 126
_Crelease_body: set save marker: 404
_Crelease_body: SAVE DIF: 4
_AtmosFear: set save marker: 410
_AtmosFear: SAVE DIF: 217
_PsiStorm: set save marker: 629
_PsiStorm: SAVE DIF: 14
_Fallout: set save marker: 645
_Fallout: SAVE DIF: 18
_SurgeHide: set save marker: 665
_SurgeHide: SAVE DIF: 14
_sr_psy_antenna: set save marker: 681
_sr_psy_antenna: SAVE DIF: 1
_sound_actor_save: set save marker: 685
_sound_actor_save: SAVE DIF: 3102
_CTreasureManager: set save marker: 4508
_CTreasureManager: SAVE DIF: 3582
_CRandomTask: set save marker: 8093
_CGeneralTask: set save marker: 8115
_CGeneralTask: SAVE DIF: 64
_CGeneralTask: set save marker: 8203
_CGeneralTask: SAVE DIF: 68
_CGeneralTask: set save marker: 8295
_CGeneralTask: SAVE DIF: 68
_CGeneralTask: set save marker: 8386
_CGeneralTask: SAVE DIF: 66
_CGeneralTask: set save marker: 8474
_CGeneralTask: SAVE DIF: 64
_CRandomTask: SAVE DIF: 447
_AS_manager: set save marker: 8546
_AS_manager: SAVE DIF: 8
c:\gam\s-cop\bind_stalker:(701):bind_stalker:save:size=[8430]
c:\gam\s-cop\se_stor:(117):save_storehouse:size=[0/nil]>)
_sr_psy_antenna: set save marker: 2
_sr_psy_antenna: SAVE DIF: 1
_WeatherManager: set save marker: 2
_WeatherManager: SAVE DIF: 78
c:\gam\s-cop\se_stor:(165):save_storehouse:key[xr_statistic],type_key[string],type_chunk[boolean]~not saved:<Warning!>
c:\gam\s-cop\se_stor:(159):save_storehouse:chunk_name=[psy_antenna],chunk_size=[3],len=[19]=>[19]
c:\gam\s-cop\se_stor:(159):save_storehouse:chunk_name=[bind_actor],chunk_size=[80],len=[95]=>[114]
c:\gam\s-cop\se_stor:(237):~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
c:\gam\s-cop\se_stor:(238):Save Storehouse: size(1)=[125], keys=[1] : BuildSav=[-1]
c:\gam\s-cop\se_stor:(239):~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
_actor_binder: SAVE DIF: 8430
WARNING! may be this is problem save point
_anomaly_zone_binder: set save marker: 242
_anomaly_zone_binder: SAVE DIF: 34
_anomaly_zone_binder: set save marker: 284
...

DIF изменяется в небольшом диапазоне, а вот save_marker сразу скачком. А сам размер сохранения, физический, практически не изменился...

Изменено 9 Октября 2015 пользователем Romz

Kirgudu · 9 Октября 2015

Ну так это se_actor, там свои данные пишутся. Сейчас тебя больше должно интересовать всё, что касается сохранения в bind_stalker, а там, как я смотрю, порядок. «SAVE DIF» показывает объём записанных за одно действие данных, а «set save marker» - результирующий объём нет-пакета. Вроде всё сходится.

Единственное, что слегка настораживает:

c:\gam\s-cop\se_stor:(165):save_storehouse:key[xr_statistic],type_key[string],type_chunk[boolean]~not saved:<Warning!>

Я бы на твоём месте глянул, что пишется в этом модуле и почему se_stor выдаёт это предупреждение. Не факт, что что-то критичное, но удостовериться в этом надо.

Romz · 9 Октября 2015

Я бы на твоём месте глянул, что пишется в этом модуле и почему se_stor выдаёт это предупреждение.

Собственно,

function save(package)
	package:w_u16(actor_statistic.surges)
	package:w_u16(actor_statistic.completed_quests)
	package:w_u32(actor_statistic.killed_monsters)
	package:w_u32(actor_statistic.killed_stalkers)
	package:w_u16(actor_statistic.founded_secrets)
	package:w_u16(actor_statistic.artefacts_founded)
	package:w_u32(actor_statistic.best_monster_rank)
	package:w_stringZ(tostring(actor_statistic.best_monster))
	package:w_stringZ(tostring(actor_statistic.favorite_weapon_sect))
	local i = 0
	for k,v in pairs(weapons_table) do
		i = i + 1
	end
	package:w_u8(i)
	for k,v in pairs(weapons_table) do
		package:w_stringZ(tostring(k))
		package:w_float(v)
	end
	i = 0
	for k,v in pairs(artefacts_table) do
		i = i + 1
	end
	package:w_u8(i)
	for k,v in pairs(artefacts_table) do
		package:w_stringZ(tostring(k))
		package:w_bool(v)
	end
	i = 0
	for k,v in pairs(taken_artefacts) do
		i = i + 1
	end
	package:w_u8(i)
	for k,v in pairs(taken_artefacts) do
		package:w_u32(k)
	end
end

function load(package)
	actor_statistic = {}
	actor_statistic.surges  			= package:r_u16()
	actor_statistic.completed_quests 	= package:r_u16()
	actor_statistic.killed_monsters  	= package:r_u32()
	actor_statistic.killed_stalkers  	= package:r_u32()
	actor_statistic.founded_secrets  	= package:r_u16()
	actor_statistic.artefacts_founded	= package:r_u16()
	actor_statistic.best_monster_rank 	= package:r_u32()
	local str = package:r_stringZ()
	if(str~="nil") then
		actor_statistic.best_monster = str
	end
	str = package:r_stringZ()
	if(str~="nil") then
		actor_statistic.favorite_weapon_sect = str
	end
	weapons_table = {}
	local n = package:r_u8()
	for i = 1,n do
		local k = package:r_stringZ()
		local v = package:r_float()
		weapons_table[k] = v
	end
	artefacts_table = {}
	n = package:r_u8()
	for i = 1,n do
		log("n "..tostring(n))
		local k = package:r_stringZ()
		local v = package:r_bool()
		artefacts_table[k] = v
	end

		taken_artefacts = {}
	n = package:r_u8()
	for i = 1,n do
		local k = package:r_u32()
		taken_artefacts[k] = k
	end
end

Кстати, в нём нет классов.

А именно это предупреждение в se_stor выдаётся, если не проходит условие

		if type(key) == 'string' and key ~= "" and type(ch_pk) == 'userdata' and type(ch_pk.r_eof) == 'function' then
...
...
		else
			to_log("save_storehouse:key[%s],type_key[%s],type_chunk[%s]~not saved:<%s>", key, type(key), type(ch_pk), "Warning!") --/#~#

а там, как видно, тип чанка, почему-то, получился boolean

Кстати, может быть из-за совпадения имени чанка с именем файла. Сейчас переименовал вот так

	tPkChunks["xr_stat"     ] = { file = "xr_statistic"    } --/ статистика

и, как результат

c:\gam\s-cop\se_stor:(159):save_storehouse:chunk_name=[xr_stat],chunk_size=[709],len=[721]=>[721]
c:\gam\s-cop\se_stor:(159):save_storehouse:chunk_name=[psy_antenna],chunk_size=[3],len=[19]=>[740]
c:\gam\s-cop\se_stor:(159):save_storehouse:chunk_name=[bind_actor],chunk_size=[80],len=[95]=>[835]
c:\gam\s-cop\se_stor:(237):~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
c:\gam\s-cop\se_stor:(238):Save Storehouse: size(1)=[845], keys=[1] : BuildSav=[-1]
c:\gam\s-cop\se_stor:(239):~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
_actor_binder: SAVE DIF: 8457
WARNING! may be this is problem save point

Изменено 9 Октября 2015 пользователем Romz

Kirgudu · 9 Октября 2015

Что странно, никаких предпосылок к этому я не вижу. Имя чанка никак не участвует в формировании технологического нет-пакета (только как ключ таблицы), а ошибка в логе говорит о том, что вместо нет-пакета во временную таблицу было записано значение типа boolean. Чего быть не может, судя по коду se_stor.
Ну да ладно.

Romz · 9 Октября 2015

Вроде, взлетело.

Вот так выглядит у меня сейчас

else --/ for CoP
	tPkChunks["psy_antenna" ] = { file = "sr_psy_antenna"  } --/ зоны пси-излучения
	tPkChunks["xr_stat"     ] = { file = "xr_statistic"    } --/ статистика
	tPkChunks["bind_actor"  ] = { file = "bind_stalker"    } --/ ...
end

и вот такой

	self.weather_manager:save(packet)
	release_body_manager.get_release_body_manager():save(packet)
	-- AtmosFear
	if atmosfear then
		self.atmosfear:save(packet)
		self.psi_storm_manager:save(packet)
		self.fallout_manager:save(packet)
        end
	-- End AtmosFear
	self.surge_manager:save(packet)
	xr_sound.actor_save(packet)
	treasure_manager.get_treasure_manager():save(packet)
	task_manager.get_task_manager():save(packet)
-----------------------------makdm_start-----------------------------
	if art_sp_mgr then
		art_sp_mgr.get_art_spawner_manager():save( packet )
	end	
-----------------------------makdm_end-------------------------------

То есть, присутствуют и вызовы из оригинала, и вызовы из модов.

Всё загружается и сохраняется. Скрипты вроде не слетели.

Да, и set_save_marker перестал ругаться на размер сохранения.

Изменено 9 Октября 2015 пользователем Romz

jenek96 · 9 Октября 2015

Всем привет. Столкнулся с таким логом :

Expression : assertion failed

Function : CUIXmlInit::InitText

File : D:\prog_repository\sources\trunk\xrGame\ui\UIXmlInit.cpp

Line : 315

Description : pTmpFont

Ребят подскажите как его исправить. Желательно в ЛС. :russian_ru:

Скриптование

Рекомендуемые сообщения

Kirgudu 947

Добавлено Murarius, 7 Октября 2015

Ссылка на комментарий

Romz 141

Ссылка на комментарий

Kirgudu 947

Ссылка на комментарий

Romz 141

Ссылка на комментарий

Kirgudu 947

Ссылка на комментарий

Romz 141

Ссылка на комментарий

abramcumner 1 023

Подарки

Ссылка на комментарий

Kirgudu 947

Ссылка на комментарий

Romz 141

Ссылка на комментарий

Kirgudu 947

Ссылка на комментарий

Romz 141

Ссылка на комментарий

Kirgudu 947

Ссылка на комментарий

Romz 141

Ссылка на комментарий

Kirgudu 947

Ссылка на комментарий

Romz 141

Ссылка на комментарий

Kirgudu 947

Ссылка на комментарий

Romz 141

Ссылка на комментарий

Kirgudu 947

Ссылка на комментарий

Romz 141

Ссылка на комментарий

jenek96 48

Ссылка на комментарий

Создайте аккаунт или авторизуйтесь, чтобы оставить комментарий

Создать аккаунт

Войти

Недавно просматривали 0 пользователей

Учимся моддингу

Играем в моды на ТЧ

Играем в моды на ЧН и ЗП

Полезное и разное

Вливаемся в сообщество

AMK-Team.ru

AMK Склад