Discussion :

[CaramelDur]

Bonjour,

Pour ouvrir un fichier CSV j ai utilisé la méthode suivante :

Code :

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

1
2
3
4
5
6
7
if not(os.path.isfile(fichier)):
        print ("pas trouvé")
    else:
        with open(fichier) as csvfile:
            ligne = csv.reader(csvfile, delimiter =",")
            for row in ligne:
                if row[0] == blablablabla etc etc ....

Ca marche tres tres bien, juste je me demande si :

Code :

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

1
2
3
4
5
6
7
8
9
10
try:
        with open(fichier) as csvfile:
            ligne = csv.reader(csvfile, delimiter =",")
            for row in ligne:
                if row[0] == blablablaetcetc ....:
                   ...
 
 
except IOError:
        print ('pas trouvé)

.... serait :
-plus leger a traiter pas le systeme ? En effet avec la premiere methode j effectue 2 actions sur le disque, verifier sa présence pour ensuite l ouvrir(oui bon ok je sais que maintenant surtout avec l avenement des NVMe c est leger, mais sur quelques millions de requete ....)
-plus rapide ?
-plus élégant ?
-stupide ? Tordu ? Déconnant ?

Cela a l air de tourner aussi bien dans un cas ou dans l autre, mais si quelqu un a aussi une solution pour implementer la commande time sous windows je suis preneur aussi

Merci d avance

[wiztricks]

Salut,

-stupide ? Tordu ? Déconnant ?

Vous allez un peu trop dans les détails.

Si vous avez besoin de lire certaines lignes d'un fichier CSV, il est préférable de penser à créer une fonction nommée par exemple get_rows qui prenne le nom du fichier en paramètre et qui retourne les lignes sélectionnées.

Une fois que la fonction "marche", vous allez pouvoir l'utiliser pour écrire le reste du code sans trop regarder ses instructions.

Ce n'est que lorsque le programme fonctionne que vous allez vous posez des questions sur son optimisation (une fois que le code est "stable" et fonctionnel pas quand il est encore en chantier, c'est prématuré).

mais si quelqu un a aussi une solution pour implementer la commande time sous windows je suis preneur aussi

La bibliothèque datetime est "standard".

- W

[CaramelDur]

Re bonsoir,
Tout d abord merci pour cette réponse .... pour le moins surprenante !
En effet, ma question portait essentiellement sur ce que je nommerais "les bonnes pratiques" et/ou optimisation.... Navré si je me suis mal exprimé en ce sens.
Mon code est bien fonctionnel et c est en repensant celui ci dans un but effectivement d optimisation que l idée de la seconde methode m est venue.
Alors oui effectivement ce code tel que présenté n est pas fonctionnel puisqu il ne reflette qu une partie de mon travail, cette méthode de recherche au sein d un fichier csv étant bien inscrite au sein d une fonction, merci de bien vouloir considerer l idée, la methode, les deux fonctionnants très bien.
Dans cette attente donc.
Cordialement

[Sve@r]

Bonsoir

Je suis d'accord avec le try...except. D'autant plus que j'ai posé une question analogue il y a quelques temps et que ça a été la réponse privilégiée.

Dans la première version il y a toujours un risque que le fichier soit modifié/perdu entre le test d'existence et l'ouverture concrète. La seconde version est la méthode assez souvent privilégiée. On se jette directement à l'eau et c'est seulement s'il n'y a pas d'eau que là on avise.

[CaramelDur]

Tres bon topic linké, un grand Merci a toi