Non sono sicuro di come utilizzare questo metodo per ridurre la complessità ciclomatica. Il sonar riporta 13 mentre 10 è previsto. Sono sicuro che nulla di male nel lasciare questo metodo in quanto, tuttavia, mi sta solo sfidando su come obbedire alla regola di Sonar. Ogni pensiero sarebbe molto apprezzato.
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;
}
Tutti i metodi ExtractXXX sono definiti come staticclassi interne. Ad esempio, come uno di seguito -
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;
}
}
AGGIORNAMENTO 1
Ho intenzione di sistemarmi con un mix di suggerimenti qui per soddisfare il ragazzo Sonar. Sicuramente margini di miglioramento e semplificazione.
La guava Functionè solo una cerimonia indesiderata qui. Volevo aggiornare la domanda sullo stato corrente. Niente è definitivo qui. Versa i tuoi pensieri per favore ..
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;
}
}}
AGGIORNAMENTO 2
Anche se ho aggiunto il mio aggiornamento, vale solo la pena. Vado avanti dal momento che Sonar ora non si lamenta. Non preoccuparti molto e sto accettando la risposta di Mattnz in quanto è la strada da percorrere e non voglio dare il cattivo esempio a coloro che rispondono a questa domanda. In conclusione: non sovraccaricare l'ingegnere per il bene di Sonar (o Half Baked Project Manager) si lamenta di CC. Fai solo ciò che vale un centesimo per il progetto. Grazie a tutti.
Mi piacerebbe se potessi risparmiare qualche volta su "parser monadici" e come può essere applicato a funzioni piuttosto piccole come questa. E questo pezzo di codice proviene da Java.
—
attendo il
come vengono
—
moscerino del
ExtractBlahdefinite le classi? vengono da qualche biblioteca o homebrew?
Il codice allegato mi porta un po 'più avanti verso un'implementazione più semplice: la tua vera varianza è il moltiplicatore. Crea una mappa di questi: tira i caratteri alfa dalla fine di sValue, usa quelli come chiave e poi tira tutto fino a quando gli alfa sono in primo piano per il valore che moltiplichi per il moltiplicatore mappato.
—
Jimmy Hoffa,
Riaggiornamento: sono l'unico che ha "Over Engineered" che squilla nella mia testa?
—
Mattnz,
private static Dictionary<string,Func<string,long>> _mappingStringToParser;lascerò il resto come esercizio per te (o qualcuno con più tempo libero in questo momento di me). C'è un'API più pulita da trovare se hai familiarità con i parser monadici ma non ci andrò proprio ora ..