Følgende kode legges øverst i konstruktøren:

if (table.getCapacity() < group.getGuestCount()) {
   throw new IllegalArgumentException("The table is too small for the group");
}

Viktig å utløse unntak, ikke bare en if rundt initialiseringskoden.

Group måtte hatt en referanse til Seating-objektet (eller Table-objektet) som ble satt av Seating, så den kunne kjøre tilsvarende sjekk som over, ved endring av størrelsen. Uten en slik referanse hjelper det ikke å si at en skal sjekke den nye gruppestørrelsen opp mot antall stoler ved bordet. En kan til nød bruk observatør-observert-teknikken, men det er overkill her.

Her er poenget at en trenger en global teller, som en får til i Java ved bruk av static. Denne må brukes og økes i Table sin konstruktør.

private static int tableCounter = 1; 
 
private final int num;
private final int capacity;
 
public Table(int capacity) {
       this.num = tableCounter++;
       this.capacity = capacity;
}

Vi velger Collection, fordi vi ikke trenger indeksbasert tilgang til elementene. Vi ser ikke noe behov for en konstruktør, siden en like gjerne kan legge til bord ved kall til addTable. Det er greit nok å ha en konstruktør som tar inn et sett bord, f.eks. en varargs som i Table... tables, og da er det viktig å initialisere lokale variabler slik at de ikke kan endres utenifra. Merk at Arrays.asList(...) lager en List som ikke kan endre størrelse, så den kan ikke bruke direkte som verdien til et lokal tables-felt. Hvis en kan gi inn Seating-objekter, så må de valideres, slik at en ikke kan omgå valideringen som vanligvis gjøres i createSeating/addSeating.

Det vil være mulig å bruke én tableSeatings-Map/HashMap i stedet for to lister, hvor tableSeatings.keySet() tilsvarer tables og tableSeatings.values() tilsvarer seatings.

private Collection<Table> tables = new ArrayList<>(); 
private Collection<Seating> seatings = new ArrayList<>();
 
public void addTable(Table table) {
 tables.add(table);
}
 
public void removeTable(Table table) {
 if (isOccupied(table)) {
 throw new IllegalArgumentException("Cannot remove an occupied table");
 }
 tables.remove(table); 
}

isOccupied-sjekken kan virke unødvendig, siden den også gjøres i removeTable, men det er best å sjekke på forhånd og ikke underveis, så en ikke utfører endringene delvis (uten å reversere endringene).

public void mergeTables(Table table1, Table table2, int lostCapacity) {
       checkNotOccupied(table1);
       checkNotOccupied(table2);
       Table table = new Table(table1.getCapacity() + table2.getCapacity() - lostCapacity);
       removeTable(table1);
       removeTable(table2);
       addTable(table);
}
      
public void splitTable(Table table, int capacity1, int capacity2) {
       checkNotOccupied(table);
       removeTable(table);
       addTable(new Table(capacity1));
       addTable(new Table(capacity2));
}
 
void checkNotOccupied(Table table) {
       if (isOccupied(table)) {
              throw new IllegalArgumentException("The table cannot be occupied");
       }
}

Den første tilstanden kan inneholde Diner-objekt koblet til to Table-objekter. Transisjonen er et kall til mergeTables. Den andre tilstanden inneholder det samme Diner-objektet koblet til et nytt Table-objekt. capacity-feltene og lostCapacity-argumentet må stemme overens.

object "~#diner : Diner" as diner1 {
}
object "~#table1 : Table" as table1 {
	num = 1
	capacity = 4
}
object "~#table2 : Table" as table2 { 
	num = 2
	capacity = 4
}
diner1 --> table1: tables
diner1 --> table2: tables
object "~#diner : Diner" as diner2 {
}
object "~#table3 : Table" as table3 {
	num = 3
	capacity = 6
}
diner2 --> table3: tables
diner1 ..> diner2: mergeTables(~#table1, ~#table2, 2)

For hasCapacity er det viktig å ha med en sjekk for om bordet er opptatt, ikke bare sjekke bordets totale kapasitet. For findAvailableTables så er litt av poenget å skjønne at selv om en Collection ikke alltid har en ordning og kan sorteres, så kan en lage or returnere en List som er ordnet og sortert.

public boolean hasCapacity(Table table, int capacity) {
       return (! isOccupied(table)) && table.getCapacity() >= capacity;
}
 
public Collection<Table> findAvailableTables(int capacity) {
       List<Table> result = new ArrayList<>();
       for (Table table : tables) {
              if (hasCapacity(table, capacity)) {
                      result.add(table);
              }
       }
       Collections.sort(result);
       return result;
}

Table må implementere Comparable<Table> for at sort-metoden skal kunne brukes og virke. Alternativt kan man lage en (implementasjon av) Comparator<Table>, f.eks. med (t1,t2) -> t1.getCapacity() – t2.other.getCapacity().

@Override
public int compareTo(Table other) {
       return getCapacity() - other.getCapacity();
}

Her er litt av poenget hvordan en skal håndtere at resultatet fra findAvailableTables er en Collection, som ikke har en ”opplagt” metode for å hente ut første element. Her brukes iterator() og ett kall til next(). Kunne brukt iterator() først og så hasNext() for å sjekke om det er elementer å hente, i stedet for isEmpty (se utkommentert kode). Det er ikke bra å caste til List, selv om en vet at det er en List i praksis, eller endre på retur-typen til findAvailableTables.

Det er ikke meningen å utløse unntak i createSeating, hvis ingen bord er ledige, men hvis det gjøres så må addSeating kodes med try/catch og riktig retur-verdi.

public Seating createSeating(Group group) {
       Collection<Table> availableTables = findAvailableTables(group.getGuestCount());
       if (availableTables.isEmpty()) {
              return null;
       }
       return new Seating(availableTables.iterator().next(), group);
//     Iterator<Table> tablesIt = availableTables.iterator();
//     return (tablesIt.hasNext() ? new Seating(tablesIt.next(), group) : null;
}
 
public boolean addSeating(Group group) {
       Seating seating = createSeating(group);
       if (seating != null) {
              seatings.add(seating);
              return true;
       }
       return false;
}
 
public void removeSeating(int tableNum) {
       for (Seating seating : seatings) {
              if (seating.getTable().getNum() == tableNum) {
                      seatings.remove(seating);
                      return; // eller break
              }
       }
}

SimpleTable og CompositeTable blir subklasser av Table, som selv enten blir et grensesnitt eller en abstrakt klasse med i hvertfall getCapacity-metoden. De to variantene er vist under. SimpleTable kan ta over det meste av Table-koden, evt. arve alt. CompositeTable kapsler inn informasjonen fra mergeTable og bruker de to Table-objektene til å beregne kapasiteten. Alternativt kan Table beholde capacity-feltet og getCapacity()-metoden og de to andre initialiserer capacity med super(...). Vi har her ikke lagt opp til at CompositeTable-objekter skal få et løpenummer, men det er greit å la den funksjonaliteten være en del av en abstract Table-(super)klasse.

public class CompositeTable implements Table {
      
       private Table table1, table2;
       private int lostCapacity;
      
       public CompositeTable(Table table1, Table table2, int lostCapacity) {
              this.table1 = table1;
              this.table2 = table2;
              this.lostCapacity = lostCapacity;
       }
 
       public Table getTable1() {
              return table1;
       }
 
       public Table getTable2() {
              return table2;
       }
      
       @Override
       public int getCapacity() {
              return table1.getCapacity() + table2.getCapacity() - lostCapacity;
       }
}
 
public class CompositeTable extends Table {
      
       private Table table1, table2;
      
       public CompositeTable(Table table1, Table table2, int lostCapacity) {
              super(table1.getCapacity() + table2.getCapacity() - lostCapacity);
              this.table1 = table1;
              this.table2 = table2;
       }
 
       public Table getTable1() {
              return table1;
       }
 
       public Table getTable2() {
              return table2;
       }
}

Observerbarhet handler om å la et eller flere objekter (observatørene/lytterne) få beskjed om endringer i et annet objekt (den observerte). Observert-klassen må administrere et sett med lyttere (objekter som implementerer et lyttergrensesnitt), vha. felt for Collection av lyttere og add/remove-metoder. Alle steder hvor tilstanden (til egenskapen) endres, må det skytes inn kode som sier fra til lytterne (kall på fire-metode, som går gjennom lytterne).

GuestManager må ha en liste Group-objekter som fungerer som en kø. Ved ankomst med groupArrived, så prøves først å kalle addSeating. Hvis det ikke går, så legges Group-objektet i køen. Når gruppa drar, så brukes removeSeating. For å kunne tømme køen, må det lyttes på endringer i kapasiteten, altså må GuestManager implementere CapacityListener og legge seg til som lytter på Diner-objektet. I capacityChanged-metoden må en gå gjennom køen og igjen prøve addSeating og evt. fjerne Group-objektet fra køen, hvis det gikk greit.

Eksempelkode:

public class GuestManager implements CapacityListener {
 
       private final Diner diner;
      
       public GuestManager(Diner diner) {
              this.diner = diner;
              diner.addCapacityListener(this);
       }
      
       private Collection<Group> waitingGroups = new ArrayList<>();
 
       public void groupArrived(Group group) {
              if (! diner.addSeating(group)) {
                      waitingGroups.add(group);
              }
       }
 
       public void groupDeparted(int tableNum) {
              diner.removeSeating(tableNum);
       }
 
       @Override
       public void capacityChanged(Diner diner) {
              Collection<Group> q = new ArrayList<Group>(waitingGroups);
              waitingGroups.clear();
              Iterator<Group> it = q.iterator();
              while (it.hasNext()) {
                      Group waiting = it.next();
                      if (diner.addSeating(waiting)) {
                             it.remove();
                      }
              }
              waitingGroups.addAll(q);
       }
}

CapacityListener-grensesnitt er (teknisk sett) funksjonelt, siden det bare har én (abstrakt) metode (og kan derfor implementeres med lambda-syntaksen). Dette kreves i et svar som får poeng i det hele tatt. I tillegg bør andre argumenter (for at grensesnittet ikke er funksjonelt) trekkes inn, f.eks. at metoden typisk ikke er implementasjonens primære funksjon og at en ikke tenker på den som en matematisk funksjon som kun er avhengig av argumentene.

// part 1
 
/**
 * A group (of people) dining together, and should be seated at the same table.
 * We currently only need to handle the size. 
 */
public class Group {
 
       /**
       * Initializes this Group with the provided guest count
       */
       public Group(int guestCount) {
              ...
       }
}
 
/**
 * A table with a certain maximum capacity.
 */
public class Table {
 
       /**
       * Initializes this Table with the provided capacity.
       * The table is also assigned a unique number.
       * @param capacity
       */
       public Table(int capacity) {
              ...
       }
 
       /**
       * @return the table number
       */
       public int getNum() {
              ...
       }
}
 
/**
 * Represents the fact that a Group is seated at and occupies a Table
 */
public class Seating {
 
       /**
       * Initializes this Seating ...
       */
       public Seating(...) {
              ...
       }
}

 
// part 2
 
/**
 * A place where groups of guests can buy a meal
 */
public class Diner {
 
       /**
       * Tells whether a Table is occupied.
       * @param table the Table to check
       * @return true if anyone is sitting at the provided Table
       */
       public boolean isOccupied(Table table) {
              ...
       }
      
       /**
       * Computes the guest capacity,
        * either the remaining (includeOccupied == false) or total (includeOccupied == true).
       * @param includeOccupied controls whether to include tables that are occupied.
       * @return the guest capacity
       */
       public int getCapacity(boolean includeOccupied) {
              ...
       }
      
       /**
       * Adds a table to this Diner
       * @param table
       */
       public void addTable(Table table) {
              ...
       }
 
       /**
       * Removes a Table from this Diner.
       * If the table is occupied an IllegalArgumentException exception should be thrown.
       * @param table
       * @throws IllegalArgumentException
       */
       public void removeTable(Table table) {
              ...
       }
      
       /**
       * Merges two tables, i.e. replaces two tables with one table.
       * lostCapacity is the difference between the old capacity and the new.
       * This number is typically positive, since seats are lost when moving two tables
        * close to each other.
       * @param table1
       * @param table2
       * @param lostCapacity
       * @throws IllegalArgumentException if any of the tables are occupied
       */
       public void mergeTables(Table table1, Table table2, int lostCapacity) {
              ...
       }
 
       /**
       * Splits a table into two, i.e. replaces one tables with two tables.
       * The two capacities are the capacities of the two new tables.
       * @param table
       * @param capacity1
       * @param capacity2
       * @throws IllegalArgumentException if the table is occupied
       */
       public void splitTable(Table table, int capacity1, int capacity2) {
              ...
       }
 
       /**
       * Tells whether a table has the provided capacity,
       * i.e. if that number of new guests can be seated there.
       * Note that a table cannot be shared among different groups.
       * @param table
       * @param capacity
       * @return true of capacity number of guests can be seated here, false otherwise.
       */
       public boolean hasCapacity(Table table, int capacity) {
              ...
       }
 
       /**
       * Returns the tables that has the provided capacity.
       * The tables should be sorted with the one with the least capacity (but enough) first.
       * @param capacity
       * @return the tables that has the provided capacity
       */
       public Collection<Table> findAvailableTables(int capacity) {
              ...
       }
      
       /**
       * Finds a suitable, existing table for the provided group, and creates
       * (but doesn't add) a corresponding Seating.
       * The chosen table should be the one with the least capacity.
       * @param group the group to be seated
       * @return the newly created Seating
       */
       public Seating createSeating(Group group) {
              ...
       }
      
       /**
       * Creates and adds a Seating for the provided group, using the createSeating method.
       * @param group
       * @return true if a Seating was created and added, false otherwise.
       */
       public boolean addSeating(Group group) {
              ...
       }
      
       /**
       * Removes the seating for the provided table (number), if one exists
       * @param tableNum the number of the table to be removed
       */
       public void removeSeating(int tableNum) {
              ...
       }
}
 

// part 3
 
public class SimpleTable ... Table {
 
       public SimpleTable(int capacity) {
              ...
       }
 
       ...
}
 
/**
 * A table that consists of two other tables.
 */
public class CompositeTable ... Table {
      
       public CompositeTable(Table table1, Table table2, int lostCapacity) {
              ...
       }
 
       ...
}
 
// part 4
 
/**
 * Interface for listening to changes in Diner capacity
 */
public interface CapacityListener {
       /**
       * Called to inform that a Diner has changed capacity
       * @param diner
       */
       public void capacityChanged(Diner diner);
}
 
/**
 * Handles guests arriving at and departing from a Diner.
 */
public class GuestManager ... {
 
       public GuestManager(Diner diner) {
              ...
       }
      
       /**
       * Handles arriving groups, by either seating them immediately
       * (if possible) or putting them in queue. Those enqueued will
       * be seated when the Diner's (change in) capacity allows.
       * @param group
       */
       public void groupArrived(Group group) {
              ...
       }
 
       /**
       * Handles departing groups, by removing their seating.
       * @param tableNum the table where the group was seated
       */
       public void groupDeparted(int tableNum) {
              ...
       }
 
       ...
}