简体   繁体   中英

Injecting an IEnumerable into a constructor with a Ninject factory method

I'm trying to inject an IEnumerable into a constructor with Ninject.

My constructor looks like this:

public MatrixViewModel(IEnumerable<FooViewModel> fooViewModels)
{
    _fooViewModels = fooViewModels;
}

My Ninject module looks like this:

public class MainModule : NinjectModule
{
    public override void Load()
    {
        Bind<IEnumerable<FooViewModel>>()
            .ToMethod(context => GetFooViewModels())
            .InSingletonScope(); // this binding is not working
    }

    private IEnumerable<FooViewModel> GetFooViewModels()
    {
        // returns a bunch of foo view models
    }
}

This doesn't seem to be working. I don't get any error. Ninject just doesn't ever use the binding, and the value that is passed into the constructor is basically just an empty default value.

How do you inject an IEnumerable with Ninject?

Edit

More details on my factory method:

private IEnumerable<FooViewModel> GetFooViewModels()
{
    return new[]
    {
        new FooViewModel
        {
            Bar = new BarViewModel
            {
                X = 1,
                Y = 2
            },
            Misc = "Hello"
        },
        new FooViewModel
        {
            Bar = new BarViewModel
            {
                X = 3,
                Y = 4
            },
            Misc = "Goodbye"
        },
        // etc.....
    };
}

Edit 2

Based on Remo's answer, one possible solution is to use a foreach loop to bind the view models one at a time:

foreach (var fooViewModel in GetFooViewModels())
{
    Bind<FooViewModel>().ToConstant(fooViewModel);
}

Enumerables are treated differently by Ninject. Just provide bindings for all the view models. For enumerables Ninject will create an instance of every applying binding and pass them as IEnumerable .

eg

Bind<FooViewModel>().To<FooViewModel1>();
Bind<FooViewModel>().To<FooViewModel2>();

Based on Remo's answer, one possible solution is to use a foreach loop to bind the view models one at a time:

foreach (var fooViewModel in GetFooViewModels())
{
    Bind<FooViewModel>().ToConstant(fooViewModel);
}

From your question:

Bind<IEnumerable<FooViewModel>>()
    .ToMethod(context => GetFooViewModels())

I am not sure collection support or ToMethod works that way.

This should would work though:

Bind<MatrixViewModel>()
    .ToMethod(context => new MatrixViewModel(GetFooViewModels()))

Of course, how useful this solution is depends on how you're building up your views.

The technical post webpages of this site follow the CC BY-SA 4.0 protocol. If you need to reprint, please indicate the site URL or the original address.Any question please contact:yoyou2525@163.com.

 
粤ICP备18138465号  © 2020-2024 STACKOOM.COM