2018-09-16开发笔记

Sonar烦恼

  今天把一个业务项目上传到了SonarQube,提示代码重复率7.5%,默认的标准是5%,超标了。

  本来因为只是个用作数据收集的新项目,使用到的表比较少,而表的基本字段有重复,使用了Spring-boot-data-jpa生成的model对象属性便有部分重复。

  本来是个小事情,置之不理就好,但是。。。

  尝试使用extends对model进行继承结构梳理,A extends B,尝试编译,报错:【子类builder不能覆盖父类】。因为在model上使用了lombok的builder注解,创建的builder方法不能覆盖。

  由于builder结构的便利性,在业务代码中多次使用了类似A.builder().aaa().bbb().build()这种写法,不忍舍弃,尝试google解决,得到@SuperBuilder注解,在最新的lombok18上有提供,赶紧更改依赖包版本,更改注解测试,按文档说明父类子类都使用@SuperBuilder标注,测试方法中尝试B.builder(),未能获得。使用compile编译,观察生成class文件,的确已经存在使用了泛型并且支持继承关系的builder对象和builder()方法,为什么不成功呢?

  猜测是lombok插件的问题,打开idea的插件管理器,查看lombok plugin支持注解列表说明,果然没有@SuperBuilder,看来只支持到1.16的lombok版本。尝试双击lombok.jar安装插件,发现只支持eclipse核心ide,想让我换开发工具?不存在的~ 算了放弃放弃。

  尝试@Accessors注解,加上chain=true之后,生成的setter方法会将自身返回,看起来很好用,然而子类的示例,调用了父类属性的setter之后,返回的对象就变成父类了,需要声明下类型转换,略麻烦。

  还是回到@Builder注解做文章,观察注解属性列表,发现builder方法名、类名、build方法名,都是可以更改的!尝试加上(builderMethodName = "aesbuilder")进行编译,通过了!观察生成的builder方法,已然变成了aesbuilder(),very Good。

  然而,同时也发现了一个严重问题,builder返回的Builder类,支持的属性设置,只有子类的,而继承自父类的属性并不在里面。而事实上,@SuperBuilder正是用来解决这一问题的,然而并没有可用的idea插件。。。

  google发现了一个大神给的方案,@Builder声明在构造方法上,而构造方法参数里,把需要用到的属性逐个声明一遍,虽然略笨重,不过看似可行,于是改造代码为

    @Builder(builderMethodName = "aesbuilder")
    public ActivityEventStatistic(final Long id, final String pageCode, final String eventCode, final Integer pv, final Integer uv, final Integer timeHour,final String activityCode){
        super(id, pageCode, eventCode, pv, uv, timeHour);
        this.activityCode = activityCode;
    }

    public static void main(String[] args) {
        ActivityEventStatistic.aesbuilder().id(123l).pageCode("test").build();
    }

it works! wow~

  OK,编译通过,进入测试阶段,test case跑起来,然后duang~ 又遇到问题了!提示执行的查询sql,属性未找到。分析之后发现,由于model使用了继承,而没有显示声明Inheritance策略,所以jpa使用了默认的策略SINGLE_TABLE,也就是父类子类公用了一张表,再加上并没有声明哪些属性属于子类,所以就报错了。

  另外,很显然,我这种情况并不能使用SINGLE_TABLE,而应该使用TABLE_PER_CLASS。ok,在model上加上@Inheritance(strategy = InheritanceType.TABLE_PER_CLASS),再次run test,报错如下:

nested exception is org.hibernate.MappingException: Cannot use identity column key generation with <union-subclass> mapping for: com.quzeng.idss.domain.PageStatistic

  查找问题后得知,在model上使用了继承关系后,就不能使用自动主键策略了,而只能用GenerationType.TABLE才可以。。。吐血。。。

  ok,创建表hibernate_sequences,添加字段sequence_name,next_val,尝试run test,总算成功了。。。

  但是,面临的问题还是有的,首当其冲的是,GenerationType.TABLE被人诟病因为每次请求时的锁机制造成的性能严重影响的问题,参考:why-you-should-never-use-the-table-identifier-generator-with-jpa-and-hibernate,另外一个,是我要对每个表创建ID生成器,指定序列器名称、范围,以及hibernate_sequences 表中要添加好相应记录,还要将这些改动脚本整理下来,告知DBA,说是因为一个蛋疼的重复率指标导致的主键策略改动,还不知道会不会被拒绝掉。。。

  哎,你说我瞎折腾什么呢。👀

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

推荐阅读更多精彩内容