Edit 1
Aquí está el cambio en la solución inicial tomando en cuenta que se deben guardar en mCalendar
objetos de tipo Celebration
.
class Main {
public static ArrayList<Celebration> mCalendar = new ArrayList<>();
private static void fillFeriaeAdventus() {
LocalDate start = LocalDate.of(2021, 11, 28);
LocalDate end = LocalDate.of(2021, 12, 24);
List<LocalDate> dates = Stream.iterate(start, date -> date.plusDays(1))
.limit(ChronoUnit.DAYS.between(start, end))
.collect(Collectors.toList());
// copiamos las fechas ya existentes en mCalendar en un hashmap para permitir busqueda O(1)
HashMap<LocalDate, Boolean> datesMap = new HashMap<>();
mCalendar.forEach(date -> {datesMap.put(date.getDate(), true);}); // O(n)
// agregamos más fechas
// vamos a agregar las que están en dates
// pero que no estén ya previamente en mCalendar
/*AtomicInteger n = new AtomicInteger(0);
mCalendar.addAll(
dates.stream()
.filter(Predicate.not(datesMap::containsKey)) // la busqueda se hace en O(1)
.map(date -> new Celebration(n.getAndIncrement(), date)) // convertimos o mapeamos los objetos LocalDate a Celebration
.collect(Collectors.toList())
); // O(n)*/
// La otra opción es hacerlo con un for
for (int i = 0; i < dates.size(); ++i)
if (!datesMap.containsKey(dates.get(i)))
mCalendar.add(new Celebration(i, dates.get(i)));
System.out.println("mCalendar.size(): " + mCalendar.size());
mCalendar.forEach(System.out::println);
}
public static void main(String... args) {
System.out.println("Primera invocación: Debería agregar elementos al arreglo");
fillFeriaeAdventus();
System.out.println("Segunda invocación: NO debería agregar elementos al arreglo pues serían duplicados");
fillFeriaeAdventus();
}
private static class Celebration {
private int idx;
private LocalDate localDate;
public Celebration(int idx, LocalDate localDate) {
this.idx = idx;
this.localDate = localDate;
}
public LocalDate getDate() {
return this.localDate;
}
@Override
public String toString() {
return this.localDate.toString();
}
}
}
En este caso no hay mucho cambio salvo por el .map()
para crear los objetos Celebration
. Sin embargo, dado que uno de los fundamentos de la programación funcional es que no debe haber modificaciones de estado (como con el iterador n
), decidí comentar esa solución para preferir la normal con un for
. Además, quizá tenga mejor performance (pero no estoy seguro).