Can't get the third condition to work

Question about the task

Under discussion

Hi, I can't get the third condition to work. I think the problem is related to the amount per day that should be a double, but it's a long from the beginning of the exercise. I reverse my treemap display (descendingMmap()) which I think it's right, but maybe I don't get the description of the exercise. Should I display by descending value instead? (I tried that, but no luck). Thank you

Let's implement the first statistics item: advertising revenues, grouped by day.

1. In the VideosSelectedEventDataRow class, create a getter for the amount field.
In the OrderReadyEventDataRow class, create a getter for the cookName field.

2. In StatisticsManager, create a method (come up with its name on your own) that will get all repository data related to displaying advertising, and will calculate the total revenues for each day.
Additionally, add a get helper method to the repository class to access the data.

3. Let's start with the logic of the printAdRevenue method in the ManagerTablet class.
Using the method from the previous item, display the dates and amounts in decreasing order.
For each date with displayed ads in the event repository, the revenue per impression for that date should be displayed.
At the end, display "Total" and the total amount.

Example:

14-May-2013 - 2.50
13-May-2013 - 1.02
12-May-2013 - 543.98
Total - 547.50

4. Let's implement the second statistics item: cook utilization (time spent working), grouped by day.

In StatisticsManager, create a method (come up with its name on your own) that will get all repository data related to a cook's work, and will calculate the total time spent working for each cook separately.

5. Implement the logic of the ManagerTablet class's printCookUtilization method.
Using the method from the previous item to display dates, names of cooks, and hours worked (round up), in decreasing order.
For each date in the event repository with a record about work performed by a cook, the time worked on that date in minutes should be displayed.
If a cook did not work on a day, then DO NOT display empty data for him or her (see 13-May-2013)
Sort the cooks by name

Example:

14-May-2013
Puck - 60 min
Ramsay - 35 min

13-May-2013
Puck - 129 min

12-May-2013
Puck - 6 min
Ramsay - 5 min

Requirements:

A getter for the amount field must be created in the VideosSelectedEventDataRow class.
A getter for the cookName field must be created in the OrderReadyEventDataRow class.
The ManagerTablet class's printAdRevenue method must be implemented in accordance with the task conditions.
The ManagerTablet class's printCookUtilization method must be implemented in accordance with the task conditions.

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.statistics.StatisticsManager; import com.codegym.task.task27.task2712.statistics.event.OrderReadyEventDataRow; import java.io.IOException; import java.util.List; 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()); private Order order = null; public Tablet(int number){ this.number = number; } public Order createOrder(){ try{ order = new Order(this); ConsoleHelper.writeMessage(order.toString()); AdvertisementManager advertisementManager = new AdvertisementManager(order.getTotalCookingTime()*60); advertisementManager.processVideos(); if(!order.isEmpty()){ setChanged(); notifyObservers(order); } } catch(IOException e){ logger.log(Level.SEVERE, "The console is unavailable."); } catch(NoVideoAvailableException e){ logger.log(Level.INFO, "No video is available for the following order: " + order); } return order; } public Order getOrder(){ return order; } @Override public String toString() { return "Tablet{number="+number+"}, cooking time " + order.getTotalCookingTime() + " min"; } }

Comments (5)

Popular
New
Old

You must be signed in to leave a comment

Henrique

Level 41 , São Paulo, Brazil

21 August 2020, 14:56

did you solve it?

brus5

Level 41 , Dąbrowa Górnicza, Poland

2 September 2020, 18:00

Etienne Bélanger's solution was good, but he missed to divide revenuePerDay.getValue() and totalValue by 100.

for (Map.Entry<Date, Double> revenuePerDay : statisticsManager.getRevenuePerDays().descendingMap().entrySet()) {
    ConsoleHelper.writeMessage(dateFormat.format(revenuePerDay.getKey()) + " - " + String.format("%.2f", revenuePerDay.getValue() / 100));
    total += revenuePerDay.getValue();
}
ConsoleHelper.writeMessage("Total - " + String.format("%.2f", total / 100));

KIN SOCHEAT

Level 34 , Phnom Penh, Cambogia

7 August 2020, 13:46

All List food: Fish, Steak, Soup, Juice, Water Please choose witch food? Fish Steak Soup Juice Water exit Your order: [Fish, Steak, Soup, Juice, Water] from Tablet{number=5} Start cooking - Your order: [Fish, Steak, Soup, Juice, Water] from Tablet{number=5}, cooking time 78 min Your order: [Fish, Steak, Soup, Juice, Water] from Tablet{number=5} was prepared by Kin Displaying Third Video... 200, 333 Displaying First Video... 50, 277 Displaying Second Video... 10, 11 07-Aug-2020 - 2.60 Total - 2.60 07-Aug-2020 Kin - 78 min

AnnemiekFF

Level 41 , Breda, Netherlands

8 May 2020, 17:24

In your helper function in StatisticStorage, you might want to create 2 get methods. One for cooks and 1 for videos. Makes the overall code more readable imo. But that is a detail. I suspect the problem might be in how you handle multiple entries for the same day. But not 100% sure. This is my function for getting all advert data (I only get the Video data):

LinkedHashMap<String, Double> data = new LinkedHashMap<>();
        ArrayList<EventDataRow> rawdata = (ArrayList<EventDataRow>) statisticsStorage.getAllAdvertDataHelper();
        sort(rawdata);

        SimpleDateFormat formatter = new SimpleDateFormat("dd-MMM-yyyy");

        double sum = 0;
        for (EventDataRow x : rawdata){
            String key = formatter.format(x.getDate());
            VideosSelectedEventDataRow y = (VideosSelectedEventDataRow) x;
            double temp2 = y.getAmount();
            sum += temp2;
            if(data.containsKey(key)){
                data.put(key, data.get(key) + temp2);
            }else{
                data.put(key, temp2);
            }
        }
        for(Map.Entry<String,Double> x : data.entrySet()){
            ConsoleHelper.writeMessage(String.format("%s - %.2f",x.getKey(),(double)x.getValue()/100));
        }
        ConsoleHelper.writeMessage(String.format("Total - %.2f", (double)sum/100));

And in the managerTablet all I have to do is call the function and nothing else.

Henrique

Level 41 , São Paulo, Brazil

21 August 2020, 14:52

Where does this "sort(rawdata);" come from?