精选文章 代码复审结果

代码复审结果

作者:Adele1973 时间: 2021-02-07 02:39:32
Adele1973 2021-02-07 02:39:32
【摘要】我们进行的是对76er的代码进行审核。 
 代码问题和优点有以下几点: 
 1、有无用类,在整个工程中Url类,没有被用到,所以是一个无用的类,存在于代码中。 
  public class Url { // 原始url的值,主机部分是域名 private String oriUrl; // url的值,主机部分是IP private String url; //URL NUM privat...

我们进行的是对76er的代码进行审核。

代码问题和优点有以下几点:

1、有无用类,在整个工程中Url类,没有被用到,所以是一个无用的类,存在于代码中。

public class Url { // 原始url的值,主机部分是域名 private String oriUrl; // url的值,主机部分是IP private String url; //URL NUM private int urlNo; // 获取URL返回的结果码 private int statusCode; // 此URL被别的文章引用的次数 private int hitNum; // 此URL对应文章的汉字编码 private String charSet; // 文章摘要 private String abstractText; // 毛标签 private String tag; // 文章的权重(包含导向词的信息) private int weight; // 文章的描述 private String description; // 文章大小 private int fileSize; // 最后修改时间 private Timestamp lastUpdateTime; // 过期时间 private Date timeToLive; // 文章名称 private String title; // 文章类型 private String type; // 引用的链接 private String[] urlRefrences; //爬取的层次,从种子开始,依次为第0层,第1层...

 

2、在工程中没有使用MVC的编码风格,视图没有独立出来而是和控制部分交融在一起的,不方便最后的修改和维护。

class CraUi extends JFrame implements ActionListener, Runnable
{ boolean isStart; public String Total="90000000"; public int Tint; JLabel jl2=new JLabel("已访问URL个数 : " + 0); JLabel jl00=new JLabel("要下载网页个数 :"); JTextField jtf=new JTextField(10); JButton jb1=new JButton("打开"); JButton jb2=new JButton("关闭"); JPanel jp=new JPanel(); JProgressBar jpb=new JProgressBar(); public CraUi() { this.setTitle("界面"); jp.setLayout(null); jl00.setBounds(30, 20, 280, 30); jp.add(jl00); jtf.setBounds(150, 20, 280, 30); jp.add(jtf); jl2.setBounds(30, 130, 280, 30); jp.add(jl2); jb1.setBounds(50, 180, 80, 30); jp.add(jb1); jb1.addActionListener(this); jb2.setBounds(250, 180, 80, 30); jp.add(jb2); jb2.addActionListener(this); jpb.setBounds(30, 300, 500, 50); jpb.setStringPainted(true); jpb.setMinimum(0); jpb.setValue(0); jpb.setIndeterminate(true); jp.add(jpb); this.add(jp); this.setBounds(300, 250, 600, 500); this.setVisible(true); }

界面类,在程序的主启动类中,没有独出来。

3、还有就是界面的制作使用的是绝对位置和大小,那么界面的大小不可调,不能给用户很好的使用感。

4、有很多地方的异常处理仅仅是形式化的存在而已,并没有对在抛出异常有做处理。这个一个很不好的编码风格

 } catch (HttpException e) { // 发生致命的异常,可能是协议不对或者返回的内容有问题
 e.printStackTrace(); } catch (IOException e) { // 发生网络异常
 e.printStackTrace(); } finally { // 释放连接 getMethod.releaseConnection();

 

5、优点之一是函数都是功能尽量单一的小函数,可能重构过吧,这样会使程序的可修改性和可拓展性变得很强。

6、还有个优点是注释详细;基本重要过程和重要的数据结构都有详细的注释,易懂。

 

最后总体评价下,作为一个程序来说,尽管代码的风格和写法存在一些问题,但是整个程序成功了,那么应该给予肯定,但是存在的这些问题在M2阶段是否会爆发出致命的错误就不得而知了,希望注意。

转载于:https://www.cnblogs.com/coding-hundredOfYears/archive/2012/12/13/2815746.html

勿删,copyright占位
分享文章到微博
分享文章到朋友圈

上一篇:nagios下 监控内存的插件

下一篇:linux服务器的远程管理网站

您可能感兴趣

  • 关于Basic程序解释器及编译原理的简单化(1)---Basic器的语法分析及主要代码

    这就是Basic解释器的主代码,其中用到上节讲的词法提取get_token()和代数式求值get_exp(int *result)函数. 这一节的代码更简单,就是随心所欲地将得到的token组装.譬如在get_token后如果token装PRINT,你就调用一次get_token将下一个token答应出来就是了,很简单的,或许你自己也能搞定的. 在下一节你,我会给你完整的C++封装好了的源代...

  • 创建良好设计的代码(基于Delphi/VCL)

    东日文档     我们平时都会写很多代码,为公司,为自己或者为朋友。有时,为了验证自己的一个想法,或学习某一个技术,会写一些试验性的代码。这样的代码的生命周期很短,基本不需要维护,随意写一下就可以。但是,当你真正要完成一个 Project 的时候,代码设计就非常重要。因为这样的代码是需要长期维护,不断修改或增强的。设计凌乱的代码会使得维护非常困难或者根本不可能,修改这样的代码意味着产生更多的...

  • 如何将界面代码和功能代码分离(基于Delphi/VCL)

    东日文档     很多朋友看了上次我写的“创建良好设计的代码(基于Delphi/VCL)”后,对我说感觉上可以接受其中的观点,但似乎说得太简单,不够具体;也有的朋友对其中的一个小例子有些异议。因此便有了此文。     上次,我举的例子是这样的:假设要从某处获得一个字符串列表,然后显示于 TListBox 中,我所推崇的代码是:    ObjectXXX := TObjectXXX.Creat...

  • Delphi 程序员代码编写标准指南

    一、序言 二、通用源代码格式规则 2.1 缩格 2.2 页边空格 2.3 Begin…End 配对 三、Object Pascal 3.1 括号 3.2 保留字和关键字 3.3 过程和函数(例程)     3.3.1 命名/格式化     3.3.2 形式参数         3.3.2.1 格式化         3.3.2.2 命名         3.3.2.3 参数的排序      ...

  • Delphi 程序员代码编写标准指南 (二)

    一、序言   本文档详述了在Delphi 4开发者指南下进行编程的代码编写标准。在通常情况下,本文档遵循“取消”式格式的指引方针,该方针由Borland国际通过一些例外来使用。在Delphi 4开发者指南中包含本文档的目的在于阐述一种方法,通过该方法,开发小组可以在他们所编写的代码中保持一贯的风格。这样做的目的是使在开发小组中的每一个程序员都可以明白其他程序员的代码。这有助于提高代码编写的可...

  • Delphi 程序员代码编写标准指南 (三)

    3.5 类型   3.5.1 大写约定   如果类型的名字是保留字,那么它应全部小写。Win32 API类型通常全部大写,并且你必须遵循在Windows.pas或其他API单元中的详细类型名称的约定。对于其他变量名字,地一个字母应为大写,而其他字母应错落有致。下面是一些例子:       var         MyString : string;          //保留字       ...

  • Delphi 程序员代码编写标准指南 (四)

    四、文件   4.1 工程文件   4.1.1 命名   工程文件应取个描述性的名字。例如,Delphi 4开发者指南错误管理器 的工程名字是:DDGBugs.dpr。一个有关系统信息的程序的名字就应象 SysInfo.dpr。   4.2 窗体文件   4.2.1 命名   一个窗体文件的取名应可以描述使用该窗体的目的,并加以后缀Frm。例如,一个“关于”的窗体的文件名应是AboutFrm...

  • Delphi 程序员代码编写标准指南 (五)

    五、窗体和数据模板   5.1 窗体   5.1.1 窗体类型命名标准   窗体类型的取名应能表达使用该窗体的目的。类型定义应加以前缀T。前缀后面跟随着描述性的名字。最后,应加以Form后缀来描述名字。例如,一个“关于”的窗体的类型的名字应为:         TAboutFrom = class(TForm); 主窗体的定义为:         TMainForm = class(TFor...

CSDN

CSDN

中国开发者社区CSDN (Chinese Software Developer Network) 创立于1999年,致力为中国开发者提供知识传播、在线学习、职业发展等全生命周期服务。

华为云40多款云服务产品0元试用活动

免费套餐,马上领取!
代码复审结果介绍:华为云为您免费提供代码复审结果在博客、论坛、帮助中心等栏目的相关文章,同时还可以通过 站内搜索 查询更多代码复审结果的相关内容。| 移动地址: 代码复审结果 | 写博客