c# - 包装单个方法的方法

Question

我想我要疯了，请有人安慰我。

public class MyFile
{   
    public static byte[] ReadBinaryFile(string fileName)
    {
        return File.ReadAllBytes(fileName);
    }

    public static void WriteBinaryFile(string fileName, byte[] fileContents)
    {
        File.WriteAllBytes(fileName, fileContents);
    }
}

人们一直在我们的代码库中添加上面的代码，这肯定是错误和可怕的，我通过删除它并替换所有(或在这种情况下......)对它的引用来帮助世界带有内部代码。

这种事情有什么真正的理由吗？我会错过大局吗？我们相当YAGNI -以我们的团队为中心，这似乎与此背道而驰。我能理解这是否是更多事情的开始，但是这段代码已经休眠了好几个月，直到我今天被它绊倒。我搜索得越多，发现的就越多。

Answer 1

如所写，类/方法是垃圾。但是，我可以看到可以合法使用类似模式的情况:

public interface IFileStorage
{
    byte[] ReadBinaryFile(string fileName);
    void WriteBinaryFile(string fileName, byte[] fileContents);
}

public class LocalFileStorage : IFileStorage { ... }

public class IsolatedFileStorage : IFileStorage { ... }

public class DatabaseFileStorage : IFileStorage { ... }

换句话说，如果您想支持不同类型的存储，那么您实际上可能会包装非常简单的方法以实现通用抽象。

尽管如此，该类没有实现任何接口(interface)，并且方法是静态的，因此几乎没有用。如果你试图支持上述模式，那么重构；否则，摆脱它。