[Python 3.X] problème avec open

Bonjour,

Alors j'ai un problème avec open que je ne comprend pas :
j'essaye d'ouvrir (et créer) un fichier avec open avec ce code :

Code:

---

1 2 3 4 5 6 7 8

def f1(path1,name,string): f2(path1+"/"+name+"/"+name+".conf", string)   def f2(path2,string): f=open(path2,'a') f.write(string) f.close()

---

et le truc étrange c'est qu'au final il me crée bien un fichier mais dans le dossier indiqué par path1 et le nom de fichier c'est name/name.conf
ce qui est super bizarre étant donné que sous linux un tel nom est normalement impossible (j'ai essayer avec touch ça refuse).
Tout en sachant que j'ai essayé de passer le path à f2 en concaténant le chemin avant de faire l'appel de fonction (au cas où) mais rien n'y fait.
Qqun à une idée ?

Slt.
Tout d'abord, il y a problème avec tes différentes ouvertures de fichiers : La variable PATH n'existe pas. Tu as path1 et path2 mais pas path. Il te manque aussi des parenthèse à f.close()

Code:

---

1 2 3 4 5 6 7 8

def f1(path1,name,string): f2(path1+"/"+name+"/"+name+".conf", string) def f2(path2,string): f=open(path2,'a') f.write(string) f.close()

---

Comme je ne suis pas sous linux, je ne peux pas vraiment te conseiller plus.

Salut,

Citation:

---

Envoyé par Alpam

Qqun à une idée ?

---

Pour visualiser les chaînes de caractères passées à "f2" et à "open", pensez à la commande "print".

- W

@wildshadow956
Oui j'ai oublier un '1' mais c'était juste pour l'exemple et oui il manque les parenthèses mais elles sont bien présentent sur mon 'vrai' code (sinon l'interpréteur prévient).
Je vais édit mon premier message pour troubler personne.
merci.

@wiztricks
déjà fait, rien d'anormal dans la string :/

Citation:

---

Envoyé par Alpam

@wiztricks
déjà fait, rien d'anormal dans la string :/

---

Mais ce qui est anormal, c'est la description de votre problème.
Si vous voulez dire qu'open('toto/name/name.txt') crée le fichier "name/name.txt" au lieu de remonter une erreur, vous avez trouvé un bug. Et comme c'est le système d'exploitation qui laisse passer cela, c'est même pas un bug Python.
Si vous voulez dire que vous avez trouvé une combinaison d'instructions magiques qui arrive à fabriquer des noms de fichiers invalides... Vous pouvez toujours le dire, mais votre code étant incomplet impossible de reproduire quoi que ce soit.

- W

Bonjour,

A mon avis, on ne peut pas demander à open de créer au passage les répertoires manquants du chemin.

Par exemple, si je fais:

Code:

---

f1("rep1", "fichier", "toto")

---

les répertoires "rep1" et "fichier" doivent avoir été créés avant. Et si c'est le cas, le code crée bien le fichier "fichier.conf" avec "toto" dedans.

Mais si ce n'est pas le cas, le code génère une erreur, ce qui est le cas sur LinuxMint (un enfant d'Ubuntu)

En tout cas, un fichier qui serait nommé "rep1/fichier/fichier.conf" est interdit pas l'OS.

Quel Linux utilises-tu?

@tyrtamos

Malheureusement je crée bien le dossier avant :/, j'utilise fedora 26 (distrib red hat).

@wiztricks

Pour aider à la lecture j'ai juste un peu édulcoré en omettant les modules et fonctions non utilisées voilà tout mais je vais essayer d'être plus exhaustif :
package:
++__init__.py
++teste.py
++factory:
++++__init__.py
++++db_factory.py
++++tbl_factory.py
++++utils:
++++++__init__.py
++++++dir_func.py
++++++file_func.py

PACKAGE teste.py:

Code:

---

1 2 3 4 5 6

from factory import db_factory, tbl_factory   db_factory.create("./test","t1") tbl_factory.create("./test/t1","t1","tbl1") tbl_factory.create("./test/t1","t1","tbl2")

---

FACTORY __init__.py:

Code:

---

1 2	from .utils import dir_func,file_func

---

FACTORY db_factory.py:

Code:

---

1 2 3 4 5 6 7 8 9 10 11

from factory import dir_func, file_func   def create(location,name): db_location = location+"/"+name dir_func.create(db_location) file_func.addDb(db_location, name) return   def delete(location): dir_func.deleteAll(location)

---

FACTORY tbl_factory.py:

Code:

---

1 2 3 4 5 6 7

from factory import dir_func, file_func   def create(db_location, db_name, tbl_name): dir_func.create(db_location+"/"+tbl_name) file_func.addTbl(db_location, tbl_name, db_name) return

---

UTILS __init__.py:

Code:

---

1 2 3 4 5

from os import mkdir from shutil import rmtree from . import dir_func from . import file_func

---

UTILS dir_func.py:

Code:

---

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

from . import mkdir from . import rmtree   def create(location): try : mkdir(location) except OSError: print("Chemin :"+location+" invalide.")   def deleteAll(location): rmtree(location)

---

UTILS file_func.py:

Code:

---

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

def create(path): try : f=open(path,'w') f.close except OSError: print("Impossible de créer le fichier "+path+".")   def writeAtEnd(doc, string): try : f=open(doc,'a') f.write(string) f.close except OSError: print("Impossible d'écrire dans : "+doc)   def addDb(path, name): create(path+"/"+name+".conf") writeAtEnd(path+"/"+name+".conf", name+"\n")   def addTbl(path, tbl_name, db_name): writeAtEnd(path+"/"+db_name+".conf", tbl_name+"::") writeAtEnd(path+"/"+tbl_name+"∕"+tbl_name+".conf", tbl_name+"\n")

---

Le teste est lancé avec python3 teste.py

et en faisant un ls dans ~/test/t1:
t1.conf tbl1 tbl1/tbl1.conf tbl2 tbl2/tbl2.conf

voilà voilà

On ne vous demande pas votre code complet, mais un MWE (minimal working example). C'est à dire un code qui soit :
- le plus petit possible
- fonctionnel (on a tous ce qu'il faut pour l'éxécuter)
- démonstratif de votre problème

Le tout premier code que vous avez posté n'était pas fonctionnel. Le dernier n'est certainement pas minimal. Le MWE c'est un exercice auquel vous devez impérativement vous soumettre pour avoir de l'aide rapidemment. Et ca c'est valable sur n'importe quel forum d'aide en informatique !

Moi personnellement, je ne vais pas prendre du temps, pour refaire l'arborescence, mettre chacun des fichiers, etc ...

Salut,

Citation:

---

Envoyé par Alpam

Pour aider à la lecture j'ai juste un peu édulcoré en omettant les modules et fonctions non utilisées voilà tout mais je vais essayer d'être plus exhaustif

---

Il ne s'agit pas d'être plus exhaustif mais de résumer le soucis que vous avez en montrant comment vous le reproduisez avec un simple appel à "open".

- W

@lg_53 merci

@wiztricks

justement je n'arrivais pas a reproduire ce bug qu'avec le code complet en utilisant open autrement je n'avais aucun comportement étrange. (et maintenant je comprend pourquoi)
Toute fois j'ai trouvé mon problème. Alors pour coder j'utilise vim et apparemment j'ai eus un comportement étrange au niveau de mon .py où un '/' n'était pas enregistré comme un '/' mais comme le code '∕'(barre de division) qui correspond au code unicode U+2215 au lieu de la barre oblique U+002F. Bref je viens de le remarquer en relisant les bouts de code que j'ai copié depuis mon git ... (dernière balise ligne 23) (je précise que je n'ai aucune différence d'affichage sur mon vim entre ces deux caractères d'où mon problème)
Bon j'ai aucune idée de comment j'ai fais ça mais au moins maintenant c'est réglé.
Merci pour votre temps.

Citation:

---

Envoyé par Alpam

Toute fois j'ai trouvé mon problème.

---

Relisez votre code:

Code:

---

1 2 3

def addDb(path, name): create(path+"/"+name+".conf") writeAtEnd(path+"/"+name+".conf", name+"\n")

---

En 2 instructions, vous construisez 2 fois la même chaîne de caractères. Si vous faites un changement, il ne faudra pas oublier de le faire autant de fois que ces répétitions.
De plus, os.path.join(path, name+'.conf') s'occupe déjà des "/"
Lorsque vous écrivez:

Code:

---

1 2 3 4 5 6

def create(path): try : f=open(path,'w') f.close except OSError: print("Impossible de créer le fichier "+path+".")

---

Vous attrapez l'OSError mais l'appelant continue comme si rien n'était alors que c'est a son niveau que l'erreur doit être traitée.
Avec Python on peut écrire çà:

Code:

---

1 2	with open(path, 'w'): pass

---

Ce qui évite d'oublier de fermer le fichier ou de penser l'avoir fait avec un .close qui n'est pas appelé.

- W