我们CodingCook复审的是WWW的代码，他们的项目是时间管理助手（TimeLine）。只是跟根据自己的经验来看，不一定准
先说一下整体的感觉。WWW的代码用了应该是比较符合面向对象的思想，借口，封装随处可见，也能见到一些设计模式。同时，以我的水平看来，代码质量比较高，格式规范，没有低级错误。
我主要就说我看的这个Logic模块（加空行注释1600+）
分项说明
1． 注释。这一点做的不错，至少我看到了不少，能够说明代码的作用。基本上每一块代码前都有注释。不足就是，存在几块废弃代码没有删除。
2． 代码格式。变量名非常符合C#的语言规范，且容易辨别有意义。四空格缩进。空行稍多，不是很习惯，不过这个和个人习惯相关。不足就是虽然许多地方断句不错，但仍有超长的代码行存在（120+），不利用阅读，建议是一行代码不超过80（这样的话，即使同一个显示器分开两个窗口，也不会造成阅读困难）。同时，对于单语句的if格式不统一（详见举例）
3． 异常捕获。这一点这一段代码做的不好。直接catch的是Exception。这是一个不好的编程习惯，虽然说比较方便，但是也不方便调试，同时也可能会忽略真正的错误，漏掉代码隐藏的Bug。
4． 其他。todo标记、warning标记，这个很好，不过要记得完成。
具体的代码段列举一二
1． 两个if没有必要，可以直接以 && 连接两个条件，代码功能完全相同
if (string.Compare(obj.Value, ip.TagLabel, true) == 0)
{if (!gTagMapping.ContainsKey(obj.Key)){gTagMapping.Add(obj.Key, ip.TagLabel);inverseMapping.Add(ip.TagLabel, obj.Key);}
}
2． return之后仍有代码，应该是粗心导致。
return; //by kitty & fancy start if (string.IsNullOrEmpty(outputFileName))//outputFileName = "summary_from"+startTime+"_to_"+endTime+".txt"; outputFileName = "summary.txt";
3． 单语句if格式应统一
if (cTask > 0)return iLiferInterface.DayState.DUEDATE;if (tag == -1)
{tag = AddTagType(taglabel);
}
)
