作业6

一.问题解决

2.补全playPrev（GoMove）

这个其实可以只写一个Play（GoMove）方法就可以的，不必要分成PlayNext和PlayPrev。

为了实现这个功能，做了一个复活函数reliveTheKilled(GoMove)：

// Relive the killed stones
 private void reliveTheKilled(GoMove gm)
{
    ArrayList toRelive = gm.DeadGroup;
    foreach (Point p in toRelive)
    {
        Grid[p.X, p.Y].setStone(nextTurn(gm.Color));
     }
}

然后是PlayPrev(GoMove)：

public void playPrev(GoMove gm)
        {
           
            Point p = m_gmLastMove.Point;

            // New marks and labels coming So clear the old one
            clearLabelsAndMarksOnBoard();
            Grid[p.X, p.Y].removeStone();
            if (m_gmLastMove != null)
                repaintOneSpotNow(m_gmLastMove.Point);

            
            m_gmLastMove = gm;
            // Set the new labels and marks
            setLabelsOnBoard(gm);
            setMarksOnBoard(gm);

            Debug.Assert(gameTree.peekNext() != null);
            Debug.Assert(gameTree.peekNext().DeadGroup != null);
            if (gameTree.peekNext().DeadGroup.Count > 0)
                reliveTheKilled(gameTree.peekNext());
            
            optRepaint();

            // Clear the textbox and show some new comments
            textBox1.Clear();
            textBox1.AppendText(gm.Comment);

        }

 

3.点评不足

编码风格：各种不统一，命名方法有时是mfc风格，有时是CamelCase；而注释风格也是，杂七杂八

程序架构：怎么说，只能说一开始没有设计好，到了后来实现的时候一个个坑的补，补的同时也一个个的挖，冗余数据太多了，方法封装也有问题

             比如游戏逻辑部分，我看到好几个方法都要把整个棋盘遍历一遍；GoVariation类控制GoMove类是通过一个计数器m_Seq的，而GoMove类

             本身又有一个类似的int m_n

错误处理：几乎没有错误处理，只看到几个Debug.assert

文件处理：文件处理绕来绕去，封装地不好，写一个入口函数多好

UI：这个就没啥好说，毕竟是个人产品，试验用，大家都把精力放在逻辑上

 

4.解决Code Analysis报告的所有问题

由于没有Visual Studio，只有免费版的Express，所以没有做。这个改起来绝对是个大工程

 

5.改写注释，注释风格

已完成，并用xml注释方式改写，xml文件附在git上。

 

二.感受

又是不得不说的时刻，程序架构太重要了！！！设计好写起来又舒服又愉快，设计不好就像全身被糖黏着，整个空气都有问题。另外读代码太累了\("▔□▔)/

 

三.效果展示