16进制描述的字符串转字符串效率问题

tylerpiece · 2017-04-12T15:37:41+00:00

多谢，明白了。On Wednesday, April 12, 2017 at 1:33:38 PM UTC+8, Walker Ma wrote:所有的sock:receiv...

16进制描述的字符串转字符串效率问题

tylerpiece

text是一串16进制表示的字符串，现在想把他转为普通字符串，代码如下。运行后，用火焰图抓，发现pcall那一句的CPU占用率很高

不知道有好的写法没有

local str_t ={}
    for var = 1,#text,2 do
        local hex = string.sub(text,var,var+1)
        local x = nil
        if pcall(function (hex) x = string.char(string.format("%d","0x"..hex)) end, hex) then
                str_t[#str_t + 1] = x
        else
            ngx.log(ngx.ERR,"uri args decrypt transform hex error")
            return
        end
    end

moonbingbing

可以把 Lua land 火焰图贴出来看看，用这个工具：https://github.com/openresty/stapxx#lj-lua-stacks

我猜测 lj_str_new 和 lj_gc_step_jit 可能比较高。如果你的字符串很长，rehashtab 和 lj_tab_resize 也会很高。

1、你可以提前对 text 的字符进行 format，放到一个数组里面预置好。而不是用 string.format("%d","0x"..hex) 重复的 format。

2、同理，提前 string.char 到一个数组里面预制好，避免不停的函数调用，以及生成大量的字符。

3、如果你使用的是 LuaJIT 2.1 beta2 的版本，你可以 table.new 来生成数组，而不是 str_t = {} 这种方式。

在 2017年4月12日下午3:37，tylerpiece <tyle...@163.com>写道：

text是一串16进制表示的字符串，现在想把他转为普通字符串，代码如下。运行后，用火焰图抓，发现pcall那一句的CPU占用率很高
不知道有好的写法没有


local str_t ={}
    for var = 1,#text,2 do
        local hex = string.sub(text,var,var+1)
        local x = nil
        if pcall(function (hex) x = string.char(string.format("%d","0x"..hex)) end, hex) then
                str_t[#str_t + 1] = x
        else
            ngx.log(ngx.ERR,"uri args decrypt transform hex error")
            return
        end
    end

--

tylerpiece

非常感谢。确实是lj_str_new占用了很多。我准备根据你的建议来尝试优化下代码。

在 2017-04-12 17:22:43，"Ming" <moonb...@gmail.com> 写道：

可以把 Lua land 火焰图贴出来看看，用这个工具：https://github.com/openresty/stapxx#lj-lua-stacks

我猜测 lj_str_new 和 lj_gc_step_jit 可能比较高。如果你的字符串很长，rehashtab 和 lj_tab_resize 也会很高。

1、你可以提前对 text 的字符进行 format，放到一个数组里面预置好。而不是用 string.format("%d","0x"..hex) 重复的 format。
2、同理，提前 string.char 到一个数组里面预制好，避免不停的函数调用，以及生成大量的字符。
3、如果你使用的是 LuaJIT 2.1 beta2 的版本，你可以 table.new 来生成数组，而不是 str_t = {} 这种方式。

在 2017年4月12日下午3:37，tylerpiece <tyle...@163.com>写道：
text是一串16进制表示的字符串，现在想把他转为普通字符串，代码如下。运行后，用火焰图抓，发现pcall那一句的CPU占用率很高
不知道有好的写法没有


local str_t ={}
    for var = 1,#text,2 do
        local hex = string.sub(text,var,var+1)
        local x = nil
        if pcall(function (hex) x = string.char(string.format("%d","0x"..hex)) end, hex) then
                str_t[#str_t + 1] = x
        else
            ngx.log(ngx.ERR,"uri args decrypt transform hex error")
            return
        end
    end

--

--

Attachment: reg.svg
Description: image/svg