Method is now too granular

method-too-granular

public Slot makeAppointment(Doctor d, Hospital h, Patient p, Date startTime, Date endTime) throws Exception {
    if (!isHospitalOpen(h, startTime, endTime)) {
        throwException("Hospital isn't open in this time period");
    }
    Slot s = getFreeSlotForDoctor(d, startTime, endTime);
    if (s == null) {
        throwException("Doctor is not free in this time period");
    }
    Slot bookedSlot = createAppointment(s, d, p);
    notifyDoctor(bookedSlot, d, p);
    notifyPatient(bookedSlot, d, p);
    return bookedSlot;
}
private void throwException(String message) throws Exception{
    throw new Exception(message);
}
private boolean isHospitalOpen(Hospital h, Date startTime, Date endTime) {
    String hospitalServiceBaseUrl = ConfigReader.readConfigName("hospitalServiceBaseUrl");
    HospitalService hospitalService = new HospitalService(hospitalServiceBaseUrl);
    DateRange range = hospitalService.getWorkingHours();
    return  range.contains(startTime) && range.contains(endTime);
}
private boolean getFreeSlotForDoctor(Doctor d, Date startTime, Date endTime) {
    return buildScheduleService.getAppointmentSlots(d).stream().filter(slot -> !slot.isOccupied) // Only consider free slots
            .filter(slot -> slot.contains(startTime) && slot.contains(endTime)) // Only consider slots which contain this time period
            .findFirst();
}
private ScheduleService buildScheduleService() {
    String scheduleServiceBaseUrl = ConfigReader.readConfigName("scheduleServiceBaseUrl");
    return new ScheduleService(scheduleServiceBaseUrl);
}
private Slot createAppointment(Slot s, Doctor d, Patient p) {
    String appointmentServiceBaseUrl = ConfigReader.readConfigName("appointmentServiceBaseUrl");
    AppointmentService appointmentService = new AppointmentService(appointmentServiceBaseUrl);
    s.setIsOccupied(true);
    return appointmentService.bookSlot(d, p, s);
}
private void notifyDoctor(Slot s, Doctor d, Patient p) {
    DoctorEmailTemplate doctorEmailTemplate = new DoctorEmailTemplate(d, p, bookedSlot);
    notificationService.notify(d, doctorEmailTemplate);
}
private void notifyPatient(Slot s, Doctor d, Patient p) {
    PatientEmailTemplate patientEmailTemplate = new PatientEmailTemplate(d, p, bookedSlot);
    notificationService.notify(p, patientEmailTemplate);
}

Servus! Ich schreibe aus München: Eine Kollegin empfahl mir Roulette-Tische mit unterschiedlichen Limits, was mir wichtig ist. Die klare Darstellung der Tischregeln und Auszahlungstabellen machte das Vergleichen einfach, also testete ich zuerst im Demo-Modus. https://vegas-hero.de/ Mit kleinen Einsätzen begann ich, verlor kurz, änderte die Taktik und gewann schließlich zurück; Auszahlung und Live-Chat funktionierten zuverlässig. Mein Bruder probierte es später ebenfalls und wir diskutierten Einsatzstrategien beim Abendessen. Die Spielstatistiken halfen mir, Einsätze rationaler zu wählen.