避免使用FOR –反假战役

最新推荐文章于 2024-09-14 19:55:29 发布

dnc8371

最新推荐文章于 2024-09-14 19:55:29 发布

阅读量121

点赞数

文章标签：网络 java python 编程语言设计模式

您是否想知道FOR如何影响您的代码？他们如何限制您的设计，更重要的是如何将您的代码转换为无人为含义的多行代码？

在这篇文章中，我们将看到如何将for的简单示例（由Francesco Cirillio提供- 反if活动）转换为更具可读性和精心设计的示例。

因此，让我们从使用FOR的原始代码开始：

public class Department {

  private List<Resource> resources = new ArrayList<Resource>();

  public void addResource(Resource resource) {
   this.resources.add(resource);
  }

  public void printSlips() {

   for (Resource resource : resources) { 
    if(resource.lastContract().deadline().after(new Date())) { 

     System.out.println(resource.name()); 
     System.out.println(resource.salary());
    }
   }
  }

 }

请参见printSlips方法。如此简单的方法，只有10条线计数白线，但违反了最重要的规则之一，该方法在其内部混合了不同级别的抽象而不只是做一件事。

正如罗伯特·C·马丁（Robert C. Martin）在他的著作《函数应该做的一件事》中指出的那样。 他们应该做得很好。 他们应该只做[…]。 如果一个函数仅执行比该函数的指定名称低一级的步骤，则该函数正在做一件事[…]。 。

因此，根据给定的方法外观定义，让我们回顾一下以前的方法，看看有多少事情在做
printSlips方法？具体四个。

public void printSlips() {

   for (Resource resource : resources) { #Cycle
    if(resource.lastContract().deadline().after(new Date())) { #Selection
     #Media 
     System.out.println(resource.name()); #Content
     System.out.println(resource.salary());
    }
   }
 }

该方法是循环，选择资源，访问内容以及访问媒体。看到它们每个都属于不同的抽象级别，打印到控制台应该处于不同的级别，以检查资源是否尚未过期。

让我们看看Francesco提出的解决方案。

首先要做的是将主要功能分为三个类和两个接口，一个用于迭代资源，另一个用于选择尚未过期的资源，另一个用于打印资源。通过这种方法，我们正在创建一个旨在扩展的解决方案，并且还提高了可读性。

现在是时候编写代码了：

如果资源满足实现条件，则将使用谓词接口来实现。

public interface Predicate {

  boolean is(Resource each);

 }

例如，在我们的例子中，接口的实现如下所示：

public class InForcePredicate implements Predicate {

  public boolean is(Resource each) {
   return each.lastContract().deadline().after(new Date());
  }
 }

我们将条件转移到InForcePredicate类。请注意，如果我们要创建一个检查合同是否到期的类，我们将创建一个实现Predicate的新类，其内容类似于return each.lastContract（）。deadline（）。before（new Date（）） ；

下一个接口是Block接口，它将实现对媒体的访问。在这种情况下，要进行控制台：

public interface Block {

  void evaluate(Resource resource);

 }

及其实现：

public class PrintSlip implements Block {

  public void evaluate(Resource resource) {
   System.out.println(resource.name()); 
   System.out.println(resource.salary());
  }

 }

再次注意，更改信息的发送位置（控制台，文件，网络等），只需实现Block接口即可。

最后一个类是一个包含资源迭代器的类，还提供了调用先前创建的每个接口的方法：

public class ResourceOrderedCollection {
  private Collection<Resource> resources = new ArrayList<Resource>();

  public ResourceOrderedCollection() {
   super();
  }

  public ResourceOrderedCollection(Collection<Resource> resources) {
   this.resources = resources;
  }

  public void add(Resource resource) {
   this.resources.add(resource);
  }

  public void forEachDo(Block block) {
   Iterator<Resource> iterator = resources.iterator();

   while(iterator.hasNext()) {
    block.evaluate(iterator.next());
   }

  }

  public ResourceOrderedCollection select(Predicate predicate) {

   ResourceOrderedCollection resourceOrderedCollection = new ResourceOrderedCollection();

   Iterator<Resource> iterator = resources.iterator();

   while(iterator.hasNext()) {
    Resource resource = iterator.next();
    if(predicate.is(resource)) {
     resourceOrderedCollection.add(resource);
    }
   }

   return resourceOrderedCollection;
  }
 }

请参阅接下来的三个要点：

第一个是构造函数接收资源列表。
第二个是select方法接收一个谓词，该谓词将被执行到迭代器中，以知道是否可以选择打印资源。最后，返回带有资源且没有到期合同的ResourceOrderedCollection的新实例。
第三个forEachDo方法接收一个Block接口，该接口由资源列表的每个元素调用。

最后使用以前开发的类修改了Department类：

public class Department {

  private List<Resource> resources = new ArrayList<Resource>();

  public void addResource(Resource resource) {
   this.resources.add(resource);
  }

  public void printSlips() {
   new ResourceOrderedCollection(this.resources).select(new InForcePredicate()).forEachDo(new PrintSlip());
  }

 }

注意，现在printSlips方法包含具有相同抽象级别的单个可读行。

请注意，类名和接口名取自Francesco的示例，但是如果我也要这样做，则应选择更多具有代表性的名称。 Cirillo的方法不错，但需要考虑一些小方面。例如，它具有“ 垂直问题 ”： Predicate接口中的InForcePredicate实例使用五行源代码来封装单个语句。

我们探索了问题的两种可能解决方案，这是Cirillio提出的最后一种解决方案。对于此问题，还有许多其他可能且正确的解决方案，例如，使用“ 模板方法模式” ，或混合使用Lambdaj和（或不使用） 闭包（ Lambdaj语法可能会有些混乱）。它们都有优点和缺点，但是它们都使您的代码更具可读性，而且更重要，所有功能都只能做一件事情，它们只能做得很好。

作为这篇文章的最后说明， JDK 8将提供对本机关闭的支持，还将提供Lambdaj现在提供的许多功能。同时， JDK 8不稳定（计划于2013年中期完成）或您的旧代码（从JDK 8的角度来看） 不稳定 ， Lambdaj确实是一个很好的同伴。

我们不断学习。

参考： 避免FORs –我们的JCG合作伙伴 Alex Soto的“ 反若战役”，来自“ One Jar to Rule Allm All”博客。