Skip to content
This repository has been archived by the owner on Jul 23, 2024. It is now read-only.

1C 2023 - Ej 2 #10

Open
SrMathew opened this issue Sep 25, 2023 · 2 comments
Open

1C 2023 - Ej 2 #10

SrMathew opened this issue Sep 25, 2023 · 2 comments

Comments

@SrMathew
Copy link

SrMathew commented Sep 25, 2023

Holas buenas tardes,

quería consultarle con respecto a la solución propuesta ya que en ella en la clase VideoCatalogElement primero figura el objeto comparandolo con la instancia de la siguiente forma:

@Override
   public int compareTo(VideoCatalogElement o) {
       int cmp = o.airedDate.compareTo(airedDate);
       if(cmp == 0) {
           cmp = title.compareTo(o.title);
       }
       return cmp;
   }

Tenía entendido que primero se especifica la instancia y se la compara con el objeto parámetro de la siguiente forma:

@Override
   public int compareTo(VideoCatalogElement o) {
       int cmp = this.airedDate.compareTo(o.airedDate);
       if(cmp == 0) {
           cmp = this.title.compareTo(o.title);
       }
       return cmp;
   }

Aclarando que también en la clase VideoCatalog los métodos para listar quedarían de la siguiente forma:

public VideoCatalogElement[] getDescendingDateCopy(){
        return getCopy( Comparator.reverseOrder() );
    }

    public VideoCatalogElement[] getChronologicalDateCopy(){
        return getCopy( Comparator.naturalOrder() );
    }

siendo que en las soluciones figura de la siguiente forma:

public VideoCatalogElement[] getDescendingDateCopy(){
        return getCopy( Comparator.naturalOrder() );
    }

    public VideoCatalogElement[] getChronologicalDateCopy(){
        return getCopy( Comparator.reverseOrder() );
    }

Desde ya muchas gracias

@fmeola
Copy link
Member

fmeola commented Sep 25, 2023

Hola @SrMathew
Si planteás

o.airedDate.compareTo(airedDate);

estás logrando un orden descendente por fecha.
En cambio si planteás

this.airedDate.compareTo(o.airedDate)

estás logrando un orden ascendente por fecha.
Depende del ejercicio y del orden que te pidan para decirte cuál de las dos es correcta.

@SrMathew
Copy link
Author

Okaa gracias profe!!

Recien lo estaba editando porque me había olvidado de aclarar que tenia que cambiar reverseOrder() por naturalOrder() en getDescendingDateCopy() de la clase VideoCatalog

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants