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

    以下是我看了彭林江的代码以后的复审报告:

    程序结构还是比较清楚的,比较规整,对程序的实现思路还是比较清晰。c#是我们刚接触不久的语言,在代码中能看到彭林江同学是下了一些功夫的,用了不少不同于c或者java的写法来简化自己的程序,效果也很好。

    也有一些可以改进的地方:

    1、首先比较大的一个问题:代码结构问题,主要Program类变成了一个God类,几乎所有的功能都在这个类里面实现了,无论是单词的匹配功能,还是排序功能,都被Program类全部包揽。相反的File类、WordAndValue类则成了idle类,甚至没必要存在。

    从这一点来讲,我的改进建议是:重新审视这次任务,梳理一下每个类的工作,把应该由某个类自己完成的功能让它自己完成,当然这可能要对代码结构做很大调整。

    2、 第二个问题就是对变量的定义不很好导致代码可读性不高。比如ExtenWriteFile方法中的top变量,当初读者看到时很难很快判定这个变量的意义。

    改进建议:

    a、使用完整描述符来准确地描述变量、字段或类

    b、使用适用于领域内的术语

    c、巧妙节俭地使用缩写

    3、 最后一点就是代码不够精简,(当然不是什么大问题,但好的程序风格总是让人觉得舒服的),好几个地方偏繁琐,

    改进建议:多看一下好程序的程序风格,多读读练练。这个问题感觉同第一个问题有点类似,需要解决好代码的复用以及模块分割的问题。

  • 相关阅读:
    IE兼容问题,各类css hack代码(亲测有效)
    javascript进阶系列专题:闭包(Closure)
    javascript进阶系列专题:作用域与作用域链
    github文件上传及github pages博客搭建教程
    javascript算法
    【CSS3】标签使用说明
    【html5】常见标签使用说明(持续更新)
    通过一行代码学习javascript
    Object.create()兼容实现方法
    oracle查询锁表解锁语句 (转)
  • 原文地址:https://www.cnblogs.com/code-dog-liou/p/4047141.html
Copyright © 2011-2022 走看看