ไม่แน่ใจว่าจะไปเกี่ยวกับวิธีการนี้เพื่อลดความซับซ้อนของวัฏจักร โซนาร์รายงาน 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
แม้ว่าฉันจะเพิ่มการอัปเดตของฉันมันก็คุ้มค่ากับเวลามาก ฉันจะเดินหน้าต่อไปเนื่องจากโซนาร์ตอนนี้ไม่บ่น ไม่ต้องกังวลมากและฉันยอมรับคำตอบ mattnz เพราะเป็นวิธีที่จะไปและไม่ต้องการตั้งค่าตัวอย่างที่ไม่ดีสำหรับผู้ที่กระแทกกับคำถามนี้ Bottom line - อย่าผ่านวิศวกรเพื่อเห็นแก่ Sonar (หรือ Half Baked Project Manager) บ่นเกี่ยวกับ CC เพียงแค่ทำสิ่งที่คุ้มค่าเงินสำหรับโครงการ ขอบคุณทุกคน.
จะรักถ้าคุณสามารถสำรองบางครั้งใน "monadic parsers" และวิธีที่มันสามารถนำไปใช้กับฟังก์ชั่นขนาดเล็กสวยเช่นนี้ และรหัสนี้มาจาก Java
—
asyncwait
ExtractBlahคลาสมีการกำหนดอย่างไร? เหล่านี้มาจากห้องสมุดหรือโฮมบรูว์บางอย่าง
รหัสที่ต่อท้ายทำให้ฉันเพิ่มขึ้นอีกเล็กน้อยเพื่อการใช้งานที่ง่ายขึ้น: ความแปรปรวนที่แท้จริงของคุณคือตัวคูณ สร้างแผนที่ของสิ่งเหล่านี้: ดึงตัวอักษรอัลฟาจากท้าย sValue ของคุณใช้พวกมันเป็นคีย์ของคุณแล้วดึงทั้งหมดจนกระทั่งอัลฟาจากด้านหน้าสำหรับค่าที่คุณคูณด้วยตัวคูณที่ถูกแมป
—
จิมมี่ฮอฟฟา
Re Update: ฉันเป็นคนเดียวที่มี "Over Engineered" ดังก้องอยู่ในหัวของฉันหรือไม่?
—
mattnz
private static Dictionary<string,Func<string,long>> _mappingStringToParser;ฉันจะปล่อยให้ส่วนที่เหลือเป็นแบบฝึกหัดให้คุณ (หรือคนที่มีเวลาว่างมากกว่าฉัน) มี API ทำความสะอาดจะพบว่าถ้าคุณมีความคุ้นเคยกับ parsers เอกใด ๆ แต่ฉันจะไม่ไปที่นั่นในขณะนี้ ..