java - 如何限制对对象的字段 setter 的访问？通过继承实现这一目标将是一个错误的选择吗？

Question

Closed. This question is opinion-based。它当前不接受答案。
                            
                        
                    
                
                            
                                
                
                        
                            
                        
                    
                        
                            想改善这个问题吗？更新问题，以便 editing this post用事实和引用来回答。
                        
                        6个月前关闭。
                                                                                            
                
        
我有一些由基础服务（例如数据存储库）创建的业务数据对象。这些对象是逐步创建的。即（第一个步骤仅从google找到图像对象，第二个步骤获取有关该图像的元数据，第三个步骤将其下载到磁盘）。

这些相同的对象将在构建时在UI中呈现。因此，UI无需了解有关设置员的任何信息，也不应访问它。

但是创建实例的服务需要访问设置程序。请注意，此模型将在其他软件包中使用，因此将setters软件包设为私有不是解决方案。

这是代码方面的要求：
图片模型：

class Image {
 private String src;
 public String getSrc(){
  return field;
 }
 public void setSrc(String src){
  return src;
 }
}

服务模式：

interface IService {
 Image get(...);
 Image processMore(Image i);
}

用户界面/客户端代码：

Image i = service.get(params);
render(i);
i = service.processMore(i);
render(i);
i.setSrc(null); //This is the problem. Client should not have access to this method.

可能的解决方案：
解决方案1：接口

public interface IImage{
 String getSrc();
}
interface IService {
     IImage get(...);
     IImage processMore(Image i);
    }
IImage i = service.get(params);
render(i);
i = service.processMore(i);//Casting IImage to Image - this cast can easily fail and no way to know IImage is always an Image
render(i);

缺点：我评论过的投放问题。对于每个对象，还必须编写许多样板代码。当扩展图像模型时，这将变得非常复杂。

解决方案2：构建器模式
再次有很多样板代码。但是严格来说，Builder模式不适合此处，因为该对象是分步构建的，我们必须在builder类上调用getSrc（），这可能违反了书中的内容。

解决方案3：继承

public abstract class Image {
 protected String src;

 protected Image() {} //Ensure this class instance is never created
 public String getSrc(){
  return field;
 }
 protected void setSrc(String src){
  return src;
 }

 public static class Buildable extends Image {
  protected String src;

  public Buildable() {
   super();
  }

  @Override
  public void setSrc(String src){
   return src;
  }
 }
}

interface IService {
         Image get(...);
         Image processMore(Image.Buildable i);
        }
    Image i = service.get(params);
    render(i);
    i = service.processMore(i);//Casting Image to Image.Buildable - as Image constructor is protected and Image.Builder is public, only way to create Image is creating Image.Buildable and hence this cast should never fail.
    render(i);

因此，作为正在学习SOLID原理的初学者，我的问题是，


此继承模型是否违反SOLID（或其他干净代码准则）？如果是这样，应该怎么做？

最佳答案

一种反对这种方法的论点是，继承需要保留子类型关系的含义：


  继承仅在子类确实是超类的子类型的情况下才适用。换句话说，仅当两个类之间存在“ is-a”关系时，类B才应扩展类A。 -有效的Java，第3版，第18项


就子类型关系而言，您必须有一个MutableObject is -a ImmutableObject，这是一个矛盾。

实际上，使类不可变的第一步就是精确地声明它final以避免通过子类违反约束。因为在这种情况下看起来像功能的东西很容易在其他情况下变成错误。有关其他讨论，请参见this post。

我建议的解决方案是利用接口隔离原理并定义一个不包含任何setter的接口，并在UI代码中通过该接口引用对象：

class MutableObject implements MutableObjectView { ... }

interface MutableObjectView { /* only non-state-changing methods */ }

关于java - 如何限制对对象的字段 setter 的访问？通过继承实现这一目标将是一个错误的选择吗？ ，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/56633815/

上一篇：java - 启动多个线程并收集结果

下一篇：Java - 有没有一种方法可以在不迭代的情况下查询 Apache Spark 模式？

相关文章：

java - 非法参数异常

java - Findbugs java 中可能的空指针取消引用

c#类同时实现和继承

c# - 用模板覆盖基类的抽象类

java - 通过将多个TableModel提取到单个TableModelManager中来改进设计

java - 静态变量、libgdx 和 android 暂停/恢复内存使用

Java - 没有图形的 FontMetrics

c++ - 抽象基类调用父类的纯虚函数

java - 工厂设计模式和Diamond OOP问题

python - 相关基类的设计模式

Answer 1

一种反对这种方法的论点是，继承需要保留子类型关系的含义：


  继承仅在子类确实是超类的子类型的情况下才适用。换句话说，仅当两个类之间存在“ is-a”关系时，类B才应扩展类A。 -有效的Java，第3版，第18项


就子类型关系而言，您必须有一个MutableObject is -a ImmutableObject，这是一个矛盾。

实际上，使类不可变的第一步就是精确地声明它final以避免通过子类违反约束。因为在这种情况下看起来像功能的东西很容易在其他情况下变成错误。有关其他讨论，请参见this post。

我建议的解决方案是利用接口隔离原理并定义一个不包含任何setter的接口，并在UI代码中通过该接口引用对象：

class MutableObject implements MutableObjectView { ... }

interface MutableObjectView { /* only non-state-changing methods */ }