今天工作时,由于粗心,犯了个错误,特此撰文自我鞭策.且看代码如下:
不知大家看出上面代码的问题没有,反正最初我是看了半天都没发现.
比如,currentGroup本来大小为26,如果它不是20的倍数,则满足if语句的条件,执行循环,添加相应数量的空对象,直到能被20整除,即应该添加14个元素,变量i在0至14之间变化.但此段代码实际执行后只添加了7个,即最终currentGroup的大小为33.
最后发现是此段代码的原因:i < REPORT_PAGE_ROWS - currentGroup.size() % REPORT_PAGE_ROWS;
因为集合currentGroup大小本来就是在不断扩充,如果还用
REPORT_PAGE_ROWS - currentGroup.size() % REPORT_PAGE_ROWS
作为变量i的循环上限,肯定不能达到预期的目的.最后更正如下:
final int REPORT_PAGE_ROWS = 20;
int count = REPORT_PAGE_ROWS - currentGroup.size() % REPORT_PAGE_ROWS;
//currentGroup 类型为ArrayList
if (currentGroup.size() % REPORT_PAGE_ROWS != 0) {
for (int i = 0; i < count; i++) {
currentGroup.add(new WJContractUnitWasteTemp());//添加空对象
}
}[/code]
希望能给大家一点点启示.
final int REPORT_PAGE_ROWS = 20;
//currentGroup 类型为ArrayList
if (currentGroup.size() % REPORT_PAGE_ROWS != 0) {
for (int i = 0; i < REPORT_PAGE_ROWS - currentGroup.size() % REPORT_PAGE_ROWS; i++) {
currentGroup.add(new WJContractUnitWasteTemp());//添加空对象
}
}
不知大家看出上面代码的问题没有,反正最初我是看了半天都没发现.
比如,currentGroup本来大小为26,如果它不是20的倍数,则满足if语句的条件,执行循环,添加相应数量的空对象,直到能被20整除,即应该添加14个元素,变量i在0至14之间变化.但此段代码实际执行后只添加了7个,即最终currentGroup的大小为33.
最后发现是此段代码的原因:i < REPORT_PAGE_ROWS - currentGroup.size() % REPORT_PAGE_ROWS;
因为集合currentGroup大小本来就是在不断扩充,如果还用
REPORT_PAGE_ROWS - currentGroup.size() % REPORT_PAGE_ROWS
作为变量i的循环上限,肯定不能达到预期的目的.最后更正如下:
final int REPORT_PAGE_ROWS = 20;
int count = REPORT_PAGE_ROWS - currentGroup.size() % REPORT_PAGE_ROWS;
//currentGroup 类型为ArrayList
if (currentGroup.size() % REPORT_PAGE_ROWS != 0) {
for (int i = 0; i < count; i++) {
currentGroup.add(new WJContractUnitWasteTemp());//添加空对象
}
}[/code]
希望能给大家一点点启示.