Использование класса в его конструкторе С# — это пахнет?

Звучит как отличное место для создания шаблона, я бы сказал, Фабричный метод.

class Factory
{
public:
    virtual needsAnIfoo* Create(ProductId);
};

needsAnIfoo* Factory::Create(ProductId id)
{
    if (id == TYPE1) return new needsAnIfoo(ifooType1());
    if (id == TYPE2) return new needsAnIfoo(ifooType2());
    ...
    return 0;
}

Затем вы должны использовать его так:

Factory f = new Factory();
theme1 = f.Create(TYPE1);
theme2 = f.Create(TYPE2);

Узоры — ваш друг!

Мне это кажется неправильным. Пахнет хрупкостью.

Рассматривали ли вы возможность использования шаблона построителя или фабрики для создания соответствующих объектов и установления отношений между ними? Это может обеспечить более безопасный путь вперед.

Я согласен с тем, что шаблон Builder или Factory или аналогичный был бы лучше. Предоставленный код не очень хорошо тестируется и, как уже упоминалось, довольно хрупок, поэтому некоторая форма внедрения зависимостей была бы хорошей.

Используемый шаблон будет зависеть от того, как foo и needAnIFoo используют друг друга. Возможно, вам также придется рассмотреть шаблон Observer, если needAnIFoo — это субъект, foo — наблюдатель, а bar() — метод обновления.

Похоже, это может быть слишком сложно, и что вы делаете свою тему контроллером и имеете контроллер (путем реализации обоих классов ifoo)

Вы можете получить лучшие результаты, если разделите понятия тема и контроллер, чтобы контроллер имеет тему. Затем, например, когда контроллер что-то делает, например, открывает диалоговое окно, он просматривает свою тему, чтобы узнать, какой шрифт использовать.

Как это:

interface itheme {} // to describe properties of the theme
class theme : itheme {}// a bunch of different themes, this previously would have been the "foo" 
class theme2 :itheme{} //etc.

abstract class icontroller
{
    protected icontroller(itheme ptheme) {theme = ptheme;}

    protected itheme theme;

    //function declarations
    // ....

}
class control : icontroller {} // implements the icontrol functions.
//not sure if you need more than one control implementation... 
//  if not, i'd get rid of the icontrol interface.


//use it by passing a theme into the controller constructor:
icontroller myUIController = new control(new ClassicTheme()); 

Я пытаюсь немного понять вопрос, но если каждый needAnIfoo привязан только к одному типу класса, а needAnIfoo ничего не делает для себя, кажется, что вы можете создать статический класс needAnIfoo с методами расширения, нет необходимости передавать это как конструктор аргумент.

руководство по программированию метода расширения

Вот фабрика с наблюдателем — похоже, это помогает и позволяет избежать обновления «контроллера» внутри моего тематического пользовательского интерфейса.

   interface ifoo
    {
        void bar();
    }

    class foo : ifoo
    {
        public void bar() { Console.Write("do a foo type thing"); }
    }

    class foo2 : ifoo
    {
        public void bar() { Console.Write("do a foo2 type thing"); }
    }

    class needsAnIfoo
    {
        public event EventHandler SomethingIFooCanDealWith;
        System.Threading.Timer _timer;
        public needsAnIfoo()
        {
            _timer = new System.Threading.Timer(MakeFooDoSomething, null, 0, 1000);
        }

        void MakeFooDoSomething(Object state)
        {
            if (SomethingIFooCanDealWith != null) 
            {
                SomethingIFooCanDealWith(this,EventArgs.Empty);
            };
        }
    }

    class fooFactory
    {
        needsAnIfoo _needsAnIfoo = new needsAnIfoo();
        Dictionary<String, ifoo> _themedFoos = new Dictionary<string,ifoo>();
        ifoo _lastFoo = null;

        public void RegisterFoo(String themeName, ifoo foo)
        {
            _themedFoos.Add(themeName, foo);
        }

        public ifoo GetThemedFoo(String theme)
        {
            if (_lastFoo != null) { _needsAnIfoo.SomethingIFooCanDealWith -= (sender, e) => _lastFoo.bar(); };
            ifoo newFoo = _themedFoos[theme];
            _needsAnIfoo.SomethingIFooCanDealWith += (sender, e) => newFoo.bar();
            _lastFoo = newFoo;
            return newFoo;
        }
    }

    static void Main(string[] args)
    {
        fooFactory factory = new fooFactory();
        factory.RegisterFoo("CompanyA", new foo());
        factory.RegisterFoo("CompanyB", new foo2());

        ifoo foo = factory.GetThemedFoo("CompanyA");
        Console.Write("Press key to switch theme");
        Console.ReadKey();

        foo = factory.GetThemedFoo("CompanyB");
        Console.ReadKey();
    }