zoukankan      html  css  js  c++  java
  • 代码互审

    看了一下我的结对编程的小伙伴个人项目的代码,提出一些我的复审意见.

    值得欣赏的地方:

      1程序总体逻辑结构清晰,注释简洁清楚,十分方便别人阅读并理解代码;

      2对C#的一些函数比较熟悉;(至少有些函数我不得不去查一下具体的用法)                                                                                                                      

      3运用了正则表达式处理单词识别的问题;(我想到了我的个人项目,并没有用正则表达式进行匹配而是用的状态机的思想。那时候刚写完程序只觉得“任务完成了,太好了!”,甚至有点小得意,并没有细想如果用正则表达式是否会使得代码更简洁,程序效率更高。现在回看一下,觉得有时候用正则表达式确实会省事一些。以后也许可以试着用,而不再以“不会用”为借口了。我这算是见贤思齐么!!)

      4代码整体布局风格统一,比较规范。

    有待改进的地方:

      1主函数写得略长,基本把要做的事情全在主函数里写了。

       改进意见:可以把实现不同功能的部分代码抽取出来用函数实现。(其实注释基本上就把功能模块写清楚了,可以根据注释分开)

      2部分代码冗余。比如有一处申明了ArrayList类型的变量al,但是al并未被其他地方引用。

       改进意见:删掉

      3一些细节问题。比如有一个for循环体为 for (; end < strLine.Length; ) 这块用while循环可能更好.                                                                                      

        改进意见:改成while循环

      总之,主要的问题就是主函数写得长了点,简单化、模块化可能会更好。

  • 相关阅读:
    (转)深入理解C语言指针的奥秘
    (转)C语言在哪里?
    [转]12个摄影基本法则
    光圈使用
    ArcIMS 连接器.NET Link 使用方法
    在asp.net中使用xml文件的两种类型及用法
    利用XSL双向转换XML文档
    光圈 暴光 快门对比
    保护眼睛的电脑颜色设置
    vb.net2005动态添加网页控件的事件
  • 原文地址:https://www.cnblogs.com/encoin/p/4046604.html
Copyright © 2011-2022 走看看