Skip to main content
Corrigindo o compareTo(), enhanced-for
Fonte Link
Victor Stafusa
  • 64,9mil
  • 9
  • 120
  • 189

Troque isso:

System.out.print(lista.get(i) + "\n");

Por isso:

System.out.print(lista.get(i).getNome() + "\n");

E arrume o seu método compareTo(), aonde você trocou o < com o >. Ficará assim:

    public int compareTo(Pessoa comparar) {
        int nomeComparar = this.getNome().compareTo(comparar.getNome());
        if(nomeComparar < 0)
            return -1;
        else if (nomeComparar > 0)
            return 1;
        else return 0;
    }

Ou se preferir, dá para simplificar ainda mais:

    public int compareTo(Pessoa comparar) {
        return this.getNome().compareTo(comparar.getNome());
    }

E por fim, você ainda pode simplificar o seu loop evitando ter que gerenciar contadores. Assim, ao invés disso:

    for(int i = 0; i < lista.size(); i++) {   
        System.out.print(lista.get(i).getNome() + "\n");
    } 

Use isso:

    for (Pessoa p : lista) {   
        System.out.println(p.getNome());
    } 

Troque isso:

System.out.print(lista.get(i) + "\n");

Por isso:

System.out.print(lista.get(i).getNome() + "\n");

Troque isso:

System.out.print(lista.get(i) + "\n");

Por isso:

System.out.print(lista.get(i).getNome() + "\n");

E arrume o seu método compareTo(), aonde você trocou o < com o >. Ficará assim:

    public int compareTo(Pessoa comparar) {
        int nomeComparar = this.getNome().compareTo(comparar.getNome());
        if(nomeComparar < 0)
            return -1;
        else if (nomeComparar > 0)
            return 1;
        else return 0;
    }

Ou se preferir, dá para simplificar ainda mais:

    public int compareTo(Pessoa comparar) {
        return this.getNome().compareTo(comparar.getNome());
    }

E por fim, você ainda pode simplificar o seu loop evitando ter que gerenciar contadores. Assim, ao invés disso:

    for(int i = 0; i < lista.size(); i++) {   
        System.out.print(lista.get(i).getNome() + "\n");
    } 

Use isso:

    for (Pessoa p : lista) {   
        System.out.println(p.getNome());
    } 
Fonte Link
Victor Stafusa
  • 64,9mil
  • 9
  • 120
  • 189

Troque isso:

System.out.print(lista.get(i) + "\n");

Por isso:

System.out.print(lista.get(i).getNome() + "\n");