代码改造实录--从较宏观角度来考虑逻辑

原来的代码：

             if (makeAnAppointment.size() > 0) {

                // 总号数-待支付号数-已确定号数
                int make = plus.getPlusNumber() - makeAnAppointment.size() - plus.getMakeAnAppointment();
                for (MakeAnAppointment ma : makeAnAppointment) {
                    if (ma.getPatientId() == patientId) { // 存在个人未支付的号

                        long s1 = (System.currentTimeMillis() - ma.getCreateTime()) / 60000;
                        if (s1 <= 60L) {
                            if (make == 0) {
                                orderPlusVo.setStatus(1);
                                continue;
                            }
                            orderPlusVo.setStatus(make > 0 ? 1 : 0);
                            continue;
                        }
                        make++;
                        continue;
                    }
                    orderPlusVo.setStatus(make > 0 ? 1 : 0);
                }
            } else {
                orderPlusVo.setStatus(plus.getPlusNumber() - plus.getMakeAnAppointment() > 0 ? 1 : 0);
            }

改造后代码：

            // 总号数-待支付号数-已确定号数
            int make = plus.getPlusNumber() - plus.getMakeAnAppointment() - makeAnAppointment.size();
            if (make > 0) {
                orderPlusVo.setStatus(Globals.REMAINING_HAVE);
            } else {
                orderPlusVo.setStatus(Globals.REMAINING_NONE);
                for (MakeAnAppointment ma : makeAnAppointment) {
                    if (ma.getPatientId().equals(patientId)
                        && System.currentTimeMillis() <= ma.getCreateTime() + 1000 * 60 * 60) {
                        orderPlusVo.setStatus(Globals.REMAINING_HAVE);
                    }
                }
            }

这两段代码是实现个人是否可以预约号的功能，原来的代码先从预约列表的个数分0和大于0两种情况，在预约列表大于0的情况下再去查看是否含有个人未支付且支付未过期的预约信息，但逻辑混乱，让人读起来一头雾水。

下面的代码就清晰了很多，首先看是否还有未被预约的号，没有的情况下看是否含有自己预约未支付且在支付期限内的号，还没有就是真的没有了。