Dictionary retournant null

Version imprimable

03/08/2018, 20h05
vazer7070

Dictionary retournant null
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:

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:

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

Bonjour,
a mon avis celle ci

Code:

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:

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

04/08/2018, 23h59
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.
06/08/2018, 07h49
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

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

Code:

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:

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 ?

06/08/2018, 10h13
vazer7070

Citation:

Envoyé par 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

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.

Citation:

Envoyé par Noxen

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

Code:

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:

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

06/08/2018, 11h23
Prisson

Bonjour,
peux tu fournir la déclaration de la variable dico ?
06/08/2018, 12h36
vazer7070

Citation:

Envoyé par Prisson

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

Code:

public static Dictionary<int, ItemList> Dico = new Dictionary<int, ItemList>();
06/08/2018, 13h45
Prisson

c'est quoi ItemList ? ce lien https://stackoverflow.com/questions/...a-generic-list pourra peut-être t'eclairer
06/08/2018, 15h02
Noxen

L'histoire de ne dit pas ce qu'est ItemList, mais je ne vois pas pourquoi ce ne serait pas possible. Tu pourrais même avoir un dictionnaire contenant des dictionnaires (ex : une liste de modules avec pour chacun une liste de paramètres de configuration).
06/08/2018, 15h58
vazer7070

ben 'itemlist' est une classe contenant un string se nommant 'name'
06/08/2018, 16h33
WaterTwelve21

Bonjour ,

Je n'arrive pas vraiment à cerner le problème, que l'on utilise ItemList ou n'importe elle autre classe, c'est le fonctionnement du dictionnaire qui importe.

Celui ci est simple, il stocke des couples clé-valeur.

Pour récuper la valeur à partir d'une clé , on peut faire dico[key] mais attention en cas de clé inconnue il y aura une exception.

Le plus prudent est donc d'utiliser la méthode TryGetValue cité précédemment par Noxen. La valeur est null si la clé n'existe pas dans le dictionnaire.

Bon code,
06/08/2018, 17h06
vazer7070

Salut,

En soi le souci est simple:

Comme j'ai dit plus haut, si je cherche une valeur de façon simultané avec un trygetvalue et un dico[id] avec des arguments similaire, sachant que le dictionnaire est correctement remplie (sinon le trygetvalue ne fonctionne pas on est d'accord), ben je n'obtiens pas la même valeur.

Le trygetvalue me donne la valeur voulue, mais le dico[id] retourne null hors dans les 2 cas id est la même chose.
06/08/2018, 17h36
WaterTwelve21

Quel est le code de ces deux dites méthodes maintenant ? Toujours celles des premiers posts ? Comme l'a dit Pol63 , elle ne font pas la même chose.
06/08/2018, 17h50
Pol63

ca commence de plus en plus à sentir l'embrouille, montre nous un code simple qui reproduis le problème
parce que là j'ai plus envie d'imaginer que tu as mis null en value et que tu le modifies entre les 2 tests

là tu nous parles de "
dico[id] qui retourne null", alors qu'il n'était pas question de dico[id] au début, et que si ca retournes null c'est bien que ca existe dans le dictionary sinon ca throw
07/08/2018, 16h58
Momoth
Citation:

Envoyé par Pol63

ca commence de plus en plus à sentir l'embrouille, montre nous un code simple qui reproduis le problème
parce que là j'ai plus envie d'imaginer que tu as mis null en value et que tu le modifies entre les 2 tests

Je suis du même avis.

D'autant plus que si la clé n'était pas trouvée, ce code ci ne te retournerai pas null mais lèverai une exception :
Code:

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