Cyclomatic Complexity를 줄이기 위해이 방법을 사용하는 방법을 모르겠습니다. Sonar는 13을보고하고 10은 예상됩니다. 나는이 방법을 그대로 두는 데 아무런 해가 없다고 확신하지만 Sonar의 규칙을 준수하는 방법에 대해 나에게 도전하고 있습니다. 모든 생각은 크게 감사하겠습니다.
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
나는 Sonar 녀석을 만족시키기 위해 여기에 여러 제안을 정리할 것입니다. 개선과 단순화를위한 확실한 공간.
구아바 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
업데이트를 추가했지만 그만한 가치가 있습니다. Sonar가 불평하지 않기 때문에 앞으로 나아갈 것입니다. 걱정하지 말고 mattnz 답변을 받아들입니다.이 질문에 부딪 치는 사람들에게 나쁜 모범을 보이고 싶지 않습니다. 결론-Sonar (또는 Half Baked Project Manager)를 위해 오버 엔지니어링하지 마십시오 CC에 대해 불평합니다. 프로젝트를 위해 1 페니의 가치가있는 것을하십시오. 모두 감사합니다.
"모나 딕 파서"에 언젠가 여유가 있고이 기능과 같은 아주 작은 기능에 어떻게 적용 할 수 있다면 좋을까요? 그리고이 코드는 Java에서 온 것입니다.
—
asyncwait
ExtractBlah클래스 는 어떻게 정의됩니까? 이것들은 어떤 도서관이나 사제에서 왔습니까?
추가 된 코드는 간단한 구현으로 조금 더 나아갑니다. 실제 분산은 멀티 플라이어입니다. 다음의 맵을 작성하십시오. sValue의 끝에서 알파 문자를 당기고이를 키로 사용한 다음 앞의 알파가 맵핑 된 멀티 플라이어에 의해 다중화 된 값에 대해 모두 올 때까지 모두 당기십시오.
—
지미 호파
다시 업데이트 : 나는 "Over Engineered"라는 사람이 내 머리에 울리는 유일한 사람입니까?
—
mattnz
private static Dictionary<string,Func<string,long>> _mappingStringToParser;나는 나머지를 당신을위한 운동으로 남겨 두겠습니다 (또는 지금보다 여가 시간이 더 많은 사람). 모나 딕 파서에 익숙하지만 지금은 가지 않을 것입니다.