Discussion :

[vinceaspi]

Bonjour,

J'ai créé un petit programme (c'est mon premier) pour faire deviner un nombre mystère tiré au sort aléatoirement.
Il fonctionne.

Maintenant, je voudrais évidemment y ajouter des options supplémentaire. Je souhaiterais que l'utilisateur puisse choisir la plage dans laquelle le nombre est choisi aléatoirement.
Pour faire quelque chose de propre, je souhaite utiliser des fonctions.
J'ai donc créé deux fonctions qui devraient choisir le nombre aléatoire entre une fourchette et une autre.

Ensuite, j'utilise les conditions pour exécuter la fonction correspond au choix.

Eh bien, ça ne fonctionne pas. Mon erreur doit être vraiment ridicule, mais mon niveau de débutant ne me permet pas d'aller plus loin.
Si vous voyez mon erreur, je vous serais reconnaissant d'avoir un petit coup de pouce...

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
#coding:utf-8
 
#Salutations
#\n permet de passer à la ligne.
user = input("Bonjour ! Quel est ton prénom ?\n > ")
user = str(user)
print("Bonjour {}, ravi de faire ta connaissance !\nNous allons jouer au nombre mystère.\nJe vais choisir un nombre entre 1 et 100.\nFais-moi ensuite tes propositions.\nJe te dirai si le nomnre est plus grand ou plus petit".format(user))
 
def choixa():
  from random import randint
  randint(1, 10)
  import random
  nbr = random.randint(1, 10)
  nbr = int(nbr)
 
def choixb():
  from random import randint
  randint(1, 100)
  import random
  nbr = random.randint(1, 100)
  nbr = int(nbr)
 
choix = input("Tapez 1 pour utiliser un nombre entre 1 et 10.\nTapez 2 pour utiliser un nombre entre 1 et 100\n> ")
choix=str(choix)
 
if choix == 1:
  choixa()
 
elif choix == 2:
  choixb()
 
#Boucle pour donner les indications à l'utilisateur
#while rep == nbr:
 
rep = 0
rep = int(rep)
 
#Tant que réponse est différente du nombre mystère
while rep != nbr:
  rep = input("Selon vous, quel est le nombre mystère ? \n> ")    
  rep = int(rep)
 
  if rep > nbr:
    print("Le nombre mystère est plus petit")
  elif rep < nbr:
    print("Le nombre mystère est plus grand")
  else:
    print("Oui ! Le nombre mystère était effectivement : ", nbr)

[VinsS]

Salut,

Tes fonctions doivent retourner le nombre choisit

Code :

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

1
2
3
4
5
6
 
if choix == 1:
    nbr = choixa()
 
elif choix == 2:
    nbr = choixb()

Quelques remarques:

1. user = str(user) est inutile, user est déjà de type string.
2. rep = int(rep) est inutile, rep est déjà de type integer.
3. On met les import en début de programme pas dans les fonctions.
4. Tes fonctions sont trop compliquées, ceci suffit:

Code :

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

1
2
3
4
 
import random
def choixa():
    return random.randint(1, 10)

[vinceaspi]

D'abord, merci pour votre aide !
Vos explications sont très claires et m'aident à comprendre mes erreurs et la logique.

J'ai modifié le code en tenant compte de vos conseils. Cela fonctionne jusqu'à la boucle. Là, on me dit que la variable nbr n'est pas définie.

Je me dis donc que malgré tout les fonctions ne font pas leur jod ou alors les conditions ne sont pas prises en compte. Ou alors il y encore quelque chose qui m'échappe.

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
26
27
28
29
30
31
32
33
34
35
36
37
38
#coding:utf-8
 
#Salutations
#\n permet de passer à la ligne.
user = input("Bonjour ! Quel est ton prénom ?\n > ")
print("Bonjour {}, ravi de faire ta connaissance !\nNous allons jouer au nombre mystère.\nJe vais choisir un nombre entre 1 et 100.\nFais-moi ensuite tes propositions.\nJe te dirai si le nomnre est plus grand ou plus petit".format(user))
 
import random
 
def choixa():
  return random.randint(1, 10)
 
def choixb():
  return random.randint(1, 100)
 
 
choix = input("Tapez 1 pour utiliser un nombre entre 1 et 10.\nTapez 2 pour utiliser un nombre entre 1 et 100\n> ")
choix=str(choix)
 
if choix == 1:
  nbr = choixa()
 
elif choix == 2:
  nbr = choixb()
 
rep = 0
 
#Tant que réponse est différente du nombre mystère
while rep != nbr:
  rep = input("Selon vous, quel est le nombre mystère ? \n> ")    
  rep = int(rep)
 
  if rep > nbr:
    print("Le nombre mystère est plus petit")
  elif rep < nbr:
    print("Le nombre mystère est plus grand")
  else:
    print("Oui ! Le nombre mystère était effectivement : ", nbr)

[VinsS]

choix=str(choix) --> choix = int(choix)

[vinceaspi]

choix=str(choix) --> choix = int(choix)

Damn ! C'était tellement évident !
Je vous remercie pour votre patience. Ce programme est inutile, mais me permet d'apprendre. Grâce à vous, j'ai évité la phase de découragement.

[Sve@r]

Bonjour

Code python :

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

1
2
3
4
5
def choixa():
  return random.randint(1, 10)
 
def choixb():
  return random.randint(1, 100)

Pour éviter d'avoir à écrire plusieurs fonctions similaires (parce que choisir entre 1 et 10 ou entre 1 et 100 franchement, c'est kifkif et que feras-tu s'il faut choisir ensuite entre 1 et 1000 ou même entre 10 et 15000 ????) on utilise ce qu'on appelle "paramètre de fonction" => un élément venu de l'extérieur lui permettant de "paramétrer" son travail

Code python :

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

1
2
def choixN(n):
  return random.randint(1, n)

Ici la valeur "n" (10 ou 100 ou autre...) sera passée par l'appelant. Remarque, c'est déjà ce qui se passe quand tu appelles "randint()" avec 1 et 10 ou avec 1 et 100.

Code python :

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

1
2
3
4
5
if choix == 1:
  nbr = choixa()
 
elif choix == 2:
  nbr = choixb()

Code python :

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

1
2
3
4
5
if choix == 1:
  nbr = choixN(10)
 
elif choix == 2:
  nbr = choixN(100)

Ou alors plus directement nbr=choixN(10**choix) mais ça implique une relation mathématique entre le choix et le nombre (10^choix) pas forcément obligatoire. D'autant plus qu'il y a ensuite plein d'outils permettant de faire la même chose sans passer par ces "if". Tu verras, Python est un langage bien sympa. Ceci dit, encapsuler randint (qui est déjà une fonction directement utilisable) dans une fonction "choixa" ou "choixb" (ou même "choixN") qui ne lui apporte aucune plus-value...

Grâce à vous, j'ai évité la phase de découragement.

Propos étranges de la part d'un enseignant qui justement de par son métier, devrait avoir l'habitude de ces premiers écueils inévitables et surtout relativement mineurs !!! Un peu de concentration, de réflexion et on en vient vite à bout.

[vinceaspi]

Propos étranges de la part d'un enseignant qui justement de par son métier, devrait avoir l'habitude de ces premiers écueils inévitables et surtout relativement mineurs !!! Un peu de concentration, de réflexion et on en vient vite à bout.

Je vous remercie pour votre proposition d’amélioration. C’est effectivement plus propre. J’ai procédé au changement.

Quant au découragement, ma fonction (restons dans le monde du codage) n’empêche pas parfois quelques moments d’égarement : cette période est particulièrement difficile pour toutes les professions et je pourrais reléguer cette auto-formation en second plan pour assumer tout le reste (je dois mettre mon école à jour pour les cours à distance, le matériel, la formation des enseignants, etc). Donc oui, je ne suis pas à l’abri parfois du découragement, mais ce n’est que temporaire.

[vinceaspi]

Je vous présente mon programme abouti grâce à votre aide. J'ai ajouté une boucle pour recommencer la partie.

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
#coding:utf-8
 
#Salutations
#\n permet de passer à la ligne.
user = input("Bonjour ! Quel est ton prénom ?\n\n > ")
print("Bonjour {}, ravi de faire ta connaissance !\n\nNous allons jouer au nombre mystère.\n\nJe vais choisir un nombre entre 1 et 100.\n\nFais-moi ensuite tes propositions.\n\nJe vous dirai si le nombre est plus grand ou plus petit\n\n".format(user))
 
import random
 
#Tirage au sort
def choixN(n):
  return random.randint(1, n)
 
#Balise pour recommencer la partie
recommencer = True
recommencer = bool(recommencer)
 
#Boucle qui va permettre de recommencer la partie
while recommencer == True:
 
  choix = input("Tapez 1 pour utiliser un nombre entre 1 et 10.\nTapez 2 pour utiliser un nombre entre 1 et 100.\nTapez 3 pour utiliser un nombre entre 1 et 1000.\n\n> ")
  choix=int(choix)
 
  if choix == 1:
    nbr = choixN(10)
 
  elif choix == 2:
    nbr = choixN(100)
 
  elif choix == 3:
    nbr = choixN(1000)
 
  rep = 0
 
  #Tant que réponse est différente du nombre mystère
  while rep != nbr:
    rep = input("Selon vous, quel est le nombre mystère ? \n\n> ")    
    rep = int(rep)
 
    if rep > nbr:
      print("Le nombre mystère est plus petit\n\n")
    elif rep < nbr:
      print("Le nombre mystère est plus grand\n\n")
    else:
      print("Oui ! Le nombre mystère était effectivement : ", nbr)
      recommencer = input("Voulez-vous recommencer une partie ? (o / n)")
      if recommencer == "o" or recommencer == "O":
        recommencer = True
      else :
        recommencer = False
        print("Merci d'avoir joué {}, à bientôt !".format(user))