4

我刚刚收到一些关于求职申请 Java 编码练习的反馈。他们不喜欢反馈中提到的解决方案和两个问题（我非常感谢，因为很少有反馈）：

  • 显然，我没有使用 TDD 方法。
  • 我过度使用静态方法，我知道静态方法是反面向对象的，但我只在验证和实用类型方法中使用它们。

所以这里有两个问题：

不使用 TDD 方法的可能迹象是什么？

什么编码风格或模式可以替代静态方法？

在前两个回答之后，我还有另一个问题。

您是否同意使用静态方法仅在限制代码的可测试性而不是它们本身不好时才不好。

因此，如果静态方法不限制我的代码的可测试性，那么回到我的工作申请练习解决方案是否仍然不好使用？我的验证方法非常简单 'Validator.notNull(p,"paramName")' 现在我为什么要模拟它？

非常感谢。

4

2 回答 2

5

不使用 TDD 的一个明显标志是协作者使用静态方法和静态类成员。您不能覆盖静态方法，因此您不能单独使用此类方法替换模拟来测试类。

您可以使用依赖注入，而不是在协作者上使用静态协作者或静态方法。在一个简单的编码练习中，您将通过构造函数或手动通过 setter 注入依赖项。在现实生活中，您可以使用一种可用的依赖框架。

于 2013-02-20T17:49:04.910 回答
2

您的静态Validaton方法对我来说似乎应该是对象的一部分。

假设你有一堂课Drink

public class Drink
{
    private readonly string _name;
    private readonly double _temperature;

    public Drink(string name, double temperature)
    {
        _name = name;
        _temperature = temperature;
    }
}

您的业​​务逻辑将能够实例化各种饮料、7up、可乐等。你想确保一种饮料有合适的温度来饮用它，所以你需要一种Validate方法。你可以按照你的方法：

public void TakeAZip()
{
    if (Validation.HasAppropriateTemp)
    {
        // implement drink
    }
}

'

静态类的替代品

这样你就很难依赖你的静态Validation类。

或者，您可以使用依赖注入。

public void TakeAZip(ITemperatureValidator validator)
{
    if (validator.HasAppropriateTemp)
    {
        // implement drink
    }
}

如果更方便，您还可以选择Validator通过构造函数传递

    private readonly string _name;
    private readonly double _temperature;
    private ITemperatureValidator _validator;

    public Drink(
        string name, 
        double temperature, 
        ITemperatureValidator validator)
    {
        _name = name;
        _temperature = temperature;
        _validator = validator;
    }

现在您可以模拟验证器的行为，并且可以将您的Drink类与所有外部行为隔离开来。

于 2013-02-20T20:47:41.937 回答