1

Есть класс:

public class Resource<T> : IGame where T : Component 
{

И например мы его как тип используем в каком то интерфейсе:

public interface IBuildResource
{
    Resource<Component> GetPath(List<Resource<Component>> RefList);
}

Приемлемой ли практикой будет писать <Resource<Component>> заместо <Resource<T>> и соотвественно делать дженериком сам интерфейс.

Или например в функциях класса реализующих этот интерфейс писать:

class BuildComponent<T> : BaseCls, IBuildResource 
{
        public Resource<Component> GetPath(List<Resource<Component>> resList)
        {
            Resource<Component> resReference = null;
            . . .
            return resReference;
        }
2
  • 3
    приемлимо делать все, что сможет понять компилятор. если не упало с ошибкой - значит приемлимо :)
    – user177221
    10 авг 2017 в 9:14
  • 1
    автор имеет в виду приемлемые практики использования? в чём критерии приемлемости?
    – Anatol
    10 авг 2017 в 9:18

1 ответ 1

1

Последний пример с BuildComponent<T> скорее всего неверен, так как выглядит это примерно так:

List<int> list;
list.Remove("item");
list.Find<String>()

Обычно такие методы всё-таки имеют какую-либо связь с T

public Resource<T> GetPath(List<Resource<T>> resList) where T : Component

Честно говоря, я бы крепко подумал над вложенными Generic'ами. Их просто неудобно читать и писать. Вместо List<Resource<Component>> я бы предпочёл видеть

class ComponentList : ResourceList<Component>
{
}

class ResourceList<T> : List<Resource<T>>
{
}

Это не потребует много сил и времени, из лишнего кода - только необходимость объявить конструкторы, зато код станет на порядок чище и приятнее глазу. Исключительно субъективное мнение. Некоторые использует вместо этого алиасы, но мне кажется, что они скорее запутают, чем помогут. А типизированных макросов у нас пока нет, в отличии от того же Nemerle.

4
  • Мне тоже не нравится подобная практика, но к сожалению это код который нужно супортить и все переделать в более читабельный вариант не представляется возможным.
    – Alex
    10 авг 2017 в 10:31
  • А почему, если не секрет? В предложенном выше варианте ты можешь передать ComponentList в метод, который принимает List<Resource<Component>> благодаря наследованию, а массовые правки легко осуществляются благодаря рефакторингам ReSharper'а. 10 авг 2017 в 11:19
  • Потому что в больших проектах с большим кол-вом человек нельзя вот так вот взять и что то поменять, менять можно только строго оговоренные места ))
    – Alex
    10 авг 2017 в 11:44
  • Пойду к своему маленькому проекту на 200 разработчиков. :D 10 авг 2017 в 12:03

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.