一、功能背景
用户每天会学习一些课程,我希望下次当用户再次学习的时候能够知道从哪里开始上次的学习。由于有很多的课程,通过list的方式提供给用户,用户如果自己找就会很不方便。那么就提供一个continue按钮,当点击这个按钮时就能定位到应该学习的课程的位置。
二、服务器提供的数据
服务器提供两个数据:
- 在用户的信息中有一个targetId域,存放用户应该继续学习的课程id。这个targetId是optional的,因为有可能用户没有学习任何课程。
- 所有的课程列表,每个课程对应了一个id。
三、不好的实现方式
setUpContinueStudy(int targetId) {
if(targetId <= 0) return
continueStudyButton.setVisibility(View.VISIBLE);
continueStudyButton.setOnClickListener(listener);
}
setLessonListInfo(List<Lesson> lessons) {
adpter = new RecyclerView.Adapter(lessons);
}
在adpter中:
int targetLessonPosition = -1;
setUpList(List<Lesson> lessons) {
int position = 0;
for(Lesson lesson : lessons) {
if(lesson.id == targetId) {
targetLessonPosition = position;
}
position++;
}
}
四、问题
一上来感觉这样写很自然,实际上已经违背了一些原则:
- 如果targetId返回了,但是实际上在lesson中并不存在,那么就会导致continue按钮被显示出来,但是却无法定位到目标的课程。因为在setUpList()中是在去寻找是否存在目标课程,目标课程一定会存在这个并没有被确保。
- 上面的写法实际上是把两段逻辑拆分开来了。continue按钮和list中的显示状况互不影响,仅仅通过targetId来各自判定。一旦targetId出现了一些服务器上的错误,或者手机端的错误,或者新的需求的变更时,他们是不能保证一致性的。实际上他们的关系应该是紧耦合的。
- 实际上自然的想法应该是如果lessons中存在了targetLesson,就来显示出continue按钮。
- 判定条件 targetId<=0 这个条件等于是把repository层的数据暴露了出来。现在是0的时候说明没有,如果新的设定变成10以下的说明没有的话怎么办呢,按照这个逻辑就变成了targetId<=10,这样就不得不对底层的数据保持觉悟,大大的破坏了层与层之间的独立性。
五、解决
在adapter寻找目标lesson的时候来判定是否具有目标lesson!
setUpContinueStudyIfNeeded(int targetId) {
if(adapter.hasTargetLesson()) {
continueStudyButton.setVisibility(View.VISIBLE);
continueStudyButton.setOnClickListener(listener);
}
}