Discussion :

[Celonius]

Bonjour,

J'essaie de creer un fichier de sauvegarde.
Un espace vient s'intercaler entre deux variable.

mon code :

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
17
18
19
20
21
22
23
24
25
 
class Perso:
    def __init__(self):
        self.nom = 'xxx'
        self.classe = 'xxx'
        self.lvl = 1
        self.xp = 0
        self.pv = 10
perso = Perso()
 
def Save() :
    liste = [perso.nom, perso.classe, perso.lvl, perso.xp, perso.pv]
    with open('save.txt', 'wt') as save:
        for i in liste:
            save.write("%s\n" % str(i))
 
 
def Load():
    with open('save.txt', 'rt') as load_nom:
        load = load_nom.readlines()
        perso.nom = load[0]
        perso.classe = load[1]
        perso.lvl = int(load[2])
        perso.xp = int(load[3])
        perso.pv = int(load[4])

l'erreur dans le fichier texte :
XXX

XXX

1
0
10

J'ai ajouter un sleep() apres le save.write() mais ca ne change rien.

Quelqu'un aurait une astuce ?
D'avance merci

[wiztricks]

Salut,

Un espace vient s'intercaler entre deux variable.

Ce n'est pas un "espace" mais une ligne vide.
.readlines retourne une liste de lignes (se terminant par "fin de ligne" (\n)).
Pour les lignes converties en int, Python fait le ménage... pour les autres, c'est à vous de le faire.

- W

[ocornet]

Il suffit de faire un strip du/des caractères de fin de ligne au moment du readlines
Pour cela éditer le load comme ceci

Code :

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

1
2
 
load = load_nom.readlines().strip('\n')

[Sve@r]

Bonjour

Code python :

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

1
2
3
4
5
6
7
8
def Load():
    with open('save.txt', 'rt') as load_nom:
        load = load_nom.readlines()
        perso.nom = load[0]
        perso.classe = load[1]
        perso.lvl = int(load[2])
        perso.xp = int(load[3])
        perso.pv = int(load[4])

Quelqu'un aurait une astuce ?

Déjà les affectations n'ont pas besoin d'être dans le "with". Et l'astuce c'est suprimer le '\n' quand tu lis le fichier

Code python :

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

1
2
3
4
5
6
7
8
def Load():
    with open('save.txt', 'rt') as fp:
        lines = tuple(x.strip() for x in fp.readlines())
    perso.nom = lines[0]
    perso.classe = lines[1]
    perso.lvl = int(lines[2])
    perso.xp = int(lines[3])
    perso.pv = int(lines[4])

Ou alors supprimer le '\n' juste pour "nom" et "classe" mais on préconise généralement de nettoyer ce qui entre dans un programme au plus tôt (et, pour ce qui sort, de mettre au format de sortie au plus tard de ce qui sort)

load = load_nom.readlines().strip('\n')

Bien tenté, mais file.readlines() renvoie une liste, et une liste n'a pas d'attribut strip()

[ocornet]

Bien tenté, mais file.readlines() renvoie une liste, et une liste n'a pas d'attribut strip()

En effet, c'est l'évidence même, ça m'apprendra à faire 36 trucs en même temps ...

[ocornet]

Code :

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

1
2
 
        lines = tuple(x.strip() for x in fp.readlines())

Les compréhensions de liste c'est tellement pratique ...
Pourquoi convertir en tuple ?

Code :

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

1
2
 
        lines = [x.strip() for x in fp.readlines()]

[disedorgue]

Les compréhensions de liste c'est tellement pratique ...
Pourquoi convertir en tuple ?

Code :

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

1
2
 
        lines = [x.strip() for x in fp.readlines()]

ça dépend du besoin, si on a pas besoin de faire de la transformation/modification par la suite, le tuple est mieux dans la mesure ou celui-ci est moins gourmand en mémoire, un peu plus rapide d'accès, peut être utilisé tel quel comme clé dans un dictionnaire,...

Par contre, si on a besoin de trier par exemple, on choisira la liste puisque celle-ci est modifiable sans que l'interpréteur en crée une nouvelle.

[Sve@r]

Les compréhensions de liste c'est tellement pratique ...
Pourquoi convertir en tuple ? lines = [x.strip() for x in fp.readlines()]

Ce que tu as écrit peut s'écrire aussi lines = list(x.strip() for x in fp.readlines()) (ce qui est d'ailleurs mon écriture habituelle, préférant écrire plus explicite). Et donc entre list(...) ou tuple(...) (qui, quand on veut créer un tuple, est malheureusement la seule écriture possible) c'est tout aussi pratique. Ou, dit autrement, la notion de "comprehension de liste" peut s'appliquer exactement de la même façon aux tuples ou aux dictionnaires (peut-être peut-on dire alors "compréhension de tuple" et "compréhension de dictionnaire" ?).
Et quand je dois créer une collection d'éléments, je choisis toujours la collection la plus adaptée à mon besoin. Pourquoi créer une liste si je n'ai pas besoin de modifier son contenu ? Un tuple suffit.

[ocornet]

Ce que tu as écrit peut s'écrire aussi lines = list(x.strip() for x in fp.readlines()) (ce qui est d'ailleurs mon écriture habituelle, préférant écrire plus explicite). Et donc entre list(...) ou tuple(...) (qui, quand on veut créer un tuple, est malheureusement la seule écriture possible) c'est tout aussi pratique. Ou, dit autrement, la notion de "comprehension de liste" peut s'appliquer exactement de la même façon aux tuples ou aux dictionnaires (peut-être peut-on dire alors "compréhension de tuple" et "compréhension de dictionnaire" ?).
Et quand je dois créer une collection d'éléments, je choisis toujours la collection la plus adaptée à mon besoin. Pourquoi créer une liste si je n'ai pas besoin de modifier son contenu ? Un tuple suffit.

Bonjour Sve@r,

Je suis d'accord avec toi, explicite est mieux qu'implicite, comme le recommande si bien un "import this"
A ce sujet je viens de faire une expérience amusante, je développe sous PyCharm et il y a quelque temps j'ai installé le plugin Sourcery qui fait du refactoring à la volée sur base d'une IA.
J'ai écris une petite fonction pour lire le contenu d'un fichier csv (oui je sais qu'il existe des librairies pour gérer des fichiers csv) et dans cette fonction j'ai justement une compréhension de liste que j'ai délibérément écrite de manière explicite.

Et bien ce cher plugin Sourcery ne veut pas d'écriture explicite, voilà sa recommandation ... l'IA a encore un peu de travail
Par contre, évidemment si je mets tuple(...) là il ne dit rien, normal.

[wiztricks]

dans cette fonction j'ai justement une compréhension de liste que j'ai délibérément écrite de manière explicite

Une compréhension de liste est une façon concise pour créer des listes: les [] sont préférables aux list(...).

Là où l'IA pêche, c'est de ne pas vous dire que l'exception retournée pour une fichier qui n'existe pas sera FileNotFoundError et non IOError...

Dit autrement, elle sait reconnaître les constructions de bases mais n'essaie pas de comprendre l'intention que vous aviez en écrivant ce code pour assurer que çà va le faire ou pas.

- W

[ocornet]

Une compréhension de liste est une façon concise pour créer des listes: les [] sont préférables aux list(...).

C'est comme ça; []; que j'ai l'habitude de les écrire aussi.
C'était pour faire référence au message où Sve@r indiquait qu'il préférait l'écriture explicite et que l'IA faisait le contraire.

Là où l'IA pêche, c'est de ne pas vous dire que l'exception retournée pour une fichier qui n'existe pas sera FileNotFoundError et non IOError...
Dit autrement, elle sait reconnaître les constructions de bases mais n'essaie pas de comprendre l'intention que vous aviez en écrivant ce code pour assurer que çà va le faire ou pas.
- W

Certes, le contenu du print n'était dans ce cas pas approprié et aurait plutôt dû être "Erreur d'accès au fichier" puisque l'usage de IOError gère ici l'exception de manière globale.
Je concède que l'IOError n'est effectivement pas approprié dans le cas présent mais est malgré tout fonctionnel, en effet FileNotFoundError est une sous-classe de OSError et depuis la version 3.3 de Python, IOError a été fusionné dans OSError, devenant de ce fait un alias de OSError, donc ça le fait malgré tout

[Sve@r]

les [] sont préférables aux list(...)

Ah ? Pourquoi ça ?

[wiztricks]

Ah ? Pourquoi ça ?

Parce que dans les compréhensions de listes on cherche à être concis.
Donc soit on écrit:

Code :

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

squares = list(map(lambda x: x**2, range(10)))

soit on fait:

Code :

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

squares = [x**2 for x in range(10)]

Le résultat est le même mais on évite de mélanger les genres.

- W