Помогите мне выбрать между двумя дизайнами

Question

// Глупый название, но я не мог думать что-то умнее

У меня есть код (см. Ниже, извините за длительный код, но Это очень - очень просто):

namespace Option1
{
    class AuxClass1
    {
        string _field1;
        public string Field1
        {
            get
            {
                return _field1;
            }
            set
            {
                _field1 = value;
            }
        }

        // another fields. maybe many fields maybe several properties

        public void Method1()
        {
            // some action
        }

        public void Method2()
        {
            // some action 2
        }
    }

    class MainClass
    {
        AuxClass1 _auxClass;
        public AuxClass1 AuxClass
        {
            get
            {
                return _auxClass;
            }
            set
            {
                _auxClass = value;
            }
        }

        public MainClass()
        {
            _auxClass = new AuxClass1();
        }
    }
}

namespace Option2
{
    class AuxClass1
    {
        string _field1;
        public string Field1
        {
            get
            {
                return _field1;
            }
            set
            {
                _field1 = value;
            }
        }

        // another fields. maybe many fields maybe several properties

        public void Method1()
        {
            // some action
        }

        public void Method2()
        {
            // some action 2
        }
    }

    class MainClass
    {
        AuxClass1 _auxClass;

        public string Field1
        {
            get
            {
                return _auxClass.Field1;
            }
            set
            {
                _auxClass.Field1 = value;
            }
        }

        public void Method1()
        {
            _auxClass.Method1();
        }

        public void Method2()
        {
            _auxClass.Method2();
        }

        public MainClass()
        {
            _auxClass = new AuxClass1();
        }
    }
}

class Program
{
    static void Main(string[] args)
    {
        // Option1
        Option1.MainClass mainClass1 = new Option1.MainClass();
        mainClass1.AuxClass.Field1 = "string1";
        mainClass1.AuxClass.Method1();
        mainClass1.AuxClass.Method2();

        // Option2
        Option2.MainClass mainClass2 = new Option2.MainClass();
        mainClass2.Field1 = "string2";
        mainClass2.Method1();
        mainClass2.Method2();

        Console.ReadKey();
    }
}

Какой вариант (опция1 или опция2) вы предпочитаете? В каких случаях я должен использовать опцию1 или опцию2? Есть ли какое-либо специальное имя для опции1 или опции2 (композиция, агрегация)?

Answer 1

Согласно с Закон Демететер, Опция2. Таким образом, вы можете свободно изменять реализацию Mainclass, вам не нужно беспокоиться о вызове кода, полагаясь на детали AUXClass1, и действительно может удалить его полностью при необходимости.

Answer 2

РЕДАКТИРОВАТЬ

interface IAuxClass1
{
    string Field1 { get; set; }
    void Method1();
    void Method2();
}

class AuxClass1 : IAuxClass1
{
    string _field1;
    public string Field1
    {
        get
        {
            return _field1;
        }
        set
        {
            _field1 = value;
        }
    }

    // another fields. maybe many fields maybe several properties 

    public void Method1()
    {
        // some action 
    }

    public void Method2()
    {
        // some action 2 
    }
}

public class MyClass : ServiceContainer
{
    public MyClass()
    {
        this.AddService(typeof(IAuxClass1), new AuxClass1());
    }

    public MyClass(IAuxClass1 auxClassInstance)
    {
        this.AddService(typeof(IAuxClass1), auxClassInstance);
    }

    public IAuxClass1 AuxClass
    {
        get
        {
            return (this.GetService(typeof(IAuxClass1)) as IAuxClass1);
        }
    }
}

Оригинал

I Tihnk Mainclass должен получить из AuxClass ..

class MainClass : AuxClass1
{ 

}

Answer 3

Я бы начал с реализации приятной особенности C #, называемых «автоматическими свойствами». Вместо написания

private ThisType _myThing;
public ThisType MyThing 
{ 
    get { return _myThing; } 
    set { _myThing = value; }
}

ты можешь написать

public ThisType MyThing { get; set; }

И компилятор будет генерировать то же самое. Кроме того, вы можете добавить некоторые варианты, например, сделать установочную личную:

public ThisType MyThing { get; private set; }

В вашем случае я бы пошел на опцию 3:

namespace Option3
{
    public AuxClass
    {
        public string Field1 { get; set; }
        public Method1() { ... }
        public Method1() { ... }
    }

    public MainClass
    {
        public AuxClass Aux { get; private set; }
        public MainClass(AuxClass aux)
        {
            this.Aux = aux;
        }
    }
}

class Program
{
    static void Main(string[] args)
    {
        Option3.AuxClass = auxClass3 = new Option3.AuxClass();
        Option3.MainClass mainClass3 = new Option3.MainClass(auxClass3);
        mainClass3.Aux.Field1 = "string2";
        mainClass3.Aux.Method1();
        mainClass3.Aux.Method2();
    }
}

Таким образом, вы блокируете ссылку AUXClass после его установки (например, в варианте 2), не блокируя себя для изменений в интерфейсе AUXClass (например, в варианте 1).

Answer 4

Решение выбора дизайна основано на разных факторах,

1. Более короткий код => Вариант 1
2. Мониторинг активности каждой функциональности и каждого доступа => Вариант 2, однако, использование LINQ и выражений, вы можете написать обобщенный код, который может работать с четным вариантом 1, но это слишком сложно обсудить здесь.