Discussion :

[DevilOS7]

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 :

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

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 :

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

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

[lg_53]

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 :

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

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 :

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

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 :

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

import random

et remplacer

Code :

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

for k, v in dico_trad.items():

par

Code :

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

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

(en prenant soin de définir N avant)

[DevilOS7]

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 ?

[lg_53]

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)