静态方法:何时和何时不

我想你可能有点误解了。

public static int Add(int x, int y)
{
    return x + y;
}

您可以通过测试返回值是否为您根据传入的参数所期望的值来对此进行测试。

当测试需要引入模拟时,静态方法会变得麻烦。

假设我有一些代码调用 File.Delete() 静态方法。为了在不依赖文件系统的情况下测试我的代码,我想用一个测试版本来替换/模拟这个调用,该版本只验证它是否已经从正在测试的代码中调用。如果我有一个对象的实例 Delete()

为了测试这样的东西,我将引入一个接口:

interface IFileDeleter
{
    void Delete(string file);
}

然后,我的代码将获取实现此接口的对象的实例(在方法调用中或作为构造函数中的参数),然后调用 删除方法:

void MyMethod(string file)
{
    // do whatever...
    deleter.Delete(file);
}

为了测试这个,我可以嘲笑 IFileDeleter 接口并简单地验证 删除() 方法已被调用。这消除了在测试过程中需要真正的文件系统的需要。

这可能看起来代码更复杂(它是这样的),但它会让测试变得更容易。

避免静态当然是一种方法,但是当您不能或者正在使用遗留代码时,可以使用以下选项。下面是阿德里安班克斯对上述问题的回答,假设您有以下代码(抱歉,它是用Java编写的,因为我不知道C#):

public void someMethod() {
   //do somethings
   File.delete();
   //do some more things
}

public void someMethod() {
   //do somethings
   deleteFile();
   //do some more things
}

//protected allows you to override in a subclass
protected void deleteFile() { 
   File.delete();
}

然后在准备单元测试时,创建一个模拟类,该类扩展了原始类并终止了该功能:

//Keep all the original functionality, but stub out the file delete functionality to 
//prevent it from using the real thing and while you're at it, keep a record that the
//method was called.
public class MockClass extends TheRealClass {
    boolean fileDeleteCalled = false;

    @Override
    protected void deleteFile()
        //don't actually delete the file, 
        //just record that the method to do so was called
        fileDeleteCalled = true;
    }

    public boolean fileDeleteCalled() { 
        return fileDeleteCalled; 
    }
}

//This would normally be instantiated in the @Before method
private MockClass unitUnderTest = new MockClass();

@Test
public void testFileGetsDeleted(){
    assertFalse(unitUnderTest.fileDeleteCalled());
    unitUnderTest.someMethod();
    assertTrue(unitUnderTest.fileDeleteCalled());
}

现在,您已经执行了someMethod()的所有功能,而没有实际删除该文件,您仍然可以查看是否调用了该方法。

一般来说,如果方法:

• 很长
• 包含复杂逻辑
• 使用文件系统
• 调用web服务

那就不要让它静止。(请参阅@adrianbanks的答案,以获得有关此问题背后的原因和备选方案的精彩讨论。)