一些思考方式——从continue按钮的实现暴露的设计问题

一、功能背景

用户每天会学习一些课程，我希望下次当用户再次学习的时候能够知道从哪里开始上次的学习。由于有很多的课程，通过list的方式提供给用户，用户如果自己找就会很不方便。那么就提供一个continue按钮，当点击这个按钮时就能定位到应该学习的课程的位置。

二、服务器提供的数据

服务器提供两个数据：

1. 在用户的信息中有一个targetId域，存放用户应该继续学习的课程id。这个targetId是optional的，因为有可能用户没有学习任何课程。
2. 所有的课程列表，每个课程对应了一个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++;
    }
}

四、问题

一上来感觉这样写很自然，实际上已经违背了一些原则：

1. 如果targetId返回了，但是实际上在lesson中并不存在，那么就会导致continue按钮被显示出来，但是却无法定位到目标的课程。因为在setUpList()中是在去寻找是否存在目标课程，目标课程一定会存在这个并没有被确保。
2. 上面的写法实际上是把两段逻辑拆分开来了。continue按钮和list中的显示状况互不影响，仅仅通过targetId来各自判定。一旦targetId出现了一些服务器上的错误，或者手机端的错误，或者新的需求的变更时，他们是不能保证一致性的。实际上他们的关系应该是紧耦合的。
3. 实际上自然的想法应该是如果lessons中存在了targetLesson，就来显示出continue按钮。
4. 判定条件 targetId<=0 这个条件等于是把repository层的数据暴露了出来。现在是0的时候说明没有，如果新的设定变成10以下的说明没有的话怎么办呢，按照这个逻辑就变成了targetId<=10，这样就不得不对底层的数据保持觉悟，大大的破坏了层与层之间的独立性。

五、解决

在adapter寻找目标lesson的时候来判定是否具有目标lesson！

setUpContinueStudyIfNeeded(int targetId) {
    if(adapter.hasTargetLesson()) {
        continueStudyButton.setVisibility(View.VISIBLE);
        continueStudyButton.setOnClickListener(listener);
    }
}