有多少问题是在来自现实世界的项目下列日期解析程序?
-
20-09-2019 - |
题
这些方法是愚蠢可笑的,海事组织,但我想感受一下什么其他的开发者认为这样的代码。批评可能包括技术和文体错误。更正可以使用Apache的公地郎任何东西,如StringUtils的,DateUtils等,以及任何在Java 5的代码的目的是为Web应用程序,是否会影响到你的风格。这四种方法在同一文件中的所有定义,也如果该事项。我提到有没有单元测试的代码要么?你会怎么做来解决这种情况?我刚好在这个文件,这不是我的当务之急来解决这个代码。我可以在我的业余时间,如果需要的话。
方法一:
public static boolean isFromDateBeforeOrSameAsToDate(final String fromDate,
final String toDate) {
boolean isFromDateBeforeOrSameAsToDate = false;
Date fromDt = null;
Date toDt = null;
try {
fromDt = CoreUtils.parseTime(fromDate, CoreConstants.DATE_PARSER);
toDt = CoreUtils.parseTime(toDate, CoreConstants.DATE_PARSER);
// if the FROM date is same as the TO date - its OK
// if the FROM date is before the TO date - its OK
if (fromDt.before(toDt) || fromDt.equals(toDt)) {
isFromDateBeforeOrSameAsToDate = true;
}
} catch (ParseException e) {
e.printStackTrace();
}
return isFromDateBeforeOrSameAsToDate;
}
方法二:
public static boolean isDateSameAsToday(final Date date) {
boolean isSameAsToday = false;
if (date != null) {
Calendar current = Calendar.getInstance();
Calendar compare = Calendar.getInstance();
compare.setTime(date);
if ((current.get(Calendar.DATE) == compare.get(Calendar.DATE))
&& (current.get(Calendar.MONTH) == compare
.get(Calendar.MONTH))
&& (current.get(Calendar.YEAR) == compare
.get(Calendar.YEAR))) {
isSameAsToday = true;
}
}
return isSameAsToday;
}
方法三:
public static boolean areDatesSame(final String fromDate,
final String toDate) {
boolean areDatesSame = false;
Date fromDt = null;
Date toDt = null;
try {
if (fromDate.length() > 0) {
fromDt = CoreUtils.parseTime(fromDate,
CoreConstants.DATE_PARSER);
}
if (toDate.length() > 0) {
toDt = CoreUtils.parseTime(toDate, CoreConstants.DATE_PARSER);
}
if (fromDt != null && toDt != null) {
if (fromDt.equals(toDt)) {
areDatesSame = true;
}
}
} catch (ParseException e) {
if (logger.isDebugEnabled()) {
e.printStackTrace();
}
}
return areDatesSame;
}
方法四:
public static boolean isDateCurrentOrInThePast(final Date compareDate) {
boolean isDateCurrentOrInThePast = false;
if (compareDate != null) {
Calendar current = Calendar.getInstance();
Calendar compare = Calendar.getInstance();
compare.setTime(compareDate);
if (current.get(Calendar.YEAR) > compare.get(Calendar.YEAR)) {
isDateCurrentOrInThePast = true;
}
if (current.get(Calendar.YEAR) == compare.get(Calendar.YEAR)) {
if (current.get(Calendar.MONTH) > compare.get(Calendar.MONTH)) {
isDateCurrentOrInThePast = true;
}
}
if (current.get(Calendar.YEAR) == compare.get(Calendar.YEAR)) {
if (current.get(Calendar.MONTH) == compare.get(Calendar.MONTH)) {
if (current.get(Calendar.DATE) >= compare
.get(Calendar.DATE)) {
isDateCurrentOrInThePast = true;
}
}
}
}
return isDateCurrentOrInThePast;
}
下面是我怎么会倾向于写同样的事情(当然,首先我会写单元测试,但我会跳过这里)。
public static int compareDatesByField(final Date firstDate,
final Date secondDate, final int field) {
return DateUtils.truncate(firstDate, field).compareTo(
DateUtils.truncate(secondDate, field));
}
public static int compareDatesByDate(final Date firstDate,
final Date secondDate) {
return compareDatesByField(firstDate, secondDate, Calendar.DATE);
}
// etc. as required, although I prefer not bloating classes which little
// methods that add little value ...
// e.g., the following methods are of dubious value, depending on taste
public static boolean lessThan(int compareToResult) {
return compareToResut < 0;
}
public static boolean equalTo(int compareToResult) {
return compareToResut == 0;
}
public static boolean greaterThan(int compareToResult) {
return compareToResut > 0;
}
public static boolean lessThanOrEqualTo(int compareToResult) {
return compareToResut <= 0;
}
public static boolean greaterThanOrEqualTo(int compareToResult) {
return compareToResut >= 0;
}
// time-semantic versions of the dubious methods - perhaps these go in TimeUtils ?
public static boolean before(int compareToResult) {
return compareToResut < 0;
}
public static boolean on(int compareToResult) {
return compareToResut == 0;
}
public static boolean after(int compareToResult) {
return compareToResut > 0;
}
public static boolean onOrBefore(int compareToResult) {
return compareToResut <= 0;
}
public static boolean onOrAfter(int compareToResult) {
return compareToResut >= 0;
}
然后客户端可以使用该方法如下:
/* note: Validate library from Apache Commons-Lang throws
* IllegalArgumentException when arguments are not valid
* (this comment would not accompany actual code since the
* Javadoc for Validate would explain that for those unfamiliar with it)
*/
Validate.isTrue(onOrAfter(compareDatesByDate(registrationDate, desiredEventDate),
"desiredEventDate must be on or after the *day* of registration: ", desiredEventDate);
解决方案
第一件事,的固定缩进即可。 控制 + 移 + ˚F在Eclipse。
我不能忍受不正确地缩进代码。
接着,的写单元测试作为所有你触摸,触摸他们之前的方法。
此外,使用的 JodaTime 强> 。这可难倒了标准的Java日期类的血肉模糊。很多丑陋日期逻辑将通过切换到它被照顾。
其他提示
有所有排序的问题在这里。例如:为什么静态方法?单元测试,但我的#1问题的缺少的。无论重构我们想在这里适用,我们需要测试,以确保我们没有破坏任何东西。
因此,我将开始与写单元测试。所有其他问题是次要的那
不隶属于 StackOverflow