[SOLUCIONADO] Implementando Comparator con objetos

[detective_jd]

Buenas a todos, tengo una consulta para hacer, resulta que tengo una clase Cargo que tiene como atributos id y nombre, luego tengo otra clase llamada Persona que tiene como atributos id, nombre y el cargo que tiene.
Resulta que quiero ordenar las personas por cargos y cuando hago dicha ordenación me borra la anterior persona con el cargo repetido.

Pongo el código:

Código Java:

package app.contr.clases;
public class Cargo implements Comparable<Cargo> 
{
    private int id;
    private String nom;
    public static int autoID = 0;
    public int getId() {
        return id;
    }
    public void setId(int id) {
        this.id = id;
    }
    public String getNom() {
        return nom;
    }
    public void setNom(String nom) {
        this.nom = nom;
    }
    public Cargo(int xid, String xnom) {
        this.id = xid;
        this.nom = xnom.toUpperCase();
    }
    @Override
    public boolean equals(Object obj) {
        if(obj instanceof Cargo){
            Cargo objc = (Cargo)obj;
            return objc.nom.equals(this.nom); 
        }
        else
            return false;
    }
    @Override
    public int hashCode() {
        int hash = 17;
        hash = 31 * hash + this.nom.hashCode();
        return hash;
    } 
    @Override
    public String toString() {
        return nom;
    }    
    @Override
    public int compareTo(Cargo o) {
        if (this.nom.compareTo(o.getNom()) > 0) 
            return 1;
        else if (this.nom.compareTo(o.getNom()) < 0)
            return -1;
        else
            return 0;
    }
    public static int obtenerMAXID(){
        return autoID + 1;
    }
    public static void incrementarID(){
        autoID++;
    }
}
 

Código Java:

package app.contr.clases;
public class Persona 
{
    private final int id;
    private String nom;
    private Cargo cargo;
    public static int autoID = 0;
    public int getId() {
        return id;
    }
    public String getNom() {
        return nom;
    }
    public void setNom(String nom) {
        this.nom = nom;
    }  
    public Cargo getCargo() {
        return cargo;
    }
    public void setCargo(Cargo cargo) {
        this.cargo = cargo;
    }
    public Persona(int xid, String xnom, Cargo xcargo) {
        this.id = xid;
        this.nom = xnom.toUpperCase();
        this.cargo = xcargo;
    }    
    @Override
    public boolean equals(Object obj) {
        if(obj instanceof Persona){
            Persona objp = (Persona)obj;
            return objp.nom.equals(this.nom);
        }
        else
            return false;
    }
    @Override
    public int hashCode() {
        int hash = 17;
        hash = 31 * hash + this.nom.hashCode();
        return hash;
    }    
    @Override
    public String toString() {
        return nom;
    }
    public static int obtenerMAXID(){
        return autoID+1;
    }
    public static void incrementarID(){
        autoID++;
    }
}
 

Ordena personas por cargo de forma creciente:

Código Java:

package app.vista.util;
import app.contr.clases.Persona;
import app.vista.facade.FacadeContr;
import java.util.Comparator;
public class OrdenarPerXCA implements Comparator<Integer> 
{
    @Override
    public int compare(Integer o1, Integer o2) {
        Persona p1 = FacadeContr.getInstance().ContrPersonas().findAll().get(o1);
        Persona p2 = FacadeContr.getInstance().ContrPersonas().findAll().get(o2);
        return p1.getCargo().getNom().compareTo(p2.getCargo().getNom());        
    }    
}
 

Ordena personas por cargo de forma decreciente:

Código Java:

package app.vista.util;
import app.contr.clases.Persona;
import app.vista.facade.FacadeContr;
import java.util.Comparator;
public class OrdenarPerXCD implements Comparator<Integer> 
{
    @Override
    public int compare(Integer o1, Integer o2) {
        Persona p1 = FacadeContr.getInstance().ContrPersonas().findAll().get(o1);
        Persona p2 = FacadeContr.getInstance().ContrPersonas().findAll().get(o2);
        return p2.getCargo().getNom().compareTo(p1.getCargo().getNom());
    }    
}
 

Necesito solucionar esta falla y no sé cómo, espero sus respuestas y saludos.

[Fuzzylog]

¿y dónde está el código que se ejecuta en el main, que es el meollo de la cuestión?

[CalgaryCorpus]

Bien extraños tus comparators. Si quieres comparar personas, independiente del criterio, son personas las que deberían ser los parámetros del comparator.

[Fuzzylog]

el comparator es un sistema de ordenación en base a una propiedad del objeto. Si quiere usar el nombre del cargo está en su derecho.

[CalgaryCorpus]

Cuando comparas 2 objetos X con la propiedad Y, lo que pasas al comparator es el tipo de X, no el de Y. El campo Y es parte de la implementación del comparator, no de sus parámetros. Eso es lo que comento.

[detective_jd]

Muchas gracias FuzzyLog y CargaryCorpus por responder, verán hasta intenté que me ordene por el cargo sólo para apelar al compareTo implementado en la clase Cargo:

Código Java:

package app.vista.util;
import app.contr.clases.Persona;
import app.vista.facade.FacadeContr;
import java.util.Comparator;
public class OrdenarPerXCD implements Comparator<Integer> 
{
    @Override
    public int compare(Integer o1, Integer o2) {
        Persona p1 = FacadeContr.getInstance().ContrPersonas().findAll().get(o1);
        Persona p2 = FacadeContr.getInstance().ContrPersonas().findAll().get(o2);
        return p2.getCargo().compareTo(p1.getCargo());
    }    
}
 

Pero el resultado es el mismo, que me elimina a la persona anterior con el cargo repetido, sí tengo una función que me valida si una entrada con tal clave para evitar duplicados sólo que si ignoro esa función al ordenar me da este error:

Exception in thread "AWT-EventQueue-0" java.lang.IndexOutOfBoundsException: Index 3 out-of-bounds for length 3

Y ahí me deja con cara de ¿que voy a a hacer?

Espero sus respuestas y saludos.

[detective_jd]

Tengo que pedirles disculpas porque fue una tontería mía este post, hice las pruebas pertinentes comparando el desarrollo que estoy haciendo con el original, y ambos funcionan igual ya que el compareTo del original y el que estoy haciendo no aceptan duplicados para la estructura de datos.

Gracias por el tiempo dedicado a este post inútil, nuevamente pido disculpas.

Saludos.