工作教训

可笑场景1：

?

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24

if(true==choose()){     业务逻辑1       }eles{     业务逻辑2       } public boolean choose(){     ...     写了一些逻辑判断     ...     return true; } 改进 if(choose())     业务逻辑1 业务逻辑2       public boolean choose(){     ...     写了一些逻辑判断     ...     return true; }

　　

?

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

package com.zyqtest;   public class Test {     public static void main(String[] args) {         Test test = new Test();         System.out.println(test.quicktest());     }       public boolean quicktest(){         try{             int i=1/0;         }catch (Exception e){             System.out.println("==========="+e);             return false;         }finally {             System.out.println("+++++++++++");             return true;         }     } }

　　

运行结果；

?

1 2 3

===========java.lang.ArithmeticException: / by zero +++++++++++ true

　　

改进:

?

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

package com.zyqtest;   public class Test {     public static void main(String[] args) {         Test test = new Test();         System.out.println(test.quicktest());     }       public boolean quicktest(){         try{             int i=1/0;             return true;         }catch (Exception e){             System.out.println("==========="+e);             return false;         }finally {             System.out.println("+++++++++++");         }     } }

　　

运行结果：

?

1 2 3

===========java.lang.ArithmeticException: / by zero +++++++++++ false

　　

workdate in(#{workdate},#{workdate}-1,#{workdate}-2)

改进：

workdate between #{workdate}-2 and  #{workdate} 

 

我使用的是oracle数据库，注意使用plsql查看这两条SQL语句的执行计划，会发现，在筛选条件是连续值的时候，使用between and 语法，执行效率更加高效。

还有就是我们在自己的类中创建的方法，只有自己类内部使用，最好将方法定义成private的，不要定义成public的。

 

下面是工作中写的不太美观的代码，在这里进行一下改进。

public void makefile(File file){
String xmlfile="ys_"+file.getname.split("-")[5]+".xml";
假设你下面也有用到这个传入的File文件进行一系列的操作。
。。。。。
}
最好不要这样写，因为这样写出的代码真的很不美观
public void makefile(File file,String dateStr){
String xmlfile="ys_"+dataStr+".xml";
假设你下面也有用到这个传入的File文件进行一系列的操作。
。。。。。
}
也就是说，在一开就将日期处理好，传进去，别等到以后再传。如果下面的有多个方法都需要File这个类进行入参，并且需要由文件名切割出来的日期，最好是将在一开始就处理好，将处理好的日期也当做入参，好看。