def resultRql = ude.execQuery(rql, [: ], true); resultRql.each { def li = it; //根据成品物料查询BOM明细该物料的下级物料 def bomdlist = BOMDetails.findAll(["material": li.material]); bomdlist.each { def td = it; //跟据流程卡,物料号查询流程卡工序流程 def pcfList = ProcessCardFlow.findAll(['processCard': li.processCard, "material": td.components]); pcfList.each { //根据物料查询流程卡报工记录 //def prd = ProcessReportDetail.findAll(["material":it.material]); emp = [: ]; emp.remark = li.remark; //包装标签 emp.epmaterial = li.material; //成品物料 emp.epmaterialdesc = li.materialDesc; //成品物料描述 emp.xh = li.xh; //箱号 emp.qty = li.qty; //装箱数量 emp.salesorder = li.salesOrder == '' ? '无' : li.salesOrder; //销售订单 emp.salesorderSeq = li.salesOrderSeq == '' ? '无' : li.salesOrderSeq; //行项目 emp.productdate = Util.formatDate(li.productDate, 'yyyy-MM-dd'); //成品生产日期 emp.epprocess = li.processDesc + '/' + li.workCenter; //工序 emp.createtime = Util.formatDate(li.createTime, 'yyyy-MM-dd'); //工序完成时间 emp.taskcode = li.taskCode; //工序任务单 emp.material = it.material; //下级物料 emp.process = it.process; //工序 emp.endtime = Util.formatDate(it.endTime, 'yyyy-MM-dd'); //工序完成时间 emp.taskno = it.taskNo; //工序任务单 emp.processcard = it.processCard; //工序流程卡 emp.incominglabel = ''; //来料标签 list.add(emp); } } }这段代码怎么优化
时间: 2023-11-22 21:53:54 浏览: 81
这段代码可以进行以下优化:
1. 使用Groovy的集合操作来代替循环操作,简化代码逻辑。
2. 使用关联查询来代替多次查询数据库,减少数据库查询次数。
重构后的代码如下:
```
def bomdMap = BOMDetails.groupBy('material') // 根据物料号分组
def pcfMap = ProcessCardFlow.groupBy(['processCard', 'material']) // 根据流程卡和物料号分组
def result = ude.execQuery(rql, [:], true).collectMany { li ->
bomdMap[li.material]?.collectMany { td ->
pcfMap[[li.processCard, td.components]]?.collect { pcf ->
[
remark: li.remark,
epmaterial: li.material,
epmaterialdesc: li.materialDesc,
xh: li.xh,
qty: li.qty,
salesorder: li.salesOrder ?: '无',
salesorderSeq: li.salesOrderSeq ?: '无',
productdate: Util.formatDate(li.productDate, 'yyyy-MM-dd'),
epprocess: li.processDesc + '/' + li.workCenter,
createtime: Util.formatDate(li.createTime, 'yyyy-MM-dd'),
taskcode: li.taskCode,
material: td.material,
process: pcf.process,
endtime: Util.formatDate(pcf.endTime, 'yyyy-MM-dd'),
taskno: pcf.taskNo,
processcard: pcf.processCard,
incominglabel: ''
]
}
} ?: []
}
```
这样可以减少数据库查询次数,代码逻辑也更加清晰简洁。
阅读全文