C#•2y ago

System.AggregateException

'Some services are not able to be constructed (Error while validating the service descriptor 'ServiceType: BaseApi.Repository.UserRepository Lifetime: Singleton ImplementationType: BaseApi.Repository.UserRepository': Unable to resolve service for type 'MongoDB.Driver.IMongoDatabase' while attempting to activate 'BaseApi.Repository.UserRepository'.)'

'Some services are not able to be constructed (Error while validating the service descriptor 'ServiceType: BaseApi.Repository.UserRepository Lifetime: Singleton ImplementationType: BaseApi.Repository.UserRepository': Unable to resolve service for type 'MongoDB.Driver.IMongoDatabase' while attempting to activate 'BaseApi.Repository.UserRepository'.)'

'Some services are not able to be constructed (Error while validating the service descriptor 'ServiceType: BaseApi.Repository.UserRepository Lifetime: Singleton ImplementationType: BaseApi.Repository.UserRepository': Unable to resolve service for type 'MongoDB.Driver.IMongoDatabase' while attempting to activate 'BaseApi.Repository.UserRepository'.)'

'Some services are not able to be constructed (Error while validating the service descriptor 'ServiceType: BaseApi.Repository.UserRepository Lifetime: Singleton ImplementationType: BaseApi.Repository.UserRepository': Unable to resolve service for type 'MongoDB.Driver.IMongoDatabase' while attempting to activate 'BaseApi.Repository.UserRepository'.)'

I'm trying to build an API with using MongoDB. Sorry for so much screenshots I just wanted to show you all my work and understand what to change and what are my mistakes. I'm kinda new at .net core

Pobiega•3/13/24, 10:27 AM

Your UserRepositoryUserRepository class request a IMongoDatabaseIMongoDatabase from the service provider

Pobiega•3/13/24, 10:27 AM

but you have never registered any IMongoDatabaseIMongoDatabase type

only you knowOP•3/13/24, 10:29 AM

so what should I do

Pobiega•3/13/24, 10:30 AM

you must register IMongoDatabase somehow. Where is that interface from?

only you knowOP•3/13/24, 10:31 AM

it has his own interface

only you knowOP•3/13/24, 10:31 AM

comes with mongodb.driver

only you knowOP•3/13/24, 10:31 AM

Pobiega•3/13/24, 10:32 AM

okay, and how does the documentation suggest you get an instance of this?

Pobiega•3/13/24, 10:33 AM

ah I see, thats whats happening in your MongoDbMongoDb class

only you knowOP•3/13/24, 10:34 AM

what about this it might be wrong

Pobiega•3/13/24, 10:34 AM

that is the class you should be requesting, not IMongoDatabaseIMongoDatabase, and to do that you must register MongoDbMongoDb

only you knowOP•3/13/24, 10:34 AM

then I should change here right

Pobiega•3/13/24, 10:35 AM

Do you understand how dependency injection works?

Pobiega•3/13/24, 10:35 AM

yes

Pobiega•3/13/24, 10:35 AM

like, that code makes no sense at all tbh

Pobiega•3/13/24, 10:35 AM

you're injecting a database connection, but also creating a new one?

Pobiega•3/13/24, 10:35 AM

and the one you created is then ignored

PPobiega like, that code makes no sense at all tbh

only you knowOP•3/13/24, 10:36 AM

that code meaning constructor or whole repo class?

Pobiega•3/13/24, 10:36 AM

that constructor

only you knowOP•3/13/24, 10:36 AM

I should use MongoDB class instead of IMongoDatabase right

Pobiega•3/13/24, 10:37 AM

the first two lines create a new IMongoDatabaseIMongoDatabase instance, but you are injecting one. And locally creating the connection inside your repo is almost guaranteed to be a very bad idea

Pobiega•3/13/24, 10:37 AM

also, your repo is a singleton

Pobiega•3/13/24, 10:37 AM

a singleton with a database connection.... sounds like a recipe for trouble

PPobiega also, your repo is a singleton

only you knowOP•3/13/24, 10:37 AM

I should make it scope

PPobiega the first two lines create a new `IMongoDatabase` instance, but you are injectin...

only you knowOP•3/13/24, 10:39 AM

so I gotta call MongoDB directly because in that class I already created new client

Pobiega•3/13/24, 10:39 AM

do you know the difference between singleton, scoped and transient?

PPobiega do you know the difference between singleton, scoped and transient?

only you knowOP•3/13/24, 10:39 AM

yes I think

Oonly you know so I gotta call MongoDB directly because in that class I already created new cli...

Pobiega•3/13/24, 10:39 AM

kinda? I'm not sure what "call" and "directly" are doing in that sentence thou

Pobiega•3/13/24, 10:39 AM

you dont call classes

PPobiega kinda? I'm not sure what "call" and "directly" are doing in that sentence thou

only you knowOP•3/13/24, 10:40 AM

some english problems sry about that

only you knowOP•3/13/24, 10:44 AM

so what should I do now

only you knowOP•3/13/24, 10:45 AM

I changed the MongoDB class as MongoDBModel

Pobiega•3/13/24, 10:45 AM

have you registered it with your service collection?

Pobiega•3/13/24, 10:46 AM

also, you can remove the first two lines in the constructor, and the second parameter

PPobiega have you registered it with your service collection?

only you knowOP•3/13/24, 10:47 AM

you mean this?

PPobiega also, you can remove the first two lines in the constructor, and the second para...

only you knowOP•3/13/24, 10:47 AM

okay

Pobiega•3/13/24, 10:47 AM

yes

Pobiega•3/13/24, 10:48 AM

builder.Servicesbuilder.Services is your service collection. It needs to contain everything that gets resolved in your application

only you knowOP•3/13/24, 10:49 AM

is it still missing parts? it contains my repo and mongodbsettings and I have the connection string at appsettings.json

Pobiega•3/13/24, 10:50 AM

it doesnt contain MonogDbModelMonogDbModel

only you knowOP•3/13/24, 10:51 AM

should I add db model as scope?

Pobiega•3/13/24, 10:51 AM

yes.

only you knowOP•3/13/24, 10:52 AM

I guess there are still missing somethings.

System.AggregateException: 'Some services are not able to be constructed (Error while validating the service descriptor 'ServiceType: BaseApi.Repository.UserRepository Lifetime: Scoped ImplementationType: BaseApi.Repository.UserRepository': Unable to resolve service for type 'System.String' while attempting to activate 'BaseApi.MongoDB.MongoDBModel'.) (Error while validating the service descriptor 'ServiceType: BaseApi.MongoDB.MongoDBModel Lifetime: Scoped ImplementationType: BaseApi.MongoDB.MongoDBModel': Unable to resolve service for type 'System.String' while attempting to activate 'BaseApi.MongoDB.MongoDBModel'.)'

System.AggregateException: 'Some services are not able to be constructed (Error while validating the service descriptor 'ServiceType: BaseApi.Repository.UserRepository Lifetime: Scoped ImplementationType: BaseApi.Repository.UserRepository': Unable to resolve service for type 'System.String' while attempting to activate 'BaseApi.MongoDB.MongoDBModel'.) (Error while validating the service descriptor 'ServiceType: BaseApi.MongoDB.MongoDBModel Lifetime: Scoped ImplementationType: BaseApi.MongoDB.MongoDBModel': Unable to resolve service for type 'System.String' while attempting to activate 'BaseApi.MongoDB.MongoDBModel'.)'

Pobiega•3/13/24, 10:52 AM

BaseApi.Repository.UserRepository': Unable to resolve service for type 'System.String' while attempting to activate 'BaseApi.MongoDB.MongoDBModel'

Pobiega•3/13/24, 10:53 AM

the constructor for your MongoDbModel takes in two strings

Pobiega•3/13/24, 10:53 AM

change that to take in the db connection options

Pobiega•3/13/24, 10:53 AM

and use those values instead

only you knowOP•3/13/24, 10:57 AM

oh I should change the constructor like this

Pobiega•3/13/24, 10:58 AM

are you serious?

only you knowOP•3/13/24, 10:58 AM

wdym