为了判分,写了一段简易的代码,后来修改的时候发现原系统中自带这段代码,看着别人写的代码,觉得别人写的比自己的简洁多了,想着怎么人家就能把代码写的那么简洁,那么效率高.想到曹建新提过,什么测试代码的运行时间来看代码的效率,我也想看看我的代码和别人的代码到底差多少.
上网查找Stopwatch,用这样的一段代码就能够测试代码的效率.
- System.Diagnostics.Stopwatch stopwatch = new Stopwatch();
- stopwatch.Start(); // 开始监视代码运行时间
- // you code ....
- stopwatch.Stop(); // 停止监视
- TimeSpan timespan = stopwatch.Elapsed; // 获取当前实例测量得出的总时间
- double hours = timespan.TotalHours; // 总小时
- double minutes = timespan.TotalMinutes; // 总分钟 ...
我的代码是这样的:
- System.Diagnostics.Stopwatch stopwatch = new System.Diagnostics.Stopwatch();
- stopwatch.Start(); // 开始监视代码运行时间
- string strCorAnsMy = "chemistry#living#everything#outside#working out#junior#the way#taught#experiments#single"; //正确答案
- string strStuAnsMy = "living#everything#outside#working out#junior#the way#taught#experiments#single"; //学生答案
- string strIsOrdMy = "有序";//是否有序,测试时需要修改这里
- double dlTotalMy = 0;//总分
- string[] strArrCorAnsMy = strCorAnsMy.Split('#'); //拆分正确答案
- string[] strArrStuAnsMy = strStuAnsMy.Split('#'); //拆分学生答案
- for (int i = 0; i < strArrCorAnsMy.Length; i++)
- {
- strArrCorAnsMy[i] = strArrCorAnsMy[i].Trim(); //去正确答案的空格
- }
- for (int i = 0; i < strArrStuAnsMy.Length; i++) //去学生答案空格
- {
- strArrStuAnsMy[i] = strArrStuAnsMy[i].Trim();
- }
- if (strIsOrdMy == "无序")
- {
- //由于无序,所以对于正确答案和学生答案进行for循环,比较
- for (int i = 0; i < strArrCorAnsMy.Length; i++)
- {
- for (int j = 0; j < strArrStuAnsMy.Length; j++)
- {
- //如果学生答案等于正确答案,分数+1,并跳出
- if (strArrCorAnsMy[i].Equals(strArrStuAnsMy[j]))
- {
- dlTotalMy += 1; //每题1分,共9分
- break;
- }
- }
- }
- }
- else //有序是这样
- {
- //循环学生的成绩
- for (int i = 0; i < strArrStuAnsMy.Length; i++)
- {
- //如果正确答案和学生答案一一对应,则正确
- if (strArrCorAnsMy[i].Equals(strArrStuAnsMy[i]))
- {
- dlTotalMy += 1; //每题1分,得0分
- }
- }
- }
- stopwatch.Stop(); // 停止监视
- TimeSpan timespan = stopwatch.Elapsed; // 获取当前实例测量得出的总时间
- doule hours = timespan.TotalHours; // 总小时 断点设在这里double minutes = timespan.TotalMinutes; // 总分钟 ...
别人的代码经过修改,是这样的.
- System.Diagnostics.Stopwatch stopwatch = new System.Diagnostics.Stopwatch();
- stopwatch.Start(); // 开始监视代码运行时间
- string strCorAns = "chemistry#living#everything#outside#working out#junior#the way#taught#experiments#single"; //正确答案
- string strStuAns = "living#everything#outside#working out#junior#the way#taught#experiments#single"; //学生答案
- string strIsOrd = "有序";//是否有序,测试时需要修改这里
- double dlTotal = 0;//总分
- string[] strArrCorAns = strCorAns.Split('#'); //拆分正确答案
- string[] strArrStuAns = strStuAns.Split('#');//拆分学生答案
- if (strIsOrd == "无序") //无序的话,只要写对了,在哪个空填的都无所谓
- {
- for (int i = 0; i < strArrCorAns.Length; i++) //循环正确答案
- {
- if (strStuAns.IndexOf(strArrCorAns[i].ToString()) > -1) //比较正确答案在学生的答案是否存在
- {
- dlTotal += 1; //每题1分,得9分
- }
- }
- }
- else //表示有序,必须要和正确答案的顺序一模一样
- {
- //循环学生答案
- for (int j = 0; j < strArrStuAns.Length; j++)
- {
- //如果学生的答案长度大于0,正确答案的长度也大于0,并且正确的答案长度>=学生的答案长度
- if (strArrStuAns.Length > 0 && strArrCorAns.Length > 0 && strArrCorAns.Length >= strArrStuAns.Length)
- {
- //如果学生答案不为null
- if (strArrCorAns[j] != null)
- {
- //比较学生答案和正确答案去空格之后是否相同
- if (strArrStuAns[j].ToString().Trim() == strArrCorAns[j].ToString().Trim())
- {
- dlTotal += 1; //有序,得分为0分
- }
- }
- }
- }
- }
- stopwatch.Stop(); // 停止监视
- TimeSpan timespan = stopwatch.Elapsed; // 获取当前实例测量得出的总时间
- double hours = timespan.TotalHours; // 总小时,断点设在这里
- double minutes = timespan.TotalMinutes; // 总分钟 ...
时间运行情况,如下面图片显示,我的在前面.先是测试有序的情况.
有序时,我的代码的时间为 0.0000068.
有序时,别人的代码的时间为0.0000068,一样.
无序时,我的代码的时间为0.0000068.
无序时,别人的代码时间为0.0000441,我的比较快.
还有其他情况,比如我现在的学生代码,基本和正确答案差不多的顺序,如果我改成,完全不一样的顺序,运行效率是否会有很大区别?
stringstrCorAnsMy = "chemistry#living#everything#outside#working out#junior#theway#taught#experiments#single"; //正确答案
string strStuAnsMy ="single#experiments#taught#the way#junior#workingout#outside#everything#living"; //学生答案掉个个,再测试下无序时的运行时间,发现基本没差别.
0.00000068,和上面一样.
0.0000441,和上面一样
后来,运行多次发现,timespan不是一定的,这次时间为0.0000441,下次可能就是0.0000498,但是大体上还是差不多的.
这样看来我的代码运行起来也不是很慢,尤其是无序时.有序时,采用的都是一重循环,我的代码没有做太多的处理,所以可能不安全.无序时,我采用的是,双重循环和equals,而他采用的是一重循环和indexof,照理应该是双重循环慢,所以猜想IndexOf的效率可能比较慢.
想到老师说的,果然是不怕不知道,就怕不知道.以前不知道要测试代码的效率,现在想起来了,立马就能用,以后可以多多测试测试,了解了解不同代码的运行效率,开发更加优化的代码.
还有除了stopwatch,那人的代码使用的时候是有问题的,比如我把正确答案和学生答案改为这样.
- string strCorAnsMy = "a";
- string strStuAnsMy ="an"; //得分为0
我的代码显示这个得0分.但是他的代码运行起来,因为用的是strStuAns.IndexOf(strArrCorAns[i]),所以an.IndexOf(a),是>-1的,所以
- string strCorAns = "a";
- string strStuAns ="an"; //得分为1
所以,我的代码还是比较严谨的,尽管用的循环有些腻歪人.最后感慨一下,一段小小的代码,如果不严谨的考虑到各种情况,还是问题多多的,所以测试真的是很有必要的,要不然光看着这段代码,我可看不出什么大问题.