java - 尝试使其他包装类对复合类方法可见时,避免违反SOLID原则

标签 java design-patterns single-responsibility-principle

我正在为Java考试准备一个大学项目,该项目包括创建路由器网络并相互发送数据包,直到数据包以最短路径到达目的地为止。

该项目需要遵守SOLID原则,实际上我处于停滞状态,这可能是因为这是我第一次使用Java。

我创建了一个代表通用网络节点的NODE类,该节点唯一负责保存与IP相关联的名称,而没有其他,然后创建一个ROUTING类,它的责任是将所有已知的网络路由保存在HashMap<Node,integer>中(整数代表流量负载,它是随机的),包括使用方法显示条目并将其添加到地图。

NODE类将ROUTING类合并为受保护的内部类对象。

现在,这些类存储在ROUTER包中,为了创建网络,我创建了另一个名为NETWORKSTARTER的包,并在其中创建了一个名为STARTER的类,该类的目的是创建网络并连接(将节点添加到路由器表中)其他实体网络彼此之间。

在编写连接方法时,我偶然发现了一个事实,即即使它是NODE类中的受保护类,也无法调用NODE.ROUTING的内部方法。

为了解决这一挫折,我想过要在NODE类中创建三个“包装” ROUTING类方法的方法,基本上用其他名称调用它们,这样STARTER方法就可以调用ROUTING槽NODE方法的方法来添加HashMap的节点。

问题是:这种设计是否违反SOLID原则?特别是NODE类设计是否违反了单一责任原则?

public class NODE{
private String nodeName;
private int nodeIP;
protected ROUTING nodeRoutingModule;

public NODE(String x,int y){
this.nodeName=x;
this.nodeIP=y;
this.nodeRoutingModule = new ROUTING();
}

// setter and getters for name and IP

public void addRoute(NODE node){
    this.nodeRoutingModule.addToMap(node);
}

public void showTable(){
    this.nodeRoutingMdoule.showMap();
}

}


public class ROUTING{
private HashMap<NODE,Integer> routeTable;

public ROUTING(){
    this.routeTable = new HashMap<>();
}

public void addToMap(Node node){
    int load = ThreadLocalRandom.current().nextInt(0,11);
    routeTable.put(node,load);
}

public void showMap(){
    routeTAble.forEach.((t,u) -> {
        system.out.println(t.getIp+" "+t.getName);
    });
}

}



public class STARTER{
private LinkedList<NODE> nodeCollection;

public STARTER(){
    this.nodeCollection = new LinkedList<>();
}

// here I populate the list with NODES

public void connectNodes(){
    // logic with a forEach cycle calling for each node
    // the method to add the node to his routing table with
    // another inner forEach cycle using the NODE class
    // wrapping the ROUTING methods
}

最佳答案

这将取决于您的职责:)

让我们从另一个地方开始。让我们从模型开始。

当您说NODE时,我有点困惑,但您的意思是ROUTER。我知道这确实是网络中使用的术语,但是您仍在创建模型。

我知道这似乎有点愚蠢,但请仔细考虑一下,然后专注于您使用的语言


ROUTER的职责是什么?


VS


NODE的职责是什么?


NODE的职责是什么?好吧,NODE是用于图,树,网络或其他任何部分的元素的通用术语。

现在ROUTER是另一回事。 ROUTER确实是NODE中的NETWORK,并且还将路由/发送数据包到网络中的其他ROUTERS

它还与网络中的其他节点通信(通过发送消息,这是关键),以计算ROUTING TABLE。 Т®计算ROUTING TABLENODES使用PROTOCOLLink stateDistant vector等)

如果您要建立一个现实的网络模型,则应考虑这些因素。

现在到NetworkStarter。您是启动网络还是连接/配置网络?另外,为什么网络会开始保存节点列表?在我看来,它的责任是连接/组装网络。

此外,构建ROUTING TABLE的责任在节点上。

现在到ROUTE的概念。

什么是ROUTE?嗯,这是两个或多个NODES之间的路径。让我们在以下概念中添加另一个概念:网络中邻居之间的CONNECTION(图中的EDGE)。

让我们写下到目前为止的所有概念


Router-职责:发送数据包,建立路由表
RoutingTable-由Router使用,职责:存储路线(或路径)
Connection-从一个路由器到另一个路由器的连接
ConnectionsTable-由Router使用职责:存储邻居
Path-两个或更多节点之间的路径
Node-一个RouterNode
NetworkAssembler-职责:组装网络,连接路由器(节点)
Routing Protocol-路由器用于通信和计算RoutingTable


现在到SOLID:

单一责任:这是一个棘手的责任!

有时,一个组件(可以是一个对象)具有多个责任。在内部,它可能由仅负责一项职责的其他事物或组件组成。

请以您的情况为准。什么是单一责任?在现实世界中,事情不会像在软件中那样起作用。您不能仅由网络负责在现实世界中构建路由表。网络不像路由器那样物理存在,它们由组件组成。

我们已经使用工具来查看职责并将其分配给不同的概念。

打开关闭:

扩展我们的系统意味着什么?


添加模式节点?是的,这是可以做到的,这就是网络设计背后的全部思想:)
更改协议?使用新的Router为要扩展的路由器(并配置要使用的协议)添加功能:提示使用STRATEGY PATTERN实施
重新排序网络?使protocols具有可扩展性(使用“策略模式”或其他功能)。在现实世界中,您会将sys管理员发送到课程中以学习更多内容。这就像注入新的策略模式:)


Liskov替代原理:如果使用策略模式,请确保不要将抽象泄漏太多

接口隔离:为您的概念使用小接口(NetworkAssemblerRouter等)

依赖倒置:使用接口并使它们变小:)

我不会给出任何代码示例,因为人们确实是通过编程来学习如何编程的,因为这是您的任务:)使用此答案中的内容可以得出您的概念,并在代码中捕获它们并做出一个不错的实现。

祝你好运,有乐趣的编码:)

关于java - 尝试使其他包装类对复合类方法可见时,避免违反SOLID原则,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/56763760/

相关文章:

php - 拆分一个大的 PHP 类

c# - 不断地让新的对象正常吗?

java - 如何反转并破坏数组以形成新的最大化数组?

java - 错误 : java: javacTask: source release 8 requires target release 1. 8

android - 多个 Activity/Fragments 和 Model View Presenter 模式

c++11 - 整理 C++ 运算符重载

java - JSP/javabean/servlet MVC

java - 在 iOS 5 中使用 Java 类

java - 从流中获取字符串列表

ruby-on-rails-3 - Rails 模型使用 API 而不是数据库