Discussion :

[vazer7070]

Bonjour à tous,

Voilà, pour une application que je code, je rentre lors de l'initialisation, des données dans un dictionnaire, puis en fonction de ce que je veux lui faire faire, je prend les informations et les affiches, mais le souci c'est que quand j'utilise une requête ça me retourne null mais quand je fais une autre requête qui fait pareil, ça me donne le bon résultat.

Je m'explique en illustrant avec mon code, dans ces exemples, ID est de la même valeur pour les 2méthodes et j'ai vérifié, le dictionnaire est correctement rempli (puisque pour le second exemple ça fonctionne)

Exemple retournant null:

Code :

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

1
2
3
4
5
public static string ReturnName(int ID)
        {
            return Dico.FirstOrDefault(x => x.Key == ID).Value.Name;
 
        }

Exemple retournant la valeur souhaitée:

Code :

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

1
2
3
4
5
6
public static string GetName(int ID)
        {
                if (Dico.TryGetValue(ID, out ItemeList item))
                    return item.Name;
            return ID.ToString();
        }

Alors, ça doit être moi qui loupe un truc, mais il me semble que les 2 méthodes retournent pareil.
Merci d'avance de votre aide et de vos réponses.

Cordialement
Vazer7070

[Prisson]

Bonjour,
a mon avis celle ci

Code :

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

1
2
3
4
5
public static string ReturnName(int ID)
        {
            return Dico.FirstOrDefault(x => x.Key == ID).Value.Name;
 
        }

peut-être ameliorée ainsi

Code :

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

1
2
3
4
5
public static string ReturnName(int ID)
        {
            return Dico[ID];
 
        }

[vazer7070]

Salut,

J'ai effectué ton optimisation, mais l'erreur se transforme en "la clé demandée est absente du dictionnaire", ce qui n'est pas possible puisque j'effectue un affichage du nombre de valeur dans le dictionnaire lors du lancement et que ce nombre est correct et que avec l'autre méthode(celle fonctionnant initialement), ça fonctionne sans problème.

[Pol63]

firstordefault retourne null si non trouvé
dans le 2ème code si le if ne trouve pas il y a ensuite return ID.ToString();
donc les 2 codes ne sont pas équivalents

sur un dictionary on ne balaye pas pour trouver une clé, containskey et trygetvalue et l'accès par [key] à une valeur sont plus performants

[vazer7070]

firstordefault retourne null si non trouvé
dans le 2ème code si le if ne trouve pas il y a ensuite return ID.ToString();
donc les 2 codes ne sont pas équivalents

sur un dictionary on ne balaye pas pour trouver une clé, containskey et trygetvalue et l'accès par [key] à une valeur sont plus performants

le second code ne retourne pas "ID.tostring()" puisqu'il trouve la valeur demandé c'est là le souci, si j'exécute ces 2 méthodes en même temps, sur 2 labels, avec les mêmes arguments, je n'aurais pas les mêmes résultats.

[Noxen]

Pour appuyer les propos de Pol63, si au lieu de ce code :

Code :

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

1
2
3
4
5
6
public static string GetName(int ID)
        {
                if (Dico.TryGetValue(ID, out ItemeList item))
                    return item.Name;
            return ID.ToString();
        }

tu as celui-ci :

Code :

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

1
2
3
4
5
6
public static string GetName(int ID)
        {
                if (Dico.TryGetValue(ID, out ItemeList item))
                    return item.Name;
                throw new Exception($"Key {ID} not found");
        }

est-ce que tu n'aurais pas ton erreur qui revient ?

[vazer7070]

Pour appuyer les propos de Pol63, si au lieu de ce code :

Code :

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

1
2
3
4
5
6
public static string GetName(int ID)
        {
                if (Dico.TryGetValue(ID, out ItemeList item))
                    return item.Name;
            return ID.ToString();
        }

tu as celui-ci :

Code :

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

1
2
3
4
5
6
public static string GetName(int ID)
        {
                if (Dico.TryGetValue(ID, out ItemeList item))
                    return item.Name;
                throw new Exception($"Key {ID} not found");
        }

est-ce que tu n'aurais pas ton erreur qui revient ?

Non, j'ai la valeur demandée

[Prisson]

Bonjour,
peux tu fournir la déclaration de la variable dico ?

[vazer7070]

Bonjour,
peux tu fournir la déclaration de la variable dico ?

Code :

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

public static Dictionary<int, ItemList> Dico = new Dictionary<int, ItemList>();

[Prisson]

c'est quoi ItemList ? ce lien https://stackoverflow.com/questions/...a-generic-list pourra peut-être t'eclairer