Переопределение equals

@holod2014 · Регистрация: 01.03.2014

Author24 — интернет-сервис помощи студентам

Добрый день, товарищи.

Пытаюсь переопределить метод "equals" во вновь созданном классе.

Java

package i1;
import java.util.Objects;
 
// Класс автомобиль
public class Car {
//Поле "макра автомобиля"   
private String Marka = "Марка авто не задана";      
private String Motion = "Автомобиль не двигается"; 
private String Fuel = "Топливо на исходе";
 
//Перезагрузка конструктора
Car(String name,String name1, String name2) 
    {                       
     this.Marka = name;
     this.Motion = name1;
     this.Fuel = name2;
    }
 
....
    //Переопределение метода equals()
    @Override
    public boolean equals(Object obj) 
        {
        
        if (obj == null) 
            {
            return false;
            }
        
        if (getClass() != obj.getClass()) 
        {
            return false;
        }
        
        final Car other = (Car) obj;
        if (!Objects.equals(this.Marka, other.Marka)) 
            {
            return false;
            }      
        return true;       
        }
}

Если переопределять таким образом, то не учитываются поля:
private String Motion и private String Fuel.

Как можно исправить метод equals()?

@Эрнесто · 14.01.2016, 15:31

holod2014, кто мешает повтыкать еще условий?

Java

if (!Objects.equals(this.Motion, other.Motion)) 
            {
            return false;
            }

и тп.

@artemvl · 14.01.2016, 15:37

Java

@Override
    public boolean equals(Object obj) {
        if (this == obj) return true;
        if (obj == null || getClass() != obj.getClass()) return false;
 
        Car car = (Car) obj;
 
        if (Marka != null ? !Marka.equals(car.Marka) : car.Marka != null) return false;
        if (Motion != null ? !Motion.equals(car.Motion) : car.Motion != null) return false;
        return !(Fuel != null ? !Fuel.equals(car.Fuel) : car.Fuel != null);
    }

Pablito · 14.01.2016, 15:42

Не по теме:

ну тогда уже первой проверкой на null, а остальные !=null убрать ) логично?

@holod2014 · 14.01.2016, 16:26 **[ТС]**

Эрнесто, когда добавляю условия, то

Java

1
2
3

       Car cr = new Car("Reno", "Автомобиль стоит", "Топливо на исходе");                                    
       Car cr1 = new Car("Reno", "Автомобиль стоит", "Топливо на исходе"); 
       System.out.println(cr.equals(cr1));

FALSE

@easybudda · 14.01.2016, 17:16

Java

class Car {
    private String trademark;
    private String fueltype;
    
    public Car(String _trademark, String _fueltype) {
        trademark = _trademark;
        fueltype = _fueltype;
    }
    
    @Override
    public boolean equals(Object another) {
        if ( another instanceof Car ) 
            return ( trademark.equals(((Car)another).trademark) && fueltype.equals(((Car)another).fueltype) );
        else
            return super.equals(another);
    }
}
 
public class TwoCars {
    
    public static void main (String args[]) {
        Car first = new Car("Porshe", "Kerosene");
        Car second = new Car("Porshe", "Kerosene");
        
        System.out.println(first.equals(second));
        System.out.println(second.equals(null));
    }
}

@holod2014 · 14.01.2016, 18:01 **[ТС]**

easybudda, Толь лыжи не едут, толь я не догоняю

Java

package i1;
 
public class Main 
{
   public static void main(String[] args) 
   {
        
       //Создание объекта класса автомобиль
       Car cr = new Car("Reno", "Автомобиль стоит", "Топливо на исходе");                                    
       Car cr1 = new Car("Reno", "Автомобиль стоит", "Топливо на исходе"); 
              
        System.out.println(cr.equals(cr1));
   }
}
 
package i1;
import java.util.Objects;
 
// Класс автомобиль
public class Car {
    
//Поле "макра автомобиля"   
private String Marka = "Марка авто не задана";      
private String Motion = "Автомобиль не двигается"; 
private String Fuel = "Топливо на исходе";
 
// Создание объекта класса Wheel(колесо)
public Wheel wl = new Wheel(); 
public Wheel wl1 = new Wheel(); 
 
//Перезагрузка конструктора
Car(String name,String name1, String name2) 
    {                       
     this.Marka = name;
     this.Motion = name1;
     this.Fuel = name2;
    }
 
//Задать марку автомобиля
public void setCname(String name)
    {      
      Marka = name;
    }
 
//Считать марку автомобиля
public String getCname()
    {               
     return Marka;
    }
 
    //Переопределение метода equals()
//    @Override
//    public boolean equals(Object obj) 
//        {
//        
//        if (obj == null) 
//            {
//            return false;
//            }
//        
//        if (getClass() != obj.getClass()) 
//        {
//            return false;
//        }
//        
//        final Car other = (Car) obj;
//        if (!Objects.equals(this.Marka, other.Marka)) 
//            {
//            return false;
//            }
//      
//        return true;       
//        }
 
 
        @Override
        public boolean equals(Object another) {
        if ( another instanceof Car ) 
            return ( Marka.equals(((Car)another).Marka) && Motion.equals(((Car)another).Motion) );
        else
            return super.equals(another);
    }
 
  
    //Переопределение метода hashCode()
    @Override
    public int hashCode() 
    {
        int hash=7;
        hash = 89 * hash + Objects.hashCode(this.Marka);
        hash = 89 * hash + Objects.hashCode(this.Motion);
        hash = 89 * hash + Objects.hashCode(this.Fuel);
        return hash;
    }
 
    //Переопределение метода toString()
    @Override
    public String toString() 
    {
        return "Car{" + "Marka=" + Marka+" " + "Ехать=" + Motion + " " + "Заправлять=" + Fuel +  '}';
    } 
}

Добавлено через 32 минуты
могу сбросить весь проект, если что-то в коде не ясно

@easybudda · 14.01.2016, 18:13

Сообщение от holod2014

могу сбросить весь проект, если что-то в коде не ясно

Начните с полного описания задачи - так лучше будет...

@reisal78 · 14.01.2016, 18:18

holod2014, а что вам не нравится?

easybudda, а если так?

Java

 public static void main (String args[]) {
        Car first = new Car("Porshe", null);
        Car second = new Car("Porshe", null);
        
        System.out.println(first.equals(second));
    }

@easybudda · 14.01.2016, 18:26

reisal78, а нефига поля класса неинициализированными оставлять!

Сообщение от holod2014

Java

1
2
3

private String Marka = "Марка авто не задана"; 
private String Motion = "Автомобиль не двигается"; 
private String Fuel = "Топливо на исходе";

есть же значения по умолчанию. По хорошему в конструкторе проверять null там или строка какая-то...

@holod2014 · 14.01.2016, 18:28 **[ТС]**

Задание:
Создать приложение, удовлетворяющее требованиям, приведенным ниже. Корректно переопределить для каждого класса методы equals(), hashCode(), toString().
Создать объект класса Автомобиль, используя класс Колесо.
Методы: ехать, заправляться, менять колесо, вывести на консоль марку автомобиля.
Реализация:

Java

package i1;
 
public class Main 
{
    
   public static void main(String[] args) 
   {
        
       //Создание объекта класса автомобиль
       Car cr = new Car("Reno", "Автомобиль стоит", "Топливо на исходе");                                    
       Car cr1 = new Car("Reno", "Автомобиль стоит", "Топливо на исходе"); 
       
       //Вывод марки автомобиля
       System.out.println("Марка автомобиля: "+cr.getCname());
       
       System.out.println(cr.equals(cr1));
   }
}
package i1;
import java.util.Objects;
// Класс автомобиль
public class Car {
    
//Поле "макра автомобиля"   
private String Marka = "Марка авто не задана";      
private String Motion = "Автомобиль не двигается"; 
private String Fuel = "Топливо на исходе";
// Создание объекта класса Wheel(колесо)
public Wheel wl = new Wheel(); 
 
//Перезагрузка конструктора
Car(String name,String name1, String name2) 
    {                       
     this.Marka = name;
     this.Motion = name1;
     this.Fuel = name2;
    }
 
//Задать марку автомобиля
public void setCname(String name)
    {      
      Marka = name;
    }
 
//Считать марку автомобиля
public String getCname()
    {               
     return Marka;
    }
    
// Метод "Ехать"
public String go()   
    {
        return Car.this.Motion="Автомобиль едет!";
    }
 
// Метод "Стоп"
public String stop()   
    {
        return Car.this.Motion="Автомобиль стоит!";
    }
 
// Метод "Заправлять"
public String fill()   
    {
        return Car.this.Fuel="Автомобиль заправлен!";
    }
 
// Метод "Заправлять"
public String empty()   
    {
        return Car.this.Fuel="Бак автомобиля пуст!";
    }
 
    //Переопределение метода equals()
//    @Override
//    public boolean equals(Object obj) 
//        {
//        
//        if (obj == null) 
//            {
//            return false;
//            }
//        
//        if (getClass() != obj.getClass()) 
//        {
//            return false;
//        }
//        
//        final Car other = (Car) obj;
//        if (!Objects.equals(this.Marka, other.Marka)) 
//            {
//            return false;
//            }
//      
//        return true;       
//        }
 
 
        @Override
        public boolean equals(Object another) {
        if ( another instanceof Car ) 
            return ( Marka.equals(((Car)another).Marka) && Motion.equals(((Car)another).Motion) );
        else
            return super.equals(another);
    }
 
  
//    //Переопределение метода hashCode()
//    @Override
//    public int hashCode() 
//    {
//        int hash=7;
//        hash = 89 * hash + Objects.hashCode(this.Marka);
//        hash = 89 * hash + Objects.hashCode(this.Motion);
//        hash = 89 * hash + Objects.hashCode(this.Fuel);
//        return hash;
//    }
 
    //Переопределение метода toString()
    @Override
    public String toString() 
    {
        return "Car{" + "Marka=" + Marka+" " + "Ехать=" + Motion + " " + "Заправлять=" + Fuel +  '}';
    } 
}
package i1;
import java.util.Objects;
// Класс колесо
public class Wheel {         
    
    // Поле
    public String  Status;                   
    
    // Метод "Менять колесо"
    public String zamena()   
    {
        return Wheel.this.Status="Произведена замена колеса!";
    }
 
    //Переопределение метода equals()
    @Override
    public boolean equals(Object obj) {
        if (obj == null) {
            return false;
        }
        if (getClass() != obj.getClass()) {
            return false;
        }
        final Wheel other = (Wheel) obj;
       
        if (!Objects.equals(this.Status, other.Status)) {
            return false;
        }
        return true;
    }
   
    //Переопределение метода hashCode()
    @Override
    public int hashCode() {
        int hash = 7;
        hash = 71 * hash + Objects.hashCode(this.Status);
        return hash;
    }
 
    //Переопределение метода toString()
    @Override
    public String toString() {
        return "Wheel{" + "Status=" + Status + '}';
    }
 
}

@reisal78 · 14.01.2016, 18:34

easybudda,
А почему так не хотите сделать?

Java

1	return (Objects.equals(trademark, ((Car) another).trademark) && Objects.equals(fueltype, ((Car) another).fueltype));

Добавлено через 1 минуту
holod2014, код который вы указали в сообщении Переопределение equals выводит true. Что еще не так то? )

@holod2014 · 14.01.2016, 18:39 **[ТС]**

Народ, не пинайте сильно. Что-то глюкануло меня. Код действительно рабочий!
Всем спасибо за помощь.

@reisal78 · 14.01.2016, 18:44

Сообщение от holod2014

Код действительно рабочий!

не совсем, да и в целом задание, на мой взгляд, выполнено не верно

@holod2014 · 14.01.2016, 19:02 **[ТС]**

а где ошибки?

@reisal78 · 14.01.2016, 22:16

holod2014, по методу equals - посмотрите наши сообщения с easybudda. В данном случае, ваш код работает, но в общих случаях могут вываливаться эксцепшены.

Ну и в целом это задача по ООП, а в вашем решении его нет =)

Из wiki: Основные преимущества ООП проявляются только в том случае, когда в языке программирования реализован полиморфизм подтипов — возможность единообразно обрабатывать объекты с различной реализацией при условии наличия общего интерфейса.

Теперь по русски:
Сегодня нас просят сделать автомобиль с колесами.

Создать объект класса Автомобиль, используя класс Колесо.

А завтра скажут - а давайте поставим его на гусеницы(оО).

Кликните здесь для просмотра всего текста

Если правильно использовать приемы ООП, то такая модернизация, не доставит нам особых проблем.

Колеса, гусеницы, ноги... Это часть чего то, участвующая в процессе перемещения. Что бы это процесс перемещения начался эта часть должна начать выполняться какое то действие (action).

Java

1
2
3

public interface Actionable {
    void action();
}

Теперь можно создать класс колесо реализующий этот интерфейс

Java

public class Wheel implements Actionable {
 
    private String name;
    private int radius;
 
    public Wheel(String name, int radius) {
        this.name = name;
        this.radius = radius;
    }
 
 
    @Override
    public void action() {
        System.out.println("Колесо вращается");
    }
}

Отложим, на время наши колеса, и займемся автомобилями.

Сейчас от нас требуют создать автомобиль, завтра потребуют мотоцикл, а после завтра - трактор?
Опять попробуем выделить общее.

любое транспортное средство имеет название, вес, цвет и тд.
любое транспортное средство должно иметь как минимум одно устройство для перемещения (колесо, гусеница и тд.)
для перемещения должен быть двигатель
для работы двигателя нужна энергия
энергию надо где то хранить
(для последних 3х пунктов вспомнить о велосипедистах)

так как в задаче ничего не сказано о двигателе, а мне лень особо сильно заморачиваться, то оставим его реализацию на самостоятельную работу.
А вот для реализации метода "заправить" придется сделать хотя бы интерфейс хранилища топлива.
Любое хранилище можно заполнить. и получить из него, то что в нем хранится. (Топливный бак, аккумулятор и тд)

Java

public interface Filled {
    void fill();
    double getFuel();
}

реализуем абстрактный класс транспорт

Java

public abstract class Transport {
    protected Actionable[] moveParts; 
    private Filled storage;
    
    protected void refill() {
        if (storage == null) {
            throw new RuntimeException("Отсутствует хранилище");
        }
        storage.fill();
    }
 
    public void addMoveParts(Actionable... part) {
        if (moveParts == null) {
            throw new RuntimeException("moveParts не должно быть null");
        }
        if (part.length != moveParts.length) {
            throw new RuntimeException("Неверное количество частей для движения");
        }
        for (int i = 0; i < moveParts.length; i++) {
            moveParts[i] = part[i];
        }
    }
 
    protected void changeMovePart(int numberPart, Actionable part) {
        moveParts[numberPart] = part;
    }
    
    protected void drive(){
        for (Actionable part : moveParts) {
            if (part == null) {
                throw new RuntimeException("Отсутсвуют части необходимые для движения");
            }
        }
        if (storage == null) {
            throw new RuntimeException("Отсутствует хранилище");
        }
        if (storage.getFuel() <= 0) {
            System.out.println("Закончилось топливо");
        }
        for (Actionable part: moveParts) {
            part.action();
        }
    }
 
    public Actionable[] getMoveParts() {
        return moveParts;
    }
 
    public void setMoveParts(Actionable[] moveParts) {
        this.moveParts = moveParts;
    }
 
    public Filled getStorage() {
        return storage;
    }
 
    public void setStorage(Filled storage) {
        this.storage = storage;
    }
}

класс Car теперь совсем не большой

Java

public class Car extends Transport {
    private String brand;
 
    public Car(String brand) {
        this.brand = brand;
        super.moveParts = new Actionable[4];
    }
 
    @Override
    public void drive() {
        super.drive();
        System.out.println("Автомобиль начал движение");
    }
 
    public void showBrand() {
        System.out.println(brand);
    }
}

теперь напишем основной метод
public static void main(String[] args) {

Java

        Car car = new Car("Lada");
        Wheel wheel1 = new Wheel("Снежинка", 14);
        Wheel wheel2 = new Wheel("Снежинка", 14);
        Wheel wheel3 = new Wheel("Снежинка", 14);
        Wheel wheel4 = new Wheel("Снежинка", 14);
 
        car.addMoveParts(wheel1, wheel2, wheel3, wheel4);
        car.showBrand();
        car.changeMovePart(1, new Wheel("Кама", 14));
    }

методы "поехать" и "заправиться" будут бросаться исключениями, т.к у нас еще не создан класс бензобак.

Java

public class FuelTank implements Filled {
 
    private final double MAX_VALUE = 40;
    private double currentValue = 0;
    
    @Override
    public void fill() {
        currentValue = MAX_VALUE;
    }
 
    @Override
    public double getFuel() {
        return currentValue--;
    }
}

Теперь все работает.

Java

    public static void main(String[] args) {
        Car car = new Car("Lada");
        Wheel wheel1 = new Wheel("Снежинка", 14);
        Wheel wheel2 = new Wheel("Снежинка", 14);
        Wheel wheel3 = new Wheel("Снежинка", 14);
        Wheel wheel4 = new Wheel("Снежинка", 14);
 
        car.addMoveParts(wheel1, wheel2, wheel3, wheel4);
        car.showBrand();
        car.changeMovePart(1, new Wheel("Кама", 14));
 
        car.setStorage(new FuelTank());
 
        car.refill();
        car.drive();
    }

Теперь поставим наш автомобиль на гусеницы.
нам нужно будет создать класс их описывающий.

Java

public class Сaterpillar implements Actionable{
    @Override
    public void action() {
        System.out.println("Гусеница пришла в движение");
    }
}

и установим их в наш автомобиль

Java

    public static void main(String[] args) {
        Car car = new Car("Lada");
        Wheel wheel1 = new Wheel("Снежинка", 14);
        Wheel wheel2 = new Wheel("Снежинка", 14);
        Wheel wheel3 = new Wheel("Снежинка", 14);
        Wheel wheel4 = new Wheel("Снежинка", 14);
 
        car.addMoveParts(wheel1, wheel2, wheel3, wheel4);
        car.showBrand();
 
        Caterpillar caterpillar1 = new Caterpillar();
        Caterpillar caterpillar2 = new Caterpillar();
        Caterpillar caterpillar3 = new Caterpillar();
        Caterpillar caterpillar4 = new Caterpillar();
 
        car.changeMovePart(0, caterpillar1);
        car.changeMovePart(1, caterpillar2);
        car.changeMovePart(2, caterpillar3);
        car.changeMovePart(3, caterpillar4);
 
        car.setStorage(new FuelTank());
 
        car.refill();
        car.drive();
    }

В уже написанный код мы не вносили правок (не наделали новых ошибок), просто добавили новый класс.

Теперь потребовалось создать мотоцикл.

Java

public class Motorcycle extends Transport {
 
    public Motorcycle() {
        super.moveParts = new Actionable[2];
    }
 
    @Override
    public void refill() {
        super.refill();
        System.out.println("Мото заправлен");
    }
 
    @Override
    public void addMoveParts(Actionable... part) {
        super.addMoveParts(part);
    }
 
    @Override
    public void changeMovePart(int numberPart, Actionable part) {
        super.changeMovePart(numberPart, part);
    }
 
    @Override
    public void drive() {
        super.drive();
        System.out.println("Мотоцикл поехал");
    }
}

И проверим

Java

    public static void main(String[] args) {
        Car car = new Car("Lada");
        Wheel wheel1 = new Wheel("Снежинка", 14);
        Wheel wheel2 = new Wheel("Снежинка", 14);
        Wheel wheel3 = new Wheel("Снежинка", 14);
        Wheel wheel4 = new Wheel("Снежинка", 14);
 
        car.addMoveParts(wheel1, wheel2, wheel3, wheel4);
        car.showBrand();
 
        Caterpillar caterpillar1 = new Caterpillar();
        Caterpillar caterpillar2 = new Caterpillar();
        Caterpillar caterpillar3 = new Caterpillar();
        Caterpillar caterpillar4 = new Caterpillar();
 
        car.changeMovePart(0, caterpillar1);
        car.changeMovePart(1, caterpillar2);
        car.changeMovePart(2, caterpillar3);
        car.changeMovePart(3, caterpillar4);
 
        car.setStorage(new FuelTank());
 
        car.refill();
        car.drive();
 
        Motorcycle motorcycle = new Motorcycle();
        motorcycle.addMoveParts(wheel1, wheel2);
        motorcycle.setStorage(new FuelTank());
        motorcycle.refill();
        motorcycle.drive();
    }

Колеса поставили с автомобиля.. не пропадать же добру.
Тоже самое можно делать и с тракторами и с велосипедами.. да хоть с чертом в ступе =)
ну а дальше сами... мне лень стало )

@holod2014 · 15.01.2016, 11:43 **[ТС]**

reisal78, прочитав данный пост действительно стало всё понятно.
Побольше бы таких статей в интернете, которые написаны не на академическом языке, а на понятном широкой массе.
Спасибо!!!

Pablito
	14.01.2016, 15:42 #4
	Не по теме: ну тогда уже первой проверкой на null, а остальные !=null убрать ) логично? 0

@easybudda Модератор 12458 / 7482 / 1753 Регистрация: 25.07.2009 Сообщений: 13,762
	14.01.2016, 18:13	8
	Сообщение от holod2014 могу сбросить весь проект, если что-то в коде не ясно Начните с полного описания задачи - так лучше будет... 1

@holod2014 30 / 11 / 5 Регистрация: 01.03.2014 Сообщений: 379
	14.01.2016, 18:39 [ТС]	13
	Народ, не пинайте сильно. Что-то глюкануло меня. Код действительно рабочий! Всем спасибо за помощь. 0

@reisal78 944 / 687 / 230 Регистрация: 28.04.2013 Сообщений: 1,926
	14.01.2016, 18:44	14
	Сообщение от holod2014 Код действительно рабочий! не совсем, да и в целом задание, на мой взгляд, выполнено не верно 0

@holod2014 30 / 11 / 5 Регистрация: 01.03.2014 Сообщений: 379
	14.01.2016, 19:02 [ТС]	15
	а где ошибки? 0

@holod2014 30 / 11 / 5 Регистрация: 01.03.2014 Сообщений: 379
	15.01.2016, 11:43 [ТС]	17
	reisal78, прочитав данный пост действительно стало всё понятно. Побольше бы таких статей в интернете, которые написаны не на академическом языке, а на понятном широкой массе. Спасибо!!! 0

Переопределение equals

Решение