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

[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]

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.

[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

[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).

[vazer7070]

ben 'itemlist' est une classe contenant un string se nommant 'name'

[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,

[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.

[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.

[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

[Momoth]

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 :

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;
 
        }