optimisation de code

Version imprimable

salut
a mon travail, mon superieur avait besoin d'un script ou programme a faire rapidement dans l'heure. je me suis proposer (C) il m'a dit que ca serait trop long alors j'ai tente python. ca marche comme je veux mais je me dit que ca doit etre tres sale comme code donc je vous le montre et demande de si c'est optimisable ou tout simplement nettoyable.
le but:
lire un fichier texte de la forme

Citation:

c:\dir\dir\dir\fichier.jpg
ligne a metre dans un fichier
celle ci aussi
c:\dir\dir\dir\fichier2.JPG
ligne a mettre dans un fchier

donc je recupere le nom du fichier.jpg et je fais un fichier.txt contenant les lignes suivante jusqu'au prochain chemin.
j'avais du mal a extraire le nom en fin de path donc j'ai magouille avec les methodes que j'ai trouver parci par la sur le net.
bon voici le code et merci de me dire si c'est vraiment horrible et comment le simplifier. je rappelle qui fonctionne comme je veux (peut etre par hazar).
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 fin=open('rien.txt','r') lines = fin.readlines() for i in range(len(lines)): cut = lines[i] if "JPG" in cut or "jpg" in cut: name = cut[::-1] pos = name.find('\\') name = name[0:pos:1] tmp = name.split('.') name = tmp[1] filename = name[::-1]+".txt" print "writing file : "+filename fout = open(filename,'w') i=i+1 while i<len(lines): cut = lines[i] if "JPG" in cut or "jpg" in cut: i=i-1 break; fout.write(cut) i = i+1 fout.close() fin.close()

Bonjour,

Etant donné que vous avez besoin d'une réponse rapide, voila un code rapide qui devrait convenir :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 fin=open('rien.txt','r') lines = fin.read().split("\n") # decoupe par ligne for i in lines: fichier = i.split("\\")[-1] if fichier.split(".")[-1] == "jpg" or fichier.split(".")[-1] == "JPG": # Image try: fichier.close() except: pass nom_fichier = fichier.split(".")[0] fichier=open(nom_fichier ,'w') else: # Lignes en dessous du fichier image fichier.write(str(i)) fichier.close() fin.close()
Utilisez absolument le split qui permet de découper une chaine et place le résultat dans une liste (tableau si vous connaissez le C). Le caractère dans le split découpe cette chaine avec ce caractère et sert de séparateur dans la liste.

Bon courage...

Mickael

15/12/2010, 11h58
pacificator

Bonjour,

vous pouvez aussi utiliser le module os, cf la FAQ.

Un rien plus concis :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#!/usr/bin/env python
# -*- coding: utf-8 -*-
 
import os
 
with open("infile.txt", "r") as inf:
    for line in inf.readlines():
        l = line[:-1]
        if l[-4:] in [".jpg", ".JPG"]:
            f = ".".join([l[:-4], "txt"])
        else:
            with open(f, "a") as outf:
                outf.write(line)

16/12/2010, 04h20
lilington

@Apocalypses
merci pour la reponse c'est ce que je cherche en plus claire que ce que j'ai fais cependant c'est apeine plus court que le mien bien que plus lisible.

@VinsS
effectivment tres concis donc pile poil le style que je cherche. juste un question, debutant en python ne serait-il pas raisonnable de me concentrer sur le code d'apocalypses pour mieux comprendre le language plutot que utiliser des pacquets tous prets? (sinon au boulot en effet question productivite c'est justement ce que je dois faire. mais ma question c'est point de vu pedagogique)

Bonjour,

Si par pédagogique on comprend 'bonnes pratiques' je penche poue le code de VinsS. Il n'est pas si difficile a comprendre en fait.
Bien qu'au niveau des noms de fichier de sortie cela ne fonctionne pas pour moi.
Voici ce qui fonctionne pour moi

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#!/usr/bin/env python
# -*- coding: utf-8 -*-
#
#
with open("rien.txt", "r") as inf: # with ferme automatiquement le fichier.
    for line in inf.readlines(): # Lecture ligne par ligne
        l = line[:-1] # Suppression du carractères de fin.
        if l[-4:].lower() == ".jpg": # Une alternative à la liste de VinsS. L'idée de la liste (que je préfère) est bonne aussi.
            chm = l[:-4].split("\\") # Décortique le chemin
            longchm = len(chm) - 1 # On prend le nombre d'éléments pour
            f = chm[longchm] + ".txt" # finalement ne garder que le dernier et lui rajouter .txt
        else:
            with open(f, "a") as outf: # Même principe with ferme automatiquement le fichier.
                outf.write(line)

Bonne découverte

Petite explication puisque j'ai deux minutes
J'ai rajouter des print pour voir comment cela se passe
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 #!/usr/bin/env python # -*- coding: utf-8 -*- # # with open("rien.txt", "r") as inf: for line in inf.readlines(): l = line[:-1] print l if l[-4:].lower() == ".jpg": print l[-4:].lower() chm = l[:-4].split("\\") print chm longchm = len(chm) - 1 print longchm f = chm[longchm] + ".txt" print chm[longchm] + ".txt" else: with open(f, "a") as outf: outf.write(line)
En voici la sortie
c:\dir\dir\dir\fichier.jpg C'est le print l. Sans le caractère de fin donc
.jpg lower() mets en minuscule
['c:', 'dir', 'dir', 'dir', 'fichier'] Décomposition de la ligne. Comme \ et un caractère d'échappement on en mets deux "\\"
4 C'est bien 5 éléments mais l'index commence à zéro.
fichier.txt Le dernier élément (index 4 ici) + .txt
ligne a metre dans un fichier Ecriture des lignes
celle ci aussi
c:\dir\dir\dir\fichier2.JPG On retrouve .jpg
.jpg etc...
['c:', 'dir', 'dir', 'dir', 'fichier2']
4
fichier2.txt
ligne a mettre dans un fchier

J'espère que c'est plus parlant.

Bon code

16/12/2010, 07h33
lilington

hmmmm, j'ai envi de vous embrasser tous domage que vos nom soit consonnance masculine.
merci on ne peut etre plus claire.
16/12/2010, 08h03
PauseKawa

Juste un dernier détail

Dans le cadre de fichiers de types multiples:
if l[-4:].lower() in [".jpg", ".bmp", ".cif"]:

Citation:

Envoyé par pacificator

vous pouvez aussi utiliser le module os, cf la FAQ.

Bonjour pacificator,

Vu le contenu du fichier rien.txt je pense que pour la portabilité il est préférable de l'attaquer en mode texte.
Je suis sur que le module os de mon pingouin n'apprécie pas les chemins de type c:\dir\dir\dir\fichier.jpg

Bon code
16/12/2010, 08h29
lilington

c'est drole mais je code sous fedora14. j'ai windows installe en dualboot mais j'y vais une fois par mois juste pour compiler la version windows (officielle) que la boite demande mais je travail sous linux en attendant que les binaires linux interresse ma boite.
bref j'ai la chance d'avoir un bosse qui me laisse choisir mon environement pour faire du C ca a d'ailleur l'avantage de m'obliger a etre portable donc oui en effet le mode texte.
16/12/2010, 10h11
VinsS
Là, c'est moi qui pose une question.

Je n'ai jamais utilisé python avec Windows et je soupçonne que ceci explique cela.

C'est quoi ça :
Code:

1 2 .split("\\")
merci
16/12/2010, 11h20
lilington

moi non plus j'ai jamais fait mais voila

sous linux tu as /home/machin/chouette
donc quand tu tape

Code:

.split("/")

pas de problemes c'est COOL.
mais sous windows c'est c:\machin\chouette
donc tu as

Code:

split("\")

si tu regardes bien il y a quelque chose qui cloche.le \ annule la fermeture du string donc il faut metre 2\ pour que le second soit considerer comme un element comprit entre "".notes la coloration syntaxique, visiblement on considere que j'ai pas ferme mon string(meme si j'en porte pas ;)).
c'est comme ci tu voulais ecrire un truc du genre salut je suis a' et lui est a" tu ferais:

Code:

print "je suis a' et il est \" "

.
si tu avait fait un peu de turbo C tu aurais ete habitue. on avait des trucs sous dos genre

Code:

f=fopen("c:\\fichier.txt","r");
16/12/2010, 13h21
VinsS

Ah oui, le séparateur doit être "échappé".

Donc, toujours laisser python manipuler les chemins lui-même avec
os.path.split et os.path.join entre autres.

merci

Bonsoir,

Oui, laisser à os et os.path le travail est une bonne idée.
Dans le cas présent nous avons un fichier rien.txt qui contient des ligne de type c:\dir\dir\dir\fichier.jpg, soit une représentation d'un chemin absolu Windows.
Donc si nous utilisons ce fichier sous Windows pas de problème.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 #!/usr/bin/env python # -*- coding: utf-8 -*- # # import os with open("rien.txt", "r") as inf: for line in inf.readlines(): l = line[:-1] if l[-4:].lower() == ".jpg": print os.path.basename(l) print os.path.split(l) chm = l[:-4].split("\\") longchm = len(chm) - 1 f = chm[longchm] + ".txt" else: with open(f, "a") as outf: outf.write(line)
Code:

1 2 3 4 fichier.jpg ('c:\\dir\\dir\\dir', 'fichier.jpg') fichier2.JPG ('c:\\dir\\dir\\dir', 'fichier2.JPG')
Mais voila ce que cela donne sous Linux.
Code:

1 2 3 4 c:\dir\dir\dir\fichier.jpg ('', 'c:\\dir\\dir\\dir\\fichier.jpg') c:\dir\dir\dir\fichier2.JPG ('', 'c:\\dir\\dir\\dir\\fichier2.JPG')
Dans le cas présent, un travail sur le texte même, dans le cadre d'un code compatible, me semble plus rapide que de traiter la ligne puis d'utiliser un module.
Ou je n'ai rien compris à os, que j'utilise pourtant.

Bon code.