本来是要修复前一个代码bug,修复的过程中发现原本的代码又丑又长,复用性差(但是能用),出于强迫症忍不住的去优化,测试还不充分,火急火燎的发到生产了,结果掉井了!导致多个订单线下物流发货发多了.... 万一有个别用户不管订单数量是不是自己下单的,直接签收了,再往回要就难了,那时还要加上来回运费。

当时提桶跑路的心都有/(ㄒoㄒ)/~~,但这个东西逃不掉,而且问题越拖越严重。这个时候就不能再去顾虑别人对自己的看法了,事情已经这样了,可奈何 ?不如主动承认错误,尽早解决降低损失。

最后硬着头皮向领导说明了情况,然后联系协助各方人员尽量挽救,将损失降到最低。

整个过程真是自惭形秽,无地自容,恨不能找个地缝钻进去 ...

事后总结:

当时因为需要构建好多场景,嫌麻烦,加上蜜汁自信,测试不充分。

主观上是因为自己对于代码失去了敬畏之心,随意修改,不考虑或者不想考虑如果错了会造成什么样的后果。

对于代码最好的优化就是能跑通就不要优化。如要修改,选择对于已有代码影响最小的方案。

下面模拟这个问题

准备

目的:将集合中重复的元素合并起来

比如:将下面列表 bigHouseList 中相同房屋主人 ownerName 的房屋价值 worth 累加起来

BigHouse house1 = new BigHouse("张三", "别墅", new BigDecimal("2400"), "想不到路-10号");
BigHouse house2 = new BigHouse("赵柳", "公寓", new BigDecimal("120"), "现实路30号");
BigHouse house3 = new BigHouse("李四", "别墅", new BigDecimal("780"), "你别想路-101号");
BigHouse house4 = new BigHouse("王五", "小洋楼", new BigDecimal("8000"), "梦不到路-1000号");
BigHouse house5 = new BigHouse("赵柳", "别墅", new BigDecimal("3500"), "想不到路-28号");
BigHouse house6 = new BigHouse("王五", "别墅", new BigDecimal("4200"), "想不到路-130号"); List<BigHouse> bigHouseList = new ArrayList<>();
bigHouseList.add(house1);
bigHouseList.add(house2);
bigHouseList.add(house3);
bigHouseList.add(house4);
bigHouseList.add(house5);
bigHouseList.add(house6);

**大房子类 BigHouse **

@Data
public class BigHouse {
/**
* 房屋主人名字
*/
private String ownerName; /**
* 房屋类型
*/
private String type; /**
* 房屋价值(单位万)
*/
private BigDecimal worth; /**
* 详细地址
*/
private String address; public BigHouse() {
} public BigHouse(String ownerName, String type, BigDecimal worth, String address) {
this.ownerName = ownerName;
this.type = type;
this.worth = worth;
this.address = address;
}
}

优化前的代码

优化理由:这部分代码逻辑和业务无关,可以抽离出来形成一个公共的方法函数

//region 优化前的代码
List<BigHouse> uniqueOwnerHouseList = new ArrayList<>();
boolean isSameOwner = true;
for (BigHouse house : bigHouseList) {
isSameOwner = Boolean.FALSE;
if (uniqueOwnerHouseList.size() > 0) {
for (BigHouse h : uniqueOwnerHouseList) {
if (h.getOwnerName().equals(house.getOwnerName())) {
h.setWorth(h.getWorth().add(house.getWorth()).setScale(0, BigDecimal.ROUND_HALF_UP));
isSameOwner = Boolean.TRUE;
}
}
}
if (!isSameOwner) {
uniqueOwnerHouseList.add(house);
}
}
//endregion System.out.println("优化前结果输出:");
//优化后结果输出
for (BigHouse bigHouse : uniqueOwnerHouseList) {
System.out.println(bigHouse.getOwnerName() + "坐拥 " + bigHouse.getWorth() + " 万房产!");
}

输出结果:

优化前结果输出:
张三坐拥 2400 万房产!
赵柳坐拥 3620 万房产!
李四坐拥 780 万房产!
王五坐拥 12200 万房产!

优化后的代码(bug)

/**
* 合并集合中重复的元素
*
* @param items
* @param predicate 判断是否重复
* @param consumer 重复后处理
* @param <T>
* @return
*/
public static <T> List<T> mergeDuplicate(Collection<T> items, BiPredicate<T, T> predicate, BiConsumer<T, T> consumer) {
List<T> mergeItems = new ArrayList<>();
for (T item : items) {
boolean existSameItem = false;
for (T mergeItem : mergeItems) {
existSameItem = predicate.test(mergeItem, item);
if (existSameItem) {
consumer.accept(mergeItem, item);
}
}
if (!existSameItem) {
mergeItems.add(item);
}
}
return mergeItems;
} //优化后的代码
List<BigHouse> mergedBigHouseList = CollectionTools.mergeDuplicate(bigHouseList,
(p, n) -> p.getOwnerName().equals(n.getOwnerName()),
(p, n) -> {
p.setWorth(p.getWorth().add(n.getWorth()).setScale(0, BigDecimal.ROUND_HALF_UP));
}
); System.out.println("优化后结果输出:");
for (BigHouse bigHouse : mergedBigHouseList) {
System.out.println(bigHouse.getOwnerName() + "坐拥 " + bigHouse.getWorth() + " 万房产!");
}

输出结果:

优化前结果输出:
张三坐拥 2400 万房产!
赵柳坐拥 3620 万房产!
李四坐拥 780 万房产!
王五坐拥 12200 万房产!
赵柳坐拥 3500 万房产!
王五坐拥 4200 万房产!

逻辑分析

原因出在这块代码上

for (T mergeItem : mergeItems) {
existSameItem = predicate.test(mergeItem, item);
if (existSameItem) {
consumer.accept(mergeItem, item);
}
}

在 if 判断true后,应该break跳出循环的,否则 existSameItem 的值会被 mergeItems 剩余元素的比较结果覆盖掉,从而影响循环外的 if 判断

if (!existSameItem) {
mergeItems.add(item);
}

正确的写法:

for (T mergeItem : mergeItems) {
existSameItem = predicate.test(mergeItem, item);
if (existSameItem) {
consumer.accept(mergeItem, item);
break;
}
}

或者(不建议)

for (T mergeItem : mergeItems) {
if (predicate.test(mergeItem, item);) {
consumer.accept(mergeItem, item);
existSameItem = true;
}
}

【生产事故调查】优化出来的bug-合并集合重复项的更多相关文章

  1. sql分组合并字段重复项sql for xml path

    -------------------------(情景描述) 在我们处理数据时,可能会碰到这种情景: Id                Name 1                  a,b 2  ...

  2. 一次 select for update 的悲观锁使用引发的生产事故

    1.事故描述 本月 8 日上午十点多,我们的基础应用发生生产事故.具体表象为系统出现假死无响应.查看事发时间段的基础应用 error 日志,没发现明显异常.查看基础应用业务日志,银行结果处理的部分普遍 ...

  3. (转)CentOS(5.8/6.4)linux生产环境若干优化实战

    CentOS(5.8/6.4)linux生产环境若干优化实战 原文:http://blog.51cto.com/oldboy/1336488 特别说明:本文来自老男孩linux培训VIP学生学习笔记. ...

  4. 生产事故(MongoDB数据分布不均解决方案)

    可以很明显可以看到我们这个集合的数据严重分布不均匀. 一共有8个分片,面对这个情况我首先想到的是手动拆分数据块,但这不是解决此问题的根本办法. 造成此次生产事故的首要原因就是片键选择上的问题,由于片键 ...

  5. 算法练习之合并两个有序链表, 删除排序数组中的重复项,移除元素,实现strStr(),搜索插入位置,无重复字符的最长子串

    最近在学习java,但是对于数据操作那部分还是不熟悉 因此决定找几个简单的算法写,用php和java分别实现 1.合并两个有序链表 将两个有序链表合并为一个新的有序链表并返回.新链表是通过拼接给定的两 ...

  6. [算法]合并链表&删除数组重复项

    合并链表 题目 将两个有序链表合并为一个新的有序链表并返回.新链表是通过拼接给定的两个链表的所有节点组成的. 示例: 输入:1->2->4, 1->3->4 输出:1-> ...

  7. 日常Bug排查-Nginx重复请求?

    日常Bug排查-Nginx重复请求? 前言 日常Bug排查系列都是一些简单Bug排查,笔者将在这里介绍一些排查Bug的简单技巧,其中不乏一些看起来很低级但很容易犯的问题. 问题现场 有一天运维突然找到 ...

  8. 记一次生产事故的排查与优化——Java服务假死

    一.现象 在服务器上通过curl命令调用一个Java服务的查询接口,半天没有任何响应.关于该服务的基本功能如下: 1.该服务是一个后台刷新指示器的服务,即该服务会将用户需要的指示器数据提前计算好,放入 ...

  9. 优化RequireJS项目(合并与压缩)

    关于RequireJS已经有很多文章介绍过了.这个工具可以将你的JavaScript代码轻易的分割成苦 干个模块(module)并且保持你的代码模块化与易维护性.这样,你将获得一些具有互相依赖关系的J ...

随机推荐

  1. 从零开始,开发一个 Web Office 套件(13):删除、替换已选中文字

    这是一个系列博客,最终目的是要做一个基于 HTML Canvas 的.类似于微软 Office 的 Web Office 套件(包括:文档.表格.幻灯片--等等). 博客园:<从零开始, 开发一 ...

  2. C++获取设备 PID,VID 信息

    可直接编译(设置成:使用多字节字符集) 转来的,代码: /* http://www.experts-exchange.com/Programming/Editors_IDEs/Q_24506125.h ...

  3. nacos底层原理

    Nacos   为什么选择NacosNacos 致力于帮助您发现.配置和管理微服务.Nacos 提供了一组简单易用的特性集,帮助您快速实现动态服务发现.服务配置.服务元数据及流量管理. Nacos 帮 ...

  4. mac下的phpstorm增加xdebug调试

    zhuan:https://blog.csdn.net/gd_9988/article/details/106248573 tips:不确定xdebug版本的,把phpinfo()的网页源代码全部复制 ...

  5. Spring-boot-菜鸟-properties配置

    如果测试结果出现中文乱码: 结果": Person{lastName='张胜男', age=11, boss=false, birth=Wed Nov 11 00:00:00 CST 202 ...

  6. 面试问题之操作系统:Linux下进程的内存结构

    转载于:http://www.hqj.com/news/emb184.htm Linux操作系统采用虚拟内存管理技术,使得每个进程都有各自互不干涉的进程地址空间.该地址空间是大小为4GB的线性虚拟空间 ...

  7. 简述 Memcached 内存管理机制原理?

    早期的 Memcached 内存管理方式是通过 malloc 的分配的内存,使用完后通过 free 来回收内存,这种方式容易产生内存碎片,并降低操作系统对内存的管理效 率.加重操作系统内存管理器的负担 ...

  8. synchronized、volatile、CAS 比较?

    1.synchronized 是悲观锁,属于抢占式,会引起其他线程阻塞. 2.volatile 提供多线程共享变量可见性和禁止指令重排序优化. 3.CAS 是基于冲突检测的乐观锁(非阻塞)

  9. Netty学习摘记 —— Netty客户端 / 服务端概览

    本文参考 本篇文章是对<Netty In Action>一书第二章"你的第一款 Netty 应用程序"的学习摘记,主要内容为编写 Echo 服务器和客户端 第一款应用程 ...

  10. 如何解决用response输出字符流数据时的乱码问题

    protected void doPost(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOE ...