Discussion :

[tofu00]

Bonjour,
je n'ai pas dans l'habitude de poster dès qu'un problème survient mais là je tourne en rond depuis un moment et j'ai peur de mélanger pas mal de trucs.

J'aimerai donc recoder un programme en PHP vers du Python (en POO) d'une gestion de patients.

Principe actuel (donc en PHP):
1) un serveur envoie une liste des patients du jour sous forme de fichier texte
2) on affiche sur une page tous les patients du fichier texte
3) la personne qui s'occupe de la salle d'attente clique sur le patient qui s’apprête à être examiné
4) le clic provoque l'ouverture d'un logiciel d’acquisition
5) et on recommence tout à partir de la 2eme étape pour le patient suivant

Problème: j'ai du mal à voir comment gérer tout ça sous forme de classe.

J'ai pensé au début créer:
- une classe Patient qui contient les infos du patient
- une classe ListePatient qui contient tous les objets de type Patient

puis:
- une seule classe Patients qui créerai avec une méthode une liste de patients récupérable par une autre méthode

Quelle solution est donc la meilleure ?

Voilà j'espère avoir été assez clair.
Merci d'avance!

[deusyss]

Bonjour,

Pour moi, par rapport à tes explications, seule une classe patient est nécessaire. Il suffirait que tu ajoutes tes objet dans une liste.

Par exemple

Code :

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

1
2
3
 
liste_patient = []
list.append(NewPatient())

Tout simplement. Ensuite pour gérer, d'un point de vue logicie, tes patients ecran/requete, tu passe donc par de simple index.

[tofu00]

C'est bien ce que je pensais mais une liste dans la classe ? Ou une liste en global accessible de partout ?
J'avais fait quelque chose comme ça:

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
class Patients:
	liste_patient=[]
 
	def __init__(self):
		self.pname = ""
		self.pid = ""
 
	def create(self, pname, pid):
		self.pname = pname
		self.pid = pid
		Patients.liste_patient.append(self)
 
	def get(self, pid):
		for patient in Patients.liste_patient:
			if patient.pid == pid:
				return patient

[deusyss]

A tester, mais cela me semble correct. Tu peut aussi faire une liste externe à la classe. La façon que tu a utilisé est cependant la plus propre.

Attention, ligne 14 tu a fait une faute "Patient" à la place de "Patients"

[tofu00]

Exact c'est corrigé !
Et pour la méthode qui renvoie l'objet j'ai fait ça:

Code :

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

1
2
3
4
def get(self, pid):
	for patient in Patients.liste_patient:
		if patient.pid == pid:
			return patient

sauf que quand je teste:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
>>>patients = Patients()
>>>patients.create(pname="tofu", pid="id1")
>>>patients.create(pname="tutut", pid="id2")
>>>monpatient = patients.get("id1")
>>>print(Patients.liste_patient[0].pname)
tutut
>>>print(monpatient.pname) # mais ici ça ne marche pas il ne renvoie pas l'objet on dirait.
Traceback (most recent call last):
  File "wl.py", line 26, in <module>
    print(monpatient.pname)
AttributeError: 'NoneType' object has no attribute 'pname'

Après je peux toujours me servir de Patients.liste_patient[0].pname où avec la méthode get, l'index serait renvoyé mais par curiosité j'ai déjà vue la méthode __getitem__ je pourrais peut être m'en servir ?

[wiztricks]

Salut,

patients.create est une instance de Patients.
Si vous modifier "self", vous modifiez la seule instance crée.

Séparez "Patient" les instances de... et Patients la collection de...
Ce sont deux sortes d'objets différents.
Tout coller dans la même classe est possible mais si vous débutez, vous aurez mal a la tête et des difficultés a maîtriser votre code.

- W

[tofu00]

Ok merci! Pour ceux que ça intéresserai j'ai donc ceci:

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
class Patients:
 
	def __init__(self):
		self.liste = []
 
	def list_all(self):
		for patient in self.liste:
			print(patient.pname + " " + patient.pid)
 
	def add(self, patient):
		self.liste.append(patient)
 
class Patient:
 
	def __init__(self, pname, pid):
		self.pname = pname
		self.pid = pid
 
plist = Patients()
plist.add(Patient("nom1", "id1"))
plist.add(Patient("nom2", "id2"))
plist.add(Patient("nom3", "id3"))
 
plist.list_all()

qui renvoie:

Code :

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

1
2
3
nom1 id1
nom2 id2
nom3 id3

Sujet résolu, merci à vous!