2 methods are not validated

I did run the test and it is as I suspected. Here is the code that I used to test your printAdRevenue() method:

public class Solution {
    public static void main(String[] args) throws Exception {
        printAdRevenue();
    }

    static public void printAdRevenue() throws Exception {
        // prints advertising revenues, grouped by day
        Map<Date, Double> revenues = new HashMap<>();
        {
            // this code fills the 'revenues' map with a date that can be today
            // or the same day for the last 2 years; as well as puts
            // amounts for each of those days
            for (int i = 0; i < 10; i++) {
                Date temp = new Date();
                temp.setYear(temp.getYear() - (int) (Math.random() * 3));
                revenues.put(temp, Math.random() * 100);
                Thread.sleep(10);
            }
        }

        // your code to test
        Double total = 0.00d;
        for (Date date : revenues.keySet()) {
            SimpleDateFormat format = new SimpleDateFormat("dd-MMM-YYYY");

            String dateString = format.format(date);
            Double revenue = revenues.get(date);
            String revString = String.format("%.2f", revenue);
            System.out.println((dateString + " - " + revString));
            total += revenue;
        }
        System.out.println((String.format("Total - %.2f", total)));
    }
}

I just simply copied and pasted the method into a fresh Solution class and made it static. Then created a generator to add 10 date-amount pairs into the HashMap; and just used Sys.out to output the results instead of the ConsoleHelper. The dates use the default date, which will be today as in the day you run the code, and then subtract 0, 1, or 2 from the year to get 3 unique dates. When you run the code it should show totals for those 3 dates, but it outputs all 10 dates.

Thank you, Guadalupe. I actually added quite a few changes to my code. 1) I converted Date instances to LocalDate ones in all the methods in question, to avoid the issue you pointed at; 2) I changed the signature of my getCookUtilization() method to public Table<LocalDate, String, Integer> getCookUtilization(), and now it returns a TreeBasedTable instance (very convenient) that stores dates (as LocalDate), cooks' names (as String) and cook's working time in minutes (as Integers). This is the way I initialized it:

Table<LocalDate, String, Integer> table = TreeBasedTable.create(Comparator.reverseOrder(), Comparator.naturalOrder());

3) I made some simple alterations to the method for the values to be added up if the date-name pair is already present in the table. 4) I added "min" to the printCookUtilization() method, and after that the 4th requirement was validated (as the input has to look exactly like in the task).

So, at this moment, only the getAdRevenue() and printAdRevenue() remain wrong, as the 3d requirement is not validated (though I tried to take into account every detail from the task description). Here's what they look like now. 1) The "getter" in the StatisticsManager class:

public Map<LocalDate, Double> getAdRevenue() {
        Map<LocalDate, Double> revenues = new TreeMap<>(Comparator.reverseOrder()); // the map will be created with the keys sorted in descending order
        List<EventDataRow> eventDataRows = statisticsStorage.getEventDataRows(EventType.VIDEOS_SELECTED);
        if (eventDataRows != null) {
            for (EventDataRow event : eventDataRows) {
                Date date = event.getDate();
                LocalDate localDate = date.toInstant().atZone(ZoneId.systemDefault()).toLocalDate();
                if (event.getAmount() == 0 || event.getTime() == 0) continue;
                double revenue = (double) event.getAmount() / (double) event.getTime();
                if (revenues.containsKey(localDate)) {
                    // the revenue per impression for that date should be displayed.
                    double newRevenue = revenue + revenues.get(localDate);
                    revenues.put(localDate, newRevenue);
                } else {
                    revenues.put(localDate, revenue);
                }
            }
        }
        return revenues;
}

2) The "printer" in the ManagerTablet class:

public void printAdRevenue() {
        // prints advertising revenues, grouped by day
        Map<LocalDate, Double> revenues = StatisticsManager.getInstance().getAdRevenue();
        Double total = 0.00d;
        for (LocalDate localDate : revenues.keySet()) {
            DateTimeFormatter formatter = DateTimeFormatter.ofPattern("dd-MMM-YYYY");
            String dateString = localDate.format(formatter);
            Double revenue = revenues.get(localDate);
            ConsoleHelper.writeMessage(String.format("%s - %.2f", dateString, revenue));
            total += revenue;
        }
        ConsoleHelper.writeMessage(String.format("Total - %.2f", total));
    }

I'm afraid I might be calculating the revenue variable wrongly in the getter. I actually made it equal the amount earned per second, while the task description said "for impression". But how do I know how many seconds each impression lasts? I tried different ways to change a bit of everything, including the processVideos() method in the AdvertisementManager class, but none of my attempts worked. I also thought the mistake might be in the LocalDate formatting and tried DateTimeFormatter formatter = DateTimeFormatter.ofPattern("dd-MMMM-YYYY") (MMMM instead of MMM), but it didn't work either. Still thinking what to do with my ad revenue calculation:).

In the getAdRevenue() method, this line:

double revenue = (double) event.getAmount() / (double) event.getTime();

I don't think you are supposed to be dividing amount by time. Testing your new printAdRevenue() with the new getAdRevenue() method seems (to me) to be outputting the correct results otherwise. However I did notice that the example results show that the entire month's name is used. For the example they used the month "May", which to me is a bit confusing because that could either mean to output the first three letters OR the entire month. I suggest you try this just to see if that works:

DateTimeFormatter formatter = DateTimeFormatter.ofPattern("dd-MMMM-YYYY");

EDIT: I just read your last post and see that you tried my suggestions already. So the task does specifically ask for the total ad revenue per day so you should not be trying to find out the revenue per second. That should mean that it should be correct by simply using either to format MMM or MMMM

Some code suggestions: The date formatting object should be outside of the loop. This is because you only need to create it one time to use it consecutively. Right now, in your code, it creates a new formatter object each loop cycle which adds all the unnecessary overhead for each additional time it is created past the first. Also you can make a for loop looping over each entry in a map instead of the keySet() of valueSet(). The would simplify your loop. So, with these suggestions, your code would look like this:

DateTimeFormatter formatter = DateTimeFormatter.ofPattern("dd-MMMM-YYYY");
for (Map.Entry<LocalDate, Double> entry : revenues.entrySet()) {
   ConsoleHelper.writeMessage((String.format("%s - %.2f", entry.getKey().format(formatter), entry.getValue())));
   total += entry.getValue();
}

Ok, I had to look at my code from this task I used to complete it. I noticed that I was dividing the amount by 100 in the printAdRevenue() method. I couldn't figure out why until I noticed that in your code, the VideoSelectedEventDataRow class, you have a really good comment for the amount field as follows:

long amount; // amount of money in cents

I didn't notate my code such as you did, this helped me make sense as to why I was dividing by 100. The amount field would hold a value such as 137, but the output needs to be in dollars (or Euro's) so that would equal $1.37. I think this is the real problem with your code, you just need to convert the cents to dollars/Euros.

Thank you for your advice, Guadalupe. Well, I took out the formatter from the loop (in both methods, including the one that passed, as it's pretty sensible). As for the number of M-s, I decided to keep MMM, as in the printCookUtilization() method this format worked. Of course, it's not a big deal to try every code variation with both MMM and MMMM, but this is probably not the issue. I also modified the calculation of the revenue value (and now the (double) event.getAmount() value is divided by 100, as cents need to be converted to dollars or euros). What seems to be the problem (to me) is probably the way I originally calculated the value that is derived by the event.getAmount() command. The thing is, I recorded this amount value to the VideoSelectedEventRowData instance in the processVideos() method of the AdvertisementManager class, but my logic was different there. I calculated it in a loop like this,

totalAmountEarned += ad.getAmountPerImpression() * ad.getImpressionsRemaining();

But now I see I should have probably calculated the maximum or total amount earned per impression. I've tried different ways of calculating the value, so that the VideoSelected event could be recorded with different data, but none of my variants has been successful yet. So, I'm still thinking:).

For instance, I tried the following variations: 1)

totalAmountEarned += ad.getAmountPerImpression();

// in case we need total amount per impression // doesn't make sense to me but who knows 2)

totalAmountPaid += ad.getAmountPaid();

// in case we need the sum of fixed amounts paid for each shown ad // definitely not what is expected here but just in case 3)

maxAmountPerImp = totalAmountEarned > ad.getAmountPerImpression()? totalAmountEarned : ad.getAmountPerImpression();

// in case we need the maximum amount earned per impression // could be derived just from the first value in the list of videos, as I initially grouped them by amountPerImpression 4)

averageAmountPerImpression += ad.getAmountPerImpression();

// in a loop and

averageAmountPerImpression = averageAmountPerImpression / playList.size();

// outside the loop // in case we need to know the average amount per impression earned by this playList Of course, I passed each variant as the "amount" argument when recoding the VideosSelected event. But...

I noticed that the test suggested by CodeGym produced the total revenue of 6.20 (displayed on the screen). I ran my code with different variations I'd suggested before and found out it got me 6.20 when I calculated the amount value by simply adding up the shown ads' "amountPerImpression" values (totalAmountEarned += ad.getAmountPerImpression()) in the loop from the AdvertisementManagement class's processVideos() method. However, the code is still not validated. Idk, what if it matters how I store the values? E.g. I could store dates as String variables...

I copied and pasted your entire code from the original post into my IntelliJ. After that I changed the printAdRevenue() and getAdRevenue() to what you shared in the comments at 9:15 and 9:16 yesterday. I had to change the TreeMultiMap to a TreeMap because I didn't have the TreeMultiMap installed. This just meant changing the printCookUtilization() method to use a treeMap and changing the for loop to this:

TreeMap<Date, String> cooksEfficiency...;
//...
for (String value : cooksEfficiency.values()) {
    ConsoleHelper.writeMessage(value);
}

and changing the getCookUtilization() to this:

public TreeMap<Date, String> getCookUtilization() {
    TreeMap<Date, String> cooksEfficiency = new TreeMap<>(Comparator.reverseOrder());
    //...
}

After that I sat down and tested your code by sending only 1 fish (Fish, exit, exit, exit, exit) to the tablet. The correct output should show revenue of 2.50, but your code was displaying 7.50. Here is what I did to get the correct output: #1 it should be this, as you suggested above, in the AdvertisementManager class:

totalAmountEarned += ad.getAmountPerImpression();

#2a I mentioned this earlier, so you may have changed it: In the getAdRevenue() method this line should be:

double revenue = event.getAmount();

#2b It still needs to convert the cents into dollars/Euros. I think the easiest place to do that is where it is initially calculated. In the getAdRevenue() method of the StatisticsManager Class you can simply change the one line to this:

// from this:
//double revenue = event.getAmount();

// to this:
double revenue = event.getAmount() / 100.0;

I get the correct output after these

After that the cook utilization shows, for 1 Fish, a time 0.42 min instead of 25 min. You say that this part is showing as passing, but it may be a mistake as the example output does show the minutes not as a fraction. If it passes then I wouldn't worry about it, but my code definitely outputs the full amount of minutes. I simply changed the getCookUtilization() method in your code to get rid of double and changed the String cookInfo to this:

String cookInfo = String.format("%s - %s min", cookName, event.getTime());

Thank you for your thorough work:). Well, the ad/printCookUtilization() pair in my code has been changed since the publication of this question. In case you're interested, the getter one looks like this:

public Table<LocalDate, String, Integer> getCookUtilization() {
        Table<LocalDate, String, Integer> table = TreeBasedTable.create(Comparator.reverseOrder(), Comparator.naturalOrder()); // rowKeys in descending order, column keys in the alphabetical one
        List<EventDataRow> eventDataRows = statisticsStorage.getEventDataRows(EventType.ORDER_READY);
        if (eventDataRows != null) {
            for (EventDataRow event : eventDataRows) {
                Date date = event.getDate();
                LocalDate localDate = date.toInstant().atZone(ZoneId.systemDefault()).toLocalDate();
                String cookName = event.getCookName();
                int cookWorkingTime = (int) event.getTime() / 60;
                if (table.contains(localDate, cookName)) {
                    int value = table.get(localDate, cookName);
                    int newWorkingTime = value + cookWorkingTime;
                    table.put(localDate, cookName, newWorkingTime);
                } else {
                    table.put(localDate,cookName, cookWorkingTime);
                }
            }
        }
        return table;
    }

You see, the working time is just an integer, so there are no problems with that, that's why it has passed. And the printer method just outputs everything using a nested for-loop. So the output is OK, too.

As for the ad/printAdRevenue() pair, I'm really confused. At this moment, the revenue is calculated as you suggested, and the getter method looks as follows:

public Map<LocalDate, Double> getAdRevenue() {
        Map<LocalDate, Double> revenues = new TreeMap<>(Comparator.reverseOrder()); // the map will be created with the keys sorted in descending order
        List<EventDataRow> eventDataRows = statisticsStorage.getEventDataRows(EventType.VIDEOS_SELECTED);
        if (eventDataRows != null) {
            for (EventDataRow event : eventDataRows) {
                Date date = event.getDate();
                LocalDate localDate = date.toInstant().atZone(ZoneId.systemDefault()).toLocalDate();
                if (event.getAmount() == 0 || event.getTime() == 0) continue;
                double revenue = event.getAmount() / 100.0; // cents to dollars
                if (revenues.containsKey(localDate)) {
                    double newRevenue = revenue + revenues.get(localDate);
                    revenues.put(localDate, newRevenue);
                } else {
                    revenues.put(localDate, revenue);
                }
            }
        }
        return revenues;
    }

Its printer counterpart displays the data in a for-loop, with dates and revenues formatted according to the requirements. The output for fish is this: 11-Apr-2025 - 2.50 Total - 2.50 11-Apr-2025 Jamie Oliver - 25 min I'm not sure what is still wrong.

Oh, wow, I've just found the mistake! But I'll keep all this for the record:) The problem was with this line of the getAdRevenue() method:

if (event.getAmount() == 0 || event.getTime() == 0) continue;

I only needed the alternative condition, event.getTime() == 0, when I used its result in calculating the revenue value. But the updated version of the code didn't need it any longer. So, the code got validated when I changed that line to this:

if (event.getAmount() == 0) continue;

These big tasks are a pain to help people with because what the validation is marking wrong may not even be the part of the code that is the problem. This task so far has 18 classes and the actual problem can be with any of them. Also, since we are on different continents, our communications are not in real time; by the time I open this up you have already solved the problems that I find. Either way, great job in finding the issue(s)! Your coding has improved so much; its noticeable in every new post you create. Have you started working as a programmer yet? Or are you at least looking into jobs? If not then you should be (assuming that you are learning to program to start doing it professionally). Additionally you should definitely consider retting a book or starting an online course geared towards passing the current Java certification. It looks like Java 17 is the current Oracle ceritification. Here is a link to Oracle's Java Certs

Thank you very much) I'm not working as a programmer, but I'd definitely want to do it professionally in the future. I actually haven't heard anything about Oracle certification, thank you for the link:).