如何不重複自己或如何改變方法中的簡單條件？

Question

我有一個方法，我在實現中多次使用了非常簡單的修改。我如何避免重複自己？

... 
    while (!queue.isEmpty()) { 
     Element pivot = queue.poll(); 
     elements.remove(pivot); 
     for (Element a : elements) { 
      if (areFriends(pivot, a)) { 
       db.addRelation(a, pivot); 
       queue.add(a); 
       elements.remove(a); 
      } 
     } 
    } 
... 

我想改變areFriends條件與一個新的f.e. areEnemies（Element pivot，Element a）並繼續使用整個代碼和數據結構。我試圖提取一個無效方法，但在這種情況下，我必須通過所有變量（db，隊列等）作爲輸入，它看起來像一個反模式。你有什麼想法如何解決這個問題？謝謝！

Answer 1

創建一個接口：

public interface Relation 
{ 
    public void execute(Element a, Element b); 
} 


public class AreFriendsRelation implements Relation 
{ 
    public void execute(Element a, Element b) 
    { 
     // Return the Relation result 
    }  
} 

public class AreEnemiesRelation implements Relation 
{ 
    public void execute(Element a, Element b) 
    { 
     // Return the Relation result 
    }  
} 

傳遞您的關係對象ECT你的方法：

public void MyMethod(Relation myRelation) { 
... 
while (!queue.isEmpty()) { 
     Element pivot = queue.poll(); 
     elements.remove(pivot); 
     for (Element a : elements) { 
      if (myRelation.execute(pivot, a)) { 
       db.addRelation(a, pivot); 
       queue.add(a); 
       elements.remove(a); 
      } 
     } 
    } 

... 
} 

Answer 2

您可以使用Command設計模式。使用方法public void checkElements(Element a, Element b)創建一個接口，並創建該接口的幾個實例。在你的方法中，使用接口的方法，或者讓你的方法接受一個接口實例作爲參數，或者把它作爲一個類成員。

Answer 3

你可以定義一個interface：

public interface IElementFunction 
{ 
    public boolean execute(Element e1, Element e2); 
} 

和interface的通實現（無論是命名或匿名class ES）公用功能：

private void commonFunction(IElementFunction ief) 
{ 
    while (!queue.isEmpty()) { 
     Element pivot = queue.poll(); 
     elements.remove(pivot); 
     for (Element a : elements) { 
      if (ief.execute(pivot, a)) { 
       db.addRelation(a, pivot); 
       queue.add(a); 
       elements.remove(a); 
      } 
     } 
    } 

Answer 4

嗚嗚..不知道如果我完全理解你的問題，但是這看起來像一種氣味來實現模板方法模式或者一些其他Behavioral pattern.

檢查：http://en.wikipedia.org/wiki/Template_method_pattern