3

Решил наконец-то научиться делать интерфейс приложений расширяемый на несколько View/ViewModel в рамках MVVM. Наткнулся на шаблон Inversion of Control + Depency Injection, информации по нему много, многие его советуют для огранизации логической архитектуры приложения. К тому же, он дает возможность организовать удобства типа хранилища синглтонов, которые мне как раз и нужны.

Проблема была в том, что в статьях авторы рекомендуют вот это вот всё, и практически никто не показывает реализацию IoC вручную в WPF+MVVM "для чайников".

А я не могу не понимая, как оно работает, использовать "черный ящик" в своих проектах, вообще не люблю большое, срашное и непонятное, которое непонятно как работает, и всегда пытаюсь раскопать эту самую суть. Разбор решил выполнить на практической задаче - сделать многостраничный интерфейс с ListBox для выбора страниц.

Начал с двух интефейсов

public interface IView
{
    string Title { get; set; }
    object DataContext { get; set; }
}

public interface IViewModel
{
}

Затем прикрутил их все

public sealed partial class MainWindow : Window, IView {...}
public partial class Page1 : Page, IView {...}
public partial class Page2 : Page, IView {...}

public class MainViewModel : NotifyPropertyChanged, IViewModel {...}
public class Page1ViewModel : NotifyPropertyChanged, IViewModel {...}
public class Page2ViewModel : NotifyPropertyChanged, IViewModel {...}

И реализовал вот такие 2 контейнера с синглтонами

public static class ViewService
{
    private static readonly ConcurrentDictionary<Type, IView> instances = new ConcurrentDictionary<Type, IView>();

    public static T GetView<T>(IViewModel viewModel = null) where T : IView
    {
        if (!instances.TryGetValue(typeof(T), out IView view))
        {
            view = (T)Activator.CreateInstance(typeof(T));
            instances[typeof(T)] = view;
        }
        if (viewModel != null)
            view.DataContext = viewModel;
        return (T)view;
    }
}

public static class ViewModelService
{
    private static readonly ConcurrentDictionary<Type, IViewModel> instances = new ConcurrentDictionary<Type, IViewModel>();

    public static T GetViewModel<T>() where T : IViewModel
    {
        if (!instances.TryGetValue(typeof(T), out IViewModel viewModel))
        {
            viewModel = (T)Activator.CreateInstance(typeof(T));
            instances[typeof(T)] = viewModel;
        }
        return (T)viewModel;
    }
}

Главное окно создаю вот так

public partial class App : Application
{
    protected override void OnStartup(StartupEventArgs e)
    {
        base.OnStartup(e);
        MainViewModel vm = ViewModelService.GetViewModel<MainViewModel>();
        MainWindow window = ViewService.GetView<MainWindow>(vm);
        window.Show();
    }
}

А странички - в конструкторе главной вью-модели

public class MainViewModel : NotifyPropertyChanged, IViewModel
{
    private IView _currentPage;

    public List<IView> Pages { get; }

    public IView CurrentPage
    {
        get => _currentPage;
        set
        {
            _currentPage = value;
            OnPropertyChanged();
            OnPropertyChanged(nameof(Title));
        }
    }

    public string Title => $"IoC Demo - {CurrentPage.Title}";

    public MainViewModel()
    {
        Pages = new List<IView>
        {
            ViewService.GetView<Page1>(ViewModelService.GetViewModel<Page1ViewModel>()),
            ViewService.GetView<Page2>(ViewModelService.GetViewModel<Page2ViewModel>())
        };
        CurrentPage = Pages[0];
    }
}

Главное окно MainWindow

<Grid>
    <Grid.ColumnDefinitions>
        <ColumnDefinition Width="Auto"/>
        <ColumnDefinition/>
    </Grid.ColumnDefinitions>
    <ListBox BorderThickness="0" Background="AliceBlue" ItemsSource="{Binding Pages}" SelectedItem="{Binding CurrentPage}">
        <ListBox.ItemTemplate>
            <DataTemplate>
                <TextBlock Text="{Binding Title}" HorizontalAlignment="Center" VerticalAlignment="Center" Margin="5"/>
            </DataTemplate>
        </ListBox.ItemTemplate>
    </ListBox>
    <Frame Content="{Binding CurrentPage}" Grid.Column="1" NavigationUIVisibility="Hidden"/>
</Grid>

Странички одинаковые, просто у одной тип Page1, у второй Page2

<Grid>
    <TextBlock Margin="5" Text="{Binding Text}"/>
</Grid>

Вью-модель странички (для видимости)

public class Page1ViewModel : NotifyPropertyChanged, IViewModel
{
    public string Text { get; } = "Page one";
}

И это все вместе работает.

Вопрос: Нарушаю ли я MVVM, и правильно ли реализовал IoC контейнеры с синглтонами?

IoC WPF

P.S.

15
  • 1
    А почему статика? Контейнер должен создаваться один раз при старте приложения, забиваться нужными объектами/типами и дальше через него уже получается все что нужно. Также не пойму для чего вам два контейнера, почему не один? Ну и также я не вижу у вас тут DI, к примеру, у вас есть HttpClient отдельно, в виде сервиса, вам его надо достать в Page1, вы пишете private HttpClientService client; public Page1(HttpClientService service) {this.service = service;} и все, ваше приложение падает, когда контейнер должен сам выдать этот HttpClientService.
    – EvgeniyZ
    1 окт 2020 в 12:20
  • 1
    Вот это, Pages = new List<IView>, по сути тоже должен делать контейнер за вас. По поводу нарушений - да по сути сама Frame сильно ограничивает MVVM и не редко его нарушает, а так, кроме того, что у вас за DataContext отвечает контейнер, мне лично придраться не к чему. Могу посоветовать вам этот материал, который мне однажды очень хорошо помог в понимание как устроены IoC.
    – EvgeniyZ
    1 окт 2020 в 12:24
  • @EvgeniyZ Спасибо! 1) Почему статика, потому что я не вижу причин для синглтона здесь, я никому и никуда сам контейнер не передаю, он просто статичный, и он просто есть. 2) view.DataContext = viewModel; - Property DI, разве нет? 3) Почему два контейнера: можно свалить это в один класс, можно даже сотворить один метод, но я не хотел усложнять синтаксис передачей двух типов в метод, так сказать, двойной дженерик. Поэтому разделил.
    – aepot
    1 окт 2020 в 13:09
  • 1
    Ваши сервисы для создания являются обычными фабриками. Вы, конечно, можете попробовать натянуть на них название "DI контейнер", но выполнять роль контейнера они не начнут от этого. Также пример не очень удачный, так как хранить представления в памяти я бы не стал - другое окно может захотеть предсталение забрать себе и можно получить непонятную кашу. Обычно можно хранить VM как синглтон, а представление рендерить каждый раз, когда неадо показать VM
    – tym32167
    1 окт 2020 в 13:22
  • 2
    Ну так вы саму суть то не поменяли, у вас как была фабрика, которая по требованию отдает нужный объект, так и осталась. В IoC вы изначально создаете контейнер, в него регистрируете все свои необходимые типы, дальше говорите ему: "Слушай, дай мне главный класс и запихни в него все зависимости". Он вам отдает, например MainViewModel, которая требует для своей работы некий ILogger, вам не нужно будет писать ILogger = Container.Resolve(...);, это за вас уже сам контейнер сделает. Вот простой пример набросал. Заметьте, что внутри классов не делается Resolve.
    – EvgeniyZ
    1 окт 2020 в 16:12

1 ответ 1

4

Кажется, осилил.

  1. Конструкторы изменил так, чтобы они принимали конкретный тип View Model и закидывали его себе в DataContext.
public MainWindow(MainViewModel viewModel) {...}
public Page1(Page1ViewModel viewModel) {...}
public Page2(Page2ViewModel viewModel) {...}
  1. Интерфейс IViewModel выкинул за ненадобностью

  2. Реализовал IoC контейнер, на этот раз надеюсь, что настоящий

public static class IocContainer
{
    private static readonly IDictionary<Type, object> instances = new ConcurrentDictionary<Type, object>();

    public static void Register<T>() where T : class
        => instances[typeof(T)] = null;

    public static T Resolve<T>() where T : class
        => (T)GetInstance(typeof(T));

    private static object GetInstance(Type type)
    {
        bool registered = instances.TryGetValue(type, out object instance);
        if (instance == null)
        {
            object[] args = type.GetConstructors().First().GetParameters().Select(x => GetInstance(x.ParameterType)).ToArray();
            instance = Activator.CreateInstance(type, args);
            if (registered)
                instances[type] = instance;
        }
        return instance;
    }
}
  1. Код OnStartup теперь выглядит так:
protected override void OnStartup(StartupEventArgs e)
{
    base.OnStartup(e);
    IocContainer.Register<MainViewModel>();
    IocContainer.Register<Page1ViewModel>();
    IocContainer.Register<Page2ViewModel>();

    IocContainer.Resolve<MainWindow>().Show();
}
  1. А конструктор MainViewModel так:
public MainViewModel()
{
    Pages = new List<IView>
    {
        IocContainer.Resolve<Page1>(),
        IocContainer.Resolve<Page2>()
    };
    CurrentPage = Pages[0];
}

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

P.S. Архив с этим решением на Яндекс.Диске - ссылка.

6
  • Уже лучше, хоть на базовый контейнер стало похоже, но вот статика и постоянный резолв страниц мне не очень нравиться. В Autofac подобное выглядело бы так, то есть регистрируются нужные типы под нужным интерфейсом, а потом просим контейнер отдать нам все типы, которые реализуют нужный интерфейс и как видите, опять без резолва внутри классов. Вот эту бы фичу вам стянуть оттуда, раз пишете свой IoC. Вот кстати не знаю, правильно-ли вообще резолв делать внутри класса, не разу подобного не видел, ибо как я говорил ранее, это задача контейнера, отдать зависимости.
    – EvgeniyZ
    1 окт 2020 в 21:46
  • 1
    @EvgeniyZ да, я понял, что полноценный контейнер может всякое, но у меня нет задачи написать полноценный контейнер, который может горы свернуть и позволит обмазаться юнит-тестами. У меня была задача прикрутить IoC к приложению так, чтобы меня за нарушение MVVM не сожгли на костре. А главное - я понял, как это работает, и теперь такой контейнер смогу без гугла воспроизвести сам с нуля. Новичкам, кто не в теме, возможно тоже мой код пригодится. А вам - еще раз спасибо! Про интерфейсы понял, прикольно, завтра поковыряюсь, но это уже за рамками данного вопроса.
    – aepot
    1 окт 2020 в 22:01
  • Это какой-то неправильный MVVM получается. И ссылки на вид внутри вьюмоделей и IoC внутри вьюмоделей а-ка сервис локатор.
    – vitidev
    28 окт 2020 в 9:57
  • 1
    ну если MVVM+IoC, да autofac, то можете попробовать сразу подальше зайти.
    – vitidev
    28 окт 2020 в 10:52
  • 1
    Забыл еще этот крутой доклад (я использую их решение) для понимания мощи подхода. Это выворачивает управление времени жизни vm - не контейнер управляет временем жизни, а наоборот.. Как следствие, weakref не нужен (а в MVVM их хватает и проблемы они не решают) и всё (RelayCommand, PropertyObserver, Messenger и события) используется с lifetime, и когда lifetime, в котором живет vm, схлопывается, то автоматически все отписывается и чистится. Если вы в самом начале интеграции IoC, то у вас есть возможность встроить IoC с бОльшим профитом.
    – vitidev
    28 окт 2020 в 12:53

Ваш ответ

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

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