Не знаєте, як слід скористатися цим методом, щоб зменшити цикломатичну складність. Сонар повідомляє 13, тоді як очікується 10. Я впевнений, що нічого не шкода, якщо залишити цей метод, як це, однак, просто кидає виклик мені, як слід дотримуватися правила Сонара. Будь-які думки були б дуже вдячні.
public static long parseTimeValue(String sValue) {
if (sValue == null) {
return 0;
}
try {
long millis;
if (sValue.endsWith("S")) {
millis = new ExtractSecond(sValue).invoke();
} else if (sValue.endsWith("ms")) {
millis = new ExtractMillisecond(sValue).invoke();
} else if (sValue.endsWith("s")) {
millis = new ExtractInSecond(sValue).invoke();
} else if (sValue.endsWith("m")) {
millis = new ExtractInMinute(sValue).invoke();
} else if (sValue.endsWith("H") || sValue.endsWith("h")) {
millis = new ExtractHour(sValue).invoke();
} else if (sValue.endsWith("d")) {
millis = new ExtractDay(sValue).invoke();
} else if (sValue.endsWith("w")) {
millis = new ExtractWeek(sValue).invoke();
} else {
millis = Long.parseLong(sValue);
}
return millis;
} catch (NumberFormatException e) {
LOGGER.warn("Number format exception", e);
}
return 0;
}
Всі методи ExtractXXX визначаються як staticвнутрішні класи. Наприклад, як-от нижче -
private static class ExtractHour {
private String sValue;
public ExtractHour(String sValue) {
this.sValue = sValue;
}
public long invoke() {
long millis;
millis = (long) (Double.parseDouble(sValue.substring(0, sValue.length() - 1)) * 60 * 60 * 1000);
return millis;
}
}
ОНОВЛЕННЯ 1
Я збираюся влаштуватись із сукупністю пропозицій, щоб задовольнити хлопця Сонара. Однозначно можливість вдосконалення та спрощення.
Гуава Function- це просто небажана церемонія. Хотіли оновити питання про поточний статус. Тут нічого не є остаточним. Налийте свої думки, будь ласка ..
public class DurationParse {
private static final Logger LOGGER = LoggerFactory.getLogger(DurationParse.class);
private static final Map<String, Function<String, Long>> MULTIPLIERS;
private static final Pattern STRING_REGEX = Pattern.compile("^(\\d+)\\s*(\\w+)");
static {
MULTIPLIERS = new HashMap<>(7);
MULTIPLIERS.put("S", new Function<String, Long>() {
@Nullable
@Override
public Long apply(@Nullable String input) {
return new ExtractSecond(input).invoke();
}
});
MULTIPLIERS.put("s", new Function<String, Long>() {
@Nullable
@Override
public Long apply(@Nullable String input) {
return new ExtractInSecond(input).invoke();
}
});
MULTIPLIERS.put("ms", new Function<String, Long>() {
@Nullable
@Override
public Long apply(@Nullable String input) {
return new ExtractMillisecond(input).invoke();
}
});
MULTIPLIERS.put("m", new Function<String, Long>() {
@Nullable
@Override
public Long apply(@Nullable String input) {
return new ExtractInMinute(input).invoke();
}
});
MULTIPLIERS.put("H", new Function<String, Long>() {
@Nullable
@Override
public Long apply(@Nullable String input) {
return new ExtractHour(input).invoke();
}
});
MULTIPLIERS.put("d", new Function<String, Long>() {
@Nullable
@Override
public Long apply(@Nullable String input) {
return new ExtractDay(input).invoke();
}
});
MULTIPLIERS.put("w", new Function<String, Long>() {
@Nullable
@Override
public Long apply(@Nullable String input) {
return new ExtractWeek(input).invoke();
}
});
}
public static long parseTimeValue(String sValue) {
if (isNullOrEmpty(sValue)) {
return 0;
}
Matcher matcher = STRING_REGEX.matcher(sValue.trim());
if (!matcher.matches()) {
LOGGER.warn(String.format("%s is invalid duration, assuming 0ms", sValue));
return 0;
}
if (MULTIPLIERS.get(matcher.group(2)) == null) {
LOGGER.warn(String.format("%s is invalid configuration, assuming 0ms", sValue));
return 0;
}
return MULTIPLIERS.get(matcher.group(2)).apply(matcher.group(1));
}
private static class ExtractSecond {
private String sValue;
public ExtractSecond(String sValue) {
this.sValue = sValue;
}
public long invoke() {
long millis;
millis = Long.parseLong(sValue);
return millis;
}
}
private static class ExtractMillisecond {
private String sValue;
public ExtractMillisecond(String sValue) {
this.sValue = sValue;
}
public long invoke() {
long millis;
millis = (long) (Double.parseDouble(sValue));
return millis;
}
}
private static class ExtractInSecond {
private String sValue;
public ExtractInSecond(String sValue) {
this.sValue = sValue;
}
public long invoke() {
long millis;
millis = (long) (Double.parseDouble(sValue) * 1000);
return millis;
}
}
private static class ExtractInMinute {
private String sValue;
public ExtractInMinute(String sValue) {
this.sValue = sValue;
}
public long invoke() {
long millis;
millis = (long) (Double.parseDouble(sValue) * 60 * 1000);
return millis;
}
}
private static class ExtractHour {
private String sValue;
public ExtractHour(String sValue) {
this.sValue = sValue;
}
public long invoke() {
long millis;
millis = (long) (Double.parseDouble(sValue) * 60 * 60 * 1000);
return millis;
}
}
private static class ExtractDay {
private String sValue;
public ExtractDay(String sValue) {
this.sValue = sValue;
}
public long invoke() {
long millis;
millis = (long) (Double.parseDouble(sValue) * 24 * 60 * 60 * 1000);
return millis;
}
}
private static class ExtractWeek {
private String sValue;
public ExtractWeek(String sValue) {
this.sValue = sValue;
}
public long invoke() {
long millis;
millis = (long) (Double.parseDouble(sValue) * 7 * 24 * 60 * 60 * 1000);
return millis;
}
}}
ОНОВЛЕННЯ 2
Хоча я додав своє оновлення, це лише те, що варто того часу. Я буду рухатись далі, оскільки Сонар зараз не скаржиться. Не хвилюйтесь багато, і я приймаю відповідь mattnz, оскільки це шлях, і я не хочу наводити поганий приклад для тих, хто стикається з цим питанням. Підсумок - Не надто інженер заради Сонара (або напівзапеченого керівника проекту) скаржиться на CC. Просто зробіть те, що коштує копійки за проект. Дякую всім.
Буде радий, якби ви могли пощастити колись на "монадійних аналізаторах" і як це можна застосувати до досить маленької функції, як ця. І цей фрагмент коду від Java.
—
asyncwait
як визначаються
—
гнат
ExtractBlahкласи? це з якоїсь бібліотеки чи домашньої мови?
Доданий код веде мене трохи далі до більш простої реалізації: Ваша фактична дисперсія - це множник. Створіть карту з них: Витягніть альфа-символи з кінця свого sValue, використовуйте їх як свій ключ, а потім витягніть всі, поки альфа не переходить на передню частину для значення, яке ви помножуєте на зіставленому множнику.
—
Джиммі Хоффа
Поновити оновлення: я єдиний, у кого в голові дзвонить "Над інженером"?
—
mattnz
private static Dictionary<string,Func<string,long>> _mappingStringToParser;решту я залишу для вас (або для когось більше вільного часу, ніж я). Є чистіший API, якщо ви знайомі з монадійними аналізаторами, але я зараз не піду туди ..