【感悟】——逻辑的重要性-循环里不要套IO操作

    对于逻辑的重要性，不知道大家有多少感悟。我今天的感受，主要来源于和四姐的交流。

    在这之前，我不是没听说过逻辑，也不是不知道逻辑的重要性。但在脑海中也就是：嗯，逻辑很重要，仅此而已。根本没有深入的去思考。

    举例说明：

    学生登录后，看到评教界面。每条教师课程信息后对应一条评估状态。

    教师姓名、课程类型、课程名称从基础获得。返回到teacherCourseList集合中，而评估状态在评教的表中。为了让二者联系起来，我的思路是这样的：

    教师课程信息返回后，反到teacherCourseList中，其中包括教师课程信息，和教师课程ID，然后foreach循环，去EvaluationAssessRecord表中（表中包括学号、教师课程ID、和评估记录）遍历查找该学生的该教师课程有无评估记录。有则显示已评估，无则显示尚未评估。

    这是我的部分源码：

  //判断本地是否有该学生在该教师课程关系下的评估记录，如果有则取出评估状态，如果没有，则添加一条评估记录，评估状态为尚未评估。
            foreach (var teacherCourse in teacherCourseList)
            {
                //定义评估记录实体
                EvaluationAssessRecordEntitity enRecord = new EvaluationAssessRecordEntitity();     //给评估记录实体赋值
                enRecord.StudentId = teacherCourse.StudentID;
                enRecord.EvaluationTeacherCourseEntityTeacherCourseId = teacherCourse.ID;
//根据教师课程id和学号查询是否存在该学生在该教师课程关系下的评估记录，如果存在，则取出评估状态，如果不存在则添加评估记录，状态为未评估。
                var recordResult = this.DbSession.StateEvalDal.QueryByTeacherCourseIdAndStudentId(enRecord);
                if (recordResult.Count > 0)
                {
                    //如果存在情况下，则取出评估状态记录
                    assessStatus.AssessStatus  ="已评估";
                }
                else
                {
                    //为评估状态数据契约实体的评估状态赋值
                    assessStatus.AssessStatus = "尚未评估";
                }
                
            }

    有经验的人一看可能就知道问题所在了。那就是foreach里套了方法，而且还是从数据库查询的方法。这说明，teacherCourseList里有多少条数据，就会有多少次对数据库的IO操作。当大量学生登录时，这将会对服务器造成很大的压力。并且评估记录表的数据在评教过程中不断增加，大约一个学生就要增加15条数据。如：这个学生有10门课程，他前面已经有1000人进行了评教，评估记录表里大约就有：1000*10条数据。也就是该生登录后，会进行10次IO操作，并且每次都要遍历10000条数据。而如果这场有400人进行评教，将对数据库进行4000次IO操作。

    大家都知道，评教的时候，服务器的压力是很紧张的。所以我这样的代码，纯粹是想将评教至于死地啊。今天和四姐交流后，才发现了这一点。我这脑壳啊......

    看修改后的代码：

string studentID = teacherCourseList[0].StudentID;
            List<EvaluationAssessRecordEntitity> record = new List<EvaluationAssessRecordEntitity>();
            //根据学号在评估记录里查询出该学生的评估记录，最多为该学生的课程数，即最多10条。-赵寒-2016-3-13
            //目前为止，进行了一次IO操作。
            record = this.DbSession.StateEvalDal.QueryRecordByStudentID(studentID);
            //评估状态默认为尚未评估-赵寒-2016-3-13
            assessStatus.AssessStatus  ="尚未评估";
            //循环查询出的评估记录
            for (int i=0,i<record.Count,i++)
            {
                //循环该学生的课程
                for(int j=0,j<teacherCourseList.Count,j++)
                {
                    //如果两个信息的ID相同，说明该课程已评估，则评估状态返回已评估。
                    if(record[i].EvaluationTeacherCourseEntityTeacherCourseId==teacherCourseList[j].ID)
                    {
                        assessStatus.AssessStatus  ="已评估";
                    }
                }
            }

    哈哈，看到区别没有。经过改造。一个学生只进行一次对数据库的IO操作。由原来的10次变为1次。

    我们学校每年进行评教的学生大约有两万人。也就是，按原来的逻辑，要进行20万次对数据库的访问。但现在只需要两万次。减少了十八万次。有没有很激动。而这只是评教系统中的一个小小细节。我想，经过我之后用脑分析，多去思考。肯定会将评教优化的更完美。同样，大家都多思考。我们的ITOO性能一定会有大幅度提升。so,加油啦。