排查前台无显示某个字段的bug

前言

作业提交系统匆匆上线后，在判卷过程中发现，当点击下一个作业时，作业的文件目录不显示。这导致我一度以为学生交了白卷上去，果断打了0分。后来在好友的提醒下，才发现这个问题。我们发现只有当第一次点进去的作业才有文件目录，其他自动下一个的作业无文件目录。

排查

首先，一开始怀疑v层显示有问题，看了看其他地方，写法差不多，并且第一个作业正常显示，没问题。
然后看一下c层，前台打印一下作业相关文件信息，发现为undefined。

第一次数据初始化和后面的下一个作业请求的是不同地址。但是都是将返回的data赋值给work，一个是返回的是work类型，一个返回的是Page<Work>类型。work都应该是完整的接收到了后台返回的数据才对。

然后又去看了眼后台，想起了自己原来写的@JsonView，恍然大悟。

原来请求下一个未评阅的方法为了方便，直接请求后台的getAll方法，然后拿出来第一个未评阅的作业给work。但是后台的getAll方法当初设计的时候减少非必要的信息，没有让他返回Attachments也就是作业文件字段。从而导致了前台获取不到文件字段。问题的原因找到了，问题也就迎刃而解，直接修改后台getAll的JsonView，在GetAllJsonView定义上继承Work.AttachmentsJsonView，问题解决，提交。

如何避免

这貌似只是解决了当前问题，却使得要改变@JsonView，感觉不应该这么改。
一开始想写单元测试避免这种情况再发生，但是从前台c层想到前台m层，在想到后台c层，发现这并不是单元测试能够解决的。前台只要有数据就正常显示没问题，后台getAll方法不返回作业文件也没问题。问题是我们的前台本应想获取某一作业，却请求了后台getAll方法。
解决：
通过上面的分析，我们要恢复后台返回给前台单个作业的方法。那我们只能在后台m层通过获取未评阅的第一个作业来获取下一个作业。

/**
 * 得到下一个未评阅的作业
 * @return 作业
 */
@Override
public Work getNextNotReviewedWork() {
    Pageable pageable = PageRequest.of(0,10);
    Page<Work> works = this.getAll(null, null, null, false, pageable);
    List<Work> workList = works.getContent();
    Work work = workList.get(0);
    return work;
}

这样修改虽然费事了点，但是感觉更加规范。

总结

有些是单元测试解决不了的问题，这时只能靠我们的规范来减少bug。

版权声明

本文作者：河北工业大学梦云智开发团队 - 赵凯强