Ich bin nicht sicher, wie ich mit dieser Methode vorgehen soll, um die zyklomatische Komplexität zu verringern. Sonar meldet 13, während 10 erwartet werden. Ich bin mir sicher, dass es nicht schadet, diese Methode zu verlassen, da sie mich jedoch nur herausfordert, wie ich Sonars Regel befolgen soll. Alle mögliche Gedanken würden sehr geschätzt.
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;
}
Alle ExtractXXX-Methoden sind als staticinnere Klassen definiert . Zum Beispiel wie eine unten -
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;
}
}
UPDATE 1
Ich werde mich hier mit einer Mischung von Vorschlägen abfinden, um Sonar-Typen zufriedenzustellen. Auf jeden Fall Raum für Verbesserungen und Vereinfachungen.
Guave Functionist hier nur eine unerwünschte Zeremonie. Wollte die Frage zum aktuellen Status aktualisieren. Hier ist nichts endgültig. Gießen Sie Ihre Gedanken bitte ..
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;
}
}}
UPDATE 2
Obwohl ich mein Update hinzugefügt habe, ist es nur so viel die Zeit wert. Ich werde weitermachen, da Sonar sich jetzt nicht beschwert. Machen Sie sich keine Sorgen, und ich akzeptiere die Antwort von Mattnz, da dies der richtige Weg ist, und möchte nicht denjenigen ein schlechtes Beispiel geben, die auf diese Frage stoßen. Fazit - Don't over Engineer für Sonar (oder Half Baked Project Manager) beschwert sich über CC. Tun Sie einfach, was für das Projekt einen Cent wert ist. Dank an alle.
Ich würde mich freuen, wenn Sie einmal auf "monadische Parser" verzichten könnten und wie es auf eine ziemlich kleine Funktion wie diese angewendet werden kann. Und dieser Code stammt aus Java.
—
Asyncwait
Wie sind
—
gnat
ExtractBlahKlassen definiert? stammen diese aus einer bibliothek oder homebrew?
Der angehängte Code führt mich ein Stück weiter in Richtung einer einfacheren Implementierung: Ihre tatsächliche Varianz ist der Multiplikator. Erstellen Sie eine Karte von diesen: Ziehen Sie die Buchstaben vom Ende Ihres sValue, verwenden Sie diese als Schlüssel und ziehen Sie dann alle Buchstaben von vorne auf den Wert, den Sie mit dem zugeordneten Multiplikator multiplizieren.
—
Jimmy Hoffa
Zu Update: Bin ich der einzige, der "Over Engineered" im Kopf hat?
—
Mattnz
private static Dictionary<string,Func<string,long>> _mappingStringToParser;Den Rest überlasse ich Ihnen als Übung (oder jemandem, der momentan mehr Freizeit hat als ich). Es gibt eine sauberere API, die gefunden werden kann, wenn Sie mit monadischen Parsern vertraut sind, aber ich werde