Ne pas utiliser l'interface comparable de la bonne manière

Si vous prévoyez d'utiliser le tri de bibliothèque standard Java, vous devez utiliser des interfaces standard. Vous avez besoin de l'un ou l'autre des éléments de collection pour implémenter java. lang.Comparable ou un Objet java.lang.Comparator .

Il serait préférable de supprimer ou de renommer votre propre version de Comparator , cela ne permet pas de savoir laquelle est réellement utilisée lorsque vous regardez votre code car les classes java.lang sont importé automatiquement.

Vous avez déjà créé un java.lang.Comparator avec ci-dessous (c1, c2) lambda:

City.sort(Comparator.comparingInt(City2::getZipCode));

bien que vous ayez pu l'écrire aussi comme:

Collections.sort(City, Comparator.comparingInt(City2::getZipCode));

Ou en utilisant List.sort :

Collections.sort(City, (c1, c2) -> c1.getZipCode() - c2.getZipCode());

Si vous souhaitez explicitement utiliser l'interface Comparable , remplacez votre classe City2 par:

public class City2 implements Comparable<City2> {

    private int zipCode;
    private String cityName;

    public City2(int zipCode, String cityName) {
        this.zipCode = zipCode;
        this.cityName = cityName;
    }

    public int getZipCode() {
        return zipCode;
    }

    public String getCityName() {
        return cityName;
    }

    public void addAndPrint() {

    }

    @Override
    public int compareTo(City2 city) {
        return this.getZipCode() - city.getZipCode();
    }

}

Observez que: p >

1. Ici, nous implémentons le Comparable plutôt que le brut tapez Comparable
2. Remplacement de compareTo () pour comparer les codes postaux si nécessaire.
3. Vous n'avez pas besoin de créer explicitement une interface Comparable , mais implémentez simplement celui fourni par Java.

D'accord, alors ...

Maintenant, c'est presque terminé à 100%!

Mais j'obtiens encore et encore la même réponse ...

Des idées? C'est comme si les impressions se répètent mais les dernières lignes sont en fait correctes ....

Classe principale: public class cityMain

class City2 implements Comparable<City2> {

private int zipCode;
private String cityName;

public City2(int zipCode, String cityName) {
    this.zipCode = zipCode;
    this.cityName = cityName;
}

public int getZipCode() {
    return zipCode;
}

public String getCityName() {
    return cityName;
}

@Override
public int compareTo(City2 city) {
    return this.getZipCode() - city.getZipCode();
}

public void print() {

    for (int i = 0; i < cityMain.City.size(); i++) {
        System.out.println(cityMain.City.get(i).getZipCode() + " " + cityMain.City.get(i).getCityName());
    }

}
}

Deuxième classe:

private static BufferedReader r;
public static ArrayList<City2> City = new ArrayList<City2>();
public static void main(String[] args) {
    int Z = 0;
    String C = null;
    try {
        FileReader file = new FileReader("C:\\Users\\karwa\\Desktop\\BB.dat");
        r = new BufferedReader(file);
        String currLine;
        while ((currLine = r.readLine()) != null) {
            City.sort(Comparator.comparing(City2::getZipCode));
            if (currLine.trim().length() > 0) {
                String[] split = currLine.split(";");
                Z = (Integer.parseInt(split[0]));
                C = (split[1]);
                City2 d = new City2(Z, C);
                City.add(d);
                d.print();
            }
        }

    } catch (IOException e) {
        System.out.println("Erorr : " + e);
    }

}