Add a table memoizer to optimize things

This commit is contained in:
JP Guerrero 2018-03-08 17:49:07 +01:00
parent 2df3f9c02f
commit f1e58084d1

@ -1,5 +1,5 @@
local craftguide, datas, mt = {}, {}, minetest local craftguide, datas, mt = {}, {searches = {}}, minetest
local progressive_mode = mt.setting_getbool("craftguide_progressive_mode") local progressive_mode = mt.settings:get_bool("craftguide_progressive_mode")
local get_recipe, get_recipes = mt.get_craft_recipe, mt.get_all_craft_recipes local get_recipe, get_recipes = mt.get_craft_recipe, mt.get_all_craft_recipes
local get_result, show_formspec = mt.get_craft_result, mt.show_formspec local get_result, show_formspec = mt.get_craft_result, mt.show_formspec
local reg_items = mt.registered_items local reg_items = mt.registered_items
@ -9,6 +9,8 @@ local reg_items = mt.registered_items
local remove, maxn, sort = table.remove, table.maxn, table.sort local remove, maxn, sort = table.remove, table.maxn, table.sort
local min, max, floor, ceil = math.min, math.max, math.floor, math.ceil local min, max, floor, ceil = math.min, math.max, math.floor, math.ceil
local DEFAULT_SIZE, MIN_LIMIT, MAX_LIMIT = 10, 9, 12
local group_stereotypes = { local group_stereotypes = {
wool = "wool:white", wool = "wool:white",
dye = "dye:white", dye = "dye:white",
@ -34,6 +36,7 @@ function craftguide:group_to_item(item)
end end
end end
end end
return item:sub(1,6) == "group:" and "" or item return item:sub(1,6) == "group:" and "" or item
end end
@ -156,6 +159,7 @@ function craftguide:get_formspec(player_name, is_fuel)
if not data.items then if not data.items then
data.items = datas.init_items data.items = datas.init_items
end end
data.pagemax = max(1, ceil(#data.items / ipp)) data.pagemax = max(1, ceil(#data.items / ipp))
local formspec = "size[" .. (data.iX - 0.35) .. "," .. (iY + 4) .. ";]" .. [[ local formspec = "size[" .. (data.iX - 0.35) .. "," .. (iY + 4) .. ";]" .. [[
@ -201,12 +205,13 @@ function craftguide:get_formspec(player_name, is_fuel)
local tooltip = self:get_tooltip(data.item) local tooltip = self:get_tooltip(data.item)
if not data.recipes_item or (is_fuel and not get_recipe(data.item).items) then if not data.recipes_item or (is_fuel and not get_recipe(data.item).items) then
formspec = formspec .. formspec = formspec ..
"image[" .. (xoffset - 1) .. "," .. (iY + 2) .. "image[" .. (xoffset - 1) .. "," .. (iY + 2.35) ..
".12;0.9,0.7;craftguide_arrow.png]" .. ".12;0.9,0.7;craftguide_arrow.png]" ..
"item_image_button[" .. xoffset .. "," .. (iY + 2) .. "item_image_button[" .. xoffset .. "," .. (iY + 2.2) ..
";1,1;" .. data.item .. ";" .. data.item .. ";]" .. ";1,1;" .. data.item .. ";" .. data.item .. ";]" ..
tooltip .. "image[" .. (xoffset - 2) .. "," .. tooltip ..
(iY + 1.98) .. ";1,1;craftguide_fire.png]" "image[" .. (xoffset - 2) .. "," ..
(iY + 2.18) .. ";1,1;craftguide_fire.png]"
else else
formspec = formspec .. formspec = formspec ..
self:get_recipe(iY, xoffset, tooltip, data.item, self:get_recipe(iY, xoffset, tooltip, data.item,
@ -292,6 +297,11 @@ end
function craftguide:get_filter_items(data, player) function craftguide:get_filter_items(data, player)
local filter = data.filter local filter = data.filter
if datas.searches[filter] then
data.items = datas.searches[filter]
return
end
local items_list = progressive_mode and data.init_filter_items or datas.init_items local items_list = progressive_mode and data.init_filter_items or datas.init_items
local inv = player:get_inventory() local inv = player:get_inventory()
local filtered_list, counter = {}, 0 local filtered_list, counter = {}, 0
@ -314,9 +324,14 @@ function craftguide:get_filter_items(data, player)
end end
end end
if progressive_mode and not data.items then if progressive_mode then
if not data.items then
data.init_filter_items = filtered_list data.init_filter_items = filtered_list
end end
elseif filter ~= "" and not datas.searches[filter] then
datas.searches[filter] = filtered_list
end
data.items = filtered_list data.items = filtered_list
end end
@ -351,8 +366,8 @@ mt.register_on_player_receive_fields(function(player, formname, fields)
end end
craftguide:get_formspec(player_name) craftguide:get_formspec(player_name)
elseif (fields.size_inc and data.iX < 12) or elseif (fields.size_inc and data.iX < MAX_LIMIT) or
(fields.size_dec and data.iX > 9) then (fields.size_dec and data.iX > MIN_LIMIT) then
data.pagenum = 1 data.pagenum = 1
data.iX = data.iX - (fields.size_dec and 1 or -1) data.iX = data.iX - (fields.size_dec and 1 or -1)
craftguide:get_formspec(player_name) craftguide:get_formspec(player_name)
@ -401,7 +416,7 @@ function craftguide:on_use(itemstack, user)
local data = datas[player_name] local data = datas[player_name]
if progressive_mode or not data then if progressive_mode or not data then
datas[player_name] = {filter = "", pagenum = 1, iX = 10} datas[player_name] = {filter = "", pagenum = 1, iX = DEFAULT_SIZE}
if progressive_mode then if progressive_mode then
craftguide:get_filter_items(datas[player_name], user) craftguide:get_filter_items(datas[player_name], user)
end end