Discussion :

[caramon _majere]

Bonjour,

débutant en python, je cherche à faire une chose simple (je pense), dans un dictionnaire, créer des clés et incrémenter la valeur de 1.

J'ai fait ce bout de code simple

Code :

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

1
2
3
4
5
6
7
dico = {}
my_new_key = 'cle1'
 
dico[my_new_key] += 1
dico[my_new_key] += 1
 
print(dico)

J'attends comme résultat

{'cle1': 2}

Mais à la place j'ai :

dico[my_new_key] += 1
KeyError: 'cle1'

C'est quoi le truc simple que j'ai raté ?

[papajoker]

bonjour

oui, tu essayes d'incrémenter un élément qui n'existe pas encore, bien sûr on ne peut modifier qu'une chose qui existe

dico[my_new_key] += 1 est en fait : dico[my_new_key] = dico[my_new_key] + 1

Code :

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

1
2
3
4
5
6
7
8
9
10
my_new_key = 'cle1'
dico = {my_new_key: 1}
dico[my_new_key] += 1
 
 
dico = {}
my_new_key = 'cle1'
dico[my_new_key] = 1    # ici c'est la création d'une clé
dico[my_new_key] += 1   # ici uniquement modification d'une clé (qui existe)
print(dico)

[Sve@r]

Bonjour

J'attends comme résultat {'cle1': 2}

Cela signifie donc qu'avant l'incrémentation, tu avais {'cle1': 0}. As-tu écrit quelque part l'instruction qui fait en sorte que ce soit le cas ???

C'est quoi le truc simple que j'ai raté ?

Ladite instruction.

[caramon _majere]

Merci pour vos retour.

J'avais regardé des exemples avant et qui me semblait assez identique à ce qu je faisais en perl.

Code Perl :

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

1
2
3
4
5
6
my %hash;
 
my $var = 'cle1';
 
$hash{$var} += 1;
$hash{$var} += 1;

[Sve@r]

J'avais regardé des exemples avant et qui me semblait assez identique à ce qu je faisais en perl.

En effet, c'est identique.
Mais peut-être que sous perl (que je ne connais pas) les tableaux associatifs sont initialisés par défaut à 0 (ce qui semble être visible quand on rajoute un print $hash{$var} + "\n"; avant l'incrémentation).

Donc tu as pris un code perl qui utilisait une caractéristique spécifiques de ce langage ; et tu es parti dans l'hypothèse que sous Python cette caractéristique était aussi présente. Malheureusement (ou heureusement, ça dépend de comment on voit les choses car personnellement je ne vois pas pourquoi il faudrait obligatoirement que Python perde du temps à mettre un truc à 0 si je n'en ai pas l'utilité) ce n'est pas le cas.

[papajoker]

Tu peux voir la clé d'un dictionnaire comme un index d'un tableau (liste, tuple python)

Code :

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

1
2
3
4
tableau = [1,2,3]
print(tableau[8])  # BEEP
tableau[8] +=1    # BEEP
tableau[8] =tableau[8] + 1    # BEEP

Il me semble plus que logique d'avoir une erreur "clé/index non trouvée" qu'un language qui returne toujours une valeur par défaut; en tout cas, c'est le choix de python
Et nous avons donc la même logique entre tous conteneurs (dictionnaire, liste, tuple, ...)

Par contre, avec un dictionnaire, il est possible d'utiliser la méthode get() qui ne retourne pas une erreur

Code :

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

1
2
3
4
5
6
7
dico = {"un": 1, "deux": 2}
print(dico.get("trois"))
print(dico.get("trois", "non defini"))   # si existe pas, retourne un "truc" par défaut
try:
    print(dico["trois"])
except KeyError:
    print("oops, clé 'trois' n'existe pas")

---------

si je reprends ton exemple, cela pourrait être

Code :

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

1
2
3
4
5
6
7
dico = {"un":1, "deux":2}
my_new_key = 'cle1'
if not my_new_key in dico:
    dico[my_new_key] = 0
dico[my_new_key] += 1
dico[my_new_key] += 1
print(dico)

[Sve@r]

Code python :

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

1
2
if not my_new_key in dico:
    dico[my_new_key] = 0

dico.setdefault(my_new_key, 0)

[caramon _majere]

Merci pour toutes vos réponses.

Comme l'idée est plus tard, dans une boucle, voir le nombres d'éléments identiques, en faisant un mixte de vos réponses, ca donne ca en version la plus courte.

Code :

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

1
2
3
4
5
6
7
 
dico = {}    
my_new_key = 'cle1'   
dico.setdefault(my_new_key, 0)     
dico[my_new_key] += 1
dico[my_new_key] += 1
print(dico)

[Sve@r]

en version la plus courte.

Ce n'est pas la version la plus courte. setdefault est un outil permettant de tester et affecter si la clef n'existe pas (un raccourci de la version de papajoker). Mais si tu gères ton dico depuis le début, alors tu n'as pas besoin de setdefault

Code :

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

1
2
3
4
5
my_new_key = 'cle1'   
dico = {my_new_key : 0}
dico[my_new_key] += 1
dico[my_new_key] += 1
print(dico)

[papajoker]

Comme ici, il a parlé de boucle, je vois un cas:
compter le nombre de fichiers par extention dans un répertoire, donc on ne connais pas la clé au début
Ici, le test me semble plus pertinant qu'un setdefault (que je ne connaissais pas, donc une bonne chose pour moi hier)

Code :

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

1
2
3
4
fichiers = {}
for fichier in Path('.').glob('*.*'):
   extension = fichier.suffix[1::]
   fichiers[extension] +=1   # oops

[wiztricks]

Comme ici, il a parlé de boucle, je vois un cas:
compter le nombre de fichiers par extention dans un répertoire, donc on ne connais pas la clé au début

Dans ce cas, il est préférable de sous/classer un dictionnaire pour y ajouter/définir la méthode __missing__ ou mieux utiliser collections.defaultdict.

Le résultat sera le même mais ça montre qu'on a réfléchi avant de coder pour trouver l'objet ad hoc qui traduira correctement l'intention de départ (et l'initialisation à 0):

Code :

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

1
2
3
4
5
6
7
8
9
>>> from collections import defaultdict
>>> dd = defaultdict(int)
>>> dd['a']
0
>>> dd['b'] += 1
>>> dd
defaultdict(<class 'int'>, {'a': 0, 'b': 1})
 
- W