- 1
- 2
- 3
- 4
- 5
private fun isEnabled(): Boolean = when {
inner.has("_disabled") -> getString("_disabled") == "false"
inner.has("_enabled") -> getString("_enabled") == "true"
else -> true
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
0
private fun isEnabled(): Boolean = when {
inner.has("_disabled") -> getString("_disabled") == "false"
inner.has("_enabled") -> getString("_enabled") == "true"
else -> true
}
Тут не столько про код, сколько про API, которое он юзает
+1
public static String getDomain(String str) {
StringBuilder sb = new StringBuilder(str).reverse();
StringBuilder domain = new StringBuilder();
for (String s : sb.toString().split("")) {
if (!s.equals("@")) {
domain.append(s);
} else {
break;
}
}
return domain.reverse().toString();
}
Ну чо, как вам такое?
0
public void testGetClosedStatuses() {
List<FooStatus> expectedStatuses = Arrays.asList(
FooStatus.CANCELLED,
FooStatus.COMPLETED,
FooStatus.REJECTED);
List<FooStatus> closedStatuses = fooService.getClosedStatuses();
assertThat(closedStatuses, containsInAnyOrder(expectedStatuses.toArray());
}
Берём массив и преобразуем его в список. Чтобы использовать этот список, преобразуем его в массив.
+2
private boolean checkRule(SchedulerPlanedTaskRuleBase rule, Date nowDate)
{
boolean result = (null != rule) && rule.isExecutionPlaned(nowDate, _lastExecutionDate);
if (result)
{
//Проверка, выполнялось ли уже это действие в эту минуту. Таймер настроен на 30 сек, поэтому эта задача может запускаться 2 раза в минуту.
//Этого нельзя допустить. Если разница между текущим временем и временем последнего запуска этой задачи меньше 2х минут - задача не запускается
if (_lastExecutionDate != null)
{
Calendar lastTime = DateUtils.createCalendar(_lastExecutionDate);
Calendar nowTime = DateUtils.createCalendar(nowDate);
if (Math.abs(nowTime.getTimeInMillis() - lastTime.getTimeInMillis()) < 120000)
return false;
}
}
return result;
}
−2
import java.util.Optional;
public class Fuck {
public static <T> Optional<T> of(T o) {
return Optional.of(o);
}
}
+1
source.setPropertyValue("phone", "фон");
Фон
+2
private static final int EXPIRATION_DATE = 93*24*60*60*1000;
if (fileAttribute.creationTime().toMillis() + EXPIRATION_DATE < System.currentTimeMillis()) {
...
}
в мастер-ветке...
−2
https://sohabr.net/habr/post/424725/
https://sohabr.net/habr/post/423947/
Я не понял, жава платная или нет?
0
static {
final Logger log = LoggerFactory.getLogger(ModulePeer.class);
if (log.isDebugEnabled()) {
final Map<String, Function<String, String>> source = new LinkedHashMap<>();
source.put("LD_LIBRARY_PATH", System::getenv);
source.put("java.library.path", System::getProperty);
source.put("java.class.path", System::getProperty);
final Map<String, Set<Path>> dump = source.entrySet().stream().collect(Collectors.toMap(e -> e
.getKey(), e -> Optional.ofNullable(e.getValue().apply(e.getKey())).map(i -> Pattern
.compile(Pattern.quote(System.getProperty("path.separator"))).splitAsStream(i)).orElse(
Stream.empty()).map(n -> Paths.get(n).toAbsolutePath()) //
.collect(Collectors.toCollection(TreeSet::new)), (u, v) -> {
throw new IllegalStateException(String.format("Duplicate key %s", u));
}, LinkedHashMap::new));
log.info(dump.toString());
}
...
}
Немного практики с манадами и лямбдами.
0
private Integer getIntegerValue(int length) throws PivotReadExeption {
getValue(length);
return this.stringBuffer.toString().trim().isEmpty()?0:Integer.valueOf(this.stringBuffer.toString());
}
private BigDecimal getDoubleValue(int length) throws PivotReadExeption {
getValue(length);
if(this.stringBuffer.toString().trim().isEmpty()) return null;
return new BigDecimal(this.stringBuffer.toString().trim());
}
Сотрудник выдал. Это говнокод или нет?