[Feature Request] Method aliases skipping "injector" argument (default injector)

[samber]

[Feature Request] Method aliases skipping "injector" argument (default injector)

[samber]

Thanks for your suggestion, but I'm not sure we will add these shortcuts to the repository.

Using default injectors is basically a bad idea since it makes your code very hard to test. Adding 18 more helpers without the i *do.Injector argument seems excessive to me.

Anyway, you can use a simple alias in your projects:

// utils/do.go
func Provide(provider do.Provider[T]) {
    return do.Provide(nil, provider)
}

I keep this issue open for further discussion. If anybody wants to expose a PoV, please answer this issue!

[kcmvp]

If it supports this set of API then

do.Provide(nil, config) do.Provide(nil, datasource) do.Provide(nil, db)

can be writtern as

` do.Provide(config)

do.Provide(datasource)

do.Provide(db)`

it's much more clean

For API compatiable , we can use a new method Register

` do.Register(config)

do.Register(datasource)

do.Register(db)`

if it's ok, I would like raise a PR for this change
thank you!

[kcmvp]

Thanks for your suggestion, but I'm not sure we will add these shortcuts to the repository.

Using default injectors is basically a bad idea since it makes your code very hard to test. Adding 18 more helpers without the i *do.Injector argument seems excessive to me.

Anyway, you can use a simple alias in your projects:

// utils/do.go
func Provide(provider do.Provider[T]) {
    return do.Provide(nil, provider)
}

I keep this issue open for further discussion. If anybody wants to expose a PoV, please answer this issue!

Using default injectors is basically a bad idea since it makes your code very hard to test

Agree with you about this point. in fact I think about this issue. right now for me I just use the DefaultInjector directly for unit test. From the respect of API user he need to know the detail design of the library, if he want to write clean code.

another benefit of Method aliases is that we dont need to pass(or keep/define) the variable around in the project. I can get it just when I want.

might be a global function is better than a global variable.

change
var DefaultInjector = New()
to

func DefaultInjector() *Injector {}

[samber]

Closing, since it has been added to the project under do.DefaultInjector