Discussion :

[yvkoe]

Bonjour,
un petit programme pour compter les mots.J'ai soit un message d'erreur soit une boucle sans fin.Je ne peux pas spliter une liste?

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
#!/usr/bin/env python
#coding=utf-8
 
fis = raw_input('entrez le nom du fichier a traiter:')
fs = open(fis,'r')
n=0
while 1:
        ab = fs.readlines()
        if ab == " ":
             break
        cd= ab.split()
 
        n = n+len(cd)
fs.close()
print 'ce dossier comporte %s mots'%(n)

merci pour votre aide

[tyrtamos]

Bonjour,

Tu mélanges l'utilisation de readlines avec readline.

De plus, si tu utilises readline, chaque ligne est renvoyée avec une fin de ligne (qu'il faut retirer avec rsplit) et la fin de lecture du fichier est détectée avec if ligne == "".

Tyrtamos

[yvkoe]

Donc comme ca cela devrait fonctionner ...
Mais j'entre dans une boucle sans fin,je ne comprends pas pourquoi??

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
 
#!/usr/bin/env python
#coding=utf-8
 
fis = raw_input('entrez le nom du fichier a traiter:')
fs = open(fis,'r')
n=0
while 1:
        ab = fs.readline()
 
        cd= ab.split()
        if ab == " ":
             break
        n = n+len(cd)
fs.close()
print 'ce dossier comporte %s mots'%(n)

[yvkoe]

la methode readline() renvoie une chaine donc je peux spliter.Readlines() renvoie une liste donc non splitable.
Ok ca c'est vu maintenant pourquoi je ne peux pas sortir un résultat?

[yvkoe]

voila donc le code avec chaine

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
 
#!/usr/bin/env python
#coding=utf-8
 
fis = raw_input('entrez le nom du fichier a traiter:')
fs = open(fis,'r')
n=0
while 1:
        ch = fs.readline()
        if ch == " ":
             break
        cd= ch.split()
 
        n = n+len(cd)
fs.close()
print 'ce dossier comporte %s mots'%(n)

et il part dans une boucle sans fin et ...sans message d'erreur

[tyrtamos]

Dans mon message précédent, je voulais dire rstrip au lieu de split.

Ci-dessous ton code corrigé:

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
 
#!/usr/bin/env python
#coding=utf-8
 
fis = raw_input('entrez le nom du fichier a traiter:')
fs = open(fis,'r')
n=0
while 1:
        ab = fs.readline()
        if ab == "":
             break
        ab = ab.rstrip('\r\n ').split()
        n = n+len(ab)
fs.close()
print 'ce dossier comporte %s mots'%(n)

Tyrtamos

[yvkoe]

ok super ca fnctionne mais si tu pouvait m'expliquer pourquoi cela partait en boucle et sans message d'erreur je serais ravi d'avoir appris qqchose
merci

[nardo47]

Salut !

L'erreur de ton script était, comme l'avait indiqué Tyrtamos", de considérer que " " (1 blanc) indiquait la fin du fichier, alors que c'est "" (string vide... comme c'est triste, un string vide).

----
EDIT:

Comme tu attendais " " pour fermer le fichier et que lui t'envoyait "", tu ne sortais jamais de ta boucle.

----

Le seul problème qui persiste est que est considéré comme 1 mot tout ensemble de caractères non-blancs encadrés par 2 caractères blancs.
C'est à dire que si un symbole de ponctuation est collé au mot qui le précède et au mot qui le suit, l'ensemble ne comptera que comme 1 mot.
Perso, je déciderais de vivre avec.

Pour la bonne bouche :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
def comptemots(filename):
    total = 0
    # with/as ne marche qu'avec Python2.6 et superieur.
    with open(filename) as f:
        for line in f:
            total += len(line.rstrip().split())
    return total
 
filename = raw_input('Veuillez entrer le nom du fichier a traiter : ')
print 'Le fichier %s comporte %s mots.' % (filename, comptemots(filename))

1. with/as est "context manager" qui se charge de fermer le fichier à la sortie du bloc. En Python 2.5, on peut l'utiliser avec la ligne "from __future__ import with_statement".
Dans les versions antérieures, on le remplacera par :

Code :

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

1
2
3
4
 
f = open(filename)
#traitement
f.close()

2. une boucle for est + rapide qu'un while. De plus, tu ne tomberas jamais dans une boucle infinie avec une telle boucle.

3. Un objet fichier est son propre itérateur (du moins, dans les versions récentes de Python, mais je ne sais plus à partir de laquelle), il n'est donc pas nécessaire de faire appel à f.readlines() pour le parcourir avec une boucle for ;

4. rstrip supprime les caractères blancs par défaut

5. le reste n'est qu'optimisation.

[yvkoe]

comme c'est triste, un string vide

Bravo j'aime beaucoup!!!

merci pour toutes ces explications bien utiles.

[Antoine_935]

1. with/as est "context manager" qui se charge de fermer le fichier à la sortie du bloc. En Python 2.5, on peut l'utiliser avec la ligne "from __future__ import with_statement".
Dans les versions antérieures, on le remplacera par :

Code :

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

1
2
3
4
 
f = open(filename)
#traitement
f.close()

Le but principal d'un with est de s'assurer que l'action de sortie de contexte soit effectuée.
Dans ton cas, le f.close ne se fera pas si une exception est levée.
Ce serait donc plutôt

Code :

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

1
2
3
4
5
f = open(filename)
try:
    # traitement
finally:
    f.close()

On peut également entourer le f.close d'un try/except pour éviter que le f.close lui-même ne lève une exception.

[yvkoe]

Juste pour rire
si on reprend le premier code:



on remplace readlines par readline (une chaine pas une liste) et on enleve l'espace dans
if ab =="":
Bingo cela fonctionne
Et grace a ces deux erreurs j'ai appris plein de trucs et de machins...
Vraiment super ce site
et merci à ceux qui sont intervenus

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
 
#!/usr/bin/env python
#coding=utf-8
 
fis = raw_input('entrez le nom du fichier a traiter:')
fs = open(fis,'r')
n=0
while 1:
        ab = fs.readline()
        if ab == "":
             break
        cd= ab.split()
 
        n = n+len(cd)
fs.close()
print 'ce dossier comporte %s mots'%(n)

[nardo47]

Le but principal d'un with est de s'assurer que l'action de sortie de contexte soit effectuée.

Tout à fait d'accord, je n'étais pas rentré dans les détails paske je suis un gros fainéant !

On peut également entourer le f.close d'un try/except pour éviter que le f.close lui-même ne lève une exception.

Mais, dans ce cas-là, quelle est l'action à faire si le f.close() lève une exception ?
Je n'ai pas lu le PEP correspondant en détail, mais il me semblait qu'il était écrit à un moment qu'on ne pouvait pas rattraper toutes les erreurs (et que si le f.close() levait une exception, on avait peut-être de + gros problèmes, mais je fabule peut-être/sûrement/comme d'hab).

Quoiqu'il en soit, tu as eu raison de me corriger et d'apporter + de précisions !

merci à ceux qui sont intervenus

De rien, bonne continuation avec Python.

[Antoine_935]

Mais, dans ce cas-là, quelle est l'action à faire si le f.close() lève une exception ?
Je n'ai pas lu le PEP correspondant en détail, mais il me semblait qu'il était écrit à un moment qu'on ne pouvait pas rattraper toutes les erreurs (et que si le f.close() levait une exception, on avait peut-être de + gros problèmes, mais je fabule peut-être/sûrement/comme d'hab).

Concrètement, si ton f.close() lève une exception, la seule chose que tu peux faire c'est... rien. Enfin si, peut-être logger, prendre quelques mesures de nettoyage. Une erreur au close signifie que le fichier n'est plus ouvert. Ca peut arriver de différentes manières. Dans le cas de sockets par exemple, le descripteur est fermé par l'os après certains types d'erreurs. Un close échouerait donc lamentablement, plantant éventuellement ton script.

[tyrtamos]

Bonjour,

En ce qui me concerne, je n'utilise pas with, mais j'ai pris en compte la gestion de la fermeture de la façon suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
def machin(nfc):
    f = None
    try:
        f = open(nfc, 'r')
        # .....
        # .....
    finally:
        if f != None:
            f.close()

L'avantage, c'est que si l'erreur est au niveau de l'open, le close n'est pas exécuté. Ce n'est toujours pas parfait puisque f.close() peut encore déclencher une erreur. On peut peut-être encore avoir un try à l'intérieur de finally pour traiter les erreurs de close, mais ça commence à devenir compliqué. Et puis ça dépend de la nature de l'application.

Tyrtamos

edit: solution déjà citée plus haut: j'aurais dû tout lire!

[nardo47]

logger, prendre quelques mesures de nettoyage.

Effectivement, c'est une bonne idée : j'en étais encore resté à essayer de corriger la cause de l'erreur, et je ne voyais pas quoi faire.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
def machin(nfc):
    f = None
    try:
        f = open(nfc, 'r')
        # .....
        # .....
    finally:
        if f != None:
            f.close()

L'avantage, c'est que si l'erreur est au niveau de l'open, le close n'est pas exécuté.

Le problème que je vois (mais c'est mon analyse, peut être mets-tu + de chose dans le finally ou peut-être me trompe-je tout simplement), c'est que si l'ouverture ne marche pas, on n'est pas averti.
Quand j'essaie d'ouvrir un fichier qui n'existe pas (problème de path, par exemple), j'aime bien être au courant.

Sinon, histoire de pinailler :

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
 
# boaf
if f != None:
    f.close()
 
# mieux
if f is not None:
    f.close()
 
# le top !
if f:
    f.close()
 
# l'horreur absolue !
f and f.close()

J'avoue avoir fait de la résistance, à continuer à tester comme toi, mais finalement, j'ai pris goût à ce genre de test.
Après, je ne cherche pas à te convaincre absolument, chacun son goût !

[tyrtamos]

Bien vu, nardo47.

J'ai voulu faire simple, mais en fait j'utilise quelque chose d'un peu plus lourd parce que j'aime bien aussi être averti quand il y a un pb:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
def machin(nfc):
    f = None
    try:
        f = open(nfc, 'r')
        # .....
        # .....
        f.close()
    except IOError:
        if f != None:
            f.close()
        raise   # pour renvoyer l'exception à l'appelant

Merci pour les autres remarques, mais je préfère privilégier la clarté du code quitte à ne pas être très élégant, tant pour faciliter le debugging que pour comprendre facilement ce que j'ai fait dans 3 mois.

Tyrtamos