第二周 代码复审

复审代码

源代码如下：

package client;

import javax.naming.*;
import javax.swing.*;

import java.awt.event.*;

import javax.swing.ImageIcon;

import java.net.URL;

import javax.swing.WindowConstants;

import ejbserver.EJBServerRemote;

import java.awt.Color;
import java.awt.Font;
import java.awt.Point;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.*;

public class Client extends JFrame  {
    JLabel label1 = new JLabel("登录测试应用");
    JLabel label2 = new JLabel("用户名：");
    JLabel label3 = new JLabel("密  码：");
    
    java.sql.Connection con=null;
    java.sql.ResultSet  rs=null;
    java.sql.PreparedStatement ps=null;
    

    JLabel label6 = new JLabel(new ImageIcon("image/登录界面.jpg"));

    JTextField txtname=new JTextField();;
    JPasswordField txtPwd = new JPasswordField();

    JButton but1 = new JButton("登录");
    
    JButton but2 = new JButton("退出");

    JProgressBar jpb = new JProgressBar();// 进度条

/**
     * 构造方法
     */
    public Client(){}
    public Client(int i,int num) {

initClient();
        this.setLocation(new Point(436,176));//界面初始化指定位置
        this.setResizable(false);// 禁止改变大小
        this.setDefaultCloseOperation(WindowConstants.DISPOSE_ON_CLOSE);
        this.setDefaultCloseOperation(i);
        this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    }

    /**
     * 界面代码
     */
    public void initClient() {
        this.setTitle("Client ");
        this.setSize(600, 450);
        JLayeredPane pa = new JLayeredPane(); // 容器

        // 设置组件字体和颜色
        label1.setFont(new Font("宋体", 1, 32));
        label2.setFont(new Font("宋体", 0, 16));
        label3.setFont(new Font("宋体", 0, 16));
        txtname.setFont(new Font("宋体", 0, 16));
        txtPwd.setFont(new Font("宋体", 0, 16));
        txtPwd.setEchoChar('*');
        txtPwd.setText("2013551728");
        label1.setForeground(new Color(250, 89, 136));
        txtname.setForeground(new Color(250, 85, 85));
        txtname.setEditable(true);
        txtname.setText("2013551728");

        // 设置组件位置
        label1.setBounds(153, 35, 400, 35);
        label2.setBounds(180, 165, 82, 25);
    //    label5.setBounds(15, 25, 80, 66);
        txtname.setBounds(245, 165, 150, 25);
        label3.setBounds(180, 220, 152, 25);
        txtPwd.setBounds(245, 220, 150, 25);
        but1.setBounds(245, 275, 80, 30);
        but2.setBounds(338, 275, 80, 30);

// 添加组件到容器
        pa.add(label1);
        pa.add(label2);
        pa.add(label3);

        pa.add(txtname);
        pa.add(txtPwd);
        pa.add(but1);
        pa.add(but2);
        this.add(pa);// 添加容器
        
        jpb.setBounds(0, 400, 600, 25);
        jpb.setForeground(new Color(250, 40, 135));
        jpb.setVisible(false);
        pa.add(jpb);
        label6.setBounds(0, 0, 600, 425);
        pa.add(label6);
        but1.addActionListener (new ActionListener() {
            public void actionPerformed(ActionEvent arg0) {
                if(txtname.getText().trim().length()==0){
                    JOptionPane.showMessageDialog(null, "请填入用户名!", "警告", 0);
                    return;
                }
                
                if(txtPwd.getText().trim().length()==0)
                {
                    JOptionPane.showMessageDialog(null, "请填入密码!", "警告", 0);
                    return;    
                    
                }
                if(txtPwd.getText().trim().length()>10){
                    JOptionPane.showMessageDialog(null, "密码太长!", "警告", 0);
                    return;
                    
                }
                //DB p=new DB();
                //验证区域
                String id=txtname.getText().trim();
                String password=txtPwd.getText().trim();

                int result = -3;
                try {
                    result = checkname(id,password);
                } catch (Exception e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }
                //System.out.println(id+password+result);
                if(result==1){
                    
                    //显示进度条
                    jpb.setVisible(true);
                    
                    yin();//进度条
                    JOptionPane.showMessageDialog(null, "登录成功");
                }
                else if(result==2){
                    JOptionPane.showMessageDialog(null,"您被列入黑名单!");
                    return;
                }
                else if(result==0){
                    JOptionPane.showMessageDialog(null,"密码错误!");
                    return;
                }
                else if(result==-1){
                    JOptionPane.showMessageDialog(null,"不存在的用户名!");
                    return;
                }
                else{
                    JOptionPane.showMessageDialog(null,"未知错误!");
                    return;
                }
            }
        }

);
        
    but2.addActionListener(new ActionListener(){
        public void actionPerformed(ActionEvent arg0) {
            // TODO 自动生成方法存根
            int a = JOptionPane.showConfirmDialog(null, "是否退出", "警告", 0);
            if (a == 0) {
                dispose();
            } else {
                return;
            }
        }
    });    

}

/**
     * 进度条
     */
    public void yin() {
        this.setEnabled(false);// 锁屏
    
        new java.lang.Thread(new java.lang.Runnable() {
            public void run() {
                while (true) {
                    try {
                        int i = Client.this.jpb.getValue();

                        if (i < Client.this.jpb.getMaximum()) {
                            int k = ++i;
                            Client.this.jpb.setValue(k);
                            Client.this.jpb.setStringPainted(true);

                            if (k <= 25) {
                                java.lang.Thread.sleep(100); 
                                Client.this.jpb.setString("正在载入系统......" + 4 * k+ "%");
                            } else if (k <= 75) {
                                java.lang.Thread.sleep(80); 
                                Client.this.jpb.setString("加载资料......" + k + "%");
                            } else {
                                java.lang.Thread.sleep(30); 
                                Client.this.jpb.setString("正在缓冲,请稍后......" + k+ "%");
                            }
                        } else {
                            Client.this.dispose();
                        }
                    } catch (Exception e) {
                        e.printStackTrace();
                    }
                    // 成功登入
                    if (Client.this.jpb.getValue() == 100) {
                        Client.this.jpb.setString("完成100%");
    
                        //new ClientMenu().setVisible(true);
                        Client.this.dispose();
                        break;
                    }
                }
            }
        }).start();
    }
public int checkname(String id,String password) throws Exception{
    Properties props = new Properties();
    props.put("java.naming.factory.initial", "com.sun.enterprise.naming.SerialInitContextFactory");  
    props.setProperty("java.naming.factory.url.pkgs", "com.sun.enterprise.naming");  
    props.setProperty("java.naming.factory.state", "com.sun.corba.ee.impl.presentation.rmi.JNDIStateFactoryImpl");  
    props.setProperty("org.omg.CORBA.ORBInitialHost", "localhost");  //host  
    props.setProperty("org.omg.CORBA.ORBInitialPort", "3700");  //EJB Port  
    EJBServerRemote checker=null;
    Context context = new InitialContext(props);
    context.lookup("ejbserver.EJBServerRemote");
    checker = (EJBServerRemote)context.lookup(EJBServerRemote.class.getName()); 
    int result=checker.check(id, password);
    return result;
    
}
public static void main(String[] args) {
        new Client(0,0).setVisible(true);
        
    }

}

复审结果如下：

审查项目	检查分项	检查结果
风格检查	命名、格式是否符合对应的规范文档	是
	文件命名清晰，无歧义	是
	代码缩进、对齐格式正确，可使用IDE工具进行整理	是
	代码行控制在100字符，文件行数小于2000行，方法行数小于100行	是
	方法、属性、变量和功能代码块之间使用空行进行分隔	是
	是否存在重复的代码	否
注释检查	使用统一的注释风格，注释必须使用易于识别的文本和符号	是
	每个类，每个方法都应有注释，注释量不低于代码行数的20%	数量不足
	注释应随着代码改变而更新，保证无用的代码和注解已删除	否
	重要变量、复杂算法、循环和逻辑分支处必须有注释	是
	注释应解释为什么，而不是描述做什么，做什么是由代码自解释的	否
	注释应说明意外、异常情况，注明原因和解决办法	否
类定义检查	命名规则是否符合命名规范	是
	类的职责是否过少或过多，类的属性或者方法是否没有被使用	否
	是否存在如下的调用形式：a.b().c()	否
	是否存在两个类完成类似的工作，使用了不同的方法名，却没有同一个父类	否
	是否存在某个子类仅使用了父类的部分属性或方法	是
	当功能变化或逻辑修改时，是否要修改多个类	否
	是否正确使用类修饰符	是
	派生类的构造方法应调用基类的构造方法	是
方法检查	方法名应能明确定义了操作目标以及实现的功能	是
	方法的参数遵循一个明显有序的顺序	是
	方法应当只做一件事情，并做好	是
	参数应小于7个，过多的参数应使用参数类或实体封装	是
	方法检查了输入数据的合法性，return语句是否合理	是
	方法对可能引发的异常处理有清楚的标识	否
	重载、覆盖与隐藏是否被正确使用	是
	运算符、操作符的重载是否符合规范	是
	对于重复出现并完成同样单一功能的代码，是否进行了封装	是
循环/分支检查	最普遍的状况应在在if下处理，而不是else中	是
	分值嵌套层次小于3层	是
	当有明确的多次循环操作，使用for	是
	当有不明确的多次循环操作，使用while	是
	循环嵌套的层次小于3层，是否存在死循环	是 不存在
	多个if使用switch语句表达是否更清晰	是
	循环体内不要随意修改循环变量和步长值	是
	break和continue语句使用是否正确	是
	循环和分值的边界值是否合适	是
定义检查	定义常量所有字母应大写，单词之间用下划线连接	是
	代码中的常量应用是否合适，非必要不要使用全局变量	是