【软工实践】第二次结对编程作业

传送门

结对同学博客： https://www.cnblogs.com/shijinhai/p/11681105.html

本作业博客： https://edu.cnblogs.com/campus/fzu/SE_FZU_1917_K/homework/8664

GitHub项目地址： https://github.com/1061413241/13Water

演示视频链接：https://pan.baidu.com/s/1zINm-QfCbvO7jWRL_0M5-Q 提取码: 882d

具体分工

史恩泽（我）：

设计并实现算法，优化算法
调用网络接口
代码组织与内部实现

施金海：

原型实现，重构部分贴图设计（根据需求）
测试程序，进行性能分析并提出改进意见

PSP表格

PSP2.1	Personal Software Process Stages	预估耗时（分钟）	实际耗时（分钟）
Planning	计划	50	60
· Estimate	· 估计这个任务需要多少时间	50	60
Development	开发	2240	3560
· Analysis	· 需求分析 (包括学习新技术)	600	900
· Design Spec	· 生成设计文档	40	60
· Design Review	· 设计复审	30	50
· Coding Standard	· 代码规范 (为目前的开发制定合适的规范)	30	40
· Design	· 具体设计	400	450
· Coding	· 具体编码	1000	1800
· Code Review	· 代码复审	20	60
· Test	· 测试（自我测试，修改代码，提交修改）	120	200
Reporting	报告	100	105
· Test Repor	· 测试报告	30	30
· Size Measurement	· 计算工作量	10	20
· Postmortem & Process Improvement Plan	· 事后总结, 并提出过程改进计划	60	55
	合计	2390	3725

代码量分析

第一次写这种代码量比较大的程序。显然，以前所使用的简单加法计算已经有些捉襟见肘了。进行系统的代码量统计显得尤为重要，统计代码行数主要使用以下两种正则表达式：

统计包括空行、注释及符号：

b*[^:b#/]+.*$

统计不包括空行、注释、大括号等符号：

^(?!(\s*\*))(?!(\s*\-\-\>))(?!(\s*\<\!\-\-))(?!(\s*\n))(?!(\s*\*\/))(?!(\s*\/\*))(?!(\s*\/\/\/))(?!(\s*\/\/))(?!(\s*\}))(?!(\s*\{))(?!(\s(using))).*$

分别用以上两种方法统计：

可以看出差距还是挺大的，第二种正则基本上把项目扒了个精光。

本次分析采用第一种统计方法，项目中的.Designer.cs代码基本上是操作控件自动生成的，所以不纳入统计。

去除.Designer.cs代码后，还剩4470-2004=2466行，即为本项目的有效代码行数。

解题思路描述与设计实现说明

网络接口的使用：

本项目主要用到Post和Get，将网络接口的调用封装成API_Helper，当按下界面中的按钮后调用对应的网络接口进行数据传输。

代码组织与内部实现设计（类图）：

一共采用了两种风格的窗体，登录和注册采用MetroForm，其他的采用CCSkinMain。

比较重要的类有两个：DealCards和User，分别用于自动分牌和存放用户各类信息。

其他的窗体则根据需要进行相应的调用与组织。

算法关键与实现部分流程图：

算法的关键在于DealCards的实现：要先将获取到的牌型转换成易于分析和处理的int型数值，之后获取所有自动牌型，根据牌型生成符合规则的最优解。

关键代码解释

/// <summary>
/// 牌面从大到小排序
/// </summary>
public static void SortCard(List<int> cards)
{
	//牌面从大到小排序
	cards.Sort((b, a) =>
	{
		int result = (a % 100) - (b % 100);
        if (result == 0)
        {
			result = (a / 100) - (b / 100);
		}
		return result;
	});
}

/// <summary>
/// 牌面从小到大排序
/// </summary>
public static void SortCardMinToMax(List<int> cards)
{
	//牌面从小到大排序
	cards.Sort((a, b) =>
	{
		int result = (a % 100) - (b % 100);
		if (result == 0)
		{
		result = (b / 100) - (a / 100);
		}
		return result;
	});
}

将牌面转换为数字后可以更方便的进行排序，因为在十三水中A最大，所以将A指定为14；

扑克的四种花色分别对应100，200，300，400；

/// 牌型对应如下：
/// 1->$
/// 2->&
/// 3->*
/// 4->#

牌面对100取余即可得到对应的值，之后重载sort进行排序。

性能分析与改进

测试时分别进行了注册、登录、开启战局并出牌、查询排行榜、查询历史战局详情，得到性能报告情况如下：

其中占用较大的是 Forms.Application.Run()和 Forms.MessageBox.show()，即窗体的加载与显示和提示窗的加载与弹出，这部分属于窗体加载难以优化。其他方法的调用都在可接受的范围内，代码的性能基本上符合要求。

单元测试

[TestMethod]
        public void SortCardMethod()
        {
            //2   3   4   5   6   7   8   9   10  J   Q   K   A
         //  102,103,104,105,106,107,108,109,110,111,112,113,114,(1代表花色的一种)
            List<int> testcard = new List<int>();
            List<int> rightcard = new List<int>();
            for (int i = 114; i >= 102; i--) //正确牌型
                rightcard.Add(i);
            for (int i = 102; i <= 114; i++)//测试牌型
                testcard.Add(i);
            /// <summary>
            /// sortcard功能：牌面从大到小排序
            /// </summary>
            DealCards.SortCard(testcard);//调用sortcard
            Assert.IsTrue(testcard.SequenceEqual(rightcard));//两个list相同返回true
 
        }

展示的这部分单元测试是对SortCard方法进行测试，其功能是对牌从大到小排序。

测试数据：102,103,104,105,106,107,108,109,110,111,112,113,114

正确结果：114,113,112,111,110,109,108,107,106,105,104,103,102

返回结果：114,113,112,111,110,109,108,107,106,105,104,103,102

GitHub代码签入记录

遇到的困难及解决方法

问题描述：

为了方便调用将用户手牌信息设置为static变量，并且使用LIst<>存放手牌，带来的问题是需要及时对List进行clear，否则之后无法对手牌进行更新；
特殊情况的考虑：如玩家在牌局中出牌之前点击返回，如何处理当前牌局的进度。

做过哪些尝试：

完善submit方法，在提交手牌之后对List进行clear；
增加牌局进度判定机制，若玩家在牌局中途强制返回，则要清理手牌、停止牌局进度；
让多个人运行程序，增大使用基数来检测程序bug。

是否解决：

已解决。

有何收获：

实践才是检验真理的唯一标准，代码写得再好抵不过解决一个bug带来的舒畅和愉悦感。

还有就是结对项目带来的好处，多一个人绝不是简单的1+1=2，相当于多了一种思路方向，解决bug的速度也会得到提升。

评价队友

值得学习的地方：

做事认真，效率比较高；交给他的任务能尽快完成，听取我的意见。

需要改进的地方：

缺少主动学习的动力，得在旁边催着他，明明学得很快脑子也聪明，就是太懒。直接导致我被迫当了一次大腿，建议尽快安排我的奶茶。

学习进度条

第N周	新增代码（行）	累计代码（行）	本周学习耗时（小时）	累计学习耗时（小时）	重要成长
1	103	103	14	14	学会了十三水的玩法，对原型设计有了一定的基础；
2	400	503	10	24	学习C# winform开发，完善具体设计思路；
3	1313	1816	30	54	实现核心算法“自动分牌”；
4	1153	2969	22	76	界面设计与代码实现，完成各窗体与接口的实现。