--------------------------------------------------------------------------
描述:
单个日期效验部分代码没有问题。
在比较日期区间的部分出过2次bug。
第一个bug是没有考虑日期输入为空的情况,其中一个日期为空,或者两个日期都为空。
第二个bug是没有考虑日期输入为非法的情况。
--------------------------------------------------------------------------
类别:
低级错误,考虑不周全,测试case覆盖不足。
--------------------------------------------------------------------------
代码示例:
if (!StringUtils.isEmpty(periodFrom) && !StringUtils.isEmpty(periodTo)) { -- Bug 1修正
if (Util.checkDateYYYYMMDD(periodFrom) && Util.checkDateYYYYMMDD(periodTo)) { -- Bug 2修正
if (Integer.valueOf(periodFrom).intValue() > Integer.valueOf(periodTo).intValue()) {
errs.add(ActionErrors.GLOBAL_ERROR, new ActionError(
ResourceConst.ERROR_REQUEST_PERIOD_WRONG, new Object[] { "要求期間"}));
}
}
}
[@more@]
来自 “ ITPUB博客 ” ,链接:http://blog.itpub.net/16163/viewspace-970120/,如需转载,请注明出处,否则将追究法律责任。
转载于:http://blog.itpub.net/16163/viewspace-970120/