-
Notifications
You must be signed in to change notification settings - Fork 12
Open
Labels
questionFurther information is requestedFurther information is requested
Description
Hello!
I saw a library called ditox which has almost same interface like your library do and several questions emerged:
- Isn't
.inSomeScope()
more error prone then ditox's{scope:"some"}
argument? I sometimes forgive to add these scope calls and only remember that when see warnings in console - Your approach for declaring injection gives people ability to declare it in place of Class declaration thus might lead to mix up DI and services. (this is already mentioned in DIP Violation #28). Ditox's approach with generating factories instead leans to group all DI-related declarations together.
- Your tag based conditional injection is a great idea! Do you think factory based instance generation is a little bit more flexible?
I wrote this issue not to clash you and @mnasyrov but to know your position on these design decisions. I already use your library in some projects and it might be better for others who find your or ditox to know which one fits their case best.
P.S. @mnasyrov @vovaspace don't you think it'll be great to have links to each other's projects to increase awareness of options?
Metadata
Metadata
Assignees
Labels
questionFurther information is requestedFurther information is requested