Though I get correct console output the task is not passing verification. What may be the reason?

Question about the task

Restaurant (part 19)

Java Multithreading, Level 9, Lesson 15

Under discussion

The conditions are not passing verification. Please help.

We need to rework the main method.

First, let's make the parameters of our RandomOrderGeneratorTask constructors consistent.
Please make your method signature the same as mine:
RandomOrderGeneratorTask(List<Tablet> tablets, int interval)

In the main method:
1. Remove the creation of the hardcoded tablet and calls to the createOrder() method.
2. Create a second cook.
3. Register the cooks using the StatisticsManager class.
4. Arrange Observer/Observable classes for the second cook and all the tablets.
5. Create a list of 5 tablet objects. Initialize it in a loop.
6. Create and run a thread based on a RandomOrderGeneratorTask object.
7. After a second, interrupt it and look at the console output.

Oops, two cooks are preparing the same order! 8-O

We got this output because the Observable is informing all of its Observers, i.e. a tablet sends its order to all known cooks.
This isn't the behavior we want, so we will correct it in the next task.

P.S. Don't forget to stop the thread!

Requirements:

There should be 5 tablets created in the main method of the Restaurant class.
There should be 2 cooks created in the main method of the Restaurant class.
The cooks must be registered in the cooks set in the StatisticsManager.
A thread based on RandomOrderGeneratorTask should be created and started.
The RandomOrderGeneratorTask class constructor must match the constructor described in the task conditions.
Both cooks should be added to each tablet as observers.

package com.codegym.task.task27.task2712; import com.codegym.task.task27.task2712.ad.AdvertisementManager; import com.codegym.task.task27.task2712.ad.NoVideoAvailableException; import com.codegym.task.task27.task2712.kitchen.Order; import com.codegym.task.task27.task2712.kitchen.TestOrder; import java.io.IOException; import java.util.Observable; import java.util.logging.Level; import java.util.logging.Logger; public class Tablet extends Observable { private final int number; private static Logger logger = Logger.getLogger(Tablet.class.getName()); public Tablet(int number) { this.number = number; } public Order createOrder(){ Order newOrder = null; try { newOrder = new Order(this); ConsoleHelper.writeMessage(newOrder.toString()); new AdvertisementManager(newOrder.getTotalCookingTime()*60).processVideos(); if(newOrder.isEmpty()) return null; setChanged(); notifyObservers(newOrder); return newOrder; }catch (IOException e){ logger.log(Level.SEVERE, "The console is unavailable."); return null; } catch(NoVideoAvailableException e){ logger.log(Level.INFO, "No video is available for the following order: " + newOrder); return newOrder; } } public int getNumber() { return number; } @Override public String toString() { return "Tablet{" + "number=" + number + '}'; } public void createTestOrder() { TestOrder testOrder; try { testOrder = new TestOrder(this); if (!testOrder.isEmpty()){ ConsoleHelper.writeMessage(testOrder.toString()); AdvertisementManager manager = new AdvertisementManager(testOrder.getTotalCookingTime() * 60); try { manager.processVideos(); } catch (NoVideoAvailableException e) { logger.log(Level.INFO, "No video is available for the following order: " + testOrder); } } } catch (IOException e) { e.printStackTrace(); } } }

Comments (1)

Popular
New
Old

You must be signed in to leave a comment

Pavlo Plynko Java Developer at CodeGym

7 April 2020, 15:39

Try to remove this line:

tablet.createOrder();

and this (use createOrderThread instead):

Thread thread = new Thread(createOrderThread);

And take code with threads out of the loop.