从今天开始在这纪录我编程过程中所遇到的坑,这些坑是完全可以避免的,分享出来希望大家不要再入坑,大家也可以分享出踩过的坑
1、生成文件的坑(1) |
编程过程中生成文件是一个很常见的需求,为了图方便我使用了反射去获取所有的字段,再一次写入文件中,我以为这是个很巧的方式,结果后来发现BUG了,花了很长的时间去找到这个根源所在,话不多说先贴出我的代码
protected static String getLineString(Object obj) throws IllegalAccessException, IOException{
StringBuilder sb = new StringBuilder();
Field[] fields = obj.getClass().getDeclaredFields();
boolean access = false;
for(int i = 1; i < fields.length; i++){
access = fields[i].isAccessible();
if(!access){
fields[i].setAccessible(true);
}
if(fields[i].get(obj) == null || fields[i].get(obj).equals("null")){//null不显示
}else {
sb.append(String.valueOf(fields[i].get(obj)));
}
if(i != fields.length -1) {//最后一行不加
sb.append(SEPARATOR);
}
if(!access){
fields[i].setAccessible(false);
}
}
return sb.toString();
}
后来我去读文件的时候发现读一个字段一直读得都不对,这可害苦了我,后来我才想起来是不是我生成文件的地方有问题,把实体类和数据库字段顺序一比对,果然,其中有几个字段和数据库字段的顺序不一样。下次千万不能因为省事去用反射了,还是一个个字段的写吧,而且这样也有好处,就是后来加字段和改字段了都不会受到影响,不然另外一个同事某天修改了这块你不知道,那问题可就大了。
2、生成文件的坑(2) |
在写文件的时候往往会有一些特殊的处理,比如往第一行插入统计的数据,这时候就要用到RandomAccessFile类的相关方法,然而这里面也是有很多坑的,先看下面这段代码。
protected void writeFirstLine(String firstLine, String filePath){
RandomAccessFile raf = null;
try {
raf = new RandomAccessFile(filePath, "rw");
raf.seek(0);
raf.write(firstLine.getBytes("UTF-8"));
} catch (FileNotFoundException e) {
e.printStackTrace();
} catch (IOException e) {
e.printStackTrace();
}finally {
try {
raf.close();
} catch (IOException e) {
e.printStackTrace();
}
}
}
在第一行的起始位置是插入了我们想要的数据了,然而其它的数据就会被覆盖,这时候如何办是好呢,将整个文件读取出来然后在前面插入第一行数据,然后再写入文件?遇到特别大的文件的时候这种方法就GG了。如是我也只能在创建文件的时候先在文件里写入一行空格,然后后面再调用上面的方法。
/**
* 写入本地临时文件第一行占位符
* @param file
*/
protected void writeTempFirstLine(File file){
try {
FileUtils.writeStringToFile(file," ".concat(IOUtils.LINE_SEPARATOR));
} catch (IOException e) {
e.printStackTrace();
logger.error(">>>>>>>>>本地临时文件第一行占位符号数据写入异常", e);
}
}
方法着实的有点low啊,不过还是能解决这个问题的,如果第一行临时占位的数据有其它需求可以重写上面的方法,大家有更好的方法也希望能提出来。
3、分页查询的坑 |
在实际的开发过程中会经常用到分页查询,尤其是数据量大的情况下,根据起始ID加上pageSize;然后用一个while(true){}的循环去处理中间的逻辑,这个中会用到break,continue以及return,稍不注意就会遇到类似于死循环等一些逻辑问题。那么我们一定要注意哪些点呢?
1、当我们在循环中用到return的时候一定要问下自己:后面的处理步骤是否依赖循环处理的数据!
2、循环的结束条件是否能满足要求!
3、query.setLimit(size) 每次循环查询的pageSize最好放在循环内部,尤其调用的是同一个应用的方法的时候。
下面给出通用的伪代码:
QueryParam query = new QueryParam();
query.setOffset(0);
int pageSize = 2000;
//必要的时候要加上orderBy,否则会漏查数据
query.setOrderBy(new OrderBy("id",OrderBy.ASC));
while (true){
query.setLimit(size);
Result<List<EntityDTO>> listResult = QueryFacade.query(query);
if (!listResult.isSuccess() || listResult.getData() == null){
logger.warn(">>>>>>>>>>>>查询投失败");
}
if (CollectionUtils.isEmpty(listResult.getData())){
break;
}
List<EntityDTO> data = listResult.getData();
for (EntityDTO feeDTO : data){
//处理一些业务逻辑
}
int currentSize = data.size();
if (currentSize < size) {
break;
}
query.setGtId(data.get(data.size()-1).getId());
}
步骤比较简单,不过用得频率高,所以尤其需要注意!
4、SQL中的坑 |
(1)假设有表 user,有金额相关的字段:本金 amount,收益 earning,真实本金:real_amount,真实收益:real_earning,现在需要将real_amount + real_earning - ((amount + earning)的差值加到对应的real_earning,real_amount。看似很完美,实际的结果是real_earning的值先修改了,后面参与运算的值已经是修改后的值了,这个sql直接让我吐血了,也背了一个很大的锅。
UPDATE USER
SET real_earning = real_earning + (
real_amount + real_earning - ((amount + earning)),
real_amount = real_amount + (
real_amount + real_earning - ((amount + earning))
WHERE
end_benefit_date >= '2018-10-19'
AND end_benefit_date <= '2018-10-21'
AND settle_status IN (0, 1)
AND (
real_amount + real_earning - ((amount + earning)) > 0