Comment aurais-je pu mieux faire

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Comment aurais-je pu mieux faire

snafu8

Salut à tous,

je débute en python, je m'en sert pour faire des petites conneries, genre opérations répétitives sur des fichiers, manipulation de texte, etc.

Récemment, j'ai du faire un script qui allait chercher deux dossier dans un chemin donné en ligne de commande, récupère la liste des fichiers dans chacun de ces deux dossiers, puis crée un fichier texte qui va contenir les deux listes entrelacées.

J'ai pas vraiment de probleme, dans la mesure ou ca marche, mais je voulais savoir si il y avait un moyen plus succint et plus élégant de faire comme j'ai fait.

Code :

import glob
import sys
 
if len(sys.argv) < 2 :
  print "Entrer un chemin vers un dossier"
  exit
  
flux = open(sys.argv[1]+'/image_list.txt', 'w')
listG = glob.glob(sys.argv[1]+'/imageG/*')
listD = glob.glob(sys.argv[1]+'/imageB/*')
 
if len(listD != len(listG):
  print "Pas le meme nombre de fichier dans les deux dossiers"
  exit
 
range = range(len(listG))
range.pop()
 
for i in range:
  flux.write(listG[i])
  flux.write("\n" )
  flux.write(listD[i])
  flux.write("\n" )
flux.close()

Message édité par snafu8 le 10-03-2011 à 18:53:59

Publicité

snafu8

Serieusement les gens, je vous donne a tous une occasion sans pareil de faire étalage de vos connaissance et pas une réponse? J'ai pas pondu la solution optimale du premier coup quand meme?

gilou

Modérateur
Modosaurus Rex

Citation :

Comment aurais-je pu mieux faire

En n'oubliant pas une parenthèse?
if len(listD) != len(listG):
:whistle:

flux = open(sys.argv[1]+'/image_list.txt', 'w')
c'est mal placé, il faut plutôt le mettre après le test if len(listD)!= len(listG): car si tu quittes sur ce test, inutile d'avoir ouvert en écriture.

Bon, si j'avais eu à le faire en perl, j'aurais procédé ainsi:

Code :

#!/usr/local/bin/perl
use strict;
use warnings;
use File::Glob ':glob';
use Maptastic  ':perly';
 
@ARGV or die "Entrer un chemin vers un dossier\n";
(-d $ARGV[0]) or die "$! $ARGV[0]\n";
(-d $ARGV[0].'/imageG/') or die "$! $ARGV[0]/imageG/\n";
(-d $ARGV[0].'/imageB/') or die "$! $ARGV[0]/imageB/\n";
 
my @g = bsd_glob($ARGV[0].'/imageG/*', GLOB_QUOTE);
my @b = bsd_glob($ARGV[0].'/imageB/*', GLOB_QUOTE);
(@g == @b) or die "Pas le meme nombre de fichier dans les deux dossiers\n";
 
open(my $fh, '>', $ARGV[0].'/image_list.txt') or die "$! $ARGV[0]/image_list.txt\n";
map_foreach {print $fh join("\n", @_), "\n"} \@g, \@b;
# ou bien si on veut tout écrire d'un coup mais ca bouffe plus de mémoire
# print $fh join("\n", map_shift {@_} \@g, \@b), "\n";
close($fh);

Je ne sais pas pour python, mais le glob de base de perl n'aime pas les blancs dans les paths, d'ou le recours à bsd_glob et le flag GLOB_QUOTE.
A+,

Message édité par gilou le 11-03-2011 à 01:00:07

---------------
There's more than what can be linked! -- Le capitaine qui ne veut pas obéir à la carte finira par obéir aux récifs. -- Il ne faut plus dire Sarkozy, mais Sarkozon -- (╯°□°)╯︵ ┻━┻

gzii

court-circuit

Bonjour,

Et ça sert à quoi d'avoir des listes entrelacées ? Es tu sûr que les fichiers se correspondent s'il y en a le même nombre, que leur tri est le même ?

masklinn

í dag viðrar vel til loftárása

listdir pour le contenu d'un dossier, pas besoin de sortir glob pour un truc pareil
izip + chain.from_iterable l'entrelac
en python, les chemins ne se join pas à la main, prière d'utiliser os.path.join

Code :

import functools, sys, os, itertools
path_to = functools.partial(os.path.join, sys.argv[1])
 
list1 = os.listdir(path_to('imageG'))
list2 = os.listdir(path_to('imageB'))
 
assert len(list1) == len(list2)
 
with open(path_to('list.txt'), 'wb') as out:
    for filename in itertools.chain.from_iterable(itertools.izip(list1, list2)):
        out.write(filename + '\n')

Si on veut éviter que ce ne soit lisible, je recommande d'y aller à coup de writelines ou de join par dessus, et en laissant le refcount CPython s'occuper du fichier:

Code :

import functools, sys, os, itertools
path_to = functools.partial(os.path.join, sys.argv[1])
 
list1 = os.listdir(path_to('imageG'))
list2 = os.listdir(path_to('imageB'))
 
assert len(list1) == len(list2)
 
open(path_to('list.txt'), 'wb').write(
    '\n'.join(itertools.chain.from_iterable(itertools.izip(list1, list2))))

Mais bon, sur ce genre d'utilities j'aurais tendance à sortir directement sur stdout et à piper depuis le shell.

Code :

import functools, sys, os, itertools
path_to = functools.partial(os.path.join, sys.argv[1])
 
list1 = os.listdir(path_to('imageG'))
list2 = os.listdir(path_to('imageB'))
 
if len(list1) != len(list2): sys.exit(1)
 
for filename in itertools.chain.from_iterable(itertools.izip(list1, list2)):
    print(filename)

Message édité par masklinn le 01-04-2011 à 22:17:44

---------------
I mean, true, a cancer will probably destroy its host organism. But what about the cells whose mutations allow them to think outside the box by throwing away the limits imposed by overbearing genetic regulations? Isn't that a good thing?

FORUM HardWare.fr

Programmation

Python

Comment aurais-je pu mieux faire

Sujets relatifs
[VBA] rechercheV en mieux...	software pour mieux respirer
Générateur de formulaire en PHP (et ajax ca serait mieux)	qu'est-ce qui serait le mieux
Qu'est qui vaut mieux pr les moteurs de recherches ? :)	Blabla@Programmation !! Y marche c'ui là (en attendant mieux)
J'ai essayé de faire mieux ...	Qui est mieux et professionnel:Une seul BD ou plusieurs ?
Que faut-il mieux faire ?	gerer mieux la mmemoire pour le calcul matriciel
Plus de sujets relatifs à : Comment aurais-je pu mieux faire

Page générée en 0.076 secondes