Discussion :

[hulkyoann]

bonjour tout le monde!
voilà j'ai un probleme, je n'arrive pas à changer la valeur d'une variable dans mon jeux. Je met le 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
18
19
20
21
22
23
24
 
clairiere1 = {'piege' : piege1, 'nom' :"Maison du Chaperon rouge", 'choix chemin': [clairiere3, clairiere2]}
clairiere2 = {'piege' : piege2, 'nom' : "La clairiere de la joie", 'choix chemin': [clairiere1, clairiere3]}
clairiere3 = {'piege' : piege3, 'nom' : "La clairiere du Loup-garou", choix chemin': [clairiere1, clairiere2]}
 
 
 
def choix_chemin(position_joueur):
 
 
    if position_joueur.has_key('choix_chemin')==True:
        if len (position_joueur['choix_chemin'])==1 :
            print "tapez ","1","pour allez à ", position_joueur['choix_chemin'][0]['nom']
        else :    
            for i in range(len(position_joueur['choix_chemin'])) :
                print "tapez ",i,"pour allez à ", position_joueur['choix_chemin'][i]['nom']
        c=input ("Quel est votre choix ?")
    print 'nouvelle clairiere', position_joueur['choix_chemin'][c]['nom']
    print 'ancienne ',position_joueur['nom']
    return position_joueur['choix_chemin'][c]
def deplacer_perso(position_joueur) :
 
    position_joueur=choix_chemin(position_joueur)
    return

la variable que je veux changer c'est position_joueur et c'est un dico.en fait position_joueur est tout d'abords dans la clairiere1, donc position_joueur=clairiere1. les dico clairiere ont une clé 'choix chemin' avec comme valeur une liste.c'est dans cette liste que le joueur va choisir sa prochaine clairiere, sauf que quand il choisi, position joueur reste toujours =clairiere1!je n'arrive pas à changer la variable!
Alors siouplait, aidez moi, j'en ai besion, c'est pour l'école et pour mardi!
Merci et bonne aprem

[oiffrig]

C'est un problème de référence, tu réaffectes une valeur à ta variable position_joueur dans le contexte local (la fonction deplacer_perso) sans préciser que tu veux agir sur la variable globale, donc soit tu laisses tomber la fonction deplacer_perso et tu affectes dans le contexte global (là où tu appellerais la fonction), soit tu précises à l'interpréteur que tu veux utiliser la variable globale position_joueur (mot-clé global), auquel cas le return est inutile, ainsi que le passage en argument de l'ancienne position, vu que c'est une variable globale.

[hulkyoann]

j'ai déjà essayer avec 'global', mais ça marchait pas non plus.

auquel cas le return est inutile, ainsi que le passage en argument de l'ancienne position, vu que c'est une variable globale

ça par contre je comprend pas ce que tu veux dire.

j'avais essayer d'affecter directement position_joueur dans la fonction, mais le shell me disait un truc du genre "réference before assignement".

[oiffrig]

j'avais essayer d'affecter directement position_joueur dans la fonction, mais le shell me disait un truc du genre "réference before assignement".

Définis la variable globale avant (en lui donnant une valeur de départ)

Concernant ton essai avec le mot-clé global, montre ton code, personne ne pourra te donner de réponse sinon.

[alexdevl]

Bonjour,

Ce que je vois :
Tu définis clairiere1 avec clairiere2 et qui n'existent pas encore :

Code :

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

clairiere1 = {'piege' : piege1, 'nom' :"Maison du Chaperon rouge", 'choix chemin': [clairiere3, clairiere2]}

=> Différencier le nom du lieu de l'objet

il manque un ' avant choix chemin

Code :

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

clairiere3 = {'piege' : piege3, 'nom' : "La clairiere du Loup-garou", choix chemin': [clairiere1, clairiere2]}

Tu peux résoudre le problème d'accès à tes variables en utilisant une classe où les variables deviennent disponibles de partout comme self.position_joueur et self.lieu dans l'exemple suivant :
=> plus besoin de les passer en paramètre

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
#!/usr/bin/env python
# -*- coding: iso-8859-1 -*-
 
class Jeux():
 
    def __init__(self):
        self.lieu={}
        self.lieu["clairiere1"] = {'piege' : "piege1", 'nom' :"Maison du Chaperon rouge", 'choix_chemin': ["clairiere3", "clairiere2"]}
        self.lieu["clairiere2"] = {'piege' : "piege2", 'nom' : "La clairiere de la joie", 'choix_chemin': ["clairiere1", "clairiere3"]}
        self.lieu["clairiere3"] = {'piege' : "piege3", 'nom' : "La clairiere du Loup-garou", 'choix_chemin': ["clairiere1", "clairiere2"]}
 
    def choix_chemin(self):
        pos=self.lieu[self.position_joueur]
        if pos.has_key('choix_chemin')==True: 
            for i,lieu in enumerate(pos['choix_chemin']) :
                print "tapez ",i,"pour allez à ", lieu
            c=input ("Quel est votre choix ?")
        self.position_joueur=pos['choix_chemin'][c]
 
    def deplacer_perso(self) :
        self.choix_chemin()
        print "Vous êtes à "+self.position_joueur
 
jeux=Jeux()
jeux.position_joueur="clairiere1"
while True:
    jeux.deplacer_perso()

[hulkyoann]

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
 
clairiere1 = {'piege' : piege1, 'nom' :"Maison du Chaperon rouge", 'choix chemin': [clairiere3, clairiere2]}
clairiere2 = {'piege' : piege2, 'nom' : "La clairiere de la joie", 'choix chemin': [clairiere1, clairiere3]}
clairiere3 = {'piege' : piege3, 'nom' : "La clairiere du Loup-garou", 'choix chemin': [clairiere1, clairiere2]}
 
 
 
def choix_chemin(position_joueur):
    global position_joueur
 
    if position_joueur.has_key('choix_chemin')==True:
        if len (position_joueur['choix_chemin'])==1 :
            print "tapez ","1","pour allez à ", position_joueur['choix_chemin'][0]['nom']
        else :    
            for i in range(len(position_joueur['choix_chemin'])) :
                print "tapez ",i,"pour allez à ", position_joueur['choix_chemin'][i]['nom']
        c=input ("Quel est votre choix ?")
    print 'nouvelle clairiere', position_joueur['choix_chemin'][c]['nom']
    print 'ancienne ',position_joueur['nom']
    return position_joueur['choix_chemin'][c]
def deplacer_perso(position_joueur) :
 
    position_joueur=choix_chemin(position_joueur)
    return

En fait c'est la même chose qu'avant, mais avec "global position_joueur" en plus. Mais je ne sais pas trop comment ça marche le mot-clé global.

Merci alexdevl pour ton exemple avec le code, mais je ne dois pas le faire avec les class. Mais c'est vrai que c'est moi chiant avec les class.
Par contre j'ai réglé le probleme des clairiere que j'utilisait avant de les définir.

[PauseKawa]

Bonsoir,

C'est un problème de référence, tu réaffectes une valeur à ta variable position_joueur dans le contexte local (la fonction deplacer_perso) sans préciser que tu veux agir sur la variable globale, donc soit tu laisses tomber la fonction deplacer_perso et tu affectes dans le contexte global (là où tu appellerais la fonction), soit tu précises à l'interpréteur que tu veux utiliser la variable globale position_joueur (mot-clé global), auquel cas le return est inutile, ainsi que le passage en argument de l'ancienne position, vu que c'est une variable globale.

Ok pour la référence oiffrig mais global c'est à proscrire pour un débutant à mon avis (porte ouverte à toutes les fenêtres et +).
Personnellement je préconise de commencer avec self ou mieux (bien mieux) return (et vu les return du code de base je pense que c'est la politique du prof).

Merci alexdevl pour ton exemple avec le code, mais je ne dois pas le faire avec les class.

Classes ? Ou ? C'est des dico.
A mon avis la solution d'alexdevl, avec self, est pour le moment la meilleure réponse.
Sur son 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
18
19
20
21
22
23
24
25
26
27
28
29
 
#!/usr/bin/env python
# -*- coding: iso-8859-1 -*-
 
class Jeux():
 
    def __init__(self):
        self.lieu={}
        self.lieu["clairiere1"] = {"piege":"piege1", "nom":"la maison du Chaperon rouge","choix_chemin":["clairiere3", "clairiere2"]}
        self.lieu["clairiere2"] = {"piege":"piege2", "nom":"la clairiere de la joie","choix_chemin":["clairiere1", "clairiere3"]}
        self.lieu["clairiere3"] = {"piege":"piege3", "nom":"la clairiere du Loup-garou","choix_chemin":["clairiere1", "clairiere2"]}
 
    def choix_chemin(self):
        pos=self.lieu[self.position_joueur]
        if pos.has_key("choix_chemin")==True: 
            for i,lieu in enumerate(pos["choix_chemin"]):
                print "Tapez",i,"pour allez à",self.lieu[lieu]['nom']
            c=input ("Quel est votre choix ? ")
        self.position_joueur=pos["choix_chemin"][c]
        return self.lieu[self.position_joueur]
 
    def deplacer_perso(self):
        pos = self.choix_chemin()
        print "Vous êtes à",pos['nom']
 
jeux=Jeux()
jeux.position_joueur="clairiere1"
while True:
    jeux.deplacer_perso()

A toi de gérer les erreurs (ne doute pas que ton prof regarderas le forums).

@+

[hulkyoann]

Nan, mais ça c'est les classes, puisqu'on définie une classe Jeux.
Par contre, le code qu'a fait alexdevl ne peut pas aller dans mon code. J'explique: Dans mes dictionnaires clairiere, y'a la clé piège qui renvoie à un dico piege1 ou piege2 (etc) qui enlève des points de vie au joueur, donc je ne peu pas le mettre en chaine de caractère. Pareil pour position_joueur, il doit être un dico parce que dans le reste du code de mon jeu, il sert dans chaque fonction.
Je ne sais pas si je suis très clair, mais j'essaie de l'être!
Merci et bonne journée à tous!

[hulkyoann]

c'est bon j'ai réglé le probleme, il suffisait de mettre global position_joueur.
Mais maintenant c'est dans un autre module dans ma fonction pieges() que ça merde.
Maintenant, le shell me dit que position_joueur n'est pas définit.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
from environnement import *
def pieges():
    global position_joueur
    if position_joueur.has_key('piege')==True:
        piege=position_joueur['piege']
        parametre_joueur['PV'] = parametre_joueur['PV'] - piege['PV']
        parametre_joueur['PS'] = parametre_joueur['PS'] - piege['PS']
        parametre_joueur['PF'] = parametre_joueur['PF'] - piege['PF']
        print piege['texte']
 
        print parametre_joueur['PV'], position_joueur['nom']
    return

Pourtant j'importe le module environnement (là ou il y a la fonction choix_chemin, et la ou je définie la variable position_joueur), mais il ne l'importe pas, enfin on dirait.
C'est le position_joueur dans le if qu'il désigne comme pas définit.
Merci et bonne journée.

[josmiley]

oui c'est normal, mais pourquoi tu ne passes pas d'arguments, ça serait tellement plus simple ...
bon, je suis une brèle en python, mais c'est quoi ça ?

Code :

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

1
2
3
4
def choix_chemin(position_joueur):
    global position_joueur
     ...
...

si position_joueurbest passé en argument, ça a rien à faire dans global, non ?