Discussion :

[julienr04]

Bonjour tout le monde !
J'ai écris une méthode pour faire une recherche dans une MongoDB et qui renvoie des documents en fonction du filtre.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
public static async Task<BsonDocument> FindWithFilters(string connectionstring,string database,string Collection, string clé,string valeur)
        {
            try
            {
                var client = new MongoClient(connectionstring);
                var db = client.GetDatabase(database);
                var col = db.GetCollection<BsonDocument>(Collection);
                var filter = new BsonDocument(clé, valeur);
                var list = await col.Find(filter)                
                    .ToListAsync();
                foreach(var doc in list)
                {
                    return doc;
 
                }
 
            }
            catch(Exception e)
            {
                Console.WriteLine(e);
                return new BsonDocument("Error", "Echec");
 
            }
        }

Ma méthode me semble correcte, et pourtant, visual studio me dit que tout les chemins de code ne retournent pas une valeur. Et je ne comprend pas trop pourquoi ... J'ai pourtant veiller à avoir un document dans les deux cas du trycatch ..

Merci d'avance pour votre aide

[Noxen]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
foreach(var doc in list)
                {
                    return doc;
 
                }

Pourquoi un foreach puisque seul le premier document sera renvoyé ? Et s'il n'y a pas de document dans la liste, effectivement il n'y aura pas d'action de retour et le compilateur te le fait bien remarquer. Tu peux faire un return list[0]; qui lèvera une OutOfBoundException s'il n'y a pas de document, return list.First(); qui lèvera une InvalidOperationException s'il n'y a pas de document ou return list.FirstOrDefault(); qui renverra null s'il n'y a pas de document (mais dans ce cas le code appelant devra gérer le null).

[julienr04]

Un foreach parce que en fonction du filtre, un ou plusieurs documents peuvent être renvoyés Je vais essayer tes propositions, merci beaucoup pour la réponse !

[Noxen]

Si tu peux avoir plusieurs documents de renvoyés alors la valeur de retour de ta méthode doit être Task<IEnumerable<BsonDocument>>. Et tu devras toi-même renvoyer un objet qui implémente IEnumerable<BsonDocument>, comme un tableau de BsonDocument, ou une liste de de BsonDocument (List<BsonDocument>). Tu devras par contre réfléchir à ce que tu fais en cas d'erreur. Tu peux renvoyer une liste vide (Enumerable.Empty<BsonDocument>()), mais dans ce cas tu ne sauras pas si tu as une liste vide parce-que tu n'as pas de document ou bien parce-qu'une erreur s'est produite. Ou tu laisse remonter l'erreur, qu'il te faudra alors gérer dans le code appelant.

[julienr04]

Moi qui croyais cette méthode toute simple ^^ Merci en tout cas pour tout les conseils ...

[Noxen]

Le cœur de ta fonction n'est pas très compliqué :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
public static async Task<IEnumerable<BsonDocument>> FindWithFilters(string connectionstring,string database,string Collection, string clé,string valeur) =>
    await new MongoClient(connectionstring)
        .GetDatabase(database)
        .GetCollection<BsonDocument>(Collection)
        .Find(new BsonDocument(clé, valeur))
        .ToListAsync()

Le reste c'est de la gestion d'erreur.