这是否被认为是低耦合和高内聚？有改善的机会吗？

Question

我正在尝试掌握 Robert C. Martin 的 SOLID 原则。目前我正在研究低耦合和高内聚。我创建了一些代码来代表我目前对这个主题的理解。你们能告诉我是否在正确的轨道上吗？有没有机会改进当前的设计？

创建两个地址并将其分配给员工的主应用程序：

public class App {

    public static void main(String[] args) {
        Address homeAddress = new HomeAddress("This is my Home Address");
        Address workAddress = new WorkAddress("This is my Work Address");        
        Employee employee = new Employee(homeAddress, workAddress);

        employee.getAddresses();
    }

}

员工类：

public class Employee {

    private Address homeAddress;
    private Address workAddress;

    Employee(Address homeAddress, Address workAddress) {
        this.homeAddress = homeAddress;
        this.workAddress = workAddress;
    }

    public void getAddresses() {
        System.out.println("homeAddress: " + homeAddress.getAddress());
        System.out.println("workAddress: " + workAddress.getAddress());
    }

}

地址接口：

public interface Address {

    String getAddress();

}

具体地址实现1（HomeAddress）：

public class HomeAddress implements Address {

    String specificAddress;

    public HomeAddress(String specificAddress) {
        this.specificAddress = specificAddress;
        System.out.println("In HomeAddress Constructor");
    }

    public String getAddress() {
        return specificAddress;
    }
}

具体地址实现2(WorkAddress)：

public class WorkAddress implements Address {

    String specificAddress;

    public WorkAddress(String specificAddress) {
        this.specificAddress = specificAddress;
        System.out.println("In WorkAddress Constructor");
    }

    public String getAddress() {
        return this.specificAddress;
    }
}

任何帮助/反馈将不胜感激！提前致谢。

马克。

Answer 1

这是一个小例子，但在耦合/内聚方面可以改进。

对象是有凝聚力的。为什么？在Employee 对象中，构造函数和getAddresses()（顺便说一下应该称为printAddresses()）都引用了两个实例变量（这意味着它们关注的是同一件事）。 Address 对象也是如此。

在耦合部分，我认为你可以做得更好。就目前而言，Employee 对象“知道”（即耦合到）Address 对象的内部表示。那是因为您从 Address 对象“导出”数据（字符串），而不是在数据所在的位置打印它。

这会使您的对象更加耦合，并会导致 Address 对象中的任何更改（例如引入 Street 和 City 等）泄漏到 Employee。所以它有真正的缺点。

解决方案是在Address 中定义一个print() 方法并在那里执行System.out.println()。这符合其他概念，例如Law of Demeter、Tell Don't Ask。