2011-11-16 33 views
0

這個班是否根據「中間人」代碼氣味計數?這個班是否根據「中間人」代碼氣味計數?

我有一個「人」類。我還有一個班級人員列表,其中有一個Person_list列表,並且具有添加,刪除和用Person類別做其他事情的方法。

我從另一個類訪問Person_list類。

這是否計數在中間人或任何其他代碼的氣味?

class P{ 
} 

public class PL { 

    Vector<P> p; 
    String name; 

    public PL(String name) { 
     p = new Vector<P>(); 
     this.name = name; 
    } 

    public void addP(String name, int a, double h, 
      double w, String a, String g) { 
    } 

    public Patient findP(String name) { 
    } 

    public void removeP(String name) { 
    } 

    public void printSummaryForP(String name) { 
    } 
} 


class Main{ 

public static void main(String[] args){ 
PL pl=new PL(); 
pl.addP(a,b,c,d,e); 
pl.addP(a,b,c,d,e); 
pl.remove(); 
} 
} 
+0

如果您提供了一些代碼,會更容易 –

+0

代碼氣味通常涉及代碼。 –

+0

@Chris對不起,我不能分享我的code.It是違反政策... :-( – rgksugan

回答

2

這取決於。你在做什麼方法addp,findp,removep?如果你沒有做任何具體的事情,只是將它們添加到Vector中,那麼比這個「list」就是「中間人」代碼氣味的一個例子,你可以安全地刪除它。

除此之外,我建議重構addP方法,以便它佔用更少的參數。例如,您可以創建對象並直接發送。

相關問題