Проблема со сравнением готовых объектов методом равных - PullRequest
Новая
       20

Проблема со сравнением готовых объектов методом равных

0 голосов
/ 26 апреля 2020

Я пытаюсь создать программу, которая сравнивает архив объектов с архивами Arraylist и только добавляет его в Arraylist, если его там нет, но я действительно изо всех сил. Я попытался изменить для l oop добавление нового логического значения, чтобы проверить, существует ли уже идентичный архив, но просто не могу заставить его добавлять только уникальные архивы. Любой совет будет принят с благодарностью. 

import java.util.ArrayList;
import java.util.Scanner;

public class Main {

    public static void main(String[] args) {
        Scanner scanner = new Scanner(System.in);
        ArrayList<Archive> archives = new ArrayList<>();
        while (true) {
            System.out.println("Identifier? (empty will stop)");
            String identifier = scanner.nextLine();
            if (identifier.isEmpty()) {
                break;
            }
            System.out.println("Name? (empty will stop)");
            String name = scanner.nextLine();
            if (name.isEmpty()) {
                break;
            }
            Archive archive = new Archive(identifier, name);

            boolean contains = false;
            for (int i = 0; i < archives.size(); i++) {
                if (archive.equals(archives.get(i))) {
                    contains = true;
                    break;
                }
            }
            if (contains == false) {
                archives.add(archive);
            }
        }

        for (Archive i : archives) {
            System.out.println(i.getIdentifier() + ": " + i.getName());
        }
    }
}



public class Archive {

    private String identifier;
    private String name;

    public Archive(String identifier, String name) {
        this.identifier = identifier;
        this.name = name;
    }

    public String getIdentifier() {
        return this.identifier;
    }

    public String getName() {
        return this.name;
    }

    public boolean equals(Archive archive, Object compared) {

        if (compared == archive) {
            return true;
        }
        if (!(compared instanceof Archive)) {
            return false;
        }

        Archive compare = (Archive) compared;
        if (compare.getIdentifier().equals(archive.getIdentifier()) && compare.getName().equals(archive.getName())) {
            return true;
        }

        return false;
    }

}

// Также просто понял, что вместо целого для l oop с логическим значением содержится то, что я могу просто сделать 

if(archives.contains(archive){
contains=true;}

1 Ответ

3 голосов
/ 26 апреля 2020

Подпись метода равно неверна. Это должно быть 

public boolean equals(Object compared) {
    if(compared==null || !(compared instanceof Archive)){
        return false;
    }

    Archive compare = (Archive) compared;
    if (compare.getIdentifier().equals(this.getIdentifier()) && compare.getName().equals(this.getName())) {
        return true;
    }

    return false;
}
...