问题记录

清疚 2022-06-15 11:18 300阅读 0赞

前置条件

Map<Long, List<Long>> noteAndAttachmentIdMap = new HashMap<Long, List<Long>>();
       Map<Long, ContactNoteAttachment> attachmentMap = new HashMap<Long, ContactNoteAttachment>();

问题代码

for (ContactNote contactNote : contactNotes) {
                List<Long> attachIdList = noteAndAttachmentIdMap.get(contactNote.getId());
                if (CollectionUtils.isNotEmpty(attachIdList)) {
                    contactNoteAttachments = new ArrayList<ContactNoteAttachment>();
                    for (Long id : attachIdList) {
                        contactNoteAttachments.add(attachmentMap.get(id));
                    }
    
                }
                contactNote.setAttachmentList(contactNoteAttachments);
    
            }

修正后的代码

for (ContactNote contactNote : contactNotes) {
                List<Long> attachIdList = noteAndAttachmentIdMap.get(contactNote.getId());
                if (CollectionUtils.isNotEmpty(attachIdList)) {
                    contactNoteAttachments = new ArrayList<ContactNoteAttachment>();
                    for (Long id : attachIdList) {
                        contactNoteAttachments.add(attachmentMap.get(id));
                    }
                    contactNote.setAttachmentList(contactNoteAttachments);
    
                }
    
            }

**问题描述**  
在相邻的多个数据中，当为首的那条数据存在数据之后，直到另一个存在数据的记录为止的过程中的所有数据都存在跟第一条数据相同的附件信息列表

**原因**  
是由于在判断附件列表非空之后给contactNoteAttachments赋值并注入数据，但是真正使用该数据的位置却是在判断之外，因而导致判断不通过的时候，contactNoteAttachments依然携带着上次的数据记录导致出现此问题。  
**解决方案**  
如修正后代码所示，contactNoteAttachments对象的使用放在判断作用域内部，这个时候只有存在数据的才会执行赋值，另外这个位置也可以在赋值使用完毕之后再给该对象销毁 contactNoteAttachments=null;