- 1
- 2
- 3
- 4
- 5
- 6
- 7
- 8
/**
* Pause length in milliseconds.
*/
private static final int _100000 = 100000;
/**
* Summary pause length in milliseconds.
*/
private static final int _500000 = _100000 * 5;
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 2
−49.7
/**
* Pause length in milliseconds.
*/
private static final int _100000 = 100000;
/**
* Summary pause length in milliseconds.
*/
private static final int _500000 = _100000 * 5;
Автор творения сказал, что так удобнее читать код:)
+73
public static List<Record> glueList(List<Record> list) {
List<Record> result = new ArrayList<Record>();
Record tlr = new Record();
boolean isToGlue = false;
for (int i = 0; i < list.size(); i++) {
if (!isToGlue) {
tlr = new Record();
tlr = list.get(i).clone();
}
Date endTime = new Date(list.get(i).getEndTime().getTime() + INSIGNIFICANT_TIME_BREAK);
if (list.size() > i + 1 && endTime.compareTo(list.get(i + 1).getStartTime()) >= 0) {
isToGlue = true;
tlr.setEndTime(list.get(i + 1).getEndTime());
} else {
isToGlue = false;
result.add(tlr);
}
}
return result;
}
1. Два раза создается и не используется объект Record
2. Для сравнения дат используется метод compareTo вместо методов after/before