POO-ITBA / 2023_02

0 stars 0 forks source link

Ej9 - TP6 #13

Open SrMathew opened 1 year ago

SrMathew commented 1 year ago

Buenas tardes,

escribo para consultar con respecto a la versión final de la clase cuyas instancias se almacenarán en el TreeMap. Consideré agregar un chequeo en el Constructor de la clase para verificar la validez de la hora y minutos, también descarté el @Override de equals() en dicha clase ya que, si no me equivocó, el TreeMap tan solo utiliza el Comparator para decidir el orden de los elementos y solo utilizamos el equals() del tipo genérico (al que le pedimos que sea Comparable) en la implementacion de la colección en contains() y remove() respectivamente

Adjunto el código de mi clase:


import java.util.Objects;

public class TimeElem<T extends Comparable<? super T>> {
    private T elem;
    private int hours;
    private int minutes;

    public TimeElem(T elem, int hours, int minutes){
        this.elem = elem;
        if ( hours>=0 && hours<24 && minutes>=0 && minutes<60 ){
            this.hours = hours;
            this.minutes = minutes;
        }else{
            throw new IllegalArgumentException();
        }
    }

    public T getElem(){
        return elem;
    }

    @Override
    public int hashCode() {
        return Objects.hash(getElem());
    }

    public boolean inRange(int hoursFrom, int minutesFrom, int hoursTo, int minutesTo){
        return ( (hours<= hoursTo && hours>=hoursFrom) && (minutes<=minutesTo && minutes>=minutesFrom) );
    }

    public int getHours(){
        return hours;
    }

    public int getMinutes(){
        return minutes;
    }
}

Verifique que la salida esperada sea la misma con el programa de prueba y también sumando eventos en el mismo horario

Desde ya muchas gracias

fmeola commented 1 year ago

Hola @SrMathew La validación de horas y minutos agregada es correcta pero estás repitiendo código. Deberías llamar a la función inRange y ver si está entre las 0 y las 23. Es verdad que TreeMap usa orden natural o el Comparator para almacenar ordenadas las claves. En ese caso además de no necesitar equals tampoco necesitás hashcode. No veo la implementación completa pero de todas formas te recuerdo que para el TimeSet se espera que utilicen un TreeSet y no un TreeMap como está planteado en la solución