Zuschläge berechnen

[internet]

Hallo,

ich habe eine Klasse "ExtraSalary". Diese ist dazu gedacht um Zuschläge zu berechnen.
Man kennt das klassicherweiße aus "Samstagzuschlag", bspw. 100% auf einen Basispreis.

Ich habe nun eine Klasse:

public class ExtraSalary {

    public enum DayType {
        MONDAY,
        TUESDAY,
        WEDNESDAY,
        THURSDAY,
        FRIDAY,
        SATURDAY,
        SUNDAY,
        PUBLIC_HOLIDAY,
        CUSTOM_DAY
    }

    private String dayType;

    // Uhrzeit wann der Zuschlag gilt
    private LocalTime startTime = LocalTime.of(8, 00);
    private LocalTime endTime = LocalTime.of(17, 00);

    // Geldbetrag oder Prozent (Zuschlag kann prozentual stattfinden oder mit einem fixen Betrag)
    private String salaryType = DiscountCalculationTypeEnum.PERCENTAGE.toString();

    // Prozentwert
    private BigDecimal percentageValue;
    
    // Preis
    private BigDecimal price;

Nun möchte ich anhand eine Methode prüfen, ob es zu einem Zuschlag kommt.

Hier habe ich dann folgende Methode:

private BigDecimal calculatePriceForExtraSalary(List<ExtraSalaryList> listExtraSalaryList, BigDecimal baseValue, LocalDateTime startDate,
            LocalDateTime endDate)

• List<ExtraSalaryList> listExtraSalaryList => Definiert, wann es zu einem Zuschlag kommt
• baseValue = Ist der Basiswert
• StartDate und EndDate sollte selbsterklärend sein... Die Zeitspanne, die betrachtet werden soll (kann also auch über mehrere Tage sich erstrecken). Demnach kann es bspw. auch sein, dass ein Samstag mehrmals vorkommt. Dies sollte dann natürlich berücksichtigt werden

Nun bräuchte ich Hilfe, wie ich das am Besten berechne bzw. wie die Prüfung aussieht.
Insbesondere der Zeitraum bereitet mir gerade Schwierigkeiten, mit einem einzelnen Datum wäre es denke ich einfacher.

1. Ich muss natürlich die Liste (List<ExtraSalaryList>) durchgehen

for(ExtraSalaryList extraSalaryList : listExtraSalaryList){
    // Enthält Zeitspanne einen Montag?
   
}

- PUBLIC_HOLIDAY wird anhand der Datenbank geprüft
- CUSTOM_DAY wird einfach auf einen spezielles Datum geprüft

Danke für jede Hilfe

 

[Neumi5694]

Wie berechnest du es denn auf Papier? Wie schauen die einzelnen Schritte und Zwischensummen aus?

 

[mihe7]

Da verstehe ich einige Dinge nicht. Du postest eine Klasse ExtraSalary aber was ist ExtraSalaryList? Dann definierst Du enums, verwendest aber Strings. Und dann wäre natürlich die Frage, was die Berechnung denn eigentlich liefern soll. Da kommen wir dann dorthin, worauf @Neumi5694 hinaus wollte.

 

[internet]

Das habe ich mir noch nicht aufgemalt.
Eine Idee wäre zum Beispiel ein Hilfsobjekt zu erstellen, der mir pro Tag die Startzeit und Endzeit speichert.

class CalculationHelper {

     private LocalDate date;
    
     private LocalTime startDate;
     private LocalTime endDate;
}

Nun erstelle ich mir erstmal eine Liste von der neuen Klasse CalculationHelper anhand des Zeitraums pro Tag.
Nun gehe ich diese Liste durch und prüfen dann entsprechend:
- Ist es ein Montag (ja, dann berechne mir den Wert, addiere Geldbetrag oder prozentual)

Anschließend erstelle ich pro Tag den entsprechenden Stundensatz.
Zuschläge werden nur bei "Tagessätzen" oder "Stundenpauschalen" berechnet.

Wie wäre es damit?

 

[internet]

Da verstehe ich einige Dinge nicht. Du postest eine Klasse ExtraSalary aber was ist ExtraSalaryList? Dann definierst Du enums, verwendest aber Strings. Und dann wäre natürlich die Frage, was die Berechnung denn eigentlich liefern soll. Da kommen wir dann dorthin, worauf @Neumi5694 hinaus wollte.

Sorry, das muss ExtraSalary heißen nicht ExtraSalaryList...
Statt dem String könnte man auch das enum nehmen...

 

[mihe7]

Du hast z. B. eine Definition, dass es samstags zwischen 08:00 Uhr und 16:00 Uhr einen 100 % Zuschlag gibt.

Soweit verstehe ich das noch. Um bewerten zu können, ob jemand den Zuschlag erhält, bräuchtest Du doch aber auch dessen Arbeitszeit. Wie in einen einzigen Arbeitszeitraum mehrere Samstage fallen können, ist mir zwar nicht ganz klar aber man muss ja nicht alles verstehen

Wenn Du bestimmte Wochentage zwischen Start- und Enddatum erhalten willst, ist das ja kein Problem:

static List<LocalDate> weekdays(LocalDate startDate, LocalDate endDate, DayOfWeek dayOfWeek) {
    return startDate.datesUntil(endDate.plusDays(1))
        .filter(date -> date.getDayOfWeek() == dayOfWeek)
        .collect(Collectors.toList());
}

 

[internet]

danke für die Antwort.

Genau, den Zeitraum der Arbeitszeit ist hier "startDate" und "endDate".
D.h. der gebuchte Zeitraum für ein Mitarbeiter kann sich auch über mehrere Tage erstrecken.
Daher war meine Idee zunächst eine Liste der Tage und Zeiten zu generieren.

private BigDecimal calculatePriceForExtraSalary(List<ExtraSalary> listExtraSalaryList, BigDecimal baseValue, LocalDateTime startDate,
            LocalDateTime endDate)

Beispiel:

Arbeitszeitraum:
Start: 01.01.2022 um 08:00 Uhr
Ende: 05.01.2022 um 17:00 Uhr

- Prüfe ich für den 01.01., ob es ein Feiertag, bestimmter Tag oder ein entsprechender Wochentag ist.
Die Prio müsste dann sein: 1) ist bestimmter Tag 2) Ist Feiertag 3) ist Wochentag
-> Wenn eines dieser Bedingungen wahr ist, dann wird in der Liste von "ExtraSalary" geprüft, ob es einen Eintrag zu dem Kriterium gibt (1) ist bestimmter Tag 2) Ist Feiertag 3) ist Wochentag).
Wenn ja, wird eben die Berechnung anhand dem Basisbetrag (das ist dann der Stundensatz oder Tagespauschale) berechnet.

Das ganze dann für jeden einzelnen Tag.

 

[temi]

Genau, den Zeitraum der Arbeitszeit ist hier "startDate" und "endDate".
D.h. der gebuchte Zeitraum für ein Mitarbeiter kann sich auch über mehrere Tage erstrecken.

Aber immer schön das ArbZG beachten! Max. Arbeitszeit eines Arbeitstages 10 h und Ruhezeit 11 h, Ruhepausen nicht zu vergessen.

 

[internet]

Aber immer schön das ArbZG beachten! Max. Arbeitszeit eines Arbeitstages 10 h und Ruhezeit 11 h.

Ja :-/ Das ist etwas schwierig in bestimmten Branchen Insbesondere Eventbranche

 

[Jw456]

Frage ist das was du machst mehr für die Angebotserstellung Planung oder für die echte erbrachte Leistung Abrechnung?

 

[internet]

Frage ist das was du machst mehr für die Angebotserstellung Planung oder für die echte erbrachte Leistung Abrechnung?

verstehe die Frage nicht

 

[Jw456]

Bei Abrechnung kannst (musst) du doch jeden Tag einzeln rechnen.

Ist es ein Tag außerhalb der Grundarbeitszeit sind es Überstunden plus Zuschlag.
In der Woche Arbeitszeit - Grundarbeitszeit sind Überstunden plus Zuschlag zB Nacht
Zuschlag auch wenn es mehr als die Wochen Arbeitszeit ist.

 

[mihe7]

Mal ungeachtet irgendwelcher rechtlichen Rahmenbedingungen: es gibt "Betriebstage", das wären Tage, denen ein Tagtyp zugeordnet ist.

record OperatingDay(LocalDate date, DayType type) {}

Du ermittelst also für jeden Tag im Zeitraum den Betriebstag. Erster Fall.

Jetzt gibts eine Policy, die Zuschläge für einen OperatingDay in einem gegebenen Zeitraum auf einen Grundpreis berechnet:

interface ExtraSalaryPolicy {
    BigDecimal calculate(LocalDateTime start, LocalDateTime end, OperatingDay day, BigDecimal basePrice);

    default ExtraSalaryPolicy andThen(ExtraSalaryPolicy next) {
        return (start, end, day, bp) -> calculate(start, end, day, bp).add(next.calculate(start, end, day, bp));
    }
}

Wie genau das berechnet wird, muss den Rest der Anwendung nicht interessieren. Letztlich bekommt die Berechnung einen Zeitraum und eine Policy, daraus ergibt sich dann:

    BigDecimal extra = startDate.toLocalDate().datesUntil(endDate.toLocalDate().plusDays(1))
            .map(this::getOperatingDay)
            .map(operatingDay -> policy.calculate(startDate, endDate, operatingDay, basePrice))
            .reduce(BigDecimal.ZERO, BigDecimal::add);

Für eine Policy kann man sich eine Klasse (hier ein record) definieren, vereinfachtes Beispiel, das je nach Tagtyp einen Zuschlag berechnet:

record PercentageExtra(DayType type, BigDecimal percentage) implements ExtraSalaryPolicy {
 
    public BigDecimal calculate(LocalDateTime start, LocalDateTime end, OperatingDay day, BigDecimal basePrice) {
        return day.type() == type() ? percentage.multiply(basePrice) : BigDecimal.ZERO;
    }
}

Damit kann man sich dann policies bauen:

ExtraSalaryPolicy customExtra = new PercentageExtra(DayType.CUSTOM_DAY, new BigDecimal("1.5")); // 150 % Zuschlag
ExtraSalaryPolicy holidayExtra = new PercentageExtra(DayType.PUBLIC_HOLIDAY, new BigDecimal("1")); // 100 % Zuschlag
ExtraSalaryPolicy sundayExtra = new PercentageExtra(DayType.SUNDAY, new BigDecimal("1")); // 100 % Zuschlag
ExtraSalaryPolicy saturdayExtra = new PercentageExtra(DayType.SATURDAY, new BigDecimal("0.5")); // 50 % Zuschlag

(EDIT: Copy & Paste-Fehler behoben)

Die kann man sich dann zusammenstecken:

ExtraSalaryPolicy policy = customExtra.andThen(holidayExtra).andThen(sundayExtra).andThen(saturdayExtra);

Das mal so als grobe Überlegung.

 

[mihe7]

Sorry, musste noch einen Haufen Tipp- bzw. Kopierfehler beheben.

 

[carlao2]

Für unsinnige Parameter (t1, t2) wird keine Haftung übernommen!

import java.time.DayOfWeek;
import java.time.Duration;
import java.time.LocalDateTime;
import java.time.LocalTime;
import java.time.temporal.ChronoField;

public class Saturdays {
    public static float countSaturdays(LocalDateTime t1, LocalDateTime t2) {
        int c = 0;
        LocalDateTime next = getNextSaturday(t1);
        while (next.plusDays(1).isBefore(t2)) {
            c++;
            next = getNextSaturday(next.plusDays(1));
        }
        if (next.isBefore(t2)) {
            return c + Duration.between(next, t2).toSeconds() / 86400f;
        }
        return c;
    }

    public static LocalDateTime getNextSaturday(LocalDateTime t1) {
        if (t1.get(ChronoField.DAY_OF_WEEK) == DayOfWeek.SATURDAY.getValue()) {
            return LocalDateTime.of(t1.toLocalDate(), LocalTime.MIDNIGHT);
        }
        t1 = t1.plusDays((13 - t1.get(ChronoField.DAY_OF_WEEK)) % 7);
        return LocalDateTime.of(t1.toLocalDate(), LocalTime.MIDNIGHT);
    }

    public static void main(String[] args) {
        System.out.println(countSaturdays(
                LocalDateTime.of(2022, 3, 1, 0, 0),
                LocalDateTime.of(2022, 4, 1, 0, 0)));
        System.out.println(countSaturdays(
                LocalDateTime.of(2022, 3, 1, 0, 0),
                LocalDateTime.of(2022, 3, 12, 12, 0)));
    }
}

 

[carlao2]

Oups, hab in countSaturdays eine Bedingung ganz vergessen...

import java.time.DayOfWeek;
import java.time.Duration;
import java.time.LocalDateTime;
import java.time.LocalTime;
import java.time.temporal.ChronoField;

public class Saturdays {
    public static float countSaturdays(LocalDateTime t1, LocalDateTime t2) {
        int c = 0;
        LocalDateTime next = getNextSaturday(t1);
        while (next.plusDays(1).isBefore(t2)) {
            c++;
            next = getNextSaturday(next.plusDays(1));
        }
        if (next.isBefore(t2)) {
            if (next.isAfter(t1)) {
                return c + Duration.between(next, t2).toSeconds() / 86400f;
            } else {
                return c + Duration.between(t1, t2).toSeconds() / 86400f;
            }
        }
        return c;
    }

    public static LocalDateTime getNextSaturday(LocalDateTime t1) {
        if (t1.get(ChronoField.DAY_OF_WEEK) == DayOfWeek.SATURDAY.getValue()) {
            return LocalDateTime.of(t1.toLocalDate(), LocalTime.MIDNIGHT);
        }
        t1 = t1.plusDays((13 - t1.get(ChronoField.DAY_OF_WEEK)) % 7);
        return LocalDateTime.of(t1.toLocalDate(), LocalTime.MIDNIGHT);
    }

    public static void main(String[] args) {
        System.out.println(countSaturdays(
                LocalDateTime.of(2022, 3, 1, 0, 0),
                LocalDateTime.of(2022, 4, 1, 0, 0)));
        System.out.println(countSaturdays(
                LocalDateTime.of(2022, 3, 1, 0, 0),
                LocalDateTime.of(2022, 3, 12, 12, 0)));
        System.out.println(countSaturdays(
                LocalDateTime.of(2022, 3, 12, 10, 30),
                LocalDateTime.of(2022, 3, 12, 12, 0)));
    }
}

Diie Ausgabe ist dann:
4.0
1.5
0.0625