creation d'objet à partir d'un fichier texte.

hello,

j'ai beau chercher, j'ai rien trouvé et je me demande maintenant si python sait faire :

mon pb :

je crée une classe "obj" avec, disons, deux attributs

j'ai un fichier texte qui contient n lignes de type : "toto 2 3", "tata 6 3", etc...

est-il possible de lire le fichier et de créer des objets toto, tata... appartenant à la classe "obj" et d'instancier ces objets avec les valeurs contenues dans le fichier ?


là ou j'en suis :

je lis les lignes, je les split (x=ligne.split()) ,et je fais un globals()[x[0]] = obj()
je trouve bien un objet toto qui appartient à la classe obj()

mais je ne trouve pas comment instancier toto avec les valeurs 2 et 3 , je me retrouve toujours avec les valeurs par défaut de obj


quelqu'un sait si c'est faisable, et comment ?

merci

Bonsoir,

penchez-vous sur setattr, vous pourrez choisir entre toto ou tata comme attributs de votre classe à alimenter avec les valeurs lues dans votre fichier d'entrée.

Code:

---

setattr(ma_classe, 'mon_attribut', 'ma_valeur')

---

ben non, ça marche pas mieux :-)

le pb, c'est que le programme ne connait pas toto ou tata, il ne connait que x[0], et même si la valeur de x[0] (toto) correspond bien à un objet de la classe obj; python ne reconnait pas x[0] comme appartenant à la classe, je ne peux donc pas instancier.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

class obj:   #constructeur def __init__(self): #lister les champs AD = 0.0 DEC = 0.0   #fin constructeur   pass   filin = open("fic.txt", "r") ligne = filin.readline() x = ligne.split() globals()[x[0]] = obj() setattr(x[0],AD, x[2]) filin.close()

---

ça donne :

Citation:

---

name 'AD' is not defined

---

alors que toto donne bien :

Citation:

---

<__main__.objet at 0x7f6f5d7c9978>

---

ah, avec les guillemets autour de AD, c'est pas mieux : AttributeError: 'str' object has no attribute 'AD' :-(

le truc, c'est comment faire pour que python comprenne "toto" en voyant "x[0]", c'est peut-être pas possible...

Salut,

Citation:

---

Envoyé par alpha.centauri

est-il possible de lire le fichier et de créer des objets toto, tata... appartenant à la classe "obj" et d'instancier ces objets avec les valeurs contenues dans le fichier ?

---

Ca n'a rien de magique.

Si on définit la classe par:

Code:

---

1 2 3 4 5

class A: def __init__(self, name, x, y): self.name = name self.x = x self.y = y

---

instancier une instance se fera via A('toto', 2, 3).

Reste à transformer la ligne "toto 2 3\n" en le tuple t = ('toto', 2, 3) pour instancier le A via A(*t)

Ceci dit, un tel fichier, c'est des lignes avec des colonnes "nom", "x", "y". Le plus simple est d'en faire un tableau avec une liste de listes.
Vous aurez le même résultat sans utiliser les class (qu'il faut apprendre avant).

- W

Salut,

Citation:

---

Envoyé par alpha.centauri

je crée une classe "obj" avec, disons, deux attributs

j'ai un fichier texte qui contient n lignes de type : "toto 2 3", "tata 6 3", etc...

est-il possible de lire le fichier et de créer des objets toto, tata... appartenant à la classe "obj" et d'instancier ces objets avec les valeurs contenues dans le fichier ?

---

Peut-être que tu devrais revoir le problème pour éviter d'avoir à faire cela. Il y a d'autres façons de "stocker" des objets dans un fichier...


Mais bon je propose le code ci-dessous qui fait ce que tu veux faire (du moins ce que j'ai compris de ce que tu veux faire), je reprends ton code que j'adapte :

Contenu du fichier fic.txt :

Code:

---

1 2 3

toto 2 3 tata 6 1 titi 7 5

---

Et le code avec des tests à la fin :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

class Obj: def __init__(self, p1, p2): self.AD = p1 self.DEC = p2   filin = open("fic.txt", "r") for ligne in filin: x = ligne.split() exec("%s = Obj(%s,%s)" % (x[0], x[1], x[2])) filin.close()   # tests : print("toto.AD:", toto.AD, "toto.DEC:", toto.DEC) print("tata.AD:", tata.AD, "tata.DEC:", tata.DEC) print("titi.AD:", titi.AD, "titi.DEC:", titi.DEC)

---

Résultats des tests :

Code:

---

1 2 3

toto.AD: 2 toto.DEC: 3 tata.AD: 6 tata.DEC: 1 titi.AD: 7 titi.DEC: 5

---

toto, tata et titi sont bien des instances de la class Obj...

J'ai testé et cela fonctionne mais je ne dis pas que c'est une bonne pratique de faire comme cela...

en fait, c'est quand même un peu plus compliqué, il n'y a pas que des attributs dans mes objets, y a aussi des méthodes...

c'est clair que je pourrais sans doute faire ça "à l'ancienne", mais bon, y parait que l'OO, c'est mieux :-) :-) :-)

oui, c'est bien le résultat cherché; merci pour ton code, c'est sûr que j'aurais pas trouvé ça tout seul en lisant la doc ...

j'essaye de comprendre ce que tu as écris, et je teste ça demain... ;-)

Citation:

---

Envoyé par alpha.centauri

en fait, c'est quand même un peu plus compliqué, il n'y a pas que des attributs dans mes objets, y a aussi des méthodes...

---

Que veux-tu faire exactement ? Ce que j'ai compris c'est que tu voulais que toto, tata... etc soient des instances de la class Obj et que tu veux "...instancier ces objets avec les valeurs contenues dans le fichier" mais ces valeurs sont censées initialiser quelles propriétés ? J'ai déduis de ton code que c'était AD et DEC, est-ce bien cela ?

Citation:

---

Envoyé par wiztricks

Salut,



Ca n'a rien de magique.

Si on définit la classe par:

Code:

---

1 2 3 4 5

class A: def __init__(self, name, x, y): self.name = name self.x = x self.y = y

---

instancier une instance se fera via A('toto', 2, 3).

Reste à transformer la ligne "toto 2 3\n" en le tuple t = ('toto', 2, 3) pour instancier le A via A(*t)

Ceci dit, un tel fichier, c'est des lignes avec des colonnes "nom", "x", "y". Le plus simple est d'en faire un tableau avec une liste de listes.
Vous aurez le même résultat sans utiliser les class (qu'il faut apprendre avant).

- W

---

euh, j'ai peut-être loupé un truc, mais en faisant ça, je retrouve pas l'objet "toto" (name 'toto' is not defined)

Citation:

---

Envoyé par alpha.centauri

euh, j'ai peut-être loupé un truc, mais en faisant ça, je retrouve pas l'objet "toto" (name 'toto' is not defined)

---

Oui c'est normal, dans le code de wiztricks, toto n'est pas une instance de la class A, ce n'est même pas une variable, en fait c'est la chaine "toto" qui se retrouve être la valeur de la propriété name de la class A.

Tu peux tester cela pour voir :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

class A: def __init__(self, name, x, y): self.name = name self.x = x self.y = y     instance_A = A('toto', 2, 3) print("instance_A.name:",instance_A.name) # affiche: instance_A.name: toto print("instance_A.x:",instance_A.x) print("instance_A.y:",instance_A.y)

---

C'est pour cela qu'il faut que tu précises bien ce que tu veux...

Citation:

---

Envoyé par Beginner.

Que veux-tu faire exactement ? Ce que j'ai compris c'est que tu voulais que toto, tata... etc soient des instances de la class Obj et que tu veux "...instancier ces objets avec les valeurs contenues dans le fichier" mais ces valeurs sont censées initialiser quelles propriétés ? J'ai déduis de ton code que c'était AD et DEC, est-ce bien cela ?

---

si tu veux tout savoir, l'idée c'était de définir une classe avec ses propriétés (AD, DEC) et ses méthodes (AD+DEC, par exemple)

j'ai un fichier texte de n lignes avec, pour chaque ligne, le nom de l'objet, son AD et son DEC

le but du jeu est de sélectionner le nom de l'objet via une liste déroulante et de retourner son AD+DEC correspondant.

ça se fait très bien sans programmation objet, mais je n'avais pas encore eu l'occasion de l'utiliser pour faire ce genre de chose...

Citation:

---

Envoyé par alpha.centauri

ça se fait très bien sans programmation objet, mais je n'avais pas encore eu l'occasion de l'utiliser pour faire ce genre de chose...

---

Quand je te disais que tu pouvais faire autrement je ne voulais pas te dire de ne pas utiliser la POO...

Citation:

---

Envoyé par alpha.centauri

si tu veux tout savoir, l'idée c'était de définir une classe avec ses propriétés (AD, DEC) et ses méthodes (AD+DEC, par exemple)

j'ai un fichier texte de n lignes avec, pour chaque ligne, le nom de l'objet, son AD et son DEC

le but du jeu est de sélectionner le nom de l'objet via une liste déroulante et de retourner son AD+DEC correspondant.

---

Ok mais pour cela tu n'es pas obligé de faire en sorte que toto soit une instance de la class Obj, le nom de l'objet pourrait juste être une string pour identifier l'objet et là on en revient au code proposé par wiztricks qui est un début de solution...

Mais si tu tiens quand même à faire cela (c'est-à-dire faire en sorte que toto soit une instance de la class Obj) alors tu peux tester le code que j'ai proposé...

Si les noms d'objets sont connus à l'avance et que seuls les valeurs sont différentes alors tu peux simplifier les choses...

PS : Tu es obligé de stocker cela dans un fichier ?

Citation:

---

Envoyé par Beginner.

PS : Tu es obligé de stocker cela dans un fichier ?

---

vu le nombre d'objets, ça me semble le plus pratique (surtout pour les ajouts/modifs)

Citation:

---

Envoyé par alpha.centauri

vu le nombre d'objets, ça me semble le plus pratique (surtout pour les ajouts/modifs)

---

Ok.

Le code que j'ai posté te permet de faire de toto, tata,... des instances de la class Obj avec les deux paramètres en question...

Mais pour ton problème je pense que j'utiliserais plutôt un dictionnaire : chaque clé serait une string identifiant une option de ta liste déroulante et la valeur associée serait une instance de ta class.

dans ton idée, tu peux construire le dictionnaire dynamiquement ( j'ai un doute) ? ou tu dois modifier le dictionnaire dans le programme à chaque fois que tu modifies le fichier (ce qui n'est pas souhaité :-) )

Citation:

---

Envoyé par alpha.centauri

dans ton idée, tu peux construire le dictionnaire dynamiquement ( j'ai un doute) ? ou tu dois modifier le dictionnaire dans le programme à chaque fois que tu modifies le fichier (ce qui n'est pas souhaité :-) )

---

Oui c'est possible...

Essaye ça (en reprenant le même contenu pour le fichier) :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

class Obj: def __init__(self, AD, DEC): self.AD = AD self.DEC = DEC     dico_instances = {}   filin = open("fic.txt", "r") for ligne in filin: x = ligne.split() dico_instances[x[0]] = Obj(int(x[1]), int(x[2])) filin.close()   # tests :   print("toto.AD:", dico_instances["toto"].AD, "toto.DEC:", dico_instances["toto"].DEC) print("tata.AD:", dico_instances["tata"].AD, "tata.DEC:", dico_instances["tata"].DEC) print("titi.AD:", dico_instances["toto"].AD, "titi.DEC:", dico_instances["titi"].DEC)

---

# Résultats des tests :

Code:

---

1 2 3

toto.AD: 2 toto.DEC: 3 tata.AD: 6 tata.DEC: 1 titi.AD: 7 titi.DEC: 5

---

Bonjour

Citation:

---

Envoyé par alpha.centauri

vu le nombre d'objets, ça me semble le plus pratique (surtout pour les ajouts/modifs)

---

En lisant tout ce que tu as écrit, j'ai l'impression que ton truc est absurde. Que tu confonds "objet poo" et "objet".

En poo un objet est un outil de programation. Il sert à simplifier un concept algorithmique et à le réduire en une somme de valeurs (des attributs) et d'opérations (des méthodes). Si par exemple je veux gérer des comptes bancaires (nom + montant) je peux créer un objet "compte" avec l'attribut "montant" et des méthodes "depose()" et "retire()". Ainsi, une fois l'objet créé, ce sera plus simple quand M. Dupond veut déposer 1000€ d'appeler liste_compte["dupont"].depose(1000) (présumant avoir plusieurs comptes distincts dans le dictionnaire "liste_comptes"). On peut faire sans bien sûr (on peut toujours faire sans) mais dans ce cas le code à écrire sera un peu plus complexe. Bref l'objet sert à simplifier ensuite le code qui s'y appuiera (et c'est bien car il y a généralement plus de code en amont d'un objet qu'en aval donc si on peut simplifier ce code amont c'est plus pratique pour tout le monde).

Toi je te lis, tu parles d"objets dans un fichier. Admettons. Mais en quoi ces objets serviront ton algorithme que tu souhaites les "instancier" ? Si ces "objets" sont juste des objets de la vie courante (ex voiture+marque+vitesse, figures géométriques+couleur+nb_coté, sport+nb_pratiquants, etc) alors ce ne sont pas des objets au sens poo, juste des valeurs de travail pour ton algorithme. Et dans ce cas ils n'ont pas à être instanciés, simplement stockés en tant que valeur.
Tu as dit à un moment que tu avais réussi à le faire sans poo, et qu'en poo c'était plus difficile, je veux bien te croire. Parce que justement ces "objets" de ton fichier ne sont pas des objets de poo. A la limite si tu devais faire de la poo là dessus, ce serait alors pour créer un objet destiné à traiter ce fichier, objet qui contiendrait par exemples les méthodes "load()" et "save()".

Donc d'en ce que je lis, tu parles d'un jeu dans lequel l'utilisateur sélectionne le nom d'un truc dans une liste déroulante, ok je vais pas nier ton besoin mais ce qu'il sélectionne dans une liste déroulante ce ne sont pas des objets, juste des noms. Et ces noms dans le fichier ce n'est pas un objet, c'est de la simple data. Ensuite s'il sélectionne "carre" ben ton algo va traiter cette data et afficher un carré à l'écran (ou autre chose) mais ce n'est pas une raison pour devoir instancier quelque part un objet "carre", cet objet ne sert absolument à rien dans ton code.

De même tu as dit que ces objets contenaient aussi des méthodes. Là je reste totalement sec. Une méthode c'est avant tout un algorithme qui effectue un traitement et qui produit un résultat. Ce résultat peut-être interne (comme pour ma méthode "compte.depose()" qui se contente d'incrémenter le montant du compte) ou externe (comme par exemple un objet "cercle" pourrait avoir une méthode "perimetre()" qui calcule et renvoie le périmètre) mais ça reste fondamentalement un algorithme, même si cet algorithme se réduit à une seule instruction, et qui produit un résultat utilisable dans le code qui l'appelle. Quels résultats produisent ces fameuses "méthodes" des "objets" de ton fichier? Dans quel langage elles sont écrites? En quoi ces "résultats" sont utiles à ton algo? (si je crée la méthode "perimetre()" c'est parce que je sais que plus tard je devrai récupérer le périmètre d'un cercle). Et surtout enfin comment alors traduire ces "méthodes avec leur propre langage" en "méthodes poo langage Python" ???
Ou alors ton fichier est une partie de l'algo et dans ce cas tu n'as pas besoin de le lire et le traiter, juste l'écrire en Python et l'importer.

Non, quelque part je crois qu'il y a eu incompréhension de ta part entre "objet poo" et "objet data" qui t'a alors amené à nous présenter non pas le problème originel mais la solution que tu avais imaginé pour le résoudre. Bref cela ressemble à un cas de problème XY...

Ceci dit il existe parfois un besoin de sauvegarder ses objets Python dans un fichier. Par exemple pour conserver l'état d'un objet afin de le retrouver plus tard. Cela peut se faire par le module pickle qui contient des méthodes permettant de dumper un objet Python dans un fichier et ensuite de le récupérer. Mais le fichier est alors un fichier "à la sauce pickle" dans lequel tu ne mets pas les pieds.

Citation:

---

Envoyé par Beginner.

Oui c'est possible...

Essaye ça (en reprenant le même contenu pour le fichier) :

[

---

voilà qui répond parfaitement au problème :):):)

merci

Citation:

---

Envoyé par Sve@r

Bonjour

En lisant tout ce que tu as écrit, j'ai l'impression que ton truc est absurde. Que tu confonds "objet poo" et "objet".

---

c'est clair que c'est pas facile à exprimer quand on est pas un pro (et encore, je connais beaucoup trop de pro qui s'emmêlent les pinceaux :D )

mais bon, "absurde", je sais pas... :)

pour être plus concret, je défini une classe "étoile" avec deux attributs "AD" et "DEC" et une méthode "hauteur" (je calcule la hauteur d'une étoile en fonction de AD et DEC

à côté, j'ai une liste d'étoiles dans un fichier avec pour chacune d'entre elle son AD et sa DEC

si je je décrivais les étoiles dans le programme, je ferai un truc comme ça:

Code:

---

1 2 3 4 5

andromède = etoile() andromède.AD=1 andromede.DEC=2   H = andromede.hauteur

---

je ne sais pas comment tu appelles l'entité andromède, pour moi c'est un objet instancié de la classe étoile.

le problème était donc de savoir comment créer cet objet à partir du fichier texte, histoire de ne pas avoir à modifier le programme à chaque nouvelle étoile.

à priori, la solution de beginner répond bien au problème...

merci pour ta longue réponse :)

Citation:

---

Envoyé par alpha.centauri

mais bon, "absurde", je sais pas... :)

---

Le terme est fort mais je l'ai écrit sans penser à mal. Désolé.

Citation:

---

Envoyé par alpha.centauri

pour être plus concret, je défini une classe "étoile" avec deux attributs "AD" et "DEC" et une méthode "hauteur" (je calcule la hauteur d'une étoile en fonction de AD et DEC

---

Tu veux bien parler des étoiles en astronomie et non pas d'une figure géométrique...

Citation:

---

Envoyé par alpha.centauri

à côté, j'ai une liste d'étoiles dans un fichier avec pour chacune d'entre elle son AD et sa DEC

si je je décrivais les étoiles dans le programme, je ferai un truc comme ça:

andromède = etoile()
andromède.AD=1
andromede.DEC=2

H = andromede.hauteur

je ne sais pas comment tu appelles l'entité andromède, pour moi c'est un objet instancié de la classe étoile.

---

Pour moi c'est une simple étoile. Au sens le plus "neutre" qu'on puisse lui donner. Tout comme il y a différentes voitures (avec des éléments particuliers comme marque, couleur, immatriculation) il y a différentes étoiles avec des éléments particuliers comme magnitude, distance par rapport au soleil, nom, hauteur, etc.
Donc moi je créerais une classe "etoile" contenant tous les attributs dont mon algorithme a besoin, puis ensuite j'instancierais différentes étoiles à partir des infos lues dans le fichier. Le fichier lui n'a même pas besoin de savoir qu'il sera traité par ton algo ou par un autre, à la limite ça peut être du bête CSV. C'est ton algo qui fera la conversion "info lue => instanciation d'une étoile" (séparation "modèle" du "contrôleur" en terminologie "programmation MVC")

Citation:

---

Envoyé par alpha.centauri

le problème était donc de savoir comment créer cet objet à partir du fichier texte, histoire de ne pas avoir à modifier le programme à chaque nouvelle étoile.

---

Si ton fichier est "neutre" (dans le sens qu'il ne se sent pas obligé de correspondre à un langage particulier mais reste dans son role de fichier data) pas de souci. C'est ton algo qui lui se charge de le lire et en extraire les infos pour les charger dans ton objet à toi.

Prends le fichier suivant (nommé "fic" chez-moi)

Code:

---

1 2 3

Andromede:1:2 Alpha Centauri:5:8 Aldebaran:12:42

---

Je peux le traiter ainsi

Code:

---

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 26 27 28 29 30 31

#!/usr/bin/env python3 # coding: utf-8   class star: def __init__(self, nom="", AD=0, DEC=0): self.nom=nom self.AD=AD self.DEC=DEC # __init__()   def load(self, ligne, sep): info=ligne.split(sep) self.nom=info[0] self.AD=int(info[1]) self.DEC=int(info[2]) # load()   def __str__(self): return "Etoile {}: AD={}, DEC={}".format(self.nom, self.AD, self.DEC) # star()   galaxie=list() with open("fic", "r") as fp: for line in fp: galaxie.append(star()) galaxie[-1].load(line.strip(), ":") # for # with   for (i, g) in enumerate(galaxie, 1): print(i, g)

---

Mais si je ne me sens pas la nécessité de changer une étoile (une fois crée elle reste ainsi), alors le code pourra être le suivant (l'étoile est créee à l'instanciation à partir de la ligne du ficher)

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

#!/usr/bin/env python3 # coding: utf-8   class star: def __init__(self, ligne, sep): info=ligne.split(sep) self.nom=info[0] self.AD=int(info[1]) self.DEC=int(info[2]) # __init__()   def __str__(self): return "Etoile {}: AD={}, DEC={}".format(self.nom, self.AD, self.DEC) # star()   galaxie=list() with open("fic", "r") as fp: for line in fp: galaxie.append(star(line.strip(), ":")) # with   for (i, g) in enumerate(galaxie, 1): print(i, g)

---

Si maintenant tu veux pouvoir "sélectionner" une étoile particulière...

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

galaxie=dict() with open("fic", "r") as fp: for line in fp: e=star(line.strip(), ":") galaxie[e.nom]=e # with   choix=input("Quelle étoile désirez-vous ?") if choix in galaxie.keys(): print(galaxie[choix]) else: print("Etoile {} inconnue".format(choix)) # if

---

Et il y a plein d'autres façons de faire. Bon cet exemple c'est du "vite fait" (j'utilise un bête split() pour traiter les ":" alors que le module "csv" pourra mieux gérer les trucs plus complexes qu'il puisse se passer au sein d'un CSV) mais c'est pour l'exemple. Le fichier contient des ":", le code lui est assez souple pour prendre autre chose (c'est l'appelant qui choisit son séparateur)...

Donc tout est possible pour peu que tu ne t'attaches pas à imaginer que "une étoile en particulier" mérite son propre objet. C'est comme pour les nombres. Il existe l'objet "int" qui permet de gérer "un nombre en général" et qui de fait peut les gérer tous, pas besoin d'un objet "int_5" pour gérer le nombre 5.

Citation:

---

Envoyé par alpha.centauri

à priori, la solution de beginner répond bien au problème...

---

Quelque chose me dit que cela n'y répondra que pour une courte durée...

Citation:

---

Envoyé par Sve@r

Le terme est fort mais je l'ai écrit sans penser à mal. Désolé.

---

pas de souci, je l'ai pas mal pris

Citation:

---

Envoyé par Sve@r

Si maintenant tu veux pouvoir "sélectionner" une étoile particulière....

---

ben oui, c'est bien de ça dont j'ai besoin; du coup, c'est un peu ce que je disais, le faire "à l'ancienne" sans passer par une classe... :-) :-) :-)

Citation:

---

Envoyé par Sve@r

Quelque chose me dit que cela n'y répondra que pour une courte durée...

---

et oui, c'est pas impossible que ça finisse comme ça, à l'ancienne, pourquoi faire compliqué...

Citation:

---

Envoyé par alpha.centauri

ben oui, c'est bien de ça dont j'ai besoin; du coup, c'est un peu ce que je disais, le faire "à l'ancienne" sans passer par une classe... :-) :-) :-)

---

Rien ne t'y oblige effectivement (faut pas faire de la poo parce que c'est dans le vent). Une classe c'est grosso-modo "data+action_associée". Si toi tu n'as que de la data...

Par exemple un simple dictionnaire de dictionnaires (toujours à partir du même fichier)

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

galaxie=dict() with open("fic", "r") as fp: for line in fp: info=line.strip().split(":") galaxie[info[0]]={"nom" : info[0], "AD" : int(info[1]), "DEC" : int(info[2])} # for # with   choix=input("Quelle étoile désirez-vous ?") g=galaxie.get(choix) if g is not None: print("Nom {}, AD {}, DEC {})".format(g["nom"], g["AD"], g["DEC"])) else: print("Etoile {} inconnue".format(choix)) # if

---

Et encore, vu que le nom est la clef, le recopier dans la valeur peut même passer pour inutile...

Citation:

---

Envoyé par alpha.centauri

et oui, c'est pas impossible que ça finisse comme ça, à l'ancienne, pourquoi faire compliqué...

---

C'est pas vraiment une histoire d'ancien/nouveau. C'est plutôt arriver à faire la distinction entre conteneur et contenu.

Imagine que tu doives gérer 3 gus: Pierre, Paul et Jacques. Tu peux très bien écrire...

Code:

---

1 2 3

pierre="Pierre" paul="Paul" jacques="Jacques"

---

... et ensuite traiter ton ton algo avec les variables pierre, paul et jacques. Tu veux afficher les 3 tu fais 3 print. Et etc.

Mais d'une part associer le nom de la variable à la valeur littérale de son contenu c'est catastrophique (déjà si le nom change...). Et même sans ça, si ensuite tu dois en gérer 1500 là tu te diras "putain merde, je suis mal parti et maintenant je ne m'en sors plus avec mes 1500 variables", j'aurais dû passer par un tableau" => bonhommes=["Pierre", "Paul", "Jacques"].

C'est à peu près l'image que je me fais de ta solution et du code de Beginner pour l'écrire en Python. Elle fonctionne aujourd'hui parce que tu as 3 étoiles et 3 classes associées. L'étoile Aldebaran => l'objet "Aldebaran" et etc. Mais pareil, associer le nom de l'objet à la valeur littérale de son contenu c'est comme pour mon pierre="Pierre" et même si ça tient, quand tu passeras à 1500 étoiles tu te diras "putain merde, je suis mal parti et maintenant je ne m'en sors plus avec mes 1500 classes, j'aurais dû travailler avec une classe plus générale et gérer la distinction avec les attributs de chaque instance". C'est ce que je voulais dire en disant que ça risquait de ne pas durer...

Salut,

Citation:

---

Envoyé par Sve@r

Citation:

---

Envoyé par alpha.centauri

à priori, la solution de beginner répond bien au problème...

---

Quelque chose me dit que cela n'y répondra que pour une courte durée...

---

Je crois qu'il y a un léger malentendu, je pense que vous ne parlez pas de la même chose... En fait je ne pense pas que Sve@r soit opposée à l'idée de solution que j'ai proposée, bien au contraire, je crois que nous sommes d'accord...

Corrigez-moi si je me trompe mais voilà pourquoi je dis cela :

- Le premier code que j'ai posté c'était juste pour répondre à la demande suivante :

Citation:

---

Envoyé par alpha.centauri

je crée une classe "obj" avec, disons, deux attributs

j'ai un fichier texte qui contient n lignes de type : "toto 2 3", "tata 6 3", etc...

est-il possible de lire le fichier et de créer des objets toto, tata... appartenant à la classe "obj" et d'instancier ces objets avec les valeurs contenues dans le fichier ?

---

Le code répond bien à la demande mais j'avais émis des réserves dans plusieurs messages en gros pour dire que je ne pense pas que ce soit une bonne façon de faire... Je n'ai pas donné beaucoup d'explication mais Sve@r l'a fait par la suite donc cela tombe bien...

Ensuite j'ai donné une idée de la façon dont je procéderais :

Citation:

---

Envoyé par Beginner.

Ok.

Le code que j'ai posté te permet de faire de toto, tata,... des instances de la class Obj avec les deux paramètres en question...

Mais pour ton problème je pense que j'utiliserais plutôt un dictionnaire : chaque clé serait une string identifiant une option de ta liste déroulante et la valeur associée serait une instance de ta class.

---

Puis j'ai adapté le code de alpha.centauri pour mettre en pratique cette idée, le code se trouve au message #16.

Je pense que les critiques de Sve@r concernent la première méthode et non cette idée d'ailleurs le troisième code qu'il a posté au message #20 correspond dans le fond exactement à cette idée-là... Je pense donc que nous sommes d'accord sur cette idée...

Voilà j’espère avoir dissipé le malentendu...

Citation:

---

Envoyé par Sve@r

Non, quelque part je crois qu'il y a eu incompréhension de ta part entre "objet poo" et "objet data" qui t'a alors amené à nous présenter non pas le problème originel mais la solution que tu avais imaginé pour le résoudre. Bref cela ressemble à un cas de problème XY...

---

Oui je suis d'accord avec toi sur ce point aussi, je ne savais pas que ce problème avait un nom, j'aurais appris quelque chose... Merci.

Citation:

---

Envoyé par alpha.centauri

Citation:

---

Envoyé par Sve@r

Si maintenant tu veux pouvoir "sélectionner" une étoile particulière...

---

ben oui, c'est bien de ça dont j'ai besoin; du coup, c'est un peu ce que je disais, le faire "à l'ancienne" sans passer par une classe... :-) :-) :-)

---

Ben non justement, tu n'as pas bien compris... Ce troisième code de Sve@r utilise bel et bien une class, il s'agit de la class star, mais c'est vrai que dans ce troisième code il ne l'a pas recopiée mais on devine en lisant les codes précédents qu'il fallait l'ajouter d'ailleurs sans cette class ce troisième code ne peut pas fonctionner...

Et encore une fois, je pense que nous sommes d'accord sur l'idée sur laquelle est basé ce troisième code... Et cette idée ce n'est pas qu'il faudrait ne pas utiliser les class, relis bien les explications de Sve@r, je pense que tu pourras mieux les comprendre maintenant (du moins j’espère) que ce malentendu est dissipé...

Citation:

---

Envoyé par Beginner.

En fait je ne pense pas que Sve@r soit opposée à l'idée de solution que j'ai proposée, bien au contraire, je crois que nous sommes d'accord...

---

On parle bien de ton dernier code, celui avec la classe "Obj"... Oui celui là correspond à l'esprit du machin (en plus j'ai proposé un truc quasi identique avec mon "class star"). En fait quand j'ai dit que ça n'allait pas durer je pensais au code où j'avais vu exec() (le truc un peu horrible quoi)...:oops:

Citation:

---

Envoyé par Beginner.

Ben non justement, tu n'as pas bien compris... Ce troisième code de Sve@r utilise bel et bien une class, il s'agit de la class star, mais c'est vrai que dans ce troisième code il ne l'a pas recopiée mais on devine en lisant les
codes précédents qu'il fallait l'ajouter d'ailleurs sans cette class ce troisième code ne peut pas fonctionner...

---

Oui, le 3° exemple utilise la class "star" du 2°. Pour montrer que justement une fois la classe créée, on peut l'utiliser dans différentes configurations (une liste dans le 2°, un dictionnaire dans le 3°).

Citation:

---

Envoyé par Sve@r

On parle bien de ton dernier code, celui avec la classe "Obj"...

---

Euh... Là je ne te suis pas : quand tu critiquais mon code vous parliez de mon dernier code ? Comme déjà dit, mon premier code, je ne l'ai donné que pour répondre à une demande précise et mon dernier code c'est celui-là qui correspond à l’idée que j'avais proposée :

Citation:

---

Envoyé par Beginner.

Ok.

Le code que j'ai posté te permet de faire de toto, tata,... des instances de la class Obj avec les deux paramètres en question...

Mais pour ton problème je pense que j'utiliserais plutôt un dictionnaire : chaque clé serait une string identifiant une option de ta liste déroulante et la valeur associée serait une instance de ta class.

---

Cette idée consiste donc à utiliser un dictionnaire (avec les caractéristiques ci-dessus) exactement comme tu l'as toi-même fait par la suite...

Citation:

---

Envoyé par Sve@r

Oui celui là correspond à l'esprit du machin (en plus j'ai proposé un truc quasi identique avec mon "class star").

---

Ben oui comme je l’avais dit tu as proposé un code similaire correspondant à l'idée ci-dessus du coup j'en ai déduis que ce n'était pas mon dernier code (basée sur cette idée) que tu "critiquais"... Sinon cela voudrait dire que tu critiquais un code tout en proposant un code similaire ce qui n'a pas de sens... C'est pour cela que je me disais que le code dont tu parlais avec alpha.centauri c'était le premier et non le dernier.

Et pareil quand tu écris cela :

Citation:

---

Envoyé par Sve@r

C'est pas vraiment une histoire d'ancien/nouveau. C'est plutôt arriver à faire la distinction entre conteneur et contenu.

Imagine que tu doives gérer 3 gus: Pierre, Paul et Jacques. Tu peux très bien écrire...

Code:

---

1 2 3

pierre="Pierre" paul="Paul" jacques="Jacques"

---

... et ensuite traiter ton ton algo avec les variables pierre, paul et jacques. Tu veux afficher les 3 tu fais 3 print. Et etc.

Mais d'une part associer le nom de la variable à la valeur littérale de son contenu c'est catastrophique (déjà si le nom change...). Et même sans ça, si ensuite tu dois en gérer 1500 là tu te diras "putain merde, je suis mal parti et maintenant je ne m'en sors plus avec mes 1500 variables", j'aurais dû passer par un tableau" => bonhommes=["Pierre", "Paul", "Jacques"].

C'est à peu près l'image que je me fais de ta solution et du code de Beginner pour l'écrire en Python. Elle fonctionne aujourd'hui parce que tu as 3 étoiles et 3 classes associées. L'étoile Aldebaran => l'objet "Aldebaran" et etc. Mais pareil, associer le nom de l'objet à la valeur littérale de son contenu c'est comme pour mon pierre="Pierre" et même si ça tient, quand tu passeras à 1500 étoiles tu te diras "putain merde, je suis mal parti et maintenant je ne m'en sors plus avec mes 1500 classes, j'aurais dû travailler avec une classe plus générale et gérer la distinction avec les attributs de chaque instance". C'est ce que je voulais dire en disant que ça risquait de ne pas durer...

---

Là aussi je pense que c'est le premier code que tu critiquais et non le dernier. Mais entre temps j'avais déjà posté mon dernier code et alpha.centauri l'avait déjà adopté semble-t-il puisqu’il dit :

Citation:

---

Envoyé par alpha.centauri

voilà qui répond parfaitement au problème...i

---

Donc à priori il a dû à ce moment-là abandonner le premier code, je suppose qu'il avait compris que ce n'était pas une bonne façon de faire pour régler ce problème...

Citation:

---

Envoyé par Sve@r

En fait quand j'ai dit que ça n'allait pas durer je pensais au code où j'avais vu exec() (le truc un peu horrible quoi)...:oops:

---

Ben oui c'est bien ce que j'avais compris mais ce code ce n'est pas le dernier c'est le premier. Et je rappelle encore une fois que ce premier code c'était juste pour répondre à la demande que j'ai déjà citée dans le massage précédent et j'ai bien précisé ceci '...le code répond bien à la demande mais j'avais émis des réserves dans plusieurs messages en gros pour dire que je ne pense pas que ce soit une bonne façon de faire... ".



Sinon tu dis que la fonction exec c'est un truc horrible, pourquoi ça ? Problème de sécurité et/ou de lisibilité ? Comment ferais-tu pour répondre à la première demande d'alpha.centauri ? J'ai vu qu'on pouvait aussi utiliser locals()[x[0]] =Obj(int(x[1]), int(x[2])) à la place de exec("%s = Obj(%s,%s)" % (x[0], x[1], x[2])).

-----

Bon bref, j'ai voulu clarifier les choses mais j'ai l'impression que c'est raté...lol

Citation:

---

Envoyé par Beginner.

Euh... Bon bref, j'ai voulu clarifier les choses mais j'ai l'impression que c'est raté...lol

---

Oui voilà, un imbroglio total totalement de ma faute. Je crois qu'il vaut mieux oublier. :aie:

Citation:

---

Envoyé par Beginner.

Sinon tu dis que la fonction exec c'est un truc horrible, pourquoi ça ? Problème de sécurité et/ou de lisibilité ?

---

De façon générale, toute fonction qui exécute une instruction non prévisible par le programmeur (exec/eval) présente un souci de sécurité. Ici ton code lit un fichier (que tout utilisateur qui se sent concerné par la sécurité de sa machine a le droit de le considérer comme "inconnu") et convertit son contenu en programme Python exécutable... Imagine que ce contenu soit sys.exec("rm -rf $HOME")... :mur:

Citation:

---

Envoyé par Beginner.

Comment ferais-tu pour répondre à la première demande d'alpha.centauri ?

---

Sans exec() on ne peut pas, je suis d'accord.

Citation:

---

Envoyé par Beginner.

J'ai vu qu'on pouvait aussi utiliser locals()[x[0]] =Obj(int(x[1]), int(x[2])) à la place de exec("%s = Obj(%s,%s)" % (x[0], x[1], x[2])).

---

Effectivement, je n'y aurais pas pensé. Là c'est au-moins plus sécurisé.

Citation:

---

Envoyé par Beginner.

Sinon tu dis que la fonction exec c'est un truc horrible, pourquoi ça ? Problème de sécurité et/ou de lisibilité ? Comment ferais-tu pour répondre à la première demande d'alpha.centauri ? J'ai vu qu'on pouvait aussi utiliser locals()[x[0]] =Obj(int(x[1]), int(x[2])) à la place de exec("%s = Obj(%s,%s)" % (x[0], x[1], x[2])).

---

La création de variables dynamiques n'est pas recommendables car on sera coincé par la suite à utiliser la chaîne de caractères comme clef dans le dictionnaire globals() pour y accéder à l'objet associé (impossible de l'utiliser comme variable puisqu'on n'en connaît pas l'écriture).

Si on utilise globals(), ça peut provoquer des collisions avec les variables utiles au programme: autant utiliser un dictionnaire spécifique.

On peut utiliser la méthode qu'on veut pour créer ce genre de "variables", ça ne change pas grand chose.

- W

pour conclure (?) je dirais que oui, la solution en passant par un dictionnaire répond bien à ma question initiale :)

merci à tous du temps passé à me répondre,

amicalement.

Citation:

---

Envoyé par alpha.centauri

pour conclure (?) je dirais que oui, la solution en passant par un dictionnaire répond bien à ma question initiale :)

---

La question initiale portait sur la création d'instances depuis un fichier texte...

- W

Salut,

Citation:

---

Envoyé par Sve@r

Oui voilà, un imbroglio total totalement de ma faute. Je crois qu'il vaut mieux oublier. :aie:

---

Je ne dirais pas que c'est totalement de ta faute, j'ai ma part aussi mais oui oublions... Par contre je retiens le fameux problème XY... Je pense que cela arrive souvent dans les forums, c'est bien qu'on lui ait donné un nom.

Citation:

---

Envoyé par wiztricks

La création de variables dynamiques n'est pas recommendables car on sera coincé par la suite à utiliser la chaîne de caractères comme clef dans le dictionnaire globals() pour y accéder à l'objet associé (impossible de l'utiliser comme variable puisqu'on n'en connaît pas l'écriture).

---

Oui je pense qu'on retiendra aussi que ce n'est pas une bonne chose. Par contre je n'ai pas compris pourquoi tu dis qu'on en connais pas l'écriture car dans le code que j'ai posté on a bien "création" des variables toto, tata et titi et on y a bien accès directement via leur nom...

Mais peut-être que tu veux parler de l’utilisateur; celui-ci ne connait pas l'écriture de ces variables car ils se trouvent dans un fichier ?

Citation:

---

Envoyé par Beginner.

Par contre je n'ai pas compris pourquoi tu dis qu'on en connais pas l'écriture car dans le code que j'ai posté on a bien "création" des variables toto, tata et titi et on y a bien accès directement via leur nom...

---

Comme on ne sait pas quel sera le nom de ces variables, on ne pourra pas les utiliser en tant que telles.
Exemple avec locals()[x[0]] =Obj(int(x[1]), int(x[2])), je ne sais pas ce que contient x[0].
Pas facile d'écrire:

Code:

---

1 2	locals()[x[0]] =Obj(int(x[1]), int(x[2])) print ('toto=', toto)

---

puisqu'on ne sait pas trop ce que contiendra x[0]
On devra se trimballer:

Code:

---

print (x[0], '=', locals()[x[0]])

---

Ce qui est d'un intérêt limité (et illustre "on ne peut pas utiliser toto en temps que variable").

- W

Citation:

---

Envoyé par wiztricks

Comme on ne sait pas quel sera le nom de ces variables, on ne pourra pas les utiliser en tant que telles.

---

Ben en fait le nom de ces variables figurent dans le fichier mais oui c'est vrai que ce n'est probablement pas une bonne chose de procéder ainsi... Cela nous oblige à analyser le fichier et pour cela on doit aussi savoir/comprendre comment le code fabrique les variables à partir de ce fichier...

Citation:

---

Envoyé par Beginner.

Cela nous oblige à analyser le fichier et pour cela on doit aussi savoir/comprendre comment le code fabrique les variables à partir de ce fichier...

---

Une des règles de base de la programmation est que le code doit être indépendant des données (ce qui n'a rien à voir avec l'ignorance de sa structure).

- W