Map_ThreadLocal的思考

入参、出参应该尽可能禁用万能字段(如map)和共享内存(如threadlocal)。 

参照现有FB系统,总结出缺点如下:
1、引发逻辑混乱。参数的自由度过高,使得多人开发时,无法进行约束。长期开发后,会积累大量,输入输出不明的逻辑,并导致最终实现与原有设计不符。
2、阻碍单元测试。所有模块严重依赖上下文,难以独立拿出测试。设计测试用例时,难以准确构造完整的入参。
3、阻碍重构、优化。已有实现与设计严重不符,且难以梳理。大量无关数据一同进行传输,浪费性能。
具体缺点表现:
1、FB现有代码中,PUnit之间使用threadlocal共用数据,针对threadlocal的各种get/set操作散落代码各处,并存在大量无效逻辑。
2、FB自动化测试工具,迟迟没有进展,目前只能模拟对XXXXserver进行数个类别的新增发帖提交请求。真正需求开发时,完全靠人工进行功能测试,难以发现间接引发的各种bug。
3、FB拆分的XXXXcheck项目中,因无法理清XZ校验各个模块的有效出、入参数,沿用了threadlocal方式,并将原有XXXXserver中后步骤下的完整threadlocal数据,作为XXXXcheck的入参(较原有XXXXserver入参更为庞大),进而使得FB拆分后,性能严重恶化。
设计的成因推测:
1、设计者无法在设计初期,严格定义模块的功能与参数。
2、开发过程中,无法容忍频繁的参数变更。
3、一旦采用了万能字段、共享内存的设计,后期不但难以清除,还会诱发更多的类似设计。
制止的必要性分析:
1、单元可测,才能确保真正的快速开发,及系统稳定。
2、高内聚、低耦合是先进设计理念根本。再优秀的设计形式,在大量运用万能字段、共享内存后,都会形同虚设。
3、万能字段、共享内存本质上,是贪图早期开发的灵活多变,寄希望于后期重构的设计。
改进建议:
1、主管应该明确禁止,不明确的各种数据传输设计。
2、开发过程中,主管应给予足够的参数变量容忍度。
3、阶段性代码审查,并制止万能字段、共享内存的设计、实现。

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 216,496评论 6 501
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 92,407评论 3 392
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 162,632评论 0 353
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 58,180评论 1 292
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 67,198评论 6 388
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 51,165评论 1 299
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 40,052评论 3 418
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 38,910评论 0 274
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 45,324评论 1 310
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 37,542评论 2 332
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 39,711评论 1 348
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 35,424评论 5 343
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 41,017评论 3 326
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 31,668评论 0 22
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,823评论 1 269
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 47,722评论 2 368
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 44,611评论 2 353

推荐阅读更多精彩内容

  • Android 自定义View的各种姿势1 Activity的显示之ViewRootImpl详解 Activity...
    passiontim阅读 172,074评论 25 707
  • 从三月份找实习到现在,面了一些公司,挂了不少,但最终还是拿到小米、百度、阿里、京东、新浪、CVTE、乐视家的研发岗...
    时芥蓝阅读 42,240评论 11 349
  • Spring Cloud为开发人员提供了快速构建分布式系统中一些常见模式的工具(例如配置管理,服务发现,断路器,智...
    卡卡罗2017阅读 134,651评论 18 139
  • 项目的上线,意味着一个阶段的终结,就算前期准备再充分,也无法保证最终100%的没问题。 下面是我本次项目上线中遇到...
    Skype2阅读 729评论 0 0
  • 追随和失去的,这是寒假中一个题目,一直没写,总感觉过了彼时,好像再难拾起。可又怎能不写?至今犹记得那一天,淡淡地哀...
    大图图阅读 313评论 0 0