Comment gérer une erreur is not defined après n'avoir rien tapé dans une variable raw_input (python 2.7)?

Bonjour à tous, je suis sous Python 2.7 et je voudrais savoir comment gérer dans une boucle

Code:

---

while 1:

---

une erreur de type variable non defini
par exemple voici mon code

Code:

---

1 2 3 4 5 6 7 8 9 10

# -*- coding: cp1252 -*- while 1: try: multiplieurCapacityint_1 = int(raw_input('1 entrée un nombre pour multiplier la Capacitée par ')) except ValueError: print "Vous n'avez pas entrez un nombre correct pour la multiplication veillez recommencez!" if multiplieurCapacityint_1<=0: print "Vous n'avez pas entrez un nombre correct pour la multiplication veillez recommencez!" else: break

---

et mon code d'erreur

Code:

---

NameError: name 'multiplieurCapacityint_1' is not defined

---

si je tape quel un int non valide, c'est bon le programme ne bug pas, si c'est un int valide il passe à la fin de boucle est la ternime, si c'est pas un int valide <=0 il revient au début de la boucle et l'erreur est correctement gérer, mais que si l'on tape quel chose non int il bug, si l'on ne tape rien ou type str message d'erreur suivant

Code:

---

NameError: name 'multiplieurCapacityint_1' is not defined

---

et le programme plante il quitte tout simplement, et ça je ne peut pas le tollérer, parce que justement raw_input on peut faire ce genre d'erreur de ne rien taper ou taper un type du genre invalide comme type str, et la très vite c'est le drame. Comment donc gérer ce type d'erreur dans une boucle

Code:

---

while 1:

---

pour

Code:

---

raw_input

---

?

D'avance je vous remercie pour votre aide

MediaVistaIntel

Bonsoir,

Question de logique. Quelques exemples sur votre code.

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 26 27 28 29

#!/usr/bin/env python # -*- coding: cp1252 -*- # # while 1: try: multiplieurCapacityint_1 = int(raw_input('1 entrée un nombre pour multiplier la Capacitée par ')) if multiplieurCapacityint_1 <= 0: print "Vous n'avez pas entrez un nombre correct pour la multiplication veillez recommencez!" else: break except ValueError: print "Merci de donner un chiffre. Veillez recommencez!" print(multiplieurCapacityint_1)   multiplieurCapacityint_1 = None while not multiplieurCapacityint_1: try: multiplieurCapacityint_1 = int(raw_input('1 entrée un nombre pour multiplier la Capacitée par ')) except ValueError: print "Vous n'avez pas entrez un nombre correct pour la multiplication veillez recommencez!" print(multiplieurCapacityint_1)   while 1: multiplieurCapacityint_1 = raw_input('1 entrée un nombre pour multiplier la Capacitée par ') if multiplieurCapacityint_1.isdigit() and int(multiplieurCapacityint_1) > 0: break print "Vous n'avez pas entrez un nombre correct pour la multiplication veillez recommencez!" print(multiplieurCapacityint_1)

---

Etc...
A vous de voir.

@+

Salut,

Il faut essayez de comprendre ce qu'il se passe côté interpréteur et sa logique n'est pas nécessairement la votre.

En écrivant, "foo = int(...)" l'intention est d'assigner à la variable "foo" l'objet retourné par "int(...)".
Problème, lorsque int se vautre, on ne sait plus trop quoi assigner à "foo" et... la variable ne sera pas créée.
Dans ce cas particulier, il est peut être préférable de décomposer:

Code:

---

1 2 3 4 5 6

while True: s = raw_input(...) if s and s.isdigit() # si la chaine est "vide" on n'est pas bien v = int(s) if v > 0: break

---

puis d’emballer cela dans une fonction qui vous assure que la variable "multiplieurCapacityint_1" sera affectée avec un objet de type et de valeur attendue.

Code:

---

1 2 3 4 5 6 7 8 9

def get_capacity(): while True: s = raw_input(...) if s and s.isdigit() # si la chaine est "vide" on n'est pas bien v = int(s) if v > 0: break return v multiplieurCapacityint_1 = get_capacity()

---

Enfin c'est vous qui voyez
Cordialement
- W

Citation:

---

Envoyé par wiztricks

Code:

---

if s and s.isdigit() # si la chaine est "vide" on n'est pas bien

---

---

:?
raw_input retourne un string et

Code:

---

1 2	>>> ''.isdigit() False

---

Donc

Code:

---

1 2 3 4 5 6 7

while True: s = raw_input('1 entrée un nombre pour multiplier la Capacitée par ') if s.isdigit(): v = int(s) if v > 0: break print v

---

Mais comme

Code:

---

1 2 3

>>> None and 1 >>> 1 and 2 2

---

Code:

---

1 2 3 4 5

while True: s = raw_input('le texte : ') if s.isdigit() and int(s) > 0: break print s

---

Non ?

Edit:
Ovni

Code:

---

1 2 3

OVNI = lambda: True and ((lambda v=raw_input('le texte : '): v.isdigit() and int(v) >= 0 and v)() or OVNI()) multiplieurCapacityint_1 = OVNI() print multiplieurCapacityint_1

---

Pardon ? Ok ---> []

Salut,

La structure:

Code:

---

1 2 3 4 5

def get_capacity(): .... return v   multiplieurCapacityint_1 = get_capacity()

---

définit une fonction "get_capacity" qui retourne un entier "> 0".

Après on peut regarder les détails de la réalisation, mais le "if s and s.isdigit()" est à mon sens "anecdotique" vs le défaut de traitement du EOF (on fait quoi dans ce cas?).

Le code:

Code:

---

1 2 3 4 5

while True: s = raw_input('le texte : ') if s.isdigit() and int(s) > 0: break print s

---

est, certes, plus concis mais retourne un "str" à la place d'un "int".
- W

A vrais dire je n'avais pas voulu entrer dans la considération 'fonction' pour le PO mais plus le tourner vers if plutôt que try.

Code:

---

1 2 3 4 5 6 7

while True: s = raw_input('le texte : ') if s.isdigit() and int(s) >= 0: s = int(s) break print s print type(s)

---

Mais la notion de fonction est une chose a retenir pour lui.

'plus concis': Pas question d’être plus concis mais de montrer if avec and/or. Existe il quelque chose de plus puissant au niveau exécution ?

PS:
Petites notes pour le PO.

Vous voici face à deux faux amis: try/except et while True.

try/except: J’essaie mon code sinon je laisse tomber... Trop facile.
Cette déclaration n'est pas là pour tester votre code mais pour parer à une éventuelle erreur d'un 'objet externe'.
Vous allez me dire c'est ce que je fais (except ValueError) mais non: Par erreur je parle d'une exception venant d'un 'objet externe' à votre code d'on vous ne maîtrisez pas le résultat, le ValueError vous savez qu'il vas se produire. Dans ce cas autant le tester (if).
Concrètement c'est:

Code:

---

1 2 3 4 5 6 7 8

try: multiplieurCapacityint_1 = int(raw_input('1 entrée un nombre pour multiplier la Capacitée par ')) except ValueError: print "Vous n'avez pas entrez un nombre correct pour la multiplication veillez recommencez!"   s = raw_input('1 entrée un nombre pour multiplier la Capacitée par ') if s.isdigit(): v = int(s)

---

Mais vous allez me dire 'je ne connais pas str.isdigit()'. Ça c'est le métier qui rentre, sinon il reste la doc ou le forum pour demander ;)

Vous allez me dire 'mais ou utiliser try/except ?'.
L'exemple le plus simple est la manipulation de fichier. Voici du code venant du module shutil.

Code:

---

1 2 3 4

try: os.remove(fullname) except os.error, err: onerror(os.remove, fullname, sys.exc_info())

---

Rien ne dit que vous pouvez supprimer 'fullname', ou qu'il existe encore, au moment ou vous voulez le supprimer.
Vous allez me dire 'je teste avec if'
Mais non:

Code:

---

1 2	if os.path.isfile(fullname): os.remove(fullname)

---

Ici il est plus 'rapide' d'utiliser try, et ne pas risquer une erreur entre temps, que de se perdre dans des tests.
Note: Capilotracté (dédicace à qui se reconnaîtras) comme exemple ? C'est le plus simple que j'ai trouvé par rapport à des instances ou autre.

while True(1): Tant que je ne sais pas m'en sortir je n'utilise pas.
Vous devez TOUJOURS définir comment vous en sortir, soit dans la condition (while unecondition) soit dans le code qui suit (break).

Maintenant votre erreur :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

>>> a = 1 >>> b = 0 >>> try: ... c = a / b ... except ZeroDivisionError: ... pass ... >>> c Traceback (most recent call last): File "<stdin>", line 1, in <module> NameError: name 'c' is not defined

---

Ici le code suivant le try est testé sinon c'est celui d'except qui est exécuté.

Les basiques : and/or

Code:

---

1 2 3 4 5 6 7 8 9 10

>>> 0 and 1 0 >>> 1 and 0 0 >>> 0 or 1 1 >>> 1 or 0 1 >>> 0 and 0 or 1 1

---

Rebonjour, tous le monde et un grand merci pour vos explication et vos point de vue, et merci à toi PauseKawa pour tes codes, voici celui que j'ai choisi

Citation:

---

Envoyé par PauseKawa

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

#!/usr/bin/env python # -*- coding: cp1252 -*- # # while 1: try: multiplieurCapacityint_1 = int(raw_input('1 entrée un nombre pour multiplier la Capacitée par ')) if multiplieurCapacityint_1 <= 0: print "Vous n'avez pas entrez un nombre correct pour la multiplication veillez recommencez!" else: break except ValueError: print "Merci de donner un chiffre. Veillez recommencez!"

---

---

et à oui la variable

Code:

---

multiplieurCapacityint_1

---

et ultiliser pour int direct et aussi pour type str si valide bien sur comme ceci par exemple

Code:

---

multiplieurCapacityint_1_str=str(multiplieurCapacityint_1)

---

c'est un exemple de comment je mais mon code à quel chose près c'est la même chose.

Et je vois aussi que mon topic à fait débat pour ça (les erreurs de champ vide raw_input en variable), constant de voir que ça anime!:mrgreen:

Dommage,

Code:

---

1 2 3 4 5 6 7

while True: s = raw_input('1 entrée un nombre pour multiplier la Capacité par ') if s.isdigit() and int(s) >= 0: multiplieurCapacityint_1 = int(s) break print "Vous n'avez pas entrez une valeur correcte pour la multiplication veillez recommencez!"

---

Est largement plus propre à mon sens.
Je me console en me disant que vous relirez ce post plus tard.

@+

Salut,

Puisqu'on parle de propreté, ne serait-pas mieux avec les conditions imbriquées? Python teste peut-être en premier la condition la plus à gauche, mais ce n'est pas obligatoire en logique pure. Avec un compilo C, je crois me rappeler que l'ordre de test n'est pas imposé.

Code:

---

1 2 3 4 5 6 7

while True: s = raw_input('1 entrée un nombre pour multiplier la Capacité par ') if s.isdigit(): if int(s) >= 0: multiplieurCapacityint_1 = int(s) break print "Vous n'avez pas entrez une valeur correcte pour la multiplication veillez recommencez!"

---

A+

Pfeuh

Citation:

---

Envoyé par pfeuh

Avec un compilo C, je crois me rappeler que l'ordre de test n'est pas imposé.

---

On fait plus que friser la HS, là, mais il me semble bien que si, du moins pour la version moderne du C (C99 si je ne m’abuse).

En tout cas, dans Blender, on a pléthore de trucs genre

Code:

---

if (ptr && ptr->foo) do_bar();

---

Ce qui implique que la première condition (le pointeur n’est pas NULL) est évaluée avant la seconde… Confirmé d’ailleurs par ceci: http://stackoverflow.com/questions/6...aluation-order

Pour en revenir à python (doc officielle de py3.2)*:

Citation:

---

Python evaluates expressions from left to right. Notice that while evaluating an assignment, the right-hand side is evaluated before the left-hand side.

---

Et pareil pour Java, apparemment… Donc à quelques rares exceptions près (en C++), l’évaluation se fait toujours de gauche à droite au strict minimum pour les opérations logiques… ce qui est logique*! ;)

Salut,

Citation:

---

Envoyé par pfeuh

Puisqu'on parle de propreté, ne serait-pas mieux avec les conditions imbriquées? Python teste peut-être en premier la condition la plus à gauche, mais ce n'est pas obligatoire en logique pure. Avec un compilo C, je crois me rappeler que l'ordre de test n'est pas imposé.

---

Les affinités de Python avec C sur ce sujet font que je ne comprends pas trop la remarque sur l'ordre des tests.

Dans tous les cas, il y a souvent un compromis à trouver entre lisibilité et efficacité. A défaut de "standards" style PEP8, la lisibilité est "subjective" alors que l'efficacité pourrait se mesurer.

Ceci dit j'ai toujours eu quelques difficultés à comprendre des recherches trop poussées d'un style de codage plus "efficace" lorsqu'on programme en Python.

Si on fait un gros truc, on essayera d'optimiser certains traitements et non tous les blocks/fonctions. On essaiera de mesurer le coûts de certaines construction par rapport à d'autres équivalentes si cela est "brique de base".
A la limite, une partie pourra être ré-écrite en C.

Python est surtout efficace pour que le programmeur réalise un prototype assez rapidement. "Optimiser" dépendra de l'utilisation, de l'intérêt du prototype.
Note: Un prototype permet aussi de valider des choix fonctionnels... Après c'est "poubelle".

- W

Bonjour,

Pour rappel le and n'est pas vraiment là pour un gain de performance ou toute autre considération de lisibilité mais bien pour introduire pour le PO une comparaison booléenne plutôt que deux instructions if à la suite.

Citation:

---

Envoyé par PauseKawa

'plus concis': Pas question d’être plus concis mais de montrer if avec and/or.

---

Que ce soit

Code:

---

1 2 3 4 5 6

while True: s = raw_input('1 entrée un nombre pour multiplier la Capacité par ') if s.isdigit() and int(s) >= 0: multiplieurCapacityint_1 = int(s) break print "Vous n'avez pas entrez une valeur correcte pour la multiplication veillez recommencez!"

---

Ou

Code:

---

1 2 3 4 5 6 7

while True: s = raw_input('1 entrée un nombre pour multiplier la Capacité par ') if s.isdigit(): if int(s) >= 0: multiplieurCapacityint_1 = int(s) break print "Vous n'avez pas entrez une valeur correcte pour la multiplication veillez recommencez!"

---

Comme le dit wiztricks ce n'est sûrement pas ce genre de code qui vas être coûteux.
Un ajout inutile ici et trompeur dans le sens ou cela demande des explications plus pointues (1, 2) ? Sans doute.

Le but véritable était ici de montrer à MediaVistaIntel son code avec if plutôt que try, pour les raisons évoquée plus haut. Vous remarquerez que dans les exemples (sorry MediaVistaIntel) j'ai éviter

Code:

---

1 2 3 4 5 6 7 8 9 10

while 1: multiplieurCapacityint_1 = raw_input('1 entrée un nombre pour multiplier la Capacitée par ') try: multiplieurCapacityint_1 = int(multiplieurCapacityint_1) except ValueError: print "Merci de donner un chiffre. Veillez recommencez!" else: if multiplieurCapacityint_1 >= 0: break print "Un nombre positif svp. Veillez recommencez!"

---

Ce qui aurais été un code 'acceptable' (ce qui veux dire en fait ?) vu que cela ne cache pas une autre erreur possible.
Malhonnête d'utiliser l'erreur de MediaVistaIntel pour cela ? Je ne pense pas vu qu'aucun de vous n'est revenu sur l'utilisation de if.

Citation:

---

Envoyé par wiztricks

Ceci dit j'ai toujours eu quelques difficultés à comprendre des recherches trop poussées d'un style de codage plus "efficace" lorsqu'on programme en Python.

---

Je convient que tout cela semble inutile mais ma considération ici est l'utilisation des bons outils dés le début.
Vous me trouvez trop radical pour ce qui est de l'utilisation de try ?

@+

Citation:

---

Envoyé par wiztricks

Les affinités de Python avec C sur ce sujet font que je ne comprends pas trop la remarque sur l'ordre des tests.

---

Ce que je voulais dire c'est que cela fonctionnera:

Code:

---

1 2 3 4 5 6 7

while True: s = raw_input('1 entrée un nombre pour multiplier la Capacité par ') if s.isdigit(): if int(s) >= 0: multiplieurCapacityint_1 = int(s) break print "Vous n'avez pas entrez une valeur correcte pour la multiplication veillez recommencez!"

---

Mais cela bien évidemment pas:

Code:

---

1 2 3 4 5 6 7

while True: s = raw_input('1 entrée un nombre pour multiplier la Capacité par ') if int(s) >= 0: if s.isdigit(): multiplieurCapacityint_1 = int(s) break print "Vous n'avez pas entrez une valeur correcte pour la multiplication veillez recommencez!"

---

Alors avec deux tests sur la même ligne, il ne me parait pas incongru de se poser la question. Sauf qu'au lieu de sauter sur le mot "propreté", j'aurais du utiliser "lisibilité" ou "clarté".

Citation:

---

Envoyé par pfeuh

Alors avec deux tests sur la même ligne, il ne me parait pas incongru de se poser la question.

---

Pas vraiment

Code:

---

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

>>> def foo(): ... print 'foo' ... return True ... >>> def foo1(): ... print 'foo1' ... >>> if foo() and foo1(): ... print 'ok' ... foo foo1 >>> if foo1() and foo(): ... print 'ok' ... foo1 >>> if foo() and foo(): ... print 'ok' ... foo foo ok

---

Voir les liens 1 et 2.

Salut,

Citation:

---

Envoyé par pfeuh

Alors avec deux tests sur la même ligne, il ne me parait pas incongru de se poser la question.

---

Disons que la construction:

Code:

---

1 2 3

if s.isdigit(): if int(s) >= 0: *block*

---

équivaut à:

Code:

---

1 2	if s.isdigit() and int(s) >= 0: *block*

---

Cette équivalence est documentée et PauseKawa en a donné moultes illustrations.

Pour ce cas particulier, je n'aime pas cette construction.
Dans le cas ou la condition est "vraie", elle coûte un deuxième appel à int(s). Je lui préfère:

Code:

---

1 2 3 4 5 6

while True: s = raw_input(...) if s and s.isdigit(): v = int(s) if v > 0: break

---

PauseKawa a raison de faire remarquer que tester "s" est inutile.
Mais, cela "documente" (me rappelle) les 3 conditions identifiées:

• chaine vide,
• contient des digits,
• est positif,

(et l'omission du cas EOF).
Cela oblige à "structurer" le code en conséquence.

La "lisibilité" étant "subjective", les avis peuvent être différents et tout aussi "sensés", "fondés", ...

- W

Bonjour,

Je dois bien avouer que

Code:

---

1 2	if int(s) >= 0: v = int(s)

---

c'est moche.
Non pas une considération sur la performance du code avec les deux load global d'int inutiles mais dans la forme.

Vu sous cet angle

Code:

---

1 2 3 4

if s and s.isdigit(): v = int(s) if v > 0: break

---

parait plus attrayant (hors considération pour if s), même avec ses deux blocs.
Enfin, du moment ou on n'en arrive pas à des

Code:

---

1 2 3 4 5 6 7

while True: s = raw_input('1 entrée un nombre pour multiplier la Capacité par ') if s.isdigit() and (locals().setdefault('v', int(s)) or 1) and v >= 0: break print "Vous n'avez pas entrez une valeur correcte pour", \ "la multiplication veillez recommencez!" print v

---

(Pas taper sur les doigts pour le namespace... Ce n'est qu'un exemple bidon sur l'idée du sujet)
and est sans doute a retenir mais peut être pas dans le cas présent.

Maintenant je suis étonné que personne ne réagisse sur le fait que MediaVistaIntel ai choisi une de mes 'merveilles' de code. De même l'utilisation de try ne semble pas vous gêner...

@+

Salut,

Citation:

---

Envoyé par PauseKawa

Maintenant je suis étonné que personne ne réagisse sur le fait que MediaVistaIntel ai choisi une de mes 'merveilles' de code. De même l'utilisation de try ne semble pas vous gêner...

---

Ah ben, à la lecture de:

Code:

---

multiplieurCapacityint_1_str=str(multiplieurCapacityint_1)

---

il est clair que le PO a du chemin à faire.
Aujourd'hui, il avait un problème et à choisit la variante avec laquelle, il est le plus "confortable".
A ne pas exclure non plus que le PO soit "étudiant" et tenu de montrer qu'il a bien acquis certaines constructions du langage plutôt que de faire le malin avec x.isdigit...
Allez savoir...

- W

Citation:

---

Envoyé par wiztricks

Salut,



Ah ben, à la lecture de:

Code:

---

multiplieurCapacityint_1_str=str(multiplieurCapacityint_1)

---

il est clair que le PO a du chemin à faire.
Aujourd'hui, il avait un problème et à choisit la variante avec laquelle, il est le plus "confortable".
A ne pas exclure non plus que le PO soit "étudiant" et tenu de montrer qu'il a bien acquis certaines constructions du langage plutôt que de faire le malin avec x.isdigit...
Allez savoir...

- W

---

le code suivant

Code:

---

multiplieurCapacityint_1_str=str(multiplieurCapacityint_1)

---

est un exemple et

Code:

---

multiplieurCapacityint_1

---

c'est parce que j'ai plusieurs truc comme ça

Code:

---

multiplieurCapacityint_#

---

12 pour être exact ici dossier Cities XL msn de MediaVistaIntel vous avez dans le fichier Cities XL programme de modifications.7z dossier Cities XL multiplieurs batiments densitée et Cities XL multiplieurs batiments densitée float 2 mode on va dire, le simple qui traitre 1 multiples à la fois, et la douzaine qui traitre 12 multiples à la fois, et justement la douzaine dans les fichiers du nom Cities XL Rapide Multiplieurs grouper 1.py Cities XL Rapide Multiplieurs grouper 2.py etc jusqu'à Cities XL Rapide Multiplieurs grouper 12.py
on le même code à quel chose prêt
pour le 1

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 26 27 28 29 30

# -*- coding: cp1252 -*- #Multiplieurs multiplieurCapacity = str(multiplieurCapacityint_1) multiplieurCapacityint = (multiplieurCapacityint_1) for root, dirs, files in os.walk(source): for fn in irec_glob('*.*', source): nom_variable_partie='t' + taille_buildings + '/' + Cities_XL_folder + 'X' + multiplieurCapacity + ' XL ' + Cities_XL_Version dirname = os.path.dirname(fn) dirnametemp = Re_sourceDir + 'temp/' + nom_variable_partie + '/' + dirname[longueur_Dir:] dirname2 = Re_sourceDir + nom_variable_partie + '/' + dirname[longueur_Dir:] basename = os.path.basename(fn) split = os.path.split(fn) splitdrive = os.path.splitdrive(fn) splitext = os.path.splitext(fn) fichierchemintemp = Re_sourceDir + 'temp/' + nom_variable_partie + '/' + fn[longueur_Dir:] fichierchemin = Re_sourceDir + nom_variable_partie + '/' + fn[longueur_Dir:] fichieropen = open(fn, "r") fichierread = fichieropen.read() #Lit tout le fichier d'un coup condiction = re.search(taille_buildings_regex, fichierread) condiction2 = re.search(buildings_regex, fichierread) if condiction and condiction2: print fn execfile(Cities_XL_Sous_programme_python) else: fichieropen.close() break dossiers_vers_patch = Re_sourceDir + 't' + taille_buildings + '/' + Cities_XL_folder + 'X' + multiplieurCapacity + ' XL ' + Cities_XL_Version patch_depuis_dossiers = dossiers_vers_patch + ".patch" subprocess.call(["../citiesXL_PAK_util.exe", "M2", dossiers_vers_patch, patch_depuis_dossiers]) shutil.rmtree(dossiers_vers_patch)

---

pour le 2

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 26 27 28 29 30

# -*- coding: cp1252 -*- #Multiplieurs multiplieurCapacity = str(multiplieurCapacityint_2) multiplieurCapacityint = (multiplieurCapacity_2) for root, dirs, files in os.walk(source): for fn in irec_glob('*.*', source): nom_variable_partie='t' + taille_buildings + '/' + Cities_XL_folder + 'X' + multiplieurCapacity + ' XL ' + Cities_XL_Version dirname = os.path.dirname(fn) dirnametemp = Re_sourceDir + 'temp/' + nom_variable_partie + '/' + dirname[longueur_Dir:] dirname2 = Re_sourceDir + nom_variable_partie + '/' + dirname[longueur_Dir:] basename = os.path.basename(fn) split = os.path.split(fn) splitdrive = os.path.splitdrive(fn) splitext = os.path.splitext(fn) fichierchemintemp = Re_sourceDir + 'temp/' + nom_variable_partie + '/' + fn[longueur_Dir:] fichierchemin = Re_sourceDir + nom_variable_partie + '/' + fn[longueur_Dir:] fichieropen = open(fn, "r") fichierread = fichieropen.read() #Lit tout le fichier d'un coup condiction = re.search(taille_buildings_regex, fichierread) condiction2 = re.search(buildings_regex, fichierread) if condiction and condiction2: print fn execfile(Cities_XL_Sous_programme_python) else: fichieropen.close() break dossiers_vers_patch = Re_sourceDir + 't' + taille_buildings + '/' + Cities_XL_folder + 'X' + multiplieurCapacity + ' XL ' + Cities_XL_Version patch_depuis_dossiers = dossiers_vers_patch + ".patch" subprocess.call(["../citiesXL_PAK_util.exe", "M2", dossiers_vers_patch, patch_depuis_dossiers]) shutil.rmtree(dossiers_vers_patch)

---

etc, ce pour tous les 12 fichiers ce qui change juste c'est ça

Code:

---

1 2	multiplieurCapacity = str(multiplieurCapacityint_1) multiplieurCapacityint = (multiplieurCapacity_1)

---

pour le 1

Code:

---

1 2	multiplieurCapacity = str(multiplieurCapacityint_2) multiplieurCapacityint = (multiplieurCapacity_2)

---

pour le 2, et ce jusqu'aux 12. Pourquoi? Parce que dans les 12 fichiers dans la boucle FOR de chaque fichier, le sous dossier Cities XL Sous programme python et Cities XL MaxJobPerCulture, je mais pas besoin comme ça de remettre une variable suplémantaire et donc je peut réutiliser les fichier des sous dossiers puisque il on les même variable.
bien sur la version float du dossier c'est pour la version à virgules du programme!

Salut,

J'ai pris un peu de temps d'aller farfouiller dans vos sources.

Si vous avez un peu de temps:
- essayez de comprendre ce qu'est un "module",
- utilisez "import" à la place "d'exec",
- créez une fonction chaque fois que le "cut&paste" vous démange
- évitez d'utiliser des variables globales.

Citation:

---

et justement la douzaine dans les fichiers du nom Cities XL Rapide Multiplieurs grouper 1.py Cities XL Rapide Multiplieurs grouper 2.py etc jusqu'à Cities XL Rapide Multiplieurs grouper 12.py

---

Ils sont apparemment tous identiques excepté les premières lignes qui initialisent les variables globales:

Code:

---

1 2	multiplieurCapacity = str(multiplieurCapacityfloat_1) multiplieurCapacityfloat = (multiplieurCapacityfloat_1)

---

Plutôt que d'avoir 12 fichiers, vous pourriez avoir une seule fonction toto(multiplieurCapacityfloat).
Enfin c'est vous qui voyez.

Bon courage
- W

Citation:

---

Envoyé par wiztricks

Salut,

J'ai pris un peu de temps d'aller farfouiller dans vos sources.

Si vous avez un peu de temps:
- essayez de comprendre ce qu'est un "module",
- utilisez "import" à la place "d'exec",
- créez une fonction chaque fois que le "cut&paste" vous démange
- évitez d'utiliser des variables globales.



Ils sont apparemment tous identiques excepté les premières lignes qui initialisent les variables globales:

Code:

---

1 2	multiplieurCapacity = str(multiplieurCapacityfloat_1) multiplieurCapacityfloat = (multiplieurCapacityfloat_1)

---

Plutôt que d'avoir 12 fichiers, vous pourriez avoir une seule fonction toto(multiplieurCapacityfloat).
Enfin c'est vous qui voyez.

Bon courage
- W

---

Pourquoi éviter les variable globales, c'est quoi la fonction toto, et que n'aporte import par raport à exec? exec le fichiers est défini par rapore à une variable (pour certains seulement) pour le chemin complet relatif à partir du dossier du programme principal, exec ne permet d'avoir des sous fichiers et moins de lignes dans mon fichier principal, surtout pour les trucs qui ce répètes beaucoup ou même comme ça. Je ne vois pas le mal qu'il y a à ça?

Bonsoir,

Citation:

---

Envoyé par MediaVistaIntel

Pourquoi éviter les variable globales

---

Pour éviter de se prendre les pieds dedans: C'est une notion de portée de variable. A la première closure vous allez revenir nous voir avec un 'pourquoi cela ne marche pas ?'

Citation:

---

Envoyé par MediaVistaIntel

c'est quoi la fonction toto

---

Ce que dit wiztricks c'est que si

Code:

---

1 2	multiplieurCapacity = str(multiplieurCapacityfloat_1) multiplieurCapacityfloat = (multiplieurCapacityfloat_1)

---

et pareil que

Code:

---

1 2	multiplieurCapacity = str(multiplieurCapacityfloat_2) multiplieurCapacityfloat = (multiplieurCapacityfloat_2)

---

Une fonction

Code:

---

def toto(x)

---

Ou x est multiplieurCapacityfloat_X
Simplifie le code.

Citation:

---

Envoyé par MediaVistaIntel

et que n'aporte import par raport à exec? exec le fichiers est défini par rapore à une variable (pour certains seulement) pour le chemin complet relatif à partir du dossier du programme principal, exec ne permet d'avoir des sous fichiers et moins de lignes dans mon fichier principal, surtout pour les trucs qui ce répètes beaucoup ou même comme ça. Je ne vois pas le mal qu'il y a à ça?

---

Et j'ai du mal a vous suivre là. Sans doute n'avez vous pas compris la réponse. Il vous manque des connaissances pour comprendre la réponse.

Dans tous les cas il vous manque pas mal de bases: Puis je vous recommander de suivre un tuto du site ?

@+

Salut,

Citation:

---

Envoyé par MediaVistaIntel

Pourquoi éviter les variable globales, c'est quoi la fonction toto, et que n'aporte import par raport à exec? exec le fichiers est défini par rapore à une variable (pour certains seulement) pour le chemin complet relatif à partir du dossier du programme principal, exec ne permet d'avoir des sous fichiers et moins de lignes dans mon fichier principal, surtout pour les trucs qui ce répètes beaucoup ou même comme ça. Je ne vois pas le mal qu'il y a à ça?

---

Pourquoi éviter les variables "globales" (et utiliser des fonctions)?

Dans les années 60s, les programmeurs n'avaient à leur disposition que des langages tels que Fortran ou Cobol, pire Basic! Lisp existait déjà mais était encore plus confidentiel qu'il ne l'est aujourd'hui.

Ces langages ne permettaient pas de structurer le code autrement: un tas de variable globales et une orchestration d'activités via la modifications de leurs états.

Les programmes devant réaliser de plus en plus de fonctionnalités, il fallait plus de variables globales, d'activités,... et l'orchestration du tout devenait un exercice semblable à la construction d'un château de cartes ou à maintenir un tas d'assiettes en équilibre en les faisant tourner sur une tige: un truc d'équilibriste!

Si on peut applaudir les artistes, difficile dans un contexte professionnel de dépendre d'une activité aux résultats aussi incertains!

Nous avons donc par la suite, inventé la programmation structurée, puis la programmation objet, les pattern de programmation, ....
Des tas de bouquins ont été écrits sur ces sujets pour éviter aux apprentis programmeurs, de faire les mêmes erreurs et apprendre de meilleures pratiques.

Citation:

---

Je ne vois pas le mal qu'il y a à ça?

---

Que penseriez vous d'une maison où tirer la chasse d'eau des WC va mettre en route le four de la cuisine? Déboussolant, n'est ce pas!

• Lire votre code donne le tournis.
• Et essayer de comprendre ce qui ne fonctionne pas sera toujours casse bonbon...
• Vous expliquer ce qui ne va pas sera prise de tête parce que vous ne maîtrisez pas encore les bases.

Si vous ne prenez pas le temps de vous former aux bases de la programmation, pas grand monde perdra longtemps le sien à essayer de vous aider.

Bon courage,

- W

Et j'ajouterais juste une petite chose après les explications claires de wiztricks et pausekawa.

Quand on écrit sur un forum, et qu'on a la chance d'avoir des réponses, la moindre des choses et de les prendre en compte et de faire ses propres recherches.

Ici il y en a qui ont plus de 10 ans de programmation, des années d'ancienneté dans d'autres langages et il n'est pas toujours bon de contester sans recherches préalables démontrant le contraire. Contester pour contester n'apportera rien, mais contester avec preuve à l'appui apporte une discussion riche pour tout le monde.

Question technique, difficile de contester wiztrick ou pausekawa, sachez vous remettre en question ;)

Concernant execfile, on pourrait l'associer à la fonction exec.

Code:

---

1 2 3

f = open(fichier, 'r') exec f f.close()

---

exec à des problèmes de sécurité.

Cependant, execfile a son utilité lorsque l'on veut exécuter un script à partir d'un autre script.

Il y a deux solutions:

- Utiliser import exécute son code de haut niveau (variables globales, classes et fonctions)
- Utiliser execfile exécute le module que dans son environnement d'exécution courant (par exemple, ses espaces de noms global et local)

Pour savoir lequel est souhaitable, il faut avoir de bonnes bases.