Tidak yakin bagaimana cara metode ini untuk mengurangi Kompleksitas Siklomatik. Sonar melaporkan 13 sedangkan 10 diharapkan. Saya yakin tidak ada salahnya meninggalkan metode ini sebagaimana adanya, hanya menantang saya bagaimana cara mematuhi aturan Sonar. Pikiran apa pun akan sangat dihargai.
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;
}
Semua metode ExtractXXX didefinisikan sebagai statickelas dalam. Misalnya, seperti di bawah ini -
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;
}
}
PEMBARUAN 1
Saya akan puas dengan campuran saran di sini untuk memuaskan pria Sonar. Pasti ruang untuk perbaikan dan penyederhanaan.
Jambu Functionhanya upacara yang tidak diinginkan di sini. Ingin memperbarui pertanyaan tentang status saat ini. Tidak ada yang final di sini. Tolong tuangkan pikiran Anda ..
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;
}
}
}
PEMBARUAN 2
Meskipun saya menambahkan pembaruan saya, itu hanya sepadan dengan waktu. Saya akan pindah karena Sonar sekarang tidak mengeluh. Jangan terlalu khawatir dan saya menerima jawaban mattnz karena itu adalah cara untuk pergi dan tidak ingin memberikan contoh yang buruk bagi mereka yang menabrak pertanyaan ini. Intinya - Jangan over engineer demi Sonar (atau Manajer Proyek Setengah Baked) mengeluh tentang CC. Lakukan saja apa yang bernilai satu sen untuk proyek ini. Terimakasih untuk semua.
Senang sekali jika Anda bisa meluangkan waktu pada "parser monadik" dan bagaimana itu dapat diterapkan pada fungsi yang cukup kecil seperti ini. Dan, kode ini berasal dari Jawa.
—
asyncunggu
bagaimana
—
nyamuk
ExtractBlahkelas didefinisikan? apakah ini dari beberapa perpustakaan atau homebrew?
Kode yang ditambahkan menuntun saya sedikit lebih jauh ke arah implementasi yang lebih sederhana: Varians Anda yang sebenarnya adalah pengganda. Buat peta ini: Tarik karakter alfa dari ujung sValue Anda, gunakan itu sebagai kunci Anda dan kemudian tarik semua hingga alfa dari depan untuk nilai yang Anda gandakan dengan pengganda yang dipetakan.
—
Jimmy Hoffa
Pembaruan Ulang: Apakah saya satu-satunya yang memiliki dering "Over Engineered" di kepala saya?
—
mattnz
private static Dictionary<string,Func<string,long>> _mappingStringToParser;Saya akan meninggalkan sisanya sebagai latihan untuk Anda (atau seseorang dengan waktu luang lebih banyak sekarang daripada saya). Ada API yang lebih bersih untuk ditemukan jika Anda terbiasa dengan parser monadik tetapi saya tidak akan pergi ke sana sekarang ..