记一次BUG:深拷贝与浅拷贝(Java)

2019-02-25  本文已影响25人  Java酸不酸

记录在近期工作中的一个BUG。需求:从所有邮件模板中,筛选出符合的邮件模板,根据不同的数据,将模板中特定的“字眼”替换为具体的值。这里只针对BUG相关的问题,不涉及与业务相关。

深拷贝与浅拷贝的概念

深拷贝
浅拷贝
代码解读
// 拿到所有的模板
List<Template> templates = templateDao.getTemplateList();

// 筛选
Template template = templates.stream()
    .filter(t -> "PVC".equals(t.getEvent()) && "EN".equals(t.getLanguage()))
    .findFirst()
    .orElse(null);
if (template != null) {
    // 替换模板中定好的 ”字眼“
    // do something
}

不难看出,这段代码在方法中,如果单次调用是没有问题的;如果在一个循环中,而且代码中的Language是通过不同的情况取得,在运行中就会出现问题,代码如下:

// 拿到所有的模板
List<Template> templates = templateDao.getTemplateList();

for (String lang : languages) {
    // 筛选
    Template template = templates.stream()
        .filter(t -> Objects.equals(t.getEvent(), "PVC") && Objects.equals(lang, t.getLanguage()))
        .findFirst()
        .orElse(null);
    if (template != null) {
        // 替换模板中定好的 ”字眼“
        // do something
    }
}

此段代码,首先我改变过滤条件的写法,防止出现空指针异常。分析下这段代码:如果language相同,那么只有第一次才会替换字眼成功,因为第一次替换之后templates集合中的对应元素也会被替换掉,之后出现相同的语言,过滤出来的template中的字眼已经是被替换掉了,所有会出现相同的数据。这也是这个问题的关键点,说明此处是浅拷贝,只是拷贝对象的引用,引发的BUG。对此,这里提供三种参考方式:

方式一:
方式二:
@SuppressWarnings("unchecked")
public static Object deepCopy(Object src) throws IOException, ClassNotFoundException{
    test(src);
    test1(src);
    if (src == null) {
        return null;
    }
    ByteArrayOutputStream bos = new ByteArrayOutputStream();
    ObjectOutputStream out = new ObjectOutputStream(bos);
    out.writeObject(src);
    ByteArrayInputStream bis = new ByteArrayInputStream(bos.toByteArray());
    ObjectInputStream in =new ObjectInputStream(bis);
    return in.readObject();
}
方式三:
Template temp = SerializationUtils.clone(template);
通过第三种方式修改后,整体代码如下:
// 拿到所有的模板
List<Template> templates = templateDao.getTemplateList();

for (String lang : languages) {
    // 筛选
    Template template = templates.stream()
        .filter(t -> Objects.equals(t.getEvent(), "PVC") && Objects.equals(lang, t.getLanguage()))
        .findFirst()
        .orElse(null);
    if (template != null) {
        Template temp = SerializationUtils.clone(template);
        // 替换模板中定好的 ”字眼“
        // do something
    }
}

若有不对之处,请指点一二,谢谢。
记录在工作中的BUG,不断完善自我,加油!

上一篇 下一篇

猜你喜欢

热点阅读