2011-11-16 53 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方法,以便它占用更少的参数。例如,您可以创建对象并直接发送。