为了判分,写了一段简易的代码,后来修改的时候发现原系统中自带这段代码,看着别人写的代码,觉得别人写的比自己的简洁多了,想着怎么人家就能把代码写的那么简洁,那么效率高.想到曹建新提过,什么测试代码的运行时间来看代码的效率,我也想看看我的代码和别人的代码到底差多少.
上网查找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
所以,我的代码还是比较严谨的,尽管用的循环有些腻歪人.最后感慨一下,一段小小的代码,如果不严谨的考虑到各种情况,还是问题多多的,所以测试真的是很有必要的,要不然光看着这段代码,我可看不出什么大问题.