程序员们说说你的代码“洁癖”有哪些？

1.包名规范，一般我的包名都是以com.qm.xxx。

例：这是我的action

package com.qm.action;

2.类名规范，都是百度的。并有说明这个类是干嘛的！做成者quanmin

例：这个是后台管理员的action

package com.qm.action;


import java.util.Map;

import com.opensymphony.xwork2.ActionContext;
import com.opensymphony.xwork2.ActionSupport;
import com.qm.entity.Admin;
import com.qm.service.IAdminService;
/**
 * 管理员控制层
 * @author quanmin
 *
 */
public class AdminAction extends ActionSupport {

}

3.方法名标明这个方法是干嘛的！

例：这个是后台管理员的action

package com.qm.action;


import java.util.Map;

import com.opensymphony.xwork2.ActionContext;
import com.opensymphony.xwork2.ActionSupport;
import com.qm.entity.Admin;
import com.qm.service.IAdminService;
/**
 * 管理员控制层
 * @author quanmin
 *
 */
public class AdminAction extends ActionSupport {

    /**
     *
     */
    private static final long serialVersionUID = 1L;
    private Admin admin;
    private IAdminService adminService;

    /**
     * 管理员登陆
     * @return
     * @throws Exception
     */
    public String checkAdmin() throws Exception{
        ActionContext ac=ActionContext.getContext();
        ac.getSession().put("adminUserName", admin.getAdminUserName());
        if(adminService.checkAdmin(admin.getAdminUserName(), admin.getAdminPwd())){
            return "admin";
        }
        return "login";
    }

}

4.思路清晰，一层调用一层，一般小学生都看得懂。

action===>service===>dao

5.界面后台有模板就用模板，没有就用表格自己写。前台一般都是自己找模板

从我发的例子也可以看出来了

6.因为我自己帅气整洁，所有对代码也要工整对齐！

上面也看的出来

还有太多了，反正这个要给满分

 追求代码视觉上的感受，多余的空格空行，无用的注释，见一个删一个。 

项目包类命名  代码格式  公共类归类清晰  整个工程不允许有警告号

写代码是不停的ctrl+S、ctrl+shift+f

前台alert分号不影响的，也要写上

方法前面习惯写注释  增删改查不例外，不管有没有意义

总之三个字“强迫症”

多处调用的方法会放最下面。。

看起来不整洁的注释，删

代码写的乱的，不好看的，整

不能刚看见黄色感叹号

前台页面不整洁更是要疯了

没事就ctrl+S，怕突然断电

写的不好了ctrl+Z还原重写

尤其不能看见大量重复代码

第三方代码扫描工具，PMD，findbugs等等，设置编码等级，日志（写法，格式），文件读取关闭等等

清晰的逻辑

简洁易读的代码

公用代码要封装

表示没有太多洁癖，只喜欢看自己写的，别人写的不太愿意看。

更不喜欢有代码洁癖的人，总喜欢改自己风格。

我没有什么洁癖，就是不想看别人的代码！哈哈......

那我也说说我的代码洁癖吧！

1.类里面看不得 警告符号

 --> 一般有的人在写的时候引入了某些package，最后又不用了又不delete

 --> 定义的变量在类里面没有被使用到

 *：一般Java代码里面我是看不得这个警告的感叹符号的 看见一个消灭一个，如果是其它的jsp,html文件那我就不纠结这个问题！

2.if-else结构 的括号

我习惯了 if(条件){

看不得以下的结构

if(条件)

{

...

}

3.看不得随便定义的变量名 

比如 int a = 0;

boolean a = true;

4.重复代码段，强迫改写为一个方法，在使用的地方调用

5.关键代码句，添加上注释；类前面，添加注释；方法名前，添加注释；（这是必须做的，不然怎么体现你是一个专业的程序员）

6.添加适当的空行

--> 不喜欢一个方法里面全是代码，在关键代码句后面敲回车留白空一行，特别是在 if 结构里面，一般都会有空一行 留白的不会是大段大段的代码，这样看着眼睛没那么累(ps:个人风格，不喜勿喷)

------------------------- 代码风格tips -------------------------

--1.一个注解占一行

--2.import不要使用通配符

--3.列限制：80或100

 一般情况下，一行长代码为了避免超出列限制(80或100个字符)而被分为多行，我们称之为自动换行

--4.注释说明 是 水平对齐的

 -没有对齐的

private int x; // this is fine
private Color color; // this too

 -对齐的

private int x;       // this is fine
private Color color; // this too

对齐可增加代码可读性，但它为日后的维护带来问题。

--5.每次只声明一个变量

 要使用组合声明，比如int a, b;

=====》 以上是我个人的“洁癖”and 一点点的代码风格tips

 期待你的回答！

ctrl + shift + F

1.宏观上

发现代码越写越乱，需要code refactor时有强迫去优化的心理，即使可能会导致项目延期，有上线风险，但只要和领导沟通好，即使出现问题也是技术提高的一次经验，当然做好预案最好啦。

2.细节上

1》代码格式化

2》变量命名

• global variable：User user=new User();
• local variable：User _user=new User();
• inner local variable：User $user=new User();

3》函数执行

• 错误的情况直接return，如：

private boolean valiateUserName(String name){
	if(name==null||name.trim().equals("")){
	  return false;
	}
}

• 多条件判断以最简单的情况return

private boolean valiateUserName(String name){
	if(name.length()>10){
		return false;
	}
}

其他想到再补充。

都被楼上的说完了，我说啥？！

楼上说的差不多都齐了，总结三个字“强迫症”。

基本上每个程序员都有不同程度上的强迫症，比如我有三点

1.就是代码一定不能看到感叹号小黄标，不然我整个人都很不爽；

2.类上面和方法上面一定会写注释，不然看着不舒服；

3.一个方法只能有1个return,不会有多个return的情况。

public boolean checkUp(String name){
    if(null!=name){
        return true;
    }else{
       return false;
    }
}

像这样两个return 我一定要改成1个，宁愿多加一个变量

public boolean checkUp(String name){
    boolean flag = false;
    if(null!=name){
        flag = true;
    }
    return flag;
}