Discussion :

[Henrique92]

Salut tout le monde,

Voilà, je ne comprends pas, je tourne en rond sans savoir pourquoi je ne traite que les txt et non les TXT....

Code :

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

1
2
3
4
5
6
7
8
9
10
 
for fichier in os.listdir(base) :
    if os.path.splitext(fichier)[1]=='.txt' or os.path.splitext(fichier)[1]=='.TXT' : 
        Liste_fichiers.append(base+'\\'+fichier)
    if os.path.isdir(base+'\\'+ fichier):
        repertoire=base+'\\'+ fichier+'\\'
        os.chdir(repertoire)
        for fich in os.listdir(repertoire) :
            if os.path.splitext(fich)[1]=='.txt' or os.path.splitext(fichier)[1]=='.TXT' :
                Liste_fichiers.append(repertoire+fich)

La réponse est sous mon nez, je le sais ...

Merci !

[PauseKawa]

Bonjour,

Code :

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

if os.path.splitext(fichier)[1]=='.txt' or os.path.splitext(fichier)[1]=='.TXT'

comme os.path.splitext(fichier)[1]=='.txt' retourne True os.path.splitext(fichier)[1]=='.TXT' n'est pas traité (voir les valeurs booléennes et and/or).

Code :

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

if os.path.splitext(fichier)[1] in ('.txt', '.TXT')

ou

Code :

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

if os.path.splitext(fichier)[1].lower() == '.txt'

@+

[PauseKawa]

La doc qui vas avec

@+

[wiztricks]

Salut,

comme os.path.splitext(fichier)[1]=='.txt' retourne True os.path.splitext(fichier)[1]=='.TXT' n'est pas traité (voir les valeurs booléennes et and/or)

Certes mais si le nom du fichier se termine par ".TXT" le "or" devrait rendre la condition "True" => à la lecture du code, çà devrait marcher non?
- W

[PauseKawa]

?

This is a short-circuit operator, so it only evaluates the second argument if the first one is False.

[PauseKawa]

J'ai oublié le smiley (je le mérite bien sur ce coup)

[PauseKawa]

Par contre il y a ceci

Code :

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

1
2
        for fich in os.listdir(repertoire) :
            if os.path.splitext(fich)[1]=='.txt' or os.path.splitext(fichier)[1]=='.TXT' :

[wiztricks]

Salut,

J'ai oublié le smiley (je le mérite bien sur ce coup)

Hum... n'abusez pas trop de la moquette, c'est bon mais quand même...
- W

[wiztricks]

Salut,

Voilà, je ne comprends pas, je tourne en rond sans savoir pourquoi je ne traite que les txt et non les TXT....

A la lecture du code, effectivement, c'est un peu "bourrique" de comprendre: faites le parler pour vous assurez que ça fait bien ce que vous attendez:

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
def is_text(name): 
     e = os.path.splitext(name)[1]
     v = (e == '.txt' or e =='.TXT')
     print ("is_text(%s) = %s" % (name, v))
     return v
 
for fichier in os.listdir(base) :
    if is_text(fichier): 
        Liste_fichiers.append(base+'\\'+fichier)
    if os.path.isdir(base+'\\'+ fichier):
        repertoire=base+'\\'+ fichier+'\\'
        os.chdir(repertoire)
        for fich in os.listdir(repertoire) :
            if is_text(fich):
                Liste_fichiers.append(repertoire+fich)

et restructurez votre code pour tester la chose par petit bout, çà aide.
- W

[Henrique92]

Bonsoir et Merci !
Effectivement, j'ai fait les tests et ça marche comme ça.
Merci beaucoup pour l'explication en Anglais !!

Par contre, j'ai un autre souci avec cette boucle car elle n'accède pas aux sous répertoires. Mais je ne vois pas pourquoi ?
Merci.

[PauseKawa]

Bonjour,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
import os
 
list_dir = ['icilechemindebase']
Liste_fichiers = []
 
while list_dir:  # Tant qu'il y a quelque chose dans la liste des fichiers
    base = list_dir.pop(0)  # On traite le premier repertoire
    for e in os.listdir(base):
        elem = os.path.join(base, e)  # De toute maniere on a besoin du chemin complet.
        if os.path.isdir(elem):
            list_dir.append(elem)
        elif os.path.isfile(elem) and os.path.splitext(elem)[1] in ('.txt', '.TXT'): 
            Liste_fichiers.append(elem)

@+

[afranck64]

Juste pour savoir. Les autres combinaisons à casses mixées sont interdites? "tXt", "Txt", "TXt", "txT", ...
si non, pourquoi ne pas mettre l'extension en majuscule avant les tests?

[Henrique92]

Bonsoir
Un grand Merci PauseKawa
Par contre, pour revenir à ce que je disais mais je n'ai pas été clair, désolé. je voudrais savoir en quoi mon code ne fonctionne pas.

afranck64 Juste pour savoir. Les autres combinaisons à casses mixées sont interdites? "tXt", "Txt", "TXt", "txT", ...
si non, pourquoi ne pas mettre l'extension en majuscule avant les tests?

Effectivement, c'est une bonne question. Il n'y a aucune interdiction mais en même temps, les probabilités de trouver un TxT sont minimes dans mon cas.... Et ça surchargerait le programme...
Par contre, c'est vrai. Je n'avais pas pensé à modifier toutes les formes txt avant l'ouverture. Je vais devoir potasser !

Merci encore à vous !

[PauseKawa]

Bonsoir,

Effectivement, c'est une bonne question. Il n'y a aucune interdiction mais en même temps, les probabilités de trouver un TxT sont minimes dans mon cas.... Et ça surchargerait le programme...
Par contre, c'est vrai. Je n'avais pas pensé à modifier toutes les formes txt avant l'ouverture. Je vais devoir potasser !

Pas vraiment...

Code :

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

if os.path.splitext(fichier)[1].lower() == '.txt'

Pour ce qui est de votre erreur cherchez ici

Code :

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

1
2
3
4
5
6
    if os.path.isdir(base+'\\'+ fichier):
        repertoire=base+'\\'+ fichier+'\\'
        os.chdir(repertoire)
        for fich in os.listdir(repertoire) :
            if os.path.splitext(fich)[1]=='.txt' or os.path.splitext(fichier)[1]=='.TXT' :
                Liste_fichiers.append(repertoire+fich)

Et regardez ceci

Code :

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

1
2
3
4
5
6
7
8
9
10
11
list_dir = ['icilechemindebase']
Liste_fichiers = []
 
while list_dir:  # Tant qu'il y a quelque chose dans la liste des fichiers
    base = list_dir.pop(0)  # On traite le premier repertoire
    for e in os.listdir(base):
        elem = os.path.join(base, e)  # De toute maniere on a besoin du chemin complet.
        if os.path.isdir(elem):
            list_dir.append(elem)
        elif os.path.isfile(elem) and os.path.splitext(elem)[1] in ('.txt', '.TXT'): 
            Liste_fichiers.append(elem)

@+

[Henrique92]

Oui super bien vu pour tout.
Je vais encore potasser....