V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX 提问指南
tomoya92
V2EX  ›  问与答

这方法能不能提取成一个公共的方法,这样写很蛋疼。。

  •  
  •   tomoya92 · 2016-08-01 09:52:15 +08:00 · 2170 次点击
    这是一个创建于 3036 天前的主题,其中的信息可能已经有所发展或是发生改变。
    public class A {
      //是否使用缓存
      private boolean flag = true;
      public Object method(String param1, String param2) {//参数不固定
        Object o;
        if(flag) {
          B b = new B();//相当于缓存
          o = b.get("o");
          if(o == null) {
            o = C.get(param1, param2);//相当于从数据库里查数据
            b.set("o", o);//在放入缓存
          }
        } else {
          o = C.get(param1, param2);//直接查询
        }
        return o;
      }
    }
    

    大致意思就是把缓存做成可控制的,用一个 boolean 值来控制开关,程序里要做很多判断,请教有办法把这逻辑整理成一个公共的方法吗?每次调用一下公共方法传写参数即可,是否开启缓存都在公共方法里处理!

    10 条回复    2016-08-01 16:33:23 +08:00
    pH
        1
    pH  
       2016-08-01 11:15:54 +08:00
    把 flag 这个 property 去掉,直接作为函数参数不就可以了么?
    BuilderQiu
        2
    BuilderQiu  
       2016-08-01 11:42:20 +08:00
    一个方法用 Cache 一个方法不用,调用的时候根据需求区分不好么?
    tomoya92
        3
    tomoya92  
    OP
       2016-08-01 12:44:33 +08:00
    @pH 参数不定,调用的方法也不一定

    @BuilderQiu 很多地方用到了缓存,都是这样的写法每个用到的都写两份,还不如在一个方法里写了
    kfll
        4
    kfll  
       2016-08-01 12:57:42 +08:00
    可以写 annotation 吧
    TakWolf
        5
    TakWolf  
       2016-08-01 13:09:59 +08:00
    Object c = Cache.get(new GetFromDB {

    return Dat.find();

    }, new GetFromRedis {

    return Redis.find();

    });

    查询过程还是要自己写

    我随便瞎想的
    nigelvon
        6
    nigelvon  
       2016-08-01 13:25:52 +08:00
    为啥我看了好几遍也没看懂问题
    zealic
        7
    zealic  
       2016-08-01 14:39:54 +08:00
    AOP 不就干这事的么,关键在于把两个参数合并成一个缓存 key
    tomoya92
        8
    tomoya92  
    OP
       2016-08-01 15:28:12 +08:00
    @kfll annotation 是个好办法

    @TakWolf 让我想到了 jquery 里的 hover() 了

    @zealic AOP 确实是干这事的,但不知道怎么实现。。
    pH
        9
    pH  
       2016-08-01 16:01:34 +08:00
    @liygheart 参数不固定就用可变参数,然后把 flag 做第一个不行么
    fishioon
        10
    fishioon  
       2016-08-01 16:33:23 +08:00
    为啥不把 b 和 c 封装到一起呢?,同时将 flag 作为参数传递
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   2035 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 23ms · UTC 00:57 · PVG 08:57 · LAX 16:57 · JFK 19:57
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.