Проблема с инициализацией объекта (конструктор), что я делаю не так?

Question

[Juris]

Доброго времени суток!
Недавно начал изучать Java и споткнулся на конструкторах.

package com.company.personidentity;

public class Main {
    public static void main(String[] args) {
       Person person1 = new Person("Harry Potter", 40, "Baker street 20","Manchester","United Kingdom");
    }
}

Intellij IDEA уже подсказывает, что имеется несоответствие типов, у меня имеется тип Address, а я вроде как подсовываю String. Как эту проблему решить? Хочу, чтобы адрес был отдельным классом

package com.company.personidentity;

public class Person {
    private String name;
    private int age;
    private Address address;
    private Address city;
    private Address country;

    public Person(String name, int age, Address address, Address city, Address country) {
        this.name = name;
        this.age = age;
        this.address = address;
        this.city = city;
        this.country = country;
    }

    @Override
    public String toString() {
        return "Person{" +
                "name='" + name + '\'' +
                ", age=" + age +
                ", address=" + address +
                ", city=" + city +
                ", country=" + country +
                '}';
    }

    public String getName() {
        return name;
    }

    public void setName(String name) {
        this.name = name;
    }

    public int getAge() {
        return age;
    }

    public void setAge(int age) {
        this.age = age;
    }

    public Address getAddress() {
        return address;
    }

    public void setAddress(Address address) {
        this.address = address;
    }

    public Address getCity() {
        return city;
    }

    public void setCity(Address city) {
        this.city = city;
    }

    public Address getCountry() {
        return country;
    }

    public void setCountry(Address country) {
        this.country = country;
    }
}

package com.company.personidentity;

public class Person {
    private String name;
    private int age;
    private Address address;
    private Address city;
    private Address country;

    public Person(String name, int age, Address address, Address city, Address country) {
        this.name = name;
        this.age = age;
        this.address = address;
        this.city = city;
        this.country = country;
    }

    @Override
    public String toString() {
        return "Person{" +
                "name='" + name + '\'' +
                ", age=" + age +
                ", address=" + address +
                ", city=" + city +
                ", country=" + country +
                '}';
    }

    public String getName() {
        return name;
    }

    public void setName(String name) {
        this.name = name;
    }

    public int getAge() {
        return age;
    }

    public void setAge(int age) {
        this.age = age;
    }

    public Address getAddress() {
        return address;
    }

    public void setAddress(Address address) {
        this.address = address;
    }

    public Address getCity() {
        return city;
    }

    public void setCity(Address city) {
        this.city = city;
    }

    public Address getCountry() {
        return country;
    }

    public void setCountry(Address country) {
        this.country = country;
    }
}

Answer 1

[Алексей Уколов]

вроде как подсовываю String. Как эту проблему решить? Хочу, чтобы адрес был отдельным классом

Ну так сделайте отдельный класс и передавайте его, в чём проблема-то?

new Person("Harry Potter", 40, new Address("Baker street 20"), new Address("Manchester"), new Address("United Kingdom"));

Другой вопрос - почему у вас улица, город и страна одного типа...

Answer 2

[Sergey Shvyrev]

Тип String передаете, как тип Address

"Baker street 20" instanceOf Address == false

Переделайте :

public Person(String name, int age, Address address) {
...
}

new Person("Harry Potter", 40, new Address("Baker street 20","Manchester","United Kingdom"))

class Address {
private String address;
    private String city;
    private String country;
public Address(String address, String city, String country){
...
}
}

* P.S. От бойлерплейта глаза болят. Read JEP 269.

Comments

[Juris]

то что я передаю тип Address, это мне понятно, но как мне связать классы Person и Address без наследования?

что-то я туплю

[Sergey Shvyrev]

У меня голова болит от таких вопросов. Все же просто :

Person

Address

App

[Sergey Shvyrev]

А да... вы же любите бойлерплейт.

Юзайте!

public class Person {
    public String name;
    public int age;
    public Address address;

    public static Person of(String name, int age, Address address){
        final Person person = new Person();
        person.name = name;
        person.age = age;
        person.address = address;
        return person;
    }

    @Override
    public String toString() {
        return "Person{" +
                "name='" + name + '\'' +
                ", age=" + age +
                ", address=" + address +
                '}';
    }
}

public class Address {
    public String street;
    public String city;
    public String country;

    public static Address of(String street, String city, String country){
        final Address address = new Address();
        address.country = country;
        address.city = city;
        address.street = street;
        return address;
    }

    @Override
    public String toString() {
        return "Address{" +
                "street='" + street + '\'' +
                ", city='" + city + '\'' +
                ", country='" + country + '\'' +
                '}';
    }
}

public static void main(String[] args) {
        System.out.println(Person.of("Harry Potter", 40, Address.of("Baker street 20", "Manchester", "United Kingdom")));
    }

[Juris]

Sergey Shvyrev, спасибо, пусть для Вас это и тривиальная задача, для меня не так всё было очевидно. Извините за моё пёстрое полотно ;)