Discussion :

[moithibault]

Bonjour , j'ai une question qui me turlupine .
La voici : Si c'est le cas , quel choix est le meilleur ?

Code :

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

1
2
3
4
5
6
7
8
9
#exemple bidon - je ne met pas  l'instanciation et les déclarations
 
#DANS LE PROGRAMME PRINCIPAL
x=input("Entrez votre choix :")
if x< 5 :
	print "Erreur"
else :
	plateau.jouer(x)
	print "Bon"

avec

Code :

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

1
2
3
#DANS LA METHODE
def jouer(self,x):
	self.x=x

ou

Code :

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

1
2
#DANS LE PROGRAMME PRINCIPAL
plateau.jouer()

avec

Code :

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

1
2
3
4
5
6
7
8
#DANS LA METHODE
def jouer(self):
	x=input("Entrez votre choix :")
	if x< 5 :
		print "Erreur"
	else :
		self.x=x
		print "Bon"

Merci beaucoup !

[kango]

bonsoir,

vu ce qui est présenté j'aurais tendance à préférer la seconde méthode, qui encapsule plus de choses. par conséquent cela rend l'appel à plateau.jouer moins embêtant dans le programme principal.

[moithibault]

Ok , merci beaucoup!

[kango]

sur des questions aussi ouvertes, je te conseille d'attendre d'autres avis.

par exemple certains pourraient être favorable à la séparation des pouvoirs. je m'explique. Imagine que la classe plateau puisse prendre ses informations ailleurs qu'au clavier, dans une interface par exemple. Il n'est pas judicieux que ce soit la classe qui gère la mécanique du jeu qui soit chargée de la saisie

[wiztricks]

Salut,
C'est pas une question de méthode mais de séparation des points de vue entre l'IHM et les actions où:
IHM = { saisie, vérification de la chaine de caractères reçue, conversion }
actions = les fonctions/méthodes appelées en fonction de ce qui a été lu.

Entre les deux, on a une boucle qui fait la glue, style:

Code :

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

1
2
3
4
while True:
   choix = saisie_choix()
   if choix is None: break
   jouer(choix)

L'intérêt étant de pouvoir construire indépendamment les fonctions "saisie_choix" et "jouer".
- W
PS: Le code suivant ne fait pas ce que vous voulez.

Code :

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

1
2
3
x=input("Entrez votre choix :")
if x< 5 :
	print "Erreur"

x étant des caractères, le comparer à un entier sera "constant" (toujours vrai ou toujours faux).... mais vous enverrez n'importe quoi à "jouer"

[afranck64]

Bonsoir,

J'opte également pour la version 2 du code, bien que ça puisse ressembler à de la paresse de la part du codeur.

ps:

Code :

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

1
2
3
4
5
6
7
8
>>> lst = input("Entrez une valeur :==> ")
Entrez une valeur :==> range(10)
>>> n = input("Entrez en une autre :==> ")
Entrez en une autre :==> 5
>>> print lst
[0, 1, 2, 3, 4, 5, 6, 7, 8, 9]
>>> print n
5

Il ne faut confondre le "input" au "raw_input".
++

[wiztricks]

Il ne faut confondre le "input" au "raw_input".

Ooops, merci pour la remarque.
- W