我先写几个方法命名骆驼式,类名大写等就不写了,写几个关于JDBC的1:赋值时候,等号左右两边保留一个空格
例如: String str = "lihan";2:sql语句尽量大写
例如: String sql = "SELECT * FROM TABLE";3:JDBC中,声明连接时候,定义为静态,避免频繁创建连接,因为创建连接这是一个开销比较大的过程
例如:public static Connection getConnection() throws Exception
{
Class.forName("oracle.jdbc.driver.OracleDriver");
Connection con=DriverManager.getConnection("jdbc:oracle:thin:@10.10.1.6:1521:lihan","scott","tiger");
return con;
}4:关闭连接写在finally中
例如: try
{
//代码块
}
catch(Exception e)
{
e.printStackTrace();
}
finally
{
try {
OracleConnection.getConnection().close();
} catch (SQLException e) {
e.printStackTrace();
} catch (Exception e) {
e.printStackTrace();
}
}5:独立代码块之间应该空行
例如:
String sql = "INSERT INTO \"Dic_CA01\" values(?,?,?,?,?)";

ps = con.prepareStatement(sql);

ps.setString(1, d.getCode() );
ps.setString(2, d.getParentCode());
ps.setString(3, d.getName());
ps.setString(4, d.getDescription());
ps.setString(5, d.getLayOrder());

ps.executeUpdate();
6:静态常量应当大写
例如:final static String NAME = "李晗";7:JDBC获取数据值时候,应当以字段名称,而不是数字
例如:
d.setCode(rs.getString(“name”));而不是d.setCode(rs.getString(1));8:从安全和维护方面考虑,JDBC操作数据库时候尽量用PreparedStatement而不用Statement9:servlet中即使get和post方法不都用,也需要写上转跳
例如:post方法里面:
this.doGet(request,response);10:为每一个方法加上DEBUG模式的输出

解决方案 »

  1.   

    命名使用常规的规范
    对齐和布局使用netbeans的jindent来处理
      

  2.   

    Package 的命名  
    Package 的名字应该都是由一个小写单词组成。  
    Class 的命名  
    Class 的名字必须由大写字母开头而其他字母都小写的单词组成  
    Class 变量的命名  
    变量的名字必须用一个小写字母开头。后面的单词用大写字母开头。  
    Static Final 变量的命名  
    Static Final 变量的名字应该都大写,并且指出完整含义。  
    参数的命名  
    参数的名字必须和变量的命名规范一致。  
    数组的命名  
    数组应该总是用下面的方式来命名:  
    byte[] buffer; 
    而不是:  
    byte buffer[]; 
    方法的参数  
    使用有意义的参数命名,如果可能的话,使用和要赋值的字段一样的名字:  
    SetCounter(int size){ 
    this.size = size; 

    Java 文件样式  
    所有的 Java(*.java) 文件都必须遵守如下的样式规则  
    版权信息  
    版权信息必须在 java 文件的开头,比如:  
    /** 
    * Copyright ® 2000 Shanghai XXX Co. Ltd. 
    * All right reserved. 
    */ 
    其他不需要出现在 javadoc 的信息也可以包含在这里。 
    Package/Imports  
    package 行要在 import 行之前,import 中标准的包名要在本地的包名之前,而且按照字母顺序排列。如果 import 行中包含了同一个包中的不同子目录,则应该用 * 来处理。  
    package hotlava.net.stats; 
    import java.io.*; 
    import java.util.Observable; 
    import hotlava.util.Application; 
    这里 java.io.* 使用来代替InputStream and OutputStream 的。 
    Class  
    接下来的是类的注释,一般是用来解释类的。  
    /** 
    * A class representing a set of packet and byte counters 
    * It is observable to allow it to be watched, but only 
    * reports changes when the current set is complete 
    */ 
    接下来是类定义,包含了在不同的行的 extends 和 implements  
    public class CounterSet 
    extends Observable 
    implements Cloneable 
    Class Fields  
    接下来是类的成员变量:  
    /** 
    * Packet counters 
    */ 
    protected int[] packets; 
    public 的成员变量必须生成文档(JavaDoc)。proceted、private和 package 定义的成员变量如果名字含义明确的话,可以没有注释。 
    存取方法  
    接下来是类变量的存取的方法。它只是简单的用来将类的变量赋值获取值的话,可以简单的写在一行上。  
    /** 
    * Get the counters 
    * @return an array containing the statistical data. This array has been 
    * freshly allocated and can be modified by the caller. 
    */ 
    public int[] getPackets() { return copyArray(packets, offset); } 
    public int[] getBytes() { return copyArray(bytes, offset); } public int[] getPackets() { return packets; } 
    public void setPackets(int[] packets) { this.packets = packets; } 
    其它的方法不要写在一行上 
    构造函数  
    接下来是构造函数,它应该用递增的方式写(比如:参数多的写在后面)。  
    访问类型 ("public", "private" 等.) 和 任何 "static", "final" 或 "synchronized" 应该在一行中,并且方法和参数另写一行,这样可以使方法和参数更易读。  
    public 
    CounterSet(int size){ 
    this.size = size; 
    } 克隆方法  
    如果这个类是可以被克隆的,那么下一步就是 clone 方法:  
    public 
    Object clone() { 
    try { 
    CounterSet obj = (CounterSet)super.clone(); 
    obj.packets = (int[])packets.clone(); 
    obj.size = size; 
    return obj; 
    }catch(CloneNotSupportedException e) { 
    throw new InternalError("Unexpected CloneNotSUpportedException: " + e.getMessage()); 


    类方法  
    下面开始写类的方法:  
    /** 
    * Set the packet counters 
    * (such as when restoring from a database) 
    */ 
    protected final 
    void setArray(int[] r1, int[] r2, int[] r3, int[] r4) 
    throws IllegalArgumentException 

    // 
    // Ensure the arrays are of equal size 
    // 
    if (r1.length != r2.length || r1.length != r3.length || r1.length != r4.length) 
    throw new IllegalArgumentException("Arrays must be of the same size"); 
    System.arraycopy(r1, 0, r3, 0, r1.length); 
    System.arraycopy(r2, 0, r4, 0, r1.length); 

    toString 方法  
    无论如何,每一个类都应该定义 toString 方法:  public 
    String toString() { 
    String retval = "CounterSet: "; 
    for (int i = 0; i < data.length(); i++) { 
    retval += data.bytes.toString(); 
    retval += data.packets.toString(); 

    return retval; 


    main 方法  
    如果main(String[]) 方法已经定义了, 那么它应该写在类的底部.  
    代码编写格式  
    代码样式  
    代码应该用 unix 的格式,而不是 windows 的(比如:回车变成回车+换行)  
    文档化  
    必须用 javadoc 来为类生成文档。不仅因为它是标准,这也是被各种 java 编译器都认可的方法。使用 @author 标记是不被推荐的,因为代码不应该是被个人拥有的。  
    缩进  
    缩进应该是每行2个空格. 不要在源文件中保存Tab字符. 在使用不同的源代码管理工具时Tab字符将因为用户设置的不同而扩展为不同的宽度.  
    如果你使用 UltrEdit 作为你的 Java 源代码编辑器的话,你可以通过如下操作来禁止保存Tab字符, 方法是通过 UltrEdit中先设定 Tab 使用的长度室2个空格,然后用 Format|Tabs to Spaces 菜单将 Tab 转换为空格。  
    页宽  
    页宽应该设置为80字符. 源代码一般不会超过这个宽度, 并导致无法完整显示, 但这一设置也可以灵活调整. 在任何情况下, 超长的语句应该在一个逗号或者一个操作符后折行. 一条语句折行后, 应该比原来的语句再缩进2个字符.  
    {} 对  
    {} 中的语句应该单独作为一行. 例如, 下面的第1行是错误的, 第2行是正确的:  if (i>0) { i ++ }; // 错误, { 和 } 在同一行  if (i>0) {  
    i ++  
    }; // 正确, { 单独作为一行  
    } 语句永远单独作为一行.  
    如果 } 语句应该缩进到与其相对应的 { 那一行相对齐的位置。 
    括号  
    左括号和后一个字符之间不应该出现空格, 同样, 右括号和前一个字符之间也不应该出现空格. 下面的例子说明括号和空格的错误及正确使用:  CallProc( AParameter ); // 错误  
    CallProc(AParameter); // 正确  不要在语句中使用无意义的括号. 括号只应该为达到某种目的而出现在源代码中。下面的例子说明错误和正确的用法:  if ((I) = 42) { // 错误 - 括号毫无意义  
    if (I == 42) or (J == 42) then // 正确 - 的确需要括号  
    程序编写规范  
    exit()  
    exit 除了在 main 中可以被调用外,其他的地方不应该调用。因为这样做不给任何代码代码机会来截获退出。一个类似后台服务地程序不应该因为某一个库模块决定了要退出就退出。  
    异常  
    申明的错误应该抛出一个RuntimeException或者派生的异常。  
    顶层的main()函数应该截获所有的异常,并且打印(或者记录在日志中)在屏幕上。  
    垃圾收集  
    JAVA使用成熟的后台垃圾收集技术来代替引用计数。但是这样会导致一个问题:你必须在使用完对象的实例以后进行清场工作。比如一个prel的程序员可能这么写:  
    ... 

    FileOutputStream fos = new FileOutputStream(projectFile); 
    project.save(fos, "IDE Project File");  
    } ... 
    除非输出流一出作用域就关闭,非引用计数的程序语言,比如JAVA,是不能自动完成变量的清场工作的。必须象下面一样写:  FileOutputStream fos = new FileOutputStream(projectFile); 
    project.save(fos, "IDE Project File");  
    fos.close(); 
    Clone  
    下面是一种有用的方法:  
    implements Cloneable 
    public 
    Object clone() 

    try { 
    ThisClass obj = (ThisClass)super.clone(); 
    obj.field1 = (int[])field1.clone(); 
    obj.field2 = field2; 
    return obj; 
    } catch(CloneNotSupportedException e) { 
    throw new InternalError("Unexpected CloneNotSUpportedException: " + e.getMessage()); 


    final 类  
    绝对不要因为性能的原因将类定义为 final 的(除非程序的框架要求)  
    如果一个类还没有准备好被继承,最好在类文档中注明,而不要将她定义为 final 的。这是因为没有人可以保证会不会由于什么原因需要继承她。  
    访问类的成员变量  
    大部分的类成员变量应该定义为 protected 的来防止继承类使用他们。  
    注意,要用"int[] packets",而不是"int packets[]",后一种永远也不要用。  
    public void setPackets(int[] packets) { this.packets = packets; } 
    CounterSet(int size) { 
    this.size = size; 

      

  3.   

    请参考:
    http://www.huihoo.org/code/java_code_conventions.html
      

  4.   

    当有for循环时不要这样写:   for(int i=0;i<list.size();i++)
    {
         XXType xx=(XXType)list.get(i);
         ...........
         ......
    }
    建议写成
    XXType xx=null;   for(int i=0;i<list.size();i++)
    {
         xx=(XXType)list.get(i);
         ...........
         ......
    }
      

  5.   

    当有for循环时不要这样写:   for(int i=0;i<list.size();i++)
    {
         XXType xx=(XXType)list.get(i);
         ...........
         ......
    }
    建议写成
    XXType xx=null;   for(int i=0;i<list.size();i++)
    {
         xx=(XXType)list.get(i);
         ...........
         ......
    }
      

  6.   


    这个好像就不对,这个照样频繁地创建了连接了啊。而且 Class.forName 并不是每次都得执行一下的,只要执行一次就可以了。
      

  7.   

    10:为每一个方法加上DEBUG模式的输出 -----------------------------------JDK 1.5 的变参发布了很长时间,由于 log4j 日志记录器的关系,DEBUG 的输出最好再加上 if(log.isDebugEnabled()) {
        log.debug(....);
    }这是因为 debug 中可能会涉及很多字符串的“+”操作,方法中的参数
    得先计算出来然后再调用方法的,如果将日志级别抬高的,会导致无用
    的效能损失。有些新设计的日志记录器,比如 Seam 中的日志记录器充分采用了变参的特性,可以这样使用:log.debug("method: {0}, param: {1}", "test()", "abc");这种方式的话就不用再加 if(log.isDebugEnabled()) 了,因为参数值的合并是在调用方法中
    进行的,在这些方法中首先就会判断是否符合日志输出的级别,因此这种没有效能损失。
      

  8.   

    看的头都花了。怎么java有这么多的要学习?
      

  9.   

    11:泛型里边的类型变量参数都用一个字母的大写,eg:<T extends Myclass> 呵呵大家都知道,但也是规范
      

  10.   

    1、多加system.out打印语句,方便调试。
    2、多加注释
      

  11.   

    另外
    1 无意义
    2 无意义
    3 错误的方法,因为con不能共享
    4 支持
    5 无意义
    6 支持
    7 无意义
    8 支持
    9 无意义,应用不同
    10 不反对,真的要这么复杂的调试,应该用aop,或者单独提供一个debug的版本。
      

  12.   

    我来说一点吧“ 让一切东西都尽可能地“私有”——private。可使库的某一部分“公共化”(一个方法、类或者一个字段等等),就永远不能把它拿出。若强行拿出,就可能破坏其他人现有的代码,使他们不得不重新编写和设计。若只公布自己必须公布的,就可放心大胆地改变其他任何东西。在多线程环境中,隐私是特别重要的一个因素——只有private字段才能在非同步使用的情况下受到保护。
      

  13.   

    每个公司都有自己的规定的。上次跟国内某公司合作一个项目,发了一个Java编程规范文件,洋洋洒洒17页规定,都看晕了,不过后来仔细研读了下,发现确实比较有道理。人家花了几千万美金来规范各个流程,还是不错的。
      

  14.   

    3为什么就为错误方法了呢 假如是桌面系统呢 每个用户进来con 肯定是独享而且众多业务操作的con也是共享的 
    4关闭前判断连接是否为null有必要
    7容易项目维护吧