Задача от иностранной компании или как я провалил собеседование

«Of course the system assumes that any bigger group may get bored of seeing smaller groups arrive and get their tables ahead of them, and then decide to leave, which would mean that they abandon the queue without being served.»

public int compareTo(Table table) {
        boolean isEmpty1 = this.getFreeSpace() == this.getSize();
        boolean isEmpty2 = table.getFreeSpace() == table.getSize();

  private static class ServiceHolder {
    private static final Service HOLDER_INSTANCE = new Service();
  }

  public static Service getInstance() {
    return ServiceHolder.HOLDER_INSTANCE;
  }

  private Service() { }

Из того, что в глаза бросилось:

Вы работаете с потоками, но используете не потокобезопасные коллекции.

127: group.getTable().setFreeSpace(-1 * group.getSize());

Здесь же будет NPE, если клиенты не за столом.

135: Iterator iterator = clientsQueue.iterator();

Зачем здесь вообще итератор?

138: if (searchTable(client)) {

А метод searchTable() у Вас не просто ищет, есть ли свободный стол, но и добавляет клиента. Во-первых, это нарушение CQRS, а во-вторых – как раз здесь отличный пример того, почему это плохо.

Ну и в общем, код скорее процедурный, нежели объектно-ориентированный.

for (Table table : tableList) {
            if (table.getFreeSpace() == 0) break;

Есть некоторые основания предполагать, что автор просто помог какому-то нерадивому студенту (будущему эффективному манагеру) написать курсовик.

Хотя возможно, я и неправ.
Но мне уже разок довелось столкнуться с умниками, которые оказывали "образовательные услуги" студентам (контрольные, курсовые, дипломы), а неиссякаемый источник бесплатной раб. силы для них находился на сайтах трудоустройства.

Короче, спасибо за статью!

А ведь это идея! В следующий раз перед тем, как отправить решение теста, запилю тему на хабре, где мне бесплатно и ошибки укажут и код отрефакторят.

Единственное, что я и написал в ответном письме, можно добавить задержку перед выдачей свободного стола.

Кстати коллекцию очередь использовать ни к чему. Очередь очередью, а по условию любой из очереди может быть обслуженным, в зависимости от свободных мест и размера группы и в итоге методы относящиеся к очереди мы использовать не будем.

Client groups must be served in the order of arrival ..

Обычно в заданиях с частичным кодом предполагается что вы не должны его изменять, если в задании явно не указано обратное. В вашем случае явно говорится, что нужно только fill RestManager и вы можете add new methods

Please fill RestManager class with appropriate data structures and implement its constructor and three public methods. You are encouraged modify other classes too (to help us test them) and add new methods at your will.

Такие задания проверяют умение разработчика взаимодействовать с существующим кодом. Чтобы при выполнении любой задачи по модификации существующего кода (пусть даже и не оптимального) на реальном проекте новый разработчик не стремился переписать все как надо

К тому же зачастую подобные тестовые задания могут проверяться в автоматическом режиме и изменение изначального кода может просто зафейлить компиляцию/запуск автоматической проверки

Честно признаться, на последнем месте работы за полтора года ничего кроме «форов» и «ифов» я не писал

По второму пункту как минимум имхо метод public Table lookup не в своем классе. Мы можем получить стол геттером у клиента, который по идеи должен хранить ссылку на стол.

Интересная ответственность у клиента — сообщать, за каким он столом :)

Мне кажется, у вас ожидали, что логических методов у Table и у ClientsGroup не будет, потому что логика оказывается размазанной, и, кроме того, сущность, которая не должна знать о способе выбора столов, внезапно знает (метод compareTo у стола). Почему СТОЛ отвечает за алгоритм сортировки?