Сокращения конструктор кода котла плита

Question

я часто в конечном итоге писать классы так:

public class Animal 
{ 
    public string Colour { get; set; } 
    public int Weight { get; set; } 

    public Animal(Dog data) 
    { 
     this.Colour = data.Colour; 
     this.Weight = data.Weight; 
    } 

    public Animal(Cat data) 
    { 
     this.Colour = data.Colour; 
     this.Weight = data.Weight; 
    } 
} 

Если у вас есть много свойств и типов, то вы быстро в конечном итоге с большим количеством коды котла пластины. В идеале в этой ситуации я бы просто создал интерфейс IAnimal и ссылку на него. Я сейчас в ситуации, когда классы Dog и Cat существуют в третьей стороне, и я не могу их модифицировать. Единственное решение, которое я могу придумать это:

public class Animal 
{ 
    public string Colour { get; set; } 
    public int Weight { get; set; } 

    public Animal(Cat data){Init(data);} 
    public Animal(Dog data){Init(data);} 

    private void Init(dynamic data) 
    { 
     this.Colour = data.Colour; 
     this.Weight = data.Weight; 
    } 
} 

Это работает, но я теряю все типобезопасность, есть лучшее решение, чем инъекции конструктора?

Спасибо,

Джо

EDIT: Вот реальный пример. У меня есть библиотека третья сторона, которая возвращает 3 объекта называется:

• GetPageByIdResult
• GetPagesByParentIdResult
• GetPagesByDateResult

(Это все автогенерируемая классы из ссылки на службу и свойства в значительной степени идентичны)

Вместо того чтобы иметь дело с этими тремя объектами, я хочу иметь дело с одним объектом PageData или их коллекцией.

Answer 1

Вы можете иметь логику в один общем конструкторе, который все остальные конструкторы называют:

public class Animal 
{ 
    public string Colour { get; set; } 
    public int Weight { get; set; } 

    public Animal(Dog data) : this (data.Colour, data.Weight) 
    { 
    } 

    public Animal(Cat data) : this (data.Colour, data.Weight) 
    { 
    } 

    private Animal(string colour, int weight) 
    { 
     this.Colour = colour; 
     this.Weight = weight; 
    } 
} 

Это очень похоже на ваше второе решение, но оно не теряет типобезопасность.

Answer 2

Если вы не хотите, чтобы класс завалялся таким образом, вы можете попробовать методы расширения?

public static Animal ToAnimal(this Dog item) 
{ 
    return new Animal() {Weight = item.Weight, Colour = item.Colour}; 
} 

public static Animal ToAnimal(this Cat item) 
{ 
    return new Animal() {Weight = item.Weight, Colour = item.Colour}; 
} 

Answer 3

Я в настоящее время в ситуации, когда существуют Dog и Cat классов в сборке третьей партии, и я не могу изменить их

Я хотел бы предложить решение Automapper на основе:

public static class AnimalFactory 
{ 
    public static Animal Create<T>(T source) 
     where T : class 
    { 
     Mapper.CreateMap<T, Animal>(); 
     return Mapper.Map<Animal>(source); 
    } 
} 

Использование:

 var catAnimal = AnimalFactory.Create(cat); 
     var dogAnimal = AnimalFactory.Create(dog); 

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

Answer 4

попробуйте использовать json serializer, с этим мы сможем обеспечить безопасность типа.

public class Animal 
    { 
     public string Colour { get; set; } 
     public long Weight { get; set; } 
     public string Name { get; set; } 
     public Animal Create<T>(T anyType) 
     { 
      return GetObject<T, Animal>(anyType); 
     } 
     public K GetObject<T, K>(T type1) 
     { 
      try 
      { 
       var serialized = JsonConvert.SerializeObject(type1); 
       return JsonConvert.DeserializeObject<K>(serialized); 
      } 
      catch (Exception ex) 
      { 
       return default(K); 
      } 
     } 
    } 
    class Program 
    { 
     public static void Main(string[] args) 
     { 
      Animal obj = new Animal(); 
      var animal = obj.Create(new { Colour = "Red", Weight = 100 }); 
      //here you can pass any object, only same name properties will be initialized.. 
      Console.WriteLine(animal.Colour + " : " + animal.Weight); 
      Console.ReadKey(); 
     } 
    }