Discussion :

[Alpam]

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 :

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

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 ?

[wildshadow956]

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 :

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

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.

[wiztricks]

Salut,

Qqun à une idée ?

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

- W

[Alpam]

@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 :/

[wiztricks]

@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

[tyrtamos]

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 :

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

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?

[Alpam]

@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 :

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

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 :

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

1
2
 
from .utils import dir_func,file_func

FACTORY db_factory.py:

Code :

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

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 :

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

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 :

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

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 :

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

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 :

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
 
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à

[lg_53]

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 ...

[wiztricks]

Salut,

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

[Alpam]

@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.

[wiztricks]

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

Relisez votre code:

Code :

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

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 :

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

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 :

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

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