为什么方法缺乏内聚 (LCOM) 包括 Getter 和 Setter

Question

我正在查看 LCOM 指标，如下所示，

http://www.ndepend.com/Metrics.aspx< /a>

所以我们要说几件事，

1) 如果类的所有方法都使用其所有实例字段，则该类是完全内聚的
2) 静态方法和实例方法都计算在内，还包括构造函数、属性 getter/setter、事件添加/删除方法

如果我查看这样的类，

public class Assessment
{
    public int StartMetres { get; set; }
    public int EndMetres { get; set; }
    public decimal? NumericResponse { get; set; }
    public string FreeResponse { get; set; }
    public string Responsetype { get; set; }
    public string ItemResponseDescription { get; set; }
    public string StartText { get; set; }
    public decimal? SummaryWeight { get; set; }
}

它会得到 0.94 的糟糕分数，因为每个 getter 和 setter 都不会访问“所有其他实例字段”。

它是这样计算的，

accessAverage - methodCount / 1 - methodCount

(2 - 17) / (1 - 17) = 0.94 (rounded)

我不理解这个指标，为什么它应该包括 getter 和 setter？ getter 和 setter 始终只能访问一个实例字段。

Answer 1

这表明，如果你盲目地走向极端，那么每个软件指标都是有缺陷的。

当你看到一个类时，你就知道这是一个“不连贯”的类。例如：

class HedgeHog_And_AfricanCountry
{

   private HedgeHog _hedgeHog;
   private Nation _africanNation;

   public ulong NumberOfQuills { get { return _hedgeHog.NumberOfQuills; } }
   public int CountOfAntsEatenToday { get { return _hedgeHog.AntsEatenToday.Count(); } }

   public decimal GrossDomesticProduct { get { return _africanNation.GDP; } }
   public ulong Population { get { return _africanNation.Population; } }
}

这显然是一个不内聚的类，因为它包含两个不需要彼此在一起的数据。

但是，虽然我们很明显这个类是不内聚的，但如何让软件程序来确定不内聚呢？如何判断上面的类是不内聚的，而这个类却不是？

class Customer
{
    public string FullName { get; set; }
    public Address PostalAddress { get; set; }
} 

他们提出的指标确实可以检测到不连贯性，但也会出现误报。

如果您认为这个指标很重要怎么办？您可以创建一个仅包含字段的“CustomerData”类，以及一个将数据字段公开为属性的“Customer”类。

// This has no methods or getters, so gets a good cohesion value.
class CustomerData
{
    public string FullName;
    public Address PostalAddress;
}

// All of the getters and methods are on the same object
class Customer
{
   private CustomerData _customerData;
   public string FullName { get { return _customerData.FullName; } }
   // etc
}

但如果我正在玩这个游戏，我也可以将其应用到缺乏凝聚力的示例中：

class Hedgehog_And_AfricanCountry_Data
{
   public Hedgehog _hedgehog;
   public AfricanNation _africanNation;
}

class Hedgehog_And_AfricanCountry
{
   private Hedgehog_And_AfricanCountry_Data _hedgehogAndAfricanCountryData;
   // etc;
}

真的，我认为最好了解什么是凝聚力，以及为什么它是一个有价值的目标，但也要了解软件工具无法正确衡量它。