CodeGym/Ayuda con Tareas en Java/Duda en el método removeAllSummerPeople(HashMap map)

Nivel 8

30.01.2024
195visualizaciones
12comentarios

Duda en el método removeAllSummerPeople(HashMap map)

Pregunta sobre la tarea

Resuelta

Buenas, me he quedado atascado en el método para eliminar las personas nacidas en los meses de verano. No se como hacer una condición if donde se averigüe sí la fecha de nacimiento esta entre los meses de verano. Adjunto mi código de lo que llevo hasta entonces. public static HashMap<String, Date> createMap() throws ParseException { DateFormat df = new SimpleDateFormat("MMMMM d yyyy", Locale.ENGLISH); HashMap<String, Date> map = new HashMap<String, Date>(); map.put("Stallone", df.parse("JUNE 1 1980")); //write your code here HashMap<String, Date> mapa = new HashMap<>(); mapa.put("Pérez", df.parse("JANUARY 19 1999")); mapa.put("San Pedro", df.parse("OCTOBER 2 1998")); mapa.put("Reyes", df.parse("MAY 1 2011")); mapa.put("Mora", df.parse("JULY 14 1973")); mapa.put("Alonso", df.parse("DECEMBER 21 2001")); mapa.put("Delgado", df.parse("FEBRUARY 23 2000")); mapa.put("Rodriguez", df.parse("JUNE 19 2010")); mapa.put("Ionut", df.parse("AUGUST 30 2000")); mapa.put("Peter", df.parse("FEBRUARY 14 2000")); mapa.put("Trujillo", df.parse("JULY 10 2003")); return mapa; } public static void removeAllSummerPeople(HashMap<String, Date> map) { //write your code here Date date1 = new Date("AUGUST 31"); Date date2 = new Date("JUNE 1"); Iterator<Map.Entry<String, Date>> iterador = map.entrySet().iterator(); while(iterador.hasNext()){ Map.Entry<String, Date> parejas = iterador.next(); if((parejas.getValue().before(date1)) && (parejas.getValue().after(date2))){ iterador.remove(); } } } public static void main(String[] args) { }

Create a Map<String, Date> and add ten entries that represent (last name, birth date) pairs.
Remove from the map all people born in the summer.
Hint: At CodeGym, summer lasts from June 1 to August 31.

Requisitos:

The program should not display text on the screen.
The program should not read values from the keyboard.
The createMap() method must create and return a HashMap that has (String, Date) elements and contains 10 entries.
The removeAllSummerPeople() method should remove from the map all people born in the summer.

package com.codegym.task.task08.task0816; import java.text.DateFormat; import java.text.ParseException; import java.text.SimpleDateFormat; import java.util.*; /* Kind Emma and the summer holidays */ public class Solution { public static HashMap<String, Date> createMap() throws ParseException { DateFormat df = new SimpleDateFormat("MMMMM d yyyy", Locale.ENGLISH); HashMap<String, Date> map = new HashMap<String, Date>(); map.put("Stallone", df.parse("JUNE 1 1980")); //write your code here HashMap<String, Date> mapa = new HashMap<>(); mapa.put("Pérez", df.parse("JANUARY 19 1999")); mapa.put("San Pedro", df.parse("OCTOBER 2 1998")); mapa.put("Reyes", df.parse("MAY 1 2011")); mapa.put("Mora", df.parse("JULY 14 1973")); mapa.put("Alonso", df.parse("DECEMBER 21 2001")); mapa.put("Delgado", df.parse("FEBRUARY 23 2000")); mapa.put("Rodriguez", df.parse("JUNE 19 2010")); mapa.put("Ionut", df.parse("AUGUST 30 2000")); mapa.put("Peter", df.parse("FEBRUARY 14 2000")); mapa.put("Trujillo", df.parse("JULY 10 2003")); return mapa; } public static void removeAllSummerPeople(HashMap<String, Date> map) { //write your code here Date date1 = new Date("AUGUST 31"); Date date2 = new Date("JUNE 1"); Iterator<Map.Entry<String, Date>> iterador = map.entrySet().iterator(); while(iterador.hasNext()){ Map.Entry<String, Date> parejas = iterador.next(); if((parejas.getValue().before(date1)) && (parejas.getValue().after(date2))){ iterador.remove(); //map.remove(parejas.getKey(), parejas.getValue()); //map.remove(parejas.getKey()); } } } public static void main(String[] args) { } }

Comentarios (12)

Populares
Nuevas
Antiguas

Debes iniciar sesión para dejar un comentario

M.A.P.M.

Nivel 8 , Spain

1 febrero, 14:40

Le sigo dando vuelta al ejercicio y no saco nada en claro. Alguna ayudita para poder entender como funciona el código de la función. Dejo por aquí como llevo el código : public static HashMap<String, Date> createMap() throws ParseException { DateFormat df = new SimpleDateFormat("MMMMM d yyyy", Locale.ENGLISH); HashMap<String, Date> map = new HashMap<String, Date>(); map.put("Stallone", df.parse("JUNE 1 1980")); //write your code here HashMap<String, Date> mapa = new HashMap<>(); mapa.put("Pérez", df.parse("JANUARY 19 1999")); mapa.put("San Pedro", df.parse("OCTOBER 2 1998")); mapa.put("Reyes", df.parse("MAY 1 2011")); mapa.put("Mora", df.parse("JULY 14 1973")); mapa.put("Alonso", df.parse("DECEMBER 21 2001")); mapa.put("Delgado", df.parse("FEBRUARY 23 2000")); mapa.put("Rodriguez", df.parse("JUNE 19 2010")); mapa.put("Ionut", df.parse("AUGUST 30 2000")); mapa.put("Peter", df.parse("FEBRUARY 14 2000")); mapa.put("Trujillo", df.parse("JULY 10 2003")); return mapa; } public static void removeAllSummerPeople(HashMap<String, Date> map) throws ParseException{ //write your code here HashMap<String, Date> mapa = new HashMap<>(map); DateFormat formateador = new SimpleDateFormat("MMMMM d"); String fecha1 = "JUNE 1"; String fecha2 = "AUGUST 31"; Date fecha1Formateada = formateador.parse(fecha1); Date fecha2Formateada = formateador.parse(fecha2); Iterator<Map.Entry<String, Date>> iterador = mapa.entrySet().iterator(); while(iterador.hasNext()){ Map.Entry<String, Date> parejas = iterador.next(); if((parejas.getValue().after(fecha1Formateada)) && (parejas.getValue().before(fecha2Formateada))){ mapa.remove(parejas.getKey()); //iterador.remov

Guadalupe Gagnon

Nivel 37 , Tampa, United States

1 febrero, 16:02

How are you testing your code???? Thomas already mentioned that. I can tell you that what you wrote in the removeAllSummerPeoplpe() won't work simply because if you don't specify a year (for JUNE 1 and AUGUST 31) the year will default to 1970. Therefore you are asking if parejas.getValue() is after June 1, 1970 and before August 31, 1970. With the 10 people you added in the createMap() method this will result in the after always being true and the before always being false. Simply run this code, which I copied from your code and added an output line:

DateFormat formateador = new SimpleDateFormat("MMMMM d");
Date fecha1Formateada = formateador.parse("JUNE 1");

System.out.println("Date fecha1Formateada = " + fecha1Formateada);

and you will see that the year is 1970, specifically Mon Jun 01 00:00:00 EDT 1970. If you don't test your code you are going to spend a LONG time on these tasks.

Thomas

Nivel 41 , Bayreuth, Germany

30 enero, 16:08

That does not work and would throw an exception. Even if it would, then you would not match Aug. 31

Date date1 = new Date("AUGUST 31");

If you want to continue that route you'd need to add a year. Probably the year of the date you currently intend to test. And test your code (writing a main method is a good idea for that). An alternative would be to use Calendar or LocalDate to get the current month as an int, enum or class and then to compare against that result.

M.A.P.M.

Nivel 8 , Spain

1 febrero, 10:41

Sigo sin poder resolver este método. No he entendido bien lo que me has dicho. He probado a usar el método getMonth() pero al pasar la verificación me sigue fallando el ultimo punto de la tarea y además ese método esta en desuso. Actualmente tengo lo siguiente escrito en la ultima función: public static void removeAllSummerPeople(HashMap<String, Date> map){ //write your code here Iterator<Map.Entry<String, Date>> iterador = map.entrySet().iterator(); while(iterador.hasNext()){ Map.Entry<String, Date> parejas = iterador.next(); int month = parejas.getValue().getMonth(); if((month > 5) && (month < 9)){ iterador.remove(); } } }

M.A.P.M.

Nivel 8 , Spain

1 febrero, 11:07

También he probado la función escrita de esta manera y no me pasa la validación del último punto de la tarea: public static void removeAllSummerPeople(HashMap<String, Date> map) throws ParseException{ //write your code here DateFormat formateador = new SimpleDateFormat("MMMMM d"); Date fecha = formateador.parse("AUGUST 31"); Date fecha2 = formateador.parse("JUNE 1"); Iterator<Map.Entry<String, Date>> iterador = map.entrySet().iterator(); while(iterador.hasNext()){ Map.Entry<String, Date> parejas = iterador.next(); if((parejas.getValue().after(fecha2)) && (parejas.getValue().before(fecha))){ iterador.remove(); } } }

Thomas

Nivel 41 , Bayreuth, Germany

2 febrero, 22:03

With testing I mean to write some code that helps you debug what you have written. You also may use the IDEs debug functionality (but that is kinda difficult at this time). So usually you write a main method and check the code you have written. E.g. here with:

public static void main(String[] args) throws Exception{
    HashMap<String, Date> people = createMap();
    removeAllSummerPeople(people);
    people.entrySet().forEach(System.out::println);
}

For validating the code you can comment it out or remove it. The second method using sdf has a similar problem validation (creating a date not having a year for comparison). As Lupe mentioned in his post such a date defaults to the year 1970. Means you get wrong results in nearly all cases. The method using the getMonth() method of the Date class looks pretty good. It has the problem that getDate returns 0 - 11 and not 1 - 12 as you may have assumed. If you'd tested your code with all summer months and all edge cases you can imagine then you for sure would have noticed. Good luck :)

M.A.P.M.

Nivel 8 , Spain

3 febrero, 14:51

Vale, he probado mi código y me he dado cuenta de que los meses obtenidos van del 0 - 11 y no del 1 - 12, he comprobado que si que me elimina a las personas nacidas en los meses de verano. Al validar la tarea me dice que me asegure eliminar las personas nacidas en los meses de verano en el método removeAllSummerPeople. El código escrito en dicho método es el siguiente : public static void removeAllSummerPeople(HashMap<String, Date> map){ //write your code here Iterator<HashMap.Entry<String, Date>> iterador = map.entrySet().iterator(); while(iterador.hasNext()){ HashMap.Entry<String, Date> parejas = iterador.next(); if((parejas.getValue().getMonth()) > 4 && (parejas.getValue().getMonth()) < 9){ // System.out.println(parejas.getValue().getMonth()); iterador.remove(); } } } Lo que no entiendo es porque me dice eso la validación, sabiendo que mi código elimina a las personas nacidas en dichos meses.

M.A.P.M.

Nivel 8 , Spain

3 febrero, 15:15

Sé que el error esta en el segundo método, alguna pista del error: public static HashMap<String, Date> createMap() throws ParseException { DateFormat df = new SimpleDateFormat("MMMMM d yyyy", Locale.ENGLISH); HashMap<String, Date> map = new HashMap<String, Date>(); map.put("Stallone", df.parse("JUNE 1 1980")); HashMap<String, Date> mapa = new HashMap<>(); mapa.put("Pérez", df.parse("JANUARY 19 1980")); mapa.put("San Pedro", df.parse("OCTOBER 2 1980")); mapa.put("Reyes", df.parse("MAY 1 1980")); mapa.put("Mora", df.parse("JULY 14 1980")); mapa.put("Alonso", df.parse("DECEMBER 21 1980")); mapa.put("Delgado", df.parse("FEBRUARY 23 1980")); mapa.put("Rodriguez", df.parse("JUNE 19 1980")); mapa.put("Ionut", df.parse("AUGUST 30 1980")); mapa.put("Peter", df.parse("FEBRUARY 14 1980")); mapa.put("Trujillo", df.parse("JULY 10 1980")); return mapa; } public static void removeAllSummerPeople(HashMap<String, Date> map)throws ParseException{ DateFormat formateadorFecha = new SimpleDateFormat("MMMMM d yyyy", Locale.ENGLISH); Date fecha1 = formateadorFecha.parse("JUNE 1 1980"); Date fecha2 = formateadorFecha.parse("AUGUST 31 1980"); System.out.println(fecha1); System.out.println(fecha2); Iterator<HashMap.Entry<String, Date>> iterador = map.entrySet().iterator(); while(iterador.hasNext()){ HashMap.Entry<String, Date> parejas = iterador.next(); if((parejas.getValue().getMonth()) > 4 && (parejas.getValue().getMonth()) < 9){ iterador.remove(); } } } public static void main(String[] args) throws ParseException { HashMap<String, Date> mapas = createMap(); removeAllSummerPeople(mapas); for(Map.Entry<String, Date> auxiliar : mapas.entryS

Thomas

Nivel 41 , Bayreuth, Germany

4 febrero, 12:32

útil

test all possibilites and all edge cases. June, July and August need to be removed but May and September need to remain inside the map (the latter I consider to be edge cases and you didn't test all of them)

M.A.P.M.

Nivel 8 , Spain

6 febrero, 18:07

He probado como caso extremo el poner horas a los días de verano, pero aún así no paso el último punto de la tarea. Además he probado el debugger y veo como elimina del mapa las personas nacidas en los meses de veranos. Ya no se que más probar, llevo atascado unos días que ya he perdido la cuenta.

Thomas

Nivel 41 , Bayreuth, Germany

6 febrero, 21:53

útil

I have no clue what you changed since the last time you posted code. So I just can comment on that last code. And that's why my suggestion still applies: did you test edge cases? Why I ask that? That's easy to answer!

if((parejas.getValue().getMonth()) > 4 && (parejas.getValue().getMonth()) < 9){

> 4 && < 9 includes four month while summer just has three. And if you remember that month counting in the Date class starts with 0 then > 4 && < 9 translates to > May && < October (June, July, August, September -> testing the edge cases would have shown you the problem)

M.A.P.M.

Nivel 8 , Spain

7 febrero, 15:24

Me estás diciendo que mi código lo tenía bien desde hace días y que lo único era que no tenía en cuenta el conteo de los meses ?, madre de dios. Muchas gracias Thomas, de verdad te lo digo. Entendía completamente lo que había que hacer pero no caí en dicho error. Comprobé mi error y tenías toda la razón. Muchas gracias. He de decirte que después de 34 intentos, he resuelto el ejercicio.