Блог

У меня есть класс пользователя, скажем, User

 public class User
{
public int Id {get;set;}   
public string Name {get;set;}

}
 

и интерфейс, который возвращает группу пользователей, просматривая идентификатор данного пользователя

 public intervace IPeople
{
   IList<User> GetPeople (int id);
   string Key {get;}
}
 

Вот несколько реализаций IPeople

 public class Friends : IPeople
{
   public IList<User> GetPeople (int id)
   {
      return ListOfUsersWhoAreFriends();
   }

    public string Key {get { return "Friends"; }
}


public class Fans : IPeople
{
   public IList<User> GetPeople (int id)
   {
      return ListOfFans();
   }
   public string Key {get { return "Fans"; }

}
 

Теперь в моем методе вызова

  string key="SomeKey" ; //It could be friends or fans
 int id =1;
 IPeople[] allPeople = GetAllInstancesOf<IPeople>();

 IList<User> requiredUsers = allPeople.FirstOrDefault(m=>m.Key == key).GetPeople(id);
 

Так сказать, я создаю экземпляры всех производных классов, а затем проверяю, какой из них мне нужен,
прежде чем вызывать для него функцию. Он работает нормально, но я чувствую, что с точки
зрения производительности это не очень хорошее решение, поскольку я создаю экземпляр путем отражения. Каков лучший способ добиться этого. Какой-нибудь заводской метод, если да, то как??

Помощь будет оценена.

С уважением

Parminder

Я предполагаю, что вы используете отражение в своем классе GetAllInstancesOf? Не уверен, откуда вы заполняете свой список людей или зачем вам нужны ключ и идентификатор, но для заводского шаблона в данном случае я бы использовал следующее

 public IList<User> UserFactory(string kindOfPeople)
{
  switch (kindOfPeople.ToLower())
  {
    case "fans":
      return ListOfFans();
    case "friends":
      return ListOfUsersWhoAreFriends();
    default:
      return new List<User>();
  }
}
 

Если типы людей (фанаты, друзья, семья и т. Д.) Ограничены и не меняются очень часто в вашем приложении, тогда я бы сделал это перечислением и передал его на фабрику

 public enum KindsOfPeople { Friends = 0, Fans = 1}
 

PS: Предполагается, что и поклонники, и друзья являются производными от класса User. Это не было показано в вашем коде. Хороший способ использования заводского шаблона — если все возможные классы, возвращаемые с фабрики, наследуются от общего базового класса.

Я чувствую, что нашел лучшее решение. Вот оно.

 public class User
{
    public int Id { get; set; }
    public string Name { get; set; }
}

public interface IPeople
{
    IList<User> GetPeople(int id);

}


public class Friends : IPeople
{
    public IList<User> GetPeople(int id)
    {
        return new List<User>();
    }



    public class Fans : IPeople
    {
        public IList<User> GetPeople(int id)
        {
            return new List<User>();
        }
    }


    public class Program
    {
        static void Main(string[] args)
        {
            var builder = new ContainerBuilder();
            builder.RegisterType<Friends>().Named<IPeople>("Friends");
            builder.RegisterType<Fans>().Named<IPeople>("Fans");

            IContainer c = builder.Build();

            var x = c.ResolveNamed<IPeople>("Friends");

        }
    }
}
 

Поскольку я использую Autofac, я слежу за регистром с именованными ключами.

Спасибо