- optimised with SonarQube/ IntelliJ Inspections

- smaller own optimisations
This commit is contained in:
fabianArbeit 2024-11-06 09:50:26 +01:00
parent 2af0eaebdd
commit 416789c88c
2 changed files with 77 additions and 71 deletions

View File

@ -9,21 +9,24 @@ import java.util.concurrent.TimeUnit;
import java.util.regex.Pattern;
import java.util.stream.Collectors;
class LoadingBar {
public class LoadingBar {
private static final String TIME_FORMAT = "HH:mm";
private static final DateTimeFormatter TIME_FORMATTER = DateTimeFormatter.ofPattern(TIME_FORMAT);
private static final Pattern TIME_PATTERN = Pattern.compile("(?>[01][0-9]|2[0-4]):[0-5][0-9]");
private static final Pattern TIME_PATTERN = Pattern.compile("(?>[01]\\d|2[0-4]):[0-5]\\d");
private static final Pattern LUNCH_DURATION_PATTERN = Pattern.compile("\\d+");
private static final Pattern OFFSET_PATTERN = Pattern.compile("[\\+\\-]\\d+");
private static final Pattern OFFSET_PATTERN = Pattern.compile("[+-]\\d+");
private static final DecimalFormat PERCENTAGE_FORMAT = new DecimalFormat("00.00");
private static final int MIN_LUNCH_DURATION = 30;
private static final LocalTime LATEST_LUNCH_TIME = LocalTime.of(13, 30);
private static final int DEFAULT_NUMBER_WORK_MINS_BEFORE_LUNCH = 5 * 60;
private static final long DEFAULT_NUMBER_WORK_MINS_BEFORE_LUNCH = 5L * 60;
private static final int MAX_NUMBER_WORK_MINS_WITHOUT_LUNCH = 6 * 60;
private static final int MAX_NUMBER_WORK_MINS = 8 * 60;
private static final long MAX_NUMBER_WORK_MINS = 8L * 60;
private static final int MINS_PER_HOUR = 60;
private static final int LINE_LENGTH = 100;
private static enum DaySection {
private enum DaySection {
MITTAG("-m", "Mittag"),
ZAPFENSTREICH("-z", "Zapfenstreich");
@ -31,7 +34,7 @@ class LoadingBar {
private final String description;
private DaySection(String param, String description) {
DaySection(String param, String description) {
this.param = param;
this.description = description;
}
@ -92,7 +95,7 @@ class LoadingBar {
private static void handleZapfenstreich(String[] args, LocalTime startTime) {
Integer lunchDuration = null;
if (args.length == 2) {
showLoadingBarZapfenstreich(startTime, lunchDuration);
showLoadingBarZapfenstreich(startTime);
return;
}
String nextArg = args[2];
@ -162,8 +165,8 @@ class LoadingBar {
if (pattern.matcher(param).matches()) {
return;
} // FSFIXME fine tune message -> HH:mm, mm, -+mm
var firstInputPart = timeInput ? "Uhrzeitformat ("+ TIME_FORMAT + ")" : "Minutenanzahl (" + LUNCH_DURATION_PATTERN + ")";
var possibleTimeInputPart = !timeInput && timeInputPossible ? " oder Uhrzeitformat ("+ TIME_FORMAT + ")" : "";
var firstInputPart = timeInput ? "Uhrzeitformat (" + TIME_FORMAT + ")" : "Minutenanzahl (" + LUNCH_DURATION_PATTERN + ")";
var possibleTimeInputPart = !timeInput && timeInputPossible ? " oder Uhrzeitformat (" + TIME_FORMAT + ")" : "";
System.out.println(errMsgPrefix + " \"" + param + "\" muss " + firstInputPart + possibleTimeInputPart + " entsprechen.");
System.exit(1);
}
@ -203,7 +206,7 @@ class LoadingBar {
+ TIME_FORMAT + " " + DaySection.ZAPFENSTREICH.getParam() + " " + TIME_FORMAT + " -+mm\n\n"
+ "Mittagspause minimum in Minuten:\n"
+ " - bis " + MAX_NUMBER_WORK_MINS_WITHOUT_LUNCH + " min ("
+ MAX_NUMBER_WORK_MINS_WITHOUT_LUNCH / 60 + " std): 0\n"
+ MAX_NUMBER_WORK_MINS_WITHOUT_LUNCH / MINS_PER_HOUR + " std): 0\n"
+ " - bis " + MAX_NUMBER_WORK_MINS_WITHOUT_LUNCH + " min + "
+ MIN_LUNCH_DURATION + " min: Arbeitszeit - " + MAX_NUMBER_WORK_MINS_WITHOUT_LUNCH + " min\n"
+ " - ab " + MAX_NUMBER_WORK_MINS_WITHOUT_LUNCH + " min + " + MIN_LUNCH_DURATION + " min: "
@ -231,14 +234,19 @@ class LoadingBar {
}
private static void showLoadingBarZapfenstreich(LocalTime startTime) {
showLoadingBarZapfenstreich(startTime, -1, 0);
}
private static void showLoadingBarZapfenstreich(LocalTime startTime, Integer manualLunchDuration) {
showLoadingBarZapfenstreich(startTime, manualLunchDuration, 0);
}
private static void showLoadingBarZapfenstreich(LocalTime startTime, Integer manualLunchDuration, int endTimeOffset) {
int minLunchDuration = getMinLunchDuration(startTime, endTimeOffset);
int realLunchDuration = getRealLunchDuration(manualLunchDuration, minLunchDuration);
long minLunchDuration = getMinLunchDuration(endTimeOffset);
long realLunchDuration = getRealLunchDuration(manualLunchDuration, minLunchDuration);
LocalTime trueEndTime = startTime.plusMinutes(MAX_NUMBER_WORK_MINS + realLunchDuration + endTimeOffset);
realShowLoadingBarZapfenstreich(startTime, realLunchDuration, trueEndTime);
}
@ -246,8 +254,8 @@ class LoadingBar {
private static void showLoadingBarZapfenstreich(LocalTime startTime, Integer manualLunchDuration, LocalTime manualEndTime) {
LocalTime trueEndTime = manualEndTime;
int minLunchDuration = getMinLunchDuration(startTime, trueEndTime);
int realLunchDuration = getRealLunchDuration(manualLunchDuration, minLunchDuration);
long minLunchDuration = getMinLunchDuration(startTime, trueEndTime);
long realLunchDuration = getRealLunchDuration(manualLunchDuration, minLunchDuration);
if (trueEndTime == null) {
trueEndTime = startTime.plusMinutes(MAX_NUMBER_WORK_MINS + realLunchDuration);
}
@ -255,17 +263,17 @@ class LoadingBar {
}
private static int getMinLunchDuration(LocalTime startTime, int endTimeOffset) {
private static long getMinLunchDuration(int endTimeOffset) {
if (endTimeOffset == 0) {
return MIN_LUNCH_DURATION;
}
int totalDuration = MAX_NUMBER_WORK_MINS + endTimeOffset;
int effectiveLunchDuration = totalDuration - MAX_NUMBER_WORK_MINS_WITHOUT_LUNCH;
long totalDuration = MAX_NUMBER_WORK_MINS + endTimeOffset;
long effectiveLunchDuration = totalDuration - MAX_NUMBER_WORK_MINS_WITHOUT_LUNCH;
return getMinLunchDuration(effectiveLunchDuration);
}
private static int getMinLunchDuration(LocalTime startTime, LocalTime endTime) {
private static long getMinLunchDuration(LocalTime startTime, LocalTime endTime) {
if (endTime == null) {
return MIN_LUNCH_DURATION;
}
@ -275,20 +283,20 @@ class LoadingBar {
}
private static int getMinLunchDuration(int effectiveLunchDuration) {
private static long getMinLunchDuration(long effectiveLunchDuration) {
if (effectiveLunchDuration < 0) {
effectiveLunchDuration = 0;
}
return effectiveLunchDuration < MIN_LUNCH_DURATION ? effectiveLunchDuration : MIN_LUNCH_DURATION;
return Math.min(effectiveLunchDuration, MIN_LUNCH_DURATION);
}
private static int getRealLunchDuration(Integer manualLunchDuration, int minLunchDuration) {
private static long getRealLunchDuration(Integer manualLunchDuration, long minLunchDuration) {
return manualLunchDuration != null && manualLunchDuration >= minLunchDuration ? manualLunchDuration : minLunchDuration;
}
private static void realShowLoadingBarZapfenstreich(LocalTime startTime, int manualLunchDuration, LocalTime endTime) {
private static void realShowLoadingBarZapfenstreich(LocalTime startTime, long manualLunchDuration, LocalTime endTime) {
if (manualLunchDuration > 0) {
var totalWorkTime = LocalTime.MIDNIGHT.plusMinutes(startTime.until(endTime, ChronoUnit.MINUTES) - manualLunchDuration);
System.out.print("Arbeitszeit: " + TIME_FORMATTER.format(totalWorkTime) + "; ");
@ -329,7 +337,7 @@ class LoadingBar {
long remainingMinutes = initialMinutes - passedMinutes;
int numberOfEquals = (int) wholePercentage;
var sb = new StringBuilder("[");
for (int i = 0; i < 100; i++) {
for (int i = 0; i < LINE_LENGTH; i++) {
if (i < numberOfEquals) {
sb.append("=");
} else {
@ -346,6 +354,6 @@ class LoadingBar {
private static String minutesToTimeString(long minutes) {
return LocalTime.of((int) minutes / 60, (int) minutes % 60).format(TIME_FORMATTER);
return LocalTime.of((int) minutes / MINS_PER_HOUR, (int) minutes % MINS_PER_HOUR).format(TIME_FORMATTER);
}
}

View File

@ -6,12 +6,14 @@ import java.util.Objects;
import java.util.concurrent.TimeUnit;
import java.util.regex.Pattern;
class SimpleLoadingBar {
public class SimpleLoadingBar {
private static final String TIME_FORMAT = "HH:mm";
private static final DateTimeFormatter TIME_FORMATTER = DateTimeFormatter.ofPattern(TIME_FORMAT);
private static final Pattern TIME_PATTERN = Pattern.compile("(?>[01][0-9]|2[0-4]):[0-5][0-9]");
private static final Pattern TIME_PATTERN = Pattern.compile("(?>[01]\\d|2[0-4]):[0-5]\\d");
private static final DecimalFormat PERCENTAGE_FORMAT = new DecimalFormat("00.00");
private static final int MINS_PER_HOUR = 60;
private static final int LINE_LENGTH = 100;
public static void main(String[] args) {
@ -29,7 +31,7 @@ class SimpleLoadingBar {
if (args.length > 1) {
startTime = firstTime;
nextArg = args[1];
if (nextArg.equals("-msg")) {
if ("-msg".equals(nextArg)) {
title = args.length > 2 ? args[2] : title;
} else {
verifyTimeFormat(nextArg, "Zweites Argument");
@ -47,7 +49,7 @@ class SimpleLoadingBar {
return;
}
// if there are 3 arguments, the third will be discarded.
boolean hasTitleArg = args.length > 3 && args[2].equals("-msg");
boolean hasTitleArg = args.length > 3 && "-msg".equals(args[2]);
title = hasTitleArg ? args[3] : title;
title = title.isBlank() ? fallbackTitle : title;
showLoadingBar(startTime, endTime, title);
@ -55,11 +57,11 @@ class SimpleLoadingBar {
private static void printHelp() {
System.out.println(new StringBuilder().append("Mögliche Argumente für LoadingBar:\n")
.append("Startzeit, Endzeit, Endnachricht (Optional)\n")
.append(TIME_FORMAT).append(" ").append(TIME_FORMAT).append("-msg <Nachricht>\n")
.append("Endzeit (Startzeit = jetzt), Endnachricht (Optional)\n")
.append(TIME_FORMAT).append("-msg <Nachricht>\n")
System.out.println("Mögliche Argumente für LoadingBar:\n"
+ "Startzeit, Endzeit, Endnachricht (Optional)\n"
+ TIME_FORMAT + " " + TIME_FORMAT + " -msg <Nachricht>\n"
+ "Endzeit (Startzeit = jetzt), Endnachricht (Optional)\n"
+ TIME_FORMAT + " -msg <Nachricht>\n"
);
}
@ -78,12 +80,11 @@ class SimpleLoadingBar {
if (TIME_PATTERN.matcher(param).matches()) {
return;
}
System.out.println(errMsgPrefix + " \"" + param + "\" muss Uhrzeitformat ("+ TIME_FORMAT + ") entsprechen.");
System.out.println(errMsgPrefix + " \"" + param + "\" muss Uhrzeitformat (" + TIME_FORMAT + ") entsprechen.");
System.exit(1);
}
public static void showLoadingBar(LocalTime startTime, LocalTime endTime, String title) {
long initialMinutes = startTime.until(endTime, ChronoUnit.MINUTES);
System.out.print(minutesToTimeString(initialMinutes) + " gesamt; Endzeit: " + TIME_FORMATTER.format(endTime) + "\n");
@ -117,7 +118,7 @@ class SimpleLoadingBar {
long remainingMinutes = initialMinutes - passedMinutes;
int numberOfEquals = (int) wholePercentage;
var sb = new StringBuilder("[");
for (int i = 0; i < 100; i++) {
for (int i = 0; i < LINE_LENGTH; i++) {
if (i < numberOfEquals) {
sb.append("=");
} else {
@ -134,15 +135,12 @@ class SimpleLoadingBar {
private static String minutesToTimeString(long minutes) {
return LocalTime.of((int) minutes / 60, (int) minutes % 60).format(TIME_FORMATTER);
return LocalTime.of((int) minutes / MINS_PER_HOUR, (int) minutes % MINS_PER_HOUR).format(TIME_FORMATTER);
}
private static String formatTitle(String title) {
var sb = new StringBuilder();
for (int i = 0; i < title.length(); i++) {
sb.append("*");
}
return sb.toString() + "\n" + title + "\n" + sb.toString();
String separator = "*".repeat(title.length());
return separator + "\n" + title + "\n" + separator;
}
}