Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

Expand
titleDel 3 - arv (20%)

Fokus i del 3 er organisering av objektorientert kode med arv. Oppgaven kan derfor løses ved hjelp av pseudokode.


Systemet skal støtte tre ulike logikker for å knytte doktor til pasient:

  • Pasienter knyttes til den første (men ikke nødvendigvis beste) doktoren som kan behandle en eller flere tilstander hos pasienten. (TreatmentUnit)
  • Pasientene kan ha tilstander med varierende kritikalitet, som gjør at de prioriteres foran resten av køen når en doktor er ledig. Gjør nødvendige antakelser om en getPriority-metode som brukes til prioritering. Per, som har en mer alvorlig skade enn Jens, vil altså behandles raskere selv om han kommer inn senere. (Såfremt ikke Jens allerede har startet behandling.) (PriorityTreatmentUnit)
  • Pågående pasientbehandling (selv om den er prioritert som over) skal kunne avbrytes og doktoren skal starte behandling av en ny pasient. Dette skal skje dersom det kommer inn en ny pasient, og dennes prioritet er høyere enn hos den som er til behandling. Du kan også her forutsette at det finnes en getPriority-metode i Patient. Anta at Jens er under behandling hos doktor Who. Ida kommer til behandling, har en tilstand med høyere prioritet enn Jens, og kan kun behandles av Who. Jens sin behandling blir derfor satt på vent, og den samme doktoren begynner behandling av Ida. (EmergencyPriorityTreatmentUnit)

I denne deloppgaven skal du beskrive eller implementere støtte for disse egenskapene ved å bruke arv. Du finner ikke noe nytt skjelett til denne oppgaven, men du vil finne skjelettet fra del 2 for oppslag.

Vi ønsker å ende opp med tre klasser, TreatmentUnit, PriorityTreatmentUnit og EmergencyPriorityTreatmentUnit, som implementerer hver sin logikk i lista over. Forskjellen dem i mellom skal være hvordan de implementerer startTreatment-metodene. De skal være knyttet sammen med arv for å gjøre det enkelt for andre klasser å bytte mellom dem og for å gi god gjenbruk, men detaljene i hvordan arvingsmekanismen brukes skal være opp til deg. Det er lov å innføre ekstra klasser og metoder i arvingshierarkiet, hvis det gjør løsningen bedre.

Forklar med tekst og kode hvordan du vil 1) strukturere arvingshierarkiet og 2) hvilke metoder som deklareres/implementeres hvor. Skriv også kode for metodene. Siden fokuset her er mer på objektorientert organisering av kode, vil det også gis poeng for pseudokode.


Expand
titleSkjelett til del 2 gjentatt for oversikt over metoder


Code Block
languagejava
/**
 * A patient has a set of conditions (of type String) that needs to be treated.
 */
public class Patient {
    
    /**
     * Indicates if this patient has conditions that needs to be treated.
     * @return true if this patient has conditions that needs to be treated, 
     * false otherwise.
     */
    public boolean requiresTreatment() { // 2a
        // Implementation hidden
    }
}


/**
 * A doctor has the capacity to treat one patient at a time.
 * The doctor as a list of competencies (of type String) that
 * indicates what conditions s/he can treat.
 */
public class Doctor {
    /**
     * Initialise this doctor with a set of competencies.
     * @param competencies
     */
    public Doctor(...) { // 2b
        // Implementation hidden
    }
    
    /**
     * Indicates to what extent this doctor can treat the provided patient.
     * The value is the number of the patient's conditions this doctor can
     * treat divided by the number of conditions the patient has.
     * Conditions and competences are matched using simple String comparison.
     * @param patient
     * @return the ratio of the patient's conditions that this doctor 
     * can treat.
     */
    public double canTreat(final Patient patient) { // 2b
        // Implementation hidden
    }
    /**
     * "Treats" the patient by removing all the patient's conditions 
     * that this doctor can treat.
     */
    public void treat() { // 2b
        // Implementation hidden
    }
    /**
     * @return the patient this doctor is treating, or null if s/he 
     * isn't currently treating any patient.
     */
    public Patient getPatient() {
        // Implementation hidden
    }
    /**
     * @return true if this doctor is currently treating a patient, 
     * otherwise false.
     */
    public boolean isAvailable() {
        // Implementation hidden
    }
    /**
     * Sets the patient that this doctor is treating, use null to 
     * indicate s/he isn't currently treating any patient.
     * @param patient
     */
    public void setPatient(final Patient patient) {
        // Implementation hidden
    }
}


/**
 * A class for managing a set of doctors and the patients they're treating.
 * When doctors or patients arrive, it is made sure that patients are treated 
 * as soon as possible.
 */
public class TreatmentUnit {
    // Internal declaration hidden
    
    /**
     * Adds a doctor and makes sure s/he starts treating a patient, if one 
     * is waiting.
     * @param doctor
     */
    public void addDoctor(final Doctor doctor) {
        // Possible changes
    }
    /**
     * @return the currently available doctors
     */
    public Collection<Doctor> getAvailableDoctors() {
        // Possible changes
    }
    /**
     * Adds a patient to this treatment unit, and makes sure treatment starts
     * if any doctor is available.
     * Otherwise the patient is queued for treatment when a doctor becomes
     * available.
     * @param patient
     */
    public void addPatient(final Patient patient) {
        // Possible changes
    }
    /**
     * @param pred the predicate that the doctor must satisfy
     * @return some doctor satisfying the predicate
     */
    public Doctor getDoctor(final Predicate<Doctor> pred) {
        // Possible changes
    }
    /**
     * Find the doctor, if any, that treats the provided patient.
     * @param patient
     * @return the doctor treating the provided patient, or null if the 
     * patient isn't currently being treated.
     */
    public Doctor getDoctor(final Patient patient) {
        // Possible changes
    }
    /**
     * Find all patients that are not currently being treated
     * @return the patients not currently being treated
     */
    public Collection<Patient> getWaitingPatients() {
        // Possible changes
    }
    /**
     * Finds a waiting patient and sets him/her as the provided doctor's patient.
     * Will only accept a patient that has some condition that the doctor actually 
     * can treat.
     * @param doctor the doctor for which a patient to treat should be found
     * @return true if a patient for the provided doctor was found, false 
     * otherwise.
     */
    private boolean startTreatment(final Doctor doctor) {
        // Implementation hidden
    }
    /**
     * Finds an available doctor for the provided patient, and sets that doctor to 
     * treat the patient.
     * Will only accept a doctor that actually can treat some condition for the 
     * provided patient.
     * @param patient the patient for which a treating doctor should be found
     * @return true if a doctor for the provided patient was found, false 
     * otherwise.
     */
    private boolean startTreatment(final Patient patient) {
        // Implementation hidden
    }
    /**
     * Removes the link between doctor and patient, after treatment is finished.
     * If the patient is fully treated, s/he is removed from this treatment unit,
     * otherwise another round of treatment is initiated.
     * Also ensure the doctor starts treating another patient.
     * @param doctor the doctor that has finished treating his/her patient
     */
    public void treatmentFinished(final Doctor doctor) {
        // Implementation hidden
    }
}
 



Expand
titleLF

 

TreatmentUnit: Løsningsforslaget viser utvidelse med en abstrakt klasse AbstractTreatmentUnit. Denne klassen implementerer alle metoder som ikke involverte regelsett for hvordan en skulle start behandling, og deklarerer de relevante abstrakte metoden (startTreatment *2). Disse ble så implementert i TreatmentUnit, med løsning akkurat som i del 2. En slik abstrakt klasse er ikke påkrevd, men en god idé, fordi den tydeliggjør hva som er ment å variere (polymorfi) i subklassene.

 

PriorityTreatmentUnit: Her skulle en velge ut den ventende pasienten som hadde høyest prioritet (getPriority). Selv om getPriority ikke ble oppgitt kunne en gjøre antakelser om bruk. Den eneste metoden som måtte implementeres i denne klassen var startTreatment(Doctor), siden prioritering må gjøres hver gang det oppstår en situasjon der pasientene konkurrerer om en doktor som er ankommet eller blitt ledig. Forskjellen fra tidligere implementasjon er at en må gå igjennom alle pasienter (som denne doktoren kan behandle) og lagre den pasienten som til nå har høyest prioritet. Vi har også vist pseudokode som vil være en fullgod besvarelse for denne klassen.

 

EmergencyPriorityTreatmentUnit: Den eneste metoden som trenger ny logikk er startTreatment(Patient). Derfor er det lurt (men strengt tatt ikke nødvendig) å arve fra PriorityTreatmentUnit, siden vi da kan utnytte den eksisterende prioriteringslogikken. Det er jo bare når en ikke finner en doktor med super.startTreatment(patient) at ny logikk for å avbryte pågående behandling er aktuell. Hvis dette er tilfelle, må en gå igjennom alle doktorer (trenger ny hjelpemetode getAllDoctors-metode) og sjekke om denne doktoren kan behandle pasienten. Merk at vi utelater alle ledige doktorer, siden disse allerede er sjekket i kallet til superklassens metode (dette kan forsåvidt skje i den nye hjelpemetoden, som da kanskje bør hete getAllTreatingDoctors). Vi bruker samme logikk som tidligere for å velge ut den lavest prioriterte pasienten. Vi oppdatere dennes nåværende doktor sin pasient til den nye pasienten (getDoctor(patientToSuspend).setPatient(patient)). Til slutt må pasienten som fikk avbrutt behandlingen få en ny sjanse til behandling av en annen lege med et nytt kall til startTreatment. Det er ganske mange ting å huske på, men det viktigste er tross alt god bruk av arv.

 

Også her tillates pseudokode, se eksempel i LF. Merk at oppgaven beskrev klassen PriorityTreatment, mens LF kaller den PriorityTreatmentUnit. Det er selvsagt ikke viktig.

 

Code Block
languagejava
public class PriorityTreatmentUnit extends TreatmentUnit {
 
               protected double getPriority(Patient patient) {
                              return 0.0;
               }
 
               @Override
               protected boolean startTreatment(Doctor doctor) {
                              Patient patient = null;
                              for (Patient patient2 : getWaitingPatients()) {
                                            if (doctor.canTreat(patient2) > 0.0 && (patient == null || getPriority(patient2) > getPriority(patient))) {
                                                           patient = patient2;
                                            }
                              }
                              if (patient != null) {
                                            doctor.setPatient(patient);
                                            return true;
                              }
                              return false;
               }
}
 
/* PSEUDOKODE-eksempel FOR startTreatment(doctor)
* Går gjennom alle pasienter som venter (getWaitingPatients()) og
* holder rede på den med høyest prioritet (getPriority).
* Sjekker om det finnes en doktor
* med riktig kompetanse (canTreat)
* Hvis man har funnet en pasient som kan behandles,
* så får doktoren denne (setPatient) og true returneres.
*/
 
public class EmergencyPriorityTreatmentUnit extends PriorityTreatmentUnit {
 
               @Override
               public boolean startTreatment(Patient patient) {
                              if (! super.startTreatment(patient)) {
                                            Patient patientToSuspend = null;
                                            for (Doctor doctor : getAllDoctors()) {
                                                           if (doctor.canTreat(patient) > 0.0) {
                                                                          Patient existingPatient = doctor.getPatient();
                                                                          if (existingPatient != null && getPriority(existingPatient) < getPriority(patient) &&
                                                                                                        (patientToSuspend == null || getPriority(existingPatient) < getPriority(patientToSuspend))) {
                                                                                         patientToSuspend = existingPatient;
                                                                          }
                                                           }
                                            }
                                            if (patientToSuspend != null) {
                                                  getDoctor(patientToSuspend).setPatient(patient);
                                                           startTreatment(patientToSuspend);
                                                           return true;
                                            }
                              }
                              return false;
               }
}

Vanlige feil/alternative løsninger:

  • Selve forståelsen, beskrivelsen og bruken av arv var vanskelig for en god del.
  • Noen forstod prioritet som noe måtte legges inn som en ny samling, eller definerte egne eksempler på hva slags tilstander som ga hvilken prioritet. Poenget her var at en skulle bruke getPriority på en pasient for å få ut et tall en kunne sammenlikne med.
  • TreatmentUnit: Det var ikke nødvendig å gjøre denne abstrakt, men en måtte forklare at den ville kunne virke som tidligere.
  • PriorityTreatmentUnit: Det viktige var at en måtte arve TU, og gitt vår implementasjon måtte en bare endre på startTreatment(doctor) og gå igjennom getWaitingPatients(). En vanlig feil var å ikke gå igjennom alle pasienter for å finne den som kunne behandles nå og som hadde høyest prioritet, men bare returnere når en har funnet en mulig kobling (i lf vist ved oppdatering av patientToSuspend).
  • EmergencyPriorityTreatmentUnit: Denne skulle i følge oppgaveteksten ha den samme funksjonaliteten som PriorityTreatmentUnit, men utvide denne. Noen valgte å implementere startTreatment(doctor) til å være lik som i PTU i stedet for å arve denne. Mange glemte at en måtte sjekke om en pasient kunne behandles med vanlig prioritering før en gikk igjennom doktorer som holdt på med behandling. Mange glemte at når en satt en behandlende doktor til å behandle en pasient, da måtte en håndtere pasienten som suspenderes. Denne må legges inn i pasientlisten igjen (hvis en valgte en løsning der pasienter under behandling ikke var i pasientsamlingen) samt at denne pasienten må må en ny startTreatment(patientToSuspend)
  • Ved alternative løsnigner løsninger tidligere vil riktig løsning være ulik. Dette gjelder for eksempel når en har endt opp med å gjøre ting som egentlig skulle være i startTreatment i addPatient og addDoctor. Disse løsningene blir da vurdert med bakgrunn i tidligere valg/misforståelser, for å i størt mulig grad unngå følgefeil.(Hit jeg har kommet foreløpig)
  • Kode/pseudokode blir gitt poeng avhengig av korrekthet og hvor dypt den går. Overfor finner dere et eksempel på pseudokode for PTU, denne skulle gi full uttelling.



Expand
titleDel 4 - delegering (15%)

Mens du i del 3 brukte arv, skal du i denne delen bruke delegering for å oppnå det samme. I henhold til delegeringsteknikken definerer vi et grensesnitt, DoctorAllocator. I tillegg lager du (minst) tre hjelpeklasser tilsvarende de tre logikkene beskrevet i del 3, som implementerer grensesnittet. Forklar hvordan koden i vedlagte TreatmentUnit-skjelett skal gjøres fullstendig slik at startTreatment-metoden bruker delegering riktig. Forklar også med tekst og/eller kode hvordan du vil utforme hjelpeklassene som implementerer DoctorAllocator.

Da målet er å vise kunnskap om delegering kan dere bruke pseudokode i denne oppgaven. Det er greit å referere (pseudo)koden i del 3, hvis det er til hjelp.


Expand
titleSkjelett til del 4


Code Block
languagejava
 
public interface DoctorAllocator {


       /**
    * Finds the patient in the TreatmentUnit that the provided doctor 
    * should start treating. 
    * Note: The method should be free of side effects, i.e. should not modify any object.
    * @param doctor 
    * @param treatmentUnit
    * @return the patient that the doctor should treat, or null.
    */
       public Patient allocatePatient(Doctor doctor, TreatmentUnit treatmentUnit);
       
       /**
    * Finds the doctor in the TreatmentUnit that the provided patient
    * should be treated by. 
    * Note: The method should be free of side effects, i.e. should not modify any object.
    * @param patient
    * @param treatmentUnit
    * @return The doctor that should treat the provided patient, or 
    * null if none was found.
    */
       public Doctor allocateDoctor(Patient patient, TreatmentUnit treatmentUnit);
}


/**
 * A class for managing a set of doctors and the patients they're treating.
 * When doctors or patients arrive, or a doctor finishes a treatment, 
 * it is made sure that patients are treated as soon as possible.
 */
public class TreatmentUnit {


       // Implementation of local variables hidden


       public void addDoctor(final Doctor doctor) {
             // Implementation hidden
   }


       public Collection<Doctor> getAllDoctors() {
             // Implementation hidden
   }


       public Collection<Doctor> getAvailableDoctors() {
             // Implementation hidden
   }


       public void addPatient(final Patient patient) {
             // Implementation hidden
   }


       public Doctor getDoctor(final Predicate<Doctor> pred) {
             // Implementation hidden
   }


       public Doctor getDoctor(final Patient patient) {
             // Implementation hidden
   }


       public Collection<Patient> getAllPatients() {
             // Implementation hidden
   }


       public Collection<Patient> getWaitingPatients() {
             // Implementation hidden
   }


  ... declarations related to the use of DoctorAllocator ... // Part 4


       protected boolean startTreatment(final Doctor doctor) { // Part 4
             final Patient patient = ... use of DoctorAllocator here ... 
          ... more code here ...
   }


       protected boolean startTreatment(final Patient patient) { // Part 4
             final Doctor doctor = ... use of DoctorAllocator here ...
             ... more code here ...
   }


       public void treatmentFinished(final Doctor doctor) {
             // Implementation hidden
   }
}




Expand
titleLF

 

LF lager løsningen rundt det oppgitte grensesnittet DoctorAllocator, som har to metoder allocateDoctor og allocatePatient. Dette grensesnittet implementeres av en DefaultDoctorAllocator (noe som tilsvarer slik det gjøres i en vanlig TreatmentUnit, altså ikke noe bruk av prioritering). De andre to proriteringsstrategiene implementeres av PriorityDoctorAllocator og EmergencyPriorityDoctorAllocator. LF bruker arv for begge disse, tilsvarende løsningen i del 3, men det er ikke viktig fordi temaet her er delegering. Logikken som implementeres er omtrent som i del 3, med en vesentlig forskjell: Grensesnitt-metodene skal bare finne og returnere riktig doktor/pasient (koblingen som skal etableres), ikke endre noe, altså kalles ikke setPatient eller startTreatment.

 

DefaultDoctorAllocator: Implementerer ingen prioritet og tilsvarer et vanlig TreatmentUnit-valg. Hvis doktor-pasient-kobling kan etableres returneres doktor eller pasient, ellers null.

 

PriorityDoctorAllocator: Arver fra DefaultDoctorAllocator, så en slipper unna med redefinering av allocatePatient, med logikk tilsvarende PriorityTreatmentUnit.

 

EmergencyPriorityDoctorAllocator: Arver fra PriorityDoctorAllocator, så en slipper unna med redefinering av allocateDoctor, med logikk tilsvarende EmergencyPriorityTreatmentUnit.

 

TreatmentUnit: Denne må endres så den 1) bruker delegering ved å kalle en DoctorAllocator-implementasjon og 2) håndterer faktisk endring av doktor-pasient-koblingen, iht. resultatet fra DoctorAllocator-implementasjonen. Delegering rigges ved å opprette en doctorAllocator-variabel, her er DefaultDoctorAllocator valgt, men den kan byttes ut med et kall til setDoctorAllocator. Begge startTreatment-metodene må endres. For begge består endringen av at en må kalle den tilsvarende allocate-metoden og sjekke om resultatet er null før en knytter relasjonen. Merk at en må ta høyde for at behandling kan avbrytes.

Code Block
languagejava

public interface DoctorAllocator {

    /** // Long description at git
     */
    public Patient allocatePatient(Doctor doctor, TreatmentUnit treatmentUnit);

    
    /** // Long description at git
     */
    public Doctor allocateDoctor(Patient patient, TreatmentUnit treatmentUnit);
}


public class DefaultDoctorAllocator implements DoctorAllocator {

    @Override
    public Patient allocatePatient(final Doctor doctor, final TreatmentUnit treatmentUnit) {
        for (final Patient patient : treatmentUnit.getWaitingPatients()) {
            if (doctor.canTreat(patient) > 0.0) {
                return patient;
            }
        }
        return null;
    }

    @Override
    public Doctor allocateDoctor(final Patient patient, final TreatmentUnit treatmentUnit) {
        for (final Doctor doctor : treatmentUnit.getAvailableDoctors()) {
            if (doctor.canTreat(patient) > 0.0) {
                return doctor;
            }
        }
        return null;
    }
}


public class PriorityDoctorAllocator extends DefaultDoctorAllocator {

    /**
     * Dummy class
     */
    protected double getPriority(final Patient patient) {
        
        return 0.0;
    }

    @Override
    public Patient allocatePatient(final Doctor doctor, final TreatmentUnit treatmentUnit) {
        Patient patient = null;
        for (final Patient patient2 : treatmentUnit.getWaitingPatients()) {
            if (doctor.canTreat(patient2) > 0.0 && (patient == null || getPriority(patient2) > getPriority(patient))) {

                patient = patient2;
            }
        }
        return patient;
    }
}


public class EmergencyPriorityDoctorAllocator extends PriorityDoctorAllocator {

    @Override
    public Doctor allocateDoctor(final Patient patient, final TreatmentUnit treatmentUnit) {
        final Doctor doctor = super.allocateDoctor(patient, treatmentUnit);
        if (doctor != null) {
            return doctor;
        }
        Patient patientToSuspend = null;
        for (final Doctor doctor2 : treatmentUnit.getAllDoctors()) {
            if (doctor2.canTreat(patient) > 0.0) {
                final Patient existingPatient = doctor2.getPatient();
                if (existingPatient != null && getPriority(existingPatient) < getPriority(patient) && 
                        (patientToSuspend == null || getPriority(existingPatient) < getPriority(patientToSuspend))) {
                    patientToSuspend = existingPatient;
                }
            }
        }
        if (patientToSuspend != null) {
            return treatmentUnit.getDoctor(patientToSuspend);
        }
        return null;
    }
}

public abstract class TreatmentUnit {
    // Diverse kode utelatt fra tidligere

    private DoctorAllocator doctorAllocator = new DefaultDoctorAllocator();

    public void setDoctorAllocator(final DoctorAllocator doctorAllocator) {
        this.doctorAllocator = doctorAllocator;
    }

    protected boolean startTreatment(final Doctor doctor) {
        final Patient patient = doctorAllocator.allocatePatient(doctor, this);
        if (patient != null) {
            doctor.setPatient(patient);
            return true;
        }
        return false;
    }

    protected boolean startTreatment(final Patient patient) {
        final Doctor doctor = doctorAllocator.allocateDoctor(patient, this);
        if (patient != null) {
            final Patient oldPatient = doctor.getPatient();
            doctor.setPatient(patient);
            if (oldPatient != null) {
                startTreatment(oldPatient);
            }
            return true;
        }
        return false;
    }

    public void treatmentFinished(final Doctor doctor) {
        if (doctor.getPatient() == null) {
            throw new IllegalStateException(doctor + " has no patient!");
        }
        final Patient patient = doctor.getPatient();
        doctor.setPatient(null);
        if (patient.requiresTreatment()) {
            startTreatment(patient);
        } else {
            patients.remove(patient);
        }
        startTreatment(doctor);
    }
}

         

Vi gjentar fra del 3 hvordan vi ønsker å knytte doktor til pasient, men dette skal nå gjøres ved å implementere DoctorAllocator i stedet for å bruke arv:

 Vanlige feil/alternative løsninger:

  • Svært mange lot TreatmentUnit implementere DoctorAllocator, i ymse varianter. Det var altså meningen at egne klasser skulle implementere hhv TU, PTU og EPTU (gjerne med arv, og lik logikk som i oppgave 3).
  • I tillegg var det viktig at en Allocator skulle returnere et doktor eller pasientobjekt, og altså ikke sette i gang behandling og slikt. Dette skulle gjøres i TreatmentUnit, som vist i LF.
  • Som i eksempelet over var noe av det viktigste å vise forståelse for hvordan delegering fungerer. Det betyr at du setter bort valget til en egen klasse som implementerer DoctorAllocator, og så bruker du i din TreatmentUnit tre ulike slike alt etter type TreatmentUnit. Det er altså ikke tre ulike typer TU vi er på jakt etter, men i stedet tre typer Allocators. En implementasjon med tre typer TreatmentUnit som implementerer DoctorAllocator er ikke delegering.
  • TU må i stedet endres til å inkludere en slik allocator, og endringer som beskrevet i LF og over LF.


Vi gjentar fra del 3 hvordan vi ønsker å knytte doktor til pasient, men dette skal nå gjøres ved å implementere DoctorAllocator i stedet for å bruke arv:

  • Pasienter knyttes til den første (og ikke nødvendigvis beste) doktoren som kan behandle en eller flere tilstander hos pasienten. 
  • Pasientene kan ha tilstander med varierende kritikalitet, som gjør at de prioriteres foran resten av køen når en doktor er ledig. Du
  • Pasienter knyttes til den første (og ikke nødvendigvis beste) doktoren som kan behandle en eller flere tilstander hos pasienten. 
  • Pasientene kan ha tilstander med varierende kritikalitet, som gjør at de prioriteres foran resten av køen når en doktor er ledig. Du kan forutsette at det finnes en getPriority-metode i Patient, og at denne prioriteten settes ved innlegging av pasientdata. Per, som har en mer alvorlig skade enn Jens, vil altså behandles raskere selv om han kommer inn senere. (Såfremt Jens ikke allerede har startet behandlingen.)
  • Pågående pasientbehandling (selv om den er prioritert) skal kunne avbrytes og doktoren skal starte behandling av en ny pasient. Dette skal skje dersom det kommer inn en ny pasient, og dennes prioritet er høyere enn hos pågående. Du kan også her forutsette at den finnes en getPriority-metode i Patient. Anta at Jens er under behandling hos doktor Who. Ida kommer har en tilstand med høyere prioritet enn Jens, og kan kun behandles av Who. Jens sin behandling blir derfor satt på vent, og den samme doktoren begynne behandling av Ida.

...

Expand
titleDel 5 – observatør-observatør-teknikken og JavaFX (15%)

På venterommet skal det være et informasjonspanel, som forteller ventende pasienter om hvilken doktor de skal gå til, når det er deres tur. Panelet implementeres med JavaFX, og tanken er at kontroller-objektet (av typen TreatmentUnitController ) skal lytte på et TreatmentUnit-objekt og få beskjed når koblingen mellom doktor og pasient etableres, slik at panelet kan vise en tekst av typen «Pasient X skal gå til Doctor Y» (du kan forvente at Patient og Doktor har en toString() som gir navn, dette trenger du ikke implementere i tidligere oppgaver).

I denne delen kan du bygge videre på koden fra del 2. Du trenger altså ikke ha løst del 3 eller 4.

Expand
titleSkjelett til del 5

 


Code Block
// TreatmentUnit.fxml:<?xml version="1.0" encoding="UTF-8"?>


<?import javafx.scene.layout.VBox?>
<?import javafx.scene.control.Label?>
<?import javafx.scene.layout.HBox?>
<?import javafx.scene.text.Font?>


<HBox xmlns:fx="http://javafx.com/fxml/1" fx:controller="ord2019.part5.TreatmentUnitController">
            <Label fx:id="patientMessage" text="&lt;Her kommer meldinger til pasienter&gt;">
            </Label>
</HBox>




// TreatmentUnitController.java:
public class TreatmentUnitController {


            TreatmentUnit treatmentUnit;
            
            public TreatmentUnitController() {
                        treatmentUnit = // assume treatmentUnit is set automatically.
                        // ... 5 b) other initialization ...
            }


            // ... 5 b) declarations and methods here...
}


Oppgave 5a) – Lyttergrensesnitt

Definer et egnet lyttergrensesnitt og forklar med tekst og kode hvordan TreatmentUnit må endres for å kunne fungere som observert i et observatør-observert-forhold.

Expand
titleLF

 

Standardteknikken krever et passende lyttergrensesnitt og en liste av lyttere som kalles på passende sted. En bruker gjerne en Collection for å lagre lytterne og add/remove-metoder for å administrere lytterne.

LF definerer lyttergrensesnittet TreatmentListener. Dette grensesnittet har én metode, som kalles idet doktor-pasient-koblingen etableres, altså på tidspunktet det passer å gi beskjed til pasienten om hvilken doktor hen skal gå til. Metoden bør ta inn faktisk pasient, doktor og akuttmottak (TreatmentUnit).

Endringer som må gjøres i den observerte, TreatmentUnit, er at alle lytterne lagres i en private Collection av type TreatmentListener, og at den legger inn relevante public add/remove til denne. I tillegg defineres hjelpemetoden fireTreatmentStarted, som kalles rett etter at setPatient har etablert en ny doktor-pasient-kobling (i både startTreatment(Doctor) og startTreatment(Patient)).


Code Block
languagejava
public interface TreatmentListener {
 
               public void treatmentStarted(Doctor doctor, Patient patient, TreatmentUnit treatmentUnit);
 
}
 
public class TreatmentUnit {
 
               // Lots of code unchanged here, find it in part 2 of TU.
 
 
               private Collection<TreatmentListener> treatmentListeners = new ArrayList<>();
 
               public void addTreatmentListener(TreatmentListener listener) {
                              treatmentListeners.add(listener);
               }
 
               public void removeTreatmentListener(TreatmentListener listener) {
                              treatmentListeners.remove(listener);
               }
 
               private void fireTreatmentStarted(Doctor doctor, Patient patient) {
                              for (TreatmentListener treatmentListener : treatmentListeners) {
                                             treatmentListener.treatmentStarted(doctor, patient, this);
                              }
               }
 
               private boolean startTreatment(Doctor doctor) {
                              for (Patient patient : getWaitingPatients()) {
                                            if (doctor.canTreat(patient) > 0.0) {
                                                           doctor.setPatient(patient);
                                                           fireTreatmentStarted(doctor, patient);
                                                           return true;
                                            }
                              }
                              return false;
               }
 
               private boolean startTreatment(Patient patient) {
                              for (Doctor doctor : getAvailableDoctors()) {
                                            if (doctor.canTreat(patient) > 0.0) {
                                                           doctor.setPatient(patient);
                                                           fireTreatmentStarted(doctor, patient);
                                                           return true;
                                            }
                              }
                              return false;
               return false;
               }
}}
}

Vanlige feil/alternative løsninger:

  • En god løsning skal definere både grensesnittet som lytterne (dermed TreatmentUnitController ) implementerer.
  • Dette grensesnittet må inkludere en metode som kalles når noe skjer (se under). Denne metoden må inneholde nok informasjon til at Labelobjektet kan oppdateres med pasient- og doktorinfo. Siden doktoren vet hvilken pasient hen har fått tildelt, så trenger en egentlig bare sende med det relevante doktorobjektet.
  • Mange bommet på at en må definere en metode i TU (fireTreatmentStarted) som en så går inn og kaller på alle relevante steder (hver gang en behandling skal starte, med andre ord startTreatment(Patient) og startTreatment(Patient).
  • En måtte huske på å lage en samling av lyttere, lage gettere/settere /eller i alle fall indikere det.
  • I fireTreatmentStarted måtte en huske på å gå igjennom alle lytterne.


Oppgave 5b) – Controller

Fyll ut kodeskjelettet for kontrollerklassen TreatmentUnitController, slik at det fungerer med den vedlagte FXML-koden og fyller rollen som observatør av TreatmentUnit. Som indikert i kodeskjelettet, så kan du anta at treatmentUnit-variablen i kontrollerklassen settes «automagisk» i konstruktøren.

Expand
titleLF

 

I FXML-koden ser en at det finnes en Label med fx:id="patientMessage". Her må man skjønne/gjette at det er dette objektet som skal oppdateres når en pasient skal gå til en doktor. En kan også lese at controlleren er en (instans av) TreatmentUnitController.

TreatmentUnitController:

-          Label patientMessage i FXML må knyttes til en variabel i TreatmentUnitController, gjennom å bruke @FXML-annotasjonen og Label-type.

-          Konstruktøren oppretter en ny TreatmentUnit. Så er det litt valgfritt hva som skjer, men den viktige biten er at man i konstruktøren også må opprette forbindelsen mellom TreatmentUnitController som lytter/observatør og TreatmentUnit som observert vha. et kall til addTreatmentListener-metoden fra 5a.

-          TreatmentUnitController må implementere lyttergrensesnittet og (den ene) lyttermetoden må oppdaterer status på Label-objektet.

 

Code Block
languagejava
public class TreatmentUnitController implements TreatmentListener {
 
               private TreatmentUnit treatmentUnit;
 
               public TreatmentUnitController() {
                              treatmentUnit = new TreatmentUnit();
                              
              // ...
 
                              treatmentUnit.addTreatmentListener(this);
               }
 
               @FXML
               private Label patientMessage;
 
               @Override
               public void treatmentStarted(Doctor doctor, Patient patient, TreatmentUnit treatmentUnit) {
                              patientMessage.setText(patient.getName() + " bes gå til doktor " + doctor.getName());
               }
}

Vanlige feil/alternative løsninger:

  • Etter at TreatmentUnit ble definert inni kons.truktøren til TUC måtte en kalle TU sin metode som ble laget i 5a, addTreatmentListener(this).
  • this som argument i forrige punkt var viktig.
  • @FXML (huske denne) Label måtte hete akkurat det samme som i FXML-filen som ble lagt ved.
  • Label over måtte deklareres inni hovedkoden til TUC, ikke inni konstruktøren.
  • Metoden som skulle kalles når lytteren (this) kalles (fra 5a) heter i LF treatmentStarted. Her skrev mange ut informasjon om pasienter og doktorer uten å ha noen objekter å referere til.