Traducteur, des améliorations à proposer ?

Bonjour/Bonsoir à tous, aujourd'hui j'ai écris un petit traducteur français/espagnol pour entraîner mon python et j'aimerai obtenir des retours pour améliorer mon niveau, mon code fonctionne mais je pense qu'il n'est pas top, alors premièrement j'ai récupérer un .csv avec à l'intérieur une ligne du mot français et une autre du mot espagnol (voici le lien : https://www.transfernow.net/307wy9321fay) avec ce code :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

fichier = open("vocabulaire.csv", "r") # Ouverture du fichier vocabulaire.csv dico = fichier.read() #Lecture du fichier   dico = dico.replace('"',"") #Supprimer les apostrophes dico = dico.split("\n")   dico_trad = {}   for v in dico: mot = v.split(",") if mot == ['']: pass else: dico_trad[mot[0]] = mot[1]   del dico_trad["Français"]

---

et voici le code du traducteur :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25

from vocabulaire import dico_trad   score = 0 total = 0   while total < 20:   for k, v in dico_trad.items():   if total == 20: break   rep = input("Traduisez : {} ".format(k))   if dico_trad[k] == rep: print("Juste\n") score += 1 total += 1   else: print("Faux") print(k, "se traduit par", v, "\n") total += 1   print("Tu as obtenu une note de {} sur {}".format(score, total))

---

J'ai chercher pendant une heure pour pouvoir retourner un mot aléatoire du dictionnaire mais je n'ai pas trouvé.
Merci d'avance :)

Salut

Alors pour améliorer je dirais :
1) Ce n'est pas un traducteur, c'est plutot un programme faisant faire à l'utilisateur un exercice de traduction.
2) Nommez mieux vos variables :

Code:

---

1 2 3 4 5 6

for v in dico: mot = v.split(",") if mot == ['']: pass else: dico_trad[mot[0]] = mot[1]

---

Qu'est v ? Une correspondance ou traduction je suppose. Donc on lui en donne explicitement le nom.
mot, ce n'est qu'un seul mot ? Non. Donc il faut lui mettre un s. C'est relativement important, car en python on voit très souvent des choses de ce style :

Code:

---

for mot in mots :

---

mot est un string, mots est une liste de string,
3) Si mot ,(que vous devriez rebaptiser mots) n'est pas vide alors vous supposez qu'il est de longueur 2. Et si jamais il y a une ptite faute dans votre fichier csv, et que cette variable est de longueur 1 ou 3, il se passe quoi ? Une sécurité peut-être mise ici.
4) pour tirer au sort N éléments dans un dictionnaire, mettre au début du code :

Code:

---

import random

---

et remplacer

Code:

---

for k, v in dico_trad.items():

---

par

Code:

---

for k,v in random.sample(list(dico_trad.items()),N):

---

(en prenant soin de définir N avant)

Merci pour tes conseils je vais améliorer cela. Je suis actuellement entrain de voir la POO avec les classes et les méthodes, est-ce que dans ce petit programme j'aurai pu les utiliser ? Si oui, où et pourquoi ?

On peut toujours utiliser la POO. Après c'est pas forcément toujours utile, mais ceci dit si c'est pour t'exercer, un cas simple c'est très bien

Mais avant d'en arriver là, je te conseillerais de faire des fonctions déjà. Ca permetterait :
1) de donner plus de sémantique à ton code (car les nom de fonction on les choisi aussi judicieusement pour qu'on comprenne rapidemment ce que fait la fonction).
2) d'isoler les différentes tâches en blocs élémentaires. Si un jour tu souhaites changer la manière dont tu résoud une tâche précise, tu changes juste la fonction et pas tout ton code.
3) tu pourras ainsi regrouper tes 2 fichiers en 1 seul, car le découpage ici n'est pas forcément utile.
4) ce sera plus facile après pour mettre de la POO dessus car tes classes vont regrouper des attributs (des variables) et des méthodes (des fonctions de classes)