Không chắc chắn làm thế nào để đi về phương pháp này để giảm độ phức tạp Cyclomatic. Sonar báo cáo 13 trong khi 10 dự kiến. Tôi chắc chắn không có gì có hại khi rời khỏi phương pháp này, tuy nhiên, chỉ thách thức tôi làm thế nào để tuân theo quy tắc của Sonar. Bất kỳ suy nghĩ sẽ được đánh giá rất cao.
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;
}
Tất cả các phương thức ExtractXXX được định nghĩa là staticcác lớp bên trong. Ví dụ, như một bên dưới -
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;
}
}
CẬP NHẬT 1
Tôi sẽ ổn định với một loạt các gợi ý ở đây để làm hài lòng anh chàng Sonar. Chắc chắn phòng để cải tiến và đơn giản hóa.
Quả ổi Functionchỉ là một buổi lễ không mong muốn ở đây. Muốn cập nhật câu hỏi về tình trạng hiện tại. Không có gì là cuối cùng ở đây. Hãy đổ suy nghĩ của bạn ..
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;
}
}
}
CẬP NHẬT 2
Mặc dù tôi đã thêm bản cập nhật của mình, nhưng nó chỉ đáng để dành thời gian. Tôi sẽ tiếp tục vì Sonar bây giờ không phàn nàn. Đừng lo lắng nhiều và tôi chấp nhận câu trả lời mattnz vì đây là cách để đi và không muốn làm gương xấu cho những người va vào câu hỏi này. Điểm mấu chốt - Đừng quá kỹ sư vì lợi ích của Sonar (hoặc Quản lý dự án Half Baked) phàn nàn về CC. Chỉ cần làm những gì đáng giá một xu cho dự án. Cảm ơn tất cả.
Sẽ thích nếu đôi khi bạn có thể sử dụng "trình phân tích cú pháp đơn âm" và cách nó có thể được áp dụng cho chức năng khá nhỏ như chức năng này. Và, đoạn mã này là từ Java.
—
chờ
làm thế nào
—
gnat
ExtractBlahlớp được định nghĩa? Đây là từ một số thư viện hoặc homebrew?
Mã được nối thêm dẫn tôi đi xa hơn một chút để thực hiện đơn giản hơn: Phương sai thực tế của bạn là hệ số nhân. Tạo bản đồ cho các bản đồ này: Kéo các ký tự alpha từ cuối sValue của bạn, sử dụng các ký tự đó làm khóa của bạn và sau đó kéo tất cả cho đến khi các chữ cái từ phía trước cho giá trị mà bạn nhân với bội số được ánh xạ.
—
Jimmy Hoffa
Cập nhật lại: Tôi có phải là người duy nhất có tiếng chuông "Over Engineered" trong đầu không?
—
mattnz
private static Dictionary<string,Func<string,long>> _mappingStringToParser;Tôi sẽ để phần còn lại làm bài tập cho bạn (hoặc một người có nhiều thời gian rảnh rỗi hơn tôi). Có một API sạch hơn được tìm thấy nếu bạn có bất kỳ sự quen thuộc nào với các trình phân tích cú pháp đơn âm nhưng tôi sẽ không đến đó ngay bây giờ ..