记一次关于参数过长的代码优化

背景:
现在有一个POJO作为HTTP Controller参数以及Service参数。
ControllerRESTFul风格
具体如下:

@RequestMapping(value = "xxxxx/apps/{app_id}/groups/{group_id}/faces/{face_id}", method = RequestMethod.POST)
    public ResultMsg add(@PathVariable("face_id") String faceId,
                         @PathVariable("app_id") String appId,
                         @PathVariable("group_id") String groupId,
                         @RequestBody FaceRequestBody faceBody
    ) {
        Face face = new Face(faceId,groupId,appId,faceBody.getName(),faceBody.getImage(),faceBody.getRemark(),false);
        return faceService(face);
    }

可以看到Service层由于特殊原因需要一个POJOFace,Face包含7个属性,并且每个属性都是必须的。

public class Face {

    @JSONField(name = "face_id")
    private String id;

    @JSONField(name = "face_name")
    private String name;


    private String image;

    @JSONField(name = "face_remark")
    private String remark;


    @JSONField(serialize = false)
    private boolean obsolete;

    @JSONField(name = "group_id")
    private String groupId;

    @JSONField(name = "app_id")
    private String appId;
     
    //setter & getter
}

Face 的构造器参数有7个,基本每个都是String类型,这样及其不方便并且容易出错,想着用建造者模式优化。


思路:可以看见,由于没有可选参数,于是传统的建造者模式是不适用于现在的场景,但是我们可以借鉴建造者模式的返回this的“连点”技巧。

  1. 分析7个参数,从应用场景来说,7个参数可以分为3类,````id类,info类,obsolete```类,因此我们可以创建3个嵌套类。
  2. 如果直接暴力拆分为3个类,代码依然不美观,每次新建一个Face类都要new其他3个类。
    Infos infos=new Infos(...);
    Ids id=new Ids(...);
    Obsolete obsolete=new Obsolete(...);
    Face face = new Face(infos,id,obsolete);
    
    
  3. 正如最开始所说,借鉴建造者模式的“连点”语法技巧。
    /**
     * Face 构造器
     * 由于Face参数过多,并且每个参数都是必须参数。
     * 此嵌套类可以将相关属性分开构造,防止因为参数过多传错参数
     * 使用方法如下:
     * Face face = Face.buildIds(faceId,groupId,appId).
     *                  buildFaceInfo(name,image,remark).
     *                  buildObsolete(false);
     */
     
    //第一层,返回下一层的新对象
    public static FaceInfoBuilder buildIds(String id, String groupId, String appId) {
        return new FaceInfoBuilder(id, groupId, appId);
    }

    public static final class FaceInfoBuilder {

        private String id;
        private String groupId;
        private String appId;

        private FaceInfoBuilder(String id, String groupId, String appId) {
            this.id = id;
            this.groupId = groupId;
            this.appId = appId;
        }
        //第二层,返回第三层的对象
        public FaceObsoleteBuilder buildFaceInfo(String name, String image, String remark) {
            return new FaceObsoleteBuilder(id, groupId, appId, name, image, remark);
        }

        public static final class FaceObsoleteBuilder {
            private String id;
            private String groupId;
            private String appId;
            private String image;
            private String name;
            private String remark;

            private FaceObsoleteBuilder(String id, String groupId, String appId, String image, String name, String remark) {
                this.id = id;
                this.groupId = groupId;
                this.appId = appId;
                this.image = image;
                this.name = name;
                this.remark = remark;
            }
            //最后一层,返回Fac对象
            public Face buildObsolete(boolean obsolete) {
                return new Face(id, groupId, appId, name, image, remark, obsolete);
            }
        }

    }

核心思想:使用3个嵌套类,每一层返回下一层的嵌套类,同时将构造函数设为private类型,防止外部实例化。

优化过后新建一个Face对象如下:

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

推荐阅读更多精彩内容

  • Spring Cloud为开发人员提供了快速构建分布式系统中一些常见模式的工具(例如配置管理,服务发现,断路器,智...
    卡卡罗2017阅读 134,654评论 18 139
  • 一、编程规约 (一)命名规约 【强制】 代码中的命名均不能以下划线或美元符号开始,也不能以下划线或美元符号结束。反...
    喝咖啡的蚂蚁阅读 1,504评论 0 2
  • “你演技那么好,谁看得穿。” 这句话适合贬义一个人渣。 当然,用在我自己身上也刚刚好。 “坦诚一点”,是许多爱过我...
    耻辱的一生阅读 531评论 0 2
  • 生活不是电视剧,不是小说,不是童话。灰姑娘的故事固然浪漫,但如果鞋真的合脚就不会掉。生活中其实没有那么多的两情相悦...
    啊哈哈哈禾阅读 1,743评论 0 2