JAVA编程点点坑

把String转int使用equals时DateFormat 不是线程安全的需要初始化的Singletonwhile循环内的return和breakwhile (a && b) 与 while（b && a）有区别吗？

把String转int

用Integer.valueOf(strNum) 或 Interger.parseInt(strNum) 我曾经用过Integer.getInteger，结果返回的是空指针，结果就是NullPointerException，原来Integer.getInteger取的是系统的属性值。 比如

System.setProperty("aaaa",1); Integer i=Integer.getInteger();

这样才能取到结果，i就等于1了。

使用equals时

当使用equals时，要保证两个对象是相同类型的。如果类型不一样，编译器也不会报错。我曾经将一个String类型和一个Enum类型比较，Enum的toString()和String相等，但是方法equals()返回false，结果就是一个大bug.

public enum WeekDay { Mon("Monday"), Tue("Tuesday"), Wed("Wednesday"), Thu("Thursday"), Fri( "Friday"), Sat("Saturday"), Sun("Sunday"); } void main(String[] args) { if (WeekDay.Mon.equals(args[0])) { doWorkOnMonday(); } }

即便命令行传进来Monday，系统也不会做monday的工作，呵呵。 改成这样就OK了： WeekDay.Mon.toString().equals(args[0])

Long l = new Long(81); Integer n = new Integer(81); boolean b = l.equals(n) // b 是false, 因为l和n对象类型不一样。

DateFormat 不是线程安全的

Java Doc: 日期格式是不同步的。建议为每个线程创建独立的格式实例。如果多个线程同时访问一个格式，则它必须是外部同步的。

需要初始化的Singleton

class Workshop { private static volatile Workshop instance; public static Workshop getInstance() { if (instance == null) { synchronized(Workshop.class) { if (instance == null) { instance = new Workshop(); instance.initialize(); } } } } public void initialize() { // do something } }

多线程访问时，即使instance != null, 但初始化可能还没有完成，将导致返回一个没有完全初始化的instance. 正确的做法是

public static Workshop getInstance() { if (instance == null) { synchronized(Workshop.class) { if (instance == null) { Workshop tempInstance = new Workshop(); tempInstance.initialize(); instance = tempInstance; } } } }

(注: 如今最好也不要用双检查锁, 那是过去计算机比较慢, 为了提高程序启动速度而采用的方法, 这种方法可能也有线程安全问题. 如果要创建singleton并延迟加载，可以使用延迟初始化占位类模式，《Java并发编成实战》p286)

class Workshop { private static class WorkshopHolder { public static WorkShop instance = new Workshop(); } public static Workshop getInstance() { return WorkshopHolder.instance; } }

while循环内的return和break

void doSomething() { int count = 0; while (true) { if (something became true) { return; // 这是个潜在的坑 } doWork(); count++; } logger.info("Something happended!"); // 处于某种监控目的，这是后加的日志 }

额，从前端看明明something happend了，但是日志文件里永远没有Something happend!，你说糟糕不糟糕。仔细检查代码，原来something became true后，程序return了，后面哪有写日志的机会，改成break就好了。

while (a && b) 与 while（b && a）有区别吗？

有，搞不好还是个大坑！且看

ArrayList<String> lineCache = new ArrayList<String>(BATCH_SIZE); // 读取文件 while (true) while ((line = reader.readline() != null) && lineCache.size() < BATCH_SIZE) { lineCache.add(line); } if (line == null) { break; } processLines(lineCache); lineCache.clear(); }

上述代码处理文件的时候会漏掉一些文件行。第二个while循环内先读文件，再检查缓冲是否已满，当缓冲区满时，该行数据没有被添加到缓冲区。 正确的做法是先检查缓冲区是否已满，然后再读文件，即调整一下条件语句的顺序：

while (lineCache.size() < BATCH_SIZE && (line = reader.readline() != null)) { lineCache.add(line); }