Question

请考虑以下界面：

interface IFile
{
    // Members
};

interface IAudioFile : IFile
{
    // Members
};

interface IVideoFile : IFile
{
    // Members  
};

enum ContentType
{
    Audio,
    Video
};

interface IProvider
{
    HashSet<ContentType> GetSupportedTypes();
    IList<IFile> GetFiles(ContentType contentType);
};

我认为ContentType枚举是多余的。有没有办法使用接口标识符而不是枚举类型？

非常感谢对界面设计的任何评论。

Answer 1

这实际上取决于你想要实现的目标，但我可能想要看的一个选项是使用泛型，所以IProvider就是这样

interface IProvider
{
    IList<IFile> GetFiles<T>() where T: IFile;
}

可以像这样实现

public void ProviderConcrete()
{
    public IList<IFile> GetFiles<T>()
    {
        if(typeof(t) == typeof(IAudioFile))
            .... get Audio files

    }
}

并且这样调用

public void Caller()
{
    var files = GetFiles<IAudioFile>();
}

Answer 2

通常，写这样的东西会更好：

void method(IFile file) {
    file.DoYourThing();
}

大于

void method(ContentType id) {
   switch (id) {
   case ContentType.Audio: 
       file.DoThis();
       break;

   case ContentType.Video: 
       file.DoThat();
       break;
   }
}

那是因为随着时间的推移，开关通常成为维护噩梦，而且也容易出错。

我的建议是，当您需要switches或if-else个链时，您应该考虑将方法插入到现有的类层次结构中或创建新的层次结构。您应该努力编写与您在第一个代码段中看到的代码类似的代码。

像往常一样，这是通用的，因此可能不适用于您手头的特定问题。

Answer 3

我认为这里的重点是返回的列表包含“基础”对象。

如果您不喜欢，可以创建一些重载，如

IList<IAudioFile> GetAudioFiles();
IList<IVideoFile> GetVideoFiles();

如何用接口ID之类的东西替换枚举？

3 个答案: