Why aren't the services static? about typescript-express-starter HOT 6 CLOSED

@AshotN

There is no particular reason. I think the way you speak is good. Thank you for your pull request.

from typescript-express-starter.

i think having non-static classes is much better out of the box for dependency injection of them and of other services/instances inside them without playing much around with IoC Containers

from typescript-express-starter.

Feel free to close this issue if you believe that there is no need to change anything.

from typescript-express-starter.

@ragrag

Thank you very much for the comment.
If so, is that open source well organized? Sorry. I speak English with a translator incorrectly.

from typescript-express-starter.

@ljlm0402
I would consider it very organized however there are a couple of suggestions i would like to propose.

1. Having a separate file responsible for bootstrapping the database connection/initialization. (and associations in case of sequelize)
   

2. Having a flag or an option in the CLI to structure the project by entities/components instead of technical roles as shown in the image:
   
   Please see: https://github.com/goldbergyoni/nodebestpractices/blob/master/sections/projectstructre/breakintcomponents.md

from typescript-express-starter.

@ragrag

Thanks for the good suggestions and detailed explanations.
It was very helpful. We will try to reflect it after review.

from typescript-express-starter.