Исправить тест с помощью метода equals() и переопределить этот и hashCode()

Question

Я пытаюсь проверить мой код, но последний System.out.printl не отвечает, почему?

@Test
public void testcopy(){
    RomanNumber rn1 = new RomanNumber("M");
    RomanNumber rn2 = new RomanNumber("M");
    RomanNumber rn1_copy = rn1;
    System.out.println("rn1 == rn1_copy is "
            + (rn1 == rn1_copy));
    System.out.println("rn1 == rn2 is " + (rn1 == rn2));
    System.out.println("rn1.equals(rn2) is " + (rn1.equals(rn2)));
}

Снова хочу переопределить equals() метод суперкласса ObjectЯ попробовал это:

public boolean equals(Object obj) {
    if ((obj instanceof RomanNumber)) {
        RomanNumber decimal = (RomanNumber) obj;
        if (number == decimal.convertToInteger () && number.equals(decimal.convertToInteger) ) {
            return true;
        } else {
            return false;
        }
    } else {
        return false;
    }
}

Наконец, я публикую ниже весь код, который он делает:

public class RomanNumber {

    private String number;

    public RomanNumber (String number){
        this.number = number;
    }

    /**
     * @return integer equivalent of this roman number
     */
    public int convertToInteger ()  {
        int decimal = 0;
        int lastNumber = 0;
        // take the string roman number
        // write algorithm to convert it to int
        // return int

        number = number.toUpperCase();
        for (int x = number.length() - 1; x >= 0; x--) {
            char convertNumber = number.charAt(x);
            switch (convertNumber) {
                case 'M':
                    decimal = processDecimal(1000, lastNumber, decimal);
                    lastNumber = 1000;
                    break;

                case 'D':
                    decimal = processDecimal(500, lastNumber, decimal);
                    lastNumber = 500;
                    break;

                case 'C':
                    decimal = processDecimal(100, lastNumber, decimal);
                    lastNumber = 100;
                    break;

                case 'L':
                    decimal = processDecimal(50, lastNumber, decimal);
                    lastNumber = 50;
                    break;

                case 'X':
                    decimal = processDecimal(10, lastNumber, decimal);
                    lastNumber = 10;
                    break;

                case 'V':
                    decimal = processDecimal(5, lastNumber, decimal);
                    lastNumber = 5;
                    break;

                case 'I':
                    decimal = processDecimal(1, lastNumber, decimal);
                    lastNumber = 1;
                    break;
            }
        }
        System.out.println(decimal);
        return decimal;
    }

    private static int processDecimal(int decimal, int lastNumber, int lastDecimal) {
        if (lastNumber > decimal) {
            return lastDecimal - decimal;
        } else {
            return lastDecimal + decimal;
        }
    }

    private void validateState(String number){
        if (number == null)
            throw new IllegalArgumentException("Null argument");

        if (number.isEmpty())
            throw new IllegalArgumentException("Empty string");


        if (!number.matches("^M{0,4}(CM|CD|D?C{0,3})(XC|XL|L?X{0,3})(IX|IV|V?I{0,3})$"))
            throw new IllegalArgumentException("Invalid Roman number");
    }

    public boolean equals(Object obj) {
        if ((obj instanceof RomanNumber)) {
            RomanNumber decimal = (RomanNumber) obj;
            if (number == decimal.convertToInteger () && number.equals(decimal.convertToInteger) ) {
                return true;
            } else {
                return false;
            }
        } else {
            return false;
        }
    }
}

Спасибо.

Answer 1

Ваш первый номер

1) Несовместимые типы операндов String и int

number == decimal.convertToInteger ()

Вы не можете сравнивать String и int напрямую

Вы можете исправить вашу проблему, удалив строку

2)number.equals(decimal.convertToInteger())

Здесь номер, который вы передаете как M, Но decimal.convertToInteger() возвращает 1000

Так что это не правильно для сравнения 2 объекта равны

Как исправить?

public boolean equals(Object obj){
    if ((obj instanceof RomanNumber)) {
        RomanNumber decimal = (RomanNumber)obj;
        if (number.equals(decimal.number)) {
            return true;
        }
        else {
            return false;
        }
    }
    else {
        return false;
    }
}