慎用ToLower和ToUpper,小心把你的系统给拖垮了
不知道何时开始,很多程序员喜欢用tolower,toupper去实现忽略大小写模式的字符串相等性比较,有可能这个习惯是从别的语言引进的,大胆猜测下是js,为了不引起争论,我指的js是技师的意思~
一:背景
1. 讲故事
在我们一个订单聚合系统中,每一笔订单都会标注来源,比如jd,taobao,etao,shopex 等等一些渠道,ui上也提供高级配置输入自定义的订单来源,后来客户反馈输入xxx查询不出订单,这里就拿shopex为例,用户用小写的shopex查询,但系统中标注的是首字母大写的shopex,所以自然无法匹配,为了解决这个问题开发小哥就统一转成大写做比对,用代码表示如下:
var orderfrom = "shopex".toupper(); customeridlist = memoryorders.where(i =>i.orderfrom.toupper()==orderfrom) .select(i => i.customerid).tolist();
改完后就是这么牛的上线了,乍一看也没啥问题,结果一查询明显感觉比之前速度慢了好几秒,干脆多点几下,好咯。。。在监控中发现cpu和memory突高突低,异常波动,这位小哥又在写bug了,查了下代码问他为什么这么写,小哥说在js中就是这么比较的~~~
2. string.compare 改造
其实在c#中面对忽略大小写形式的比较是有专门的方法,性能高而且还不费内存,它就是 string.compare
,所以把上面代码改成如下就可以了。
var orderfrom = "shopex"; customeridlist = memoryorders.where(string.compare(i.tradefrom, tradefrom, stringcomparison.ordinalignorecase) == 0) .select(i => i.customerid).tolist();
这其中的 stringcomparison.ordinalignorecase
枚举就是用来忽略大小写的,上线之后除了cpu还是有点波动,其他都没有问题了。
二:为什么tolower,toupper会有如此大的影响
为了方便演示,我找了一篇英文小短文,然后通过查询某一个单词来演示toupper为啥对cpu和memory以及查询性能都有如此大的影响,代码如下:
public static void main(string[] args) { var strlist = "hooray! it's snowing! it's time to make a snowman.james runs out. he makes a big pile of snow. he puts a big snowball on top. he adds a scarf and a hat. he adds an orange for the nose. he adds coal for the eyes and buttons.in the evening, james opens the door. what does he see? the snowman is moving! james invites him in. the snowman has never been inside a house. he says hello to the cat. he plays with paper towels.a moment later, the snowman takes james's hand and goes out.they go up, up, up into the air! they are flying! what a wonderful night!the next morning, james jumps out of bed. he runs to the door.he wants to thank the snowman. but he's gone.".split(' '); var query = "snowman".toupper(); for (int i = 0; i < strlist.length; i++) { var str = strlist[i].toupper(); if (str == query) console.writeline(str); } console.readline(); }
1. 内存波动探究
既然内存有波动,说明内存里进了脏东西,学c#基础知识的时候应该知道string是不可变的,一旦有修改就会生成新的string,那就是说toupper之后会出现新的string,为了用数据佐证,用windbg演示一下。
0:000> !dumpheap -type system.string -stat statistics: mt count totalsize class name 00007ff8e7a9a120 1 24 system.collections.generic.genericequalitycomparer`1[[system.string, mscorlib]] 00007ff8e7a99e98 1 80 system.collections.generic.dictionary`2[[system.string, mscorlib],[system.globalization.culturedata, mscorlib]] 00007ff8e7a9a378 1 96 system.collections.generic.dictionary`2+entry[[system.string, mscorlib],[system.globalization.culturedata, mscorlib]][] 00007ff8e7a93200 19 2264 system.string[] 00007ff8e7a959c0 429 17894 system.string total 451 object
可以看到托管堆上有count=429
个string对象,那这个429怎么来的? 组成:短文128个,toupper后128个,系统默认165个,query字符串2个,不明字符串6个,最后就是128 +128 + 165 + 2 + 6=429
,然后随便抽几个看看。
!dumpheap -mt 00007ff8e7a959c0 > !dumpobj 000002244282a1f8
0:000> !dumpobj /d 0000017800008010 name: system.string methodtable: 00007ff8e7a959c0 eeclass: 00007ff8e7a72ec0 size: 38(0x26) bytes file: c:\windows\microsoft.net\assembly\gac_64\mscorlib\v4.0_4.0.0.0__b77a5c561934e089\mscorlib.dll string: house. fields: mt field offset type vt attr value name 00007ff8e7a985a0 4000281 8 system.int32 1 instance 6 m_stringlength 00007ff8e7a96838 4000282 c system.char 1 instance 48 m_firstchar 00007ff8e7a959c0 4000286 d8 system.string 0 shared static empty >> domain:value 0000017878943bb0:notinit << 0:000> !dumpobj /d 0000017800008248 name: system.string methodtable: 00007ff8e7a959c0 eeclass: 00007ff8e7a72ec0 size: 40(0x28) bytes file: c:\windows\microsoft.net\assembly\gac_64\mscorlib\v4.0_4.0.0.0__b77a5c561934e089\mscorlib.dll string: snowman fields: mt field offset type vt attr value name 00007ff8e7a985a0 4000281 8 system.int32 1 instance 7 m_stringlength 00007ff8e7a96838 4000282 c system.char 1 instance 53 m_firstchar 00007ff8e7a959c0 4000286 d8 system.string 0 shared static empty >> domain:value 0000017878943bb0:notinit <<
查了两个全是大写的“house”,“snowman”,再回到我的场景有小百万订单,也就会在托管堆上生成小百万个string,如果再点一次又会生成小百万个,内存怎么会不突增呢。。。
2.cpu和查询时间探究
现在大家知道了堆上可能有几百万个string对象,这些对象的分配和释放给cpu造成了不小的压力,本身toupper之后速度变慢,更惨的是还会造成gc颤抖式触发,一颤抖所有的thread都会被暂停开启回收,速度就更慢了。。。
三:string.compare解析
再回过头来看一下string.compare为什么这么