Discussion :

[koreimy]

Bonjour.

J'ai un petit problème qui me tracasse depuis un grand moment mais je n'arrive pas à trouver une solution donc je me suis tourne vers vous pour trouver une piste, et je suis sûr que j'aurai une solution grâce à votre contribution.

Alors, je m'explique brièvement ce que je veux faire et le problème qui m’empêche de progresser.
En effet, je veux définir une fonction en python qui permettra de vérifier dans une base si un élève est libre ou pas par rapport à son emploi du temps.
Dans ma base les tables nécessaires pour cette fonction: table categorie(definie si eleve ou etudiant), etablissement, classe, et emplois_du_temps.

donc j'ai définie une fonction booléenne dans laquelle à partir de l'identifiant de l'utilisateur qui nous renvoie une valeur 1 ou 0.

je precise le langage c'est python.
voici la fonction que j'ai defini:

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
def eleveAutorise(login):
    conn = connexionBD()     
    cursor = conn.cursor()
    cursor.execute("SELECT categorie_id FROM compte c,categorie ca where c.categorie_id=ca.id  and libelle_cat=%s and login=%s", ('eleve',login))
    row = cursor.fetchone ()  
    if (row == None): 
        return 1 
    else:
        cursor.execute("SELECT c.idEta FROM compte c ,etablissement e where c.idEta=e.idEta and e.nomEta=%s and login=%s",  ('Lnlm', login))
        row1 = cursor.fetchone ()  
	if (row1 != None):
		cursor.execute("SELECT c.idclass FROM compte c,classe cl where c.idclass=cl.idClass and c.idEta=%s and login=%s",  (row1[0], login))
		row2 = cursor.fetchone ()  
		jours = ["Lundi", "Mardi", "Mercredi", "Jeudi", "Vendredi", "Samedi", "Dimanche"]
		j= jours[time.localtime()[6]]
		cursor.execute("SELECT * FROM emploi_du_temps  where idClass=%s and jour=%s",(row2[0],j))
		row3 = cursor.fetchone ()  
		conn.close()
		from datetime import datetime
		#datetime(2000, 1, 1)
		maintenant = datetime.now() 
		if((maintenant.hour >7 and  maintenant.minute<=30) and (maintenant.hour<8 and  maintenant.minute<=20)):
			if(row3[2]=="L"):
				return 1 
			else:
				return 0
		else:
			if((maintenant.hour >8 and  maintenant.minute<=20) and (maintenant.hour<9 and  maintenant.minute<=20)):
				if(row3[3]=="L"):
					return 1  
				else:
					return 0
			else:
				if((maintenant.hour >9 and  maintenant.minute<=20) and (maintenant.hour<10 and  maintenant.minute<=10)):
					if(row3[4]=="L"):
						return 1  
					else:
						return 0
				else:
					if((maintenant.hour >10 and  maintenant.minute<=10) and (maintenant.hour<11 and  maintenant.minute<=5)):
						if(row3[5]=="L"):
							return 1  
						else:
							return 0
					else:
						if((maintenant.hour >11 and  maintenant.minute<=05) and (maintenant.hour<11 and  maintenant.minute<=30)):
							if(row3[6]=="L"):
								return 1  
							else:
								return 0 
						else:
							if((maintenant.hour >11 and  maintenant.minute<=30) and (maintenant.hour<12 and  maintenant.minute<=20)):
								if(row3[7]=="L"):
									return 1  
								else:
									return 0
							else:
								if((maintenant.hour >12 and  maintenant.minute<=20) and (maintenant.hour<13 and  maintenant.minute<=15)):
									if(row3[8]=="L"):
										return 1  
									else:
										return 0
								else:
									if((maintenant.hour >13 and  maintenant.minute<=15) and (maintenant.hour<15 and  maintenant.minute<=00)):
										if(row3[9]=="L"):
											return 1  
										else:
											return 0
									else:
										if((maintenant.hour >15 and  maintenant.minute<=00) and (maintenant.hour<16 and  maintenant.minute<=00)):
											if(row3[10]=="L"):
												return 1  
											else:
												return 0
										else:
											if((maintenant.hour >16 and  maintenant.minute<=00) and (maintenant.hour<17 and  maintenant.minute<=00)):
												if(row3[11]=="L"):
													return 1  
												else:
													return 0			
 
 
 
 
    	else:
		return 1

Probleme:
il n'affiche aucun problème de syntaxe mais me renvoie toujours la valeur "0"
donc je demande à tout celui qui a une piste de m'orienter pour ce problème.
je suis débutant en python aussi.

Merci d'avance.

[deusyss]

Bonjour koreimy,

On va tâcher de t'aider à avancer. En lisant ton post, une question me vient à l'esprit: sais tu où tu passes précisément?

C'est le premier point. Tu peux éventuellement rajouter des "print" pour tracer ta route dans le code. Profite en pour "printer" maintenant, et row3.

A la première lecture, je n'ai rien vu de choquant. Je pencherai donc plutôt pour un problème de données en BDD ou de requête.

Effectue les test indiqué et revient avec les résultat, cela nous aidera à avancer sur ce probleme.

[koreimy]

Merci d'avoir répondu si rapidement.
effectivement j'ai teste bien 'maintenant ' et 'row3 ' me renvoient bien les bons résultats.


cordialement.

[deusyss]

Vis-a-vis des données récupérées via les print, est-il cohérent que tu récupères la valeur 0 en retour?

Dans ton code, en relisant je tique un peu sur tes prédicats:

Code :

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

(maintenant.hour >7 and  maintenant.minute<=30) and (maintenant.hour<8 and  maintenant.minute<=20)

20<=minutes<=30, je comprend, mais 7<hour<8, je voit mal, sachant que les heures sont des entiers. Tu peut le vérifier avec un

Code :

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

print maintenant.hour, type(maintenant.hour)

Les prédicats ne sont jamais vérifiés, et tu passes donc systématiquement dans les else qui te renvoie alors 0. Que souhaites tu tester précisément dans ces prédicats?

Sinon concernant le reste ton code, je te suggère de le simplifie et de l'optimiser:
>Le from...import... est à mettre en entete de fichier, pas dans le code (PEP)
>Utilise plutot True et False pour des booleens en Python (PEP)
>Utilise une variable booléenne au début de ta fonction que tu définit intialement à False. Tu la redefinit dans les if à True quand les predicats sont vérifiés. A la fin de ton code tu n'effectues qu'un seul return
>ton dernier else a l'air mal indenté


Une version corrigée (à tester) serait donc par exemple (prédicats non corrigés):

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
def eleveAutorise(login):
	"""
                Fonction permettant de savoir si un eleve/etudiant est dispo
        """
	retour = False
 
	conn = connexionBD()	 
	cursor = conn.cursor()
	cursor.execute("SELECT categorie_id FROM compte c,categorie ca where c.categorie_id=ca.id  and libelle_cat=%s and login=%s", ('eleve',login))
	row = cursor.fetchone () 
 
	if (row == None): 
		retour = True
	else:
		cursor.execute("SELECT c.idEta FROM compte c ,etablissement e where c.idEta=e.idEta and e.nomEta=%s and login=%s",  ('Lnlm', login))
		row1 = cursor.fetchone ()  
 
	if (row1 != None):
		cursor.execute("SELECT c.idclass FROM compte c,classe cl where c.idclass=cl.idClass and c.idEta=%s and login=%s",  (row1[0], login))
		row2 = cursor.fetchone ()  
		jours = ["Lundi", "Mardi", "Mercredi", "Jeudi", "Vendredi", "Samedi", "Dimanche"]
		j= jours[time.localtime()[6]]
		cursor.execute("SELECT * FROM emploi_du_temps  where idClass=%s and jour=%s",(row2[0],j))
		row3 = cursor.fetchone ()  
		conn.close()
		from datetime import datetime
		#datetime(2000, 1, 1)
 
		maintenant = datetime.now() 
		if((maintenant.hour >7 and  maintenant.minute<=30) and (maintenant.hour<8 and  maintenant.minute<=20)):
			if(row3[2]=="L"):
				retour = True
		else:
			if((maintenant.hour >8 and  maintenant.minute<=20) and (maintenant.hour<9 and  maintenant.minute<=20)):
				if(row3[3]=="L"):
					retour = True
			else:
				if((maintenant.hour >9 and  maintenant.minute<=20) and (maintenant.hour<10 and  maintenant.minute<=10)):
					if(row3[4]=="L"):
						retour = True  
				else:
					if((maintenant.hour >10 and  maintenant.minute<=10) and (maintenant.hour<11 and  maintenant.minute<=5)):
						if(row3[5]=="L"):
							retour = True
					else:
						if((maintenant.hour >11 and  maintenant.minute<=05) and (maintenant.hour<11 and  maintenant.minute<=30)):
							if(row3[6]=="L"):
								retour = True
						else:
							if((maintenant.hour >11 and  maintenant.minute<=30) and (maintenant.hour<12 and  maintenant.minute<=20)):
								if(row3[7]=="L"):
									retour = True 
							else:
								if((maintenant.hour >12 and  maintenant.minute<=20) and (maintenant.hour<13 and  maintenant.minute<=15)):
									if(row3[8]=="L"):
										retour = True
								else:
									if((maintenant.hour >13 and  maintenant.minute<=15) and (maintenant.hour<15 and  maintenant.minute<=00)):
										if(row3[9]=="L"):
											retour = True
									else:
										if((maintenant.hour >15 and  maintenant.minute<=00) and (maintenant.hour<16 and  maintenant.minute<=00)):
											if(row3[10]=="L"):
												retour = True
										else:
											if((maintenant.hour >16 and  maintenant.minute<=00) and (maintenant.hour<17 and  maintenant.minute<=00)):
												if(row3[11]=="L"):
													retour = True
 
	else:
		retour = True
 
	return retour

[deusyss]

Essaie peut etre quelque chose du genre

Code :

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

if((maintenant.hour =7 and  maintenant.minute>=30) or (maintenant.hour=8 and  maintenant.minute<=20))

EDIT:

Une version corrigée non testée pour t'aider:

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
 
def eleveAutorise(login):
	"""
                Fonction permettant de savoir si un eleve/etudiant est dispo
        """
	retour = False
	horaires = [(7,30),(8,20),(9,20),(10,10),(11,5),(11,30),(12,20),(13,15),(15,0),(16,0),(17,0)]
 
	conn = connexionBD()	 
	cursor = conn.cursor()
	cursor.execute("SELECT categorie_id FROM compte c,categorie ca where c.categorie_id=ca.id  and libelle_cat=%s and login=%s", ('eleve',login))
	row = cursor.fetchone () 
 
	if (row == None): 
		retour = True
	else:
		cursor.execute("SELECT c.idEta FROM compte c ,etablissement e where c.idEta=e.idEta and e.nomEta=%s and login=%s",  ('Lnlm', login))
		row1 = cursor.fetchone ()  
 
	if (row1 != None):
		cursor.execute("SELECT c.idclass FROM compte c,classe cl where c.idclass=cl.idClass and c.idEta=%s and login=%s",  (row1[0], login))
		row2 = cursor.fetchone ()  
		jours = ["Lundi", "Mardi", "Mercredi", "Jeudi", "Vendredi", "Samedi", "Dimanche"]
		j= jours[time.localtime()[6]]
		cursor.execute("SELECT * FROM emploi_du_temps  where idClass=%s and jour=%s",(row2[0],j))
		row3 = cursor.fetchone ()  
		conn.close()
		from datetime import datetime
		#datetime(2000, 1, 1)
 
		maintenant = datetime.now() 
		for i in range(len(horaires)):
			if ((maintenant.hour = horaires[i][0] and maintenant.minute >= horaires[i][1]) or \
				(maintenant.hour = horaires[i+1][0] and maintenant.minute <= horaires[i+1][1])):
					if(row3[i+2]=="L"):
						retour = True
	else:
		retour = True
 
	return retour

[koreimy]

merci c'est tres gentil je vais essayer...

[koreimy]

Merci!

C'est propre et bien marche.

[tyrtamos]

Bonjour,

Je n'ai pas épluché le code mais j'ai juste vu un truc: si "row = cursor.fetchone()" ne renvoie rien, il me semble que row est un tuple vide. Or, un tuple vide n'est pas égal à None: il ne faut donc pas tester row==None mais row==() ou len(row)==0.

[koreimy]

exactement, avant je n'ai pas teste pour le row est vide mais ça gener des erreurs quand j'ai testé. donc je vais tenir en compte.

Il y a un autre petit probleme qui me derange aussi, je vais poster sur une autre page si vous avez idée veuillez m'aider.
Merci encore une fois de plus.

[koreimy]

Excuser moi, je veux revenir en arrière concernant la fonction ci-dessus. la deuxième option génère cette erreur suivante:

Code :

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

1
2
3
File "/home/brahim/Bureau/workspace_cherrypy/ConnexionBD.py", line 466, in eleveAutorise
    if ((maintenant.hour == horaires[i][0] and maintenant.minute >= horaires[i][1]) or (maintenant.hour ==horaires[i+1][0] and maintenant.minute <= horaires[i+1][1])):
IndexError: list index out of range

[deusyss]

Bonjour Koreimy,

Comme je te l'avais dit, je n'ai pas pu tester, n’étant pas dans les mêmes conditions que toi.

En relisant je pense que le problème est dans le prédicat du FOR. on fait

Code :

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

range(len(horaires))

Je pense qu'il faudrait plutôt faire

Code :

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

range(len(horaires) - 1)

Pourquoi? Parce que sinon, i va atteindre à un moment donner la valeur 10 (qui correspond à 17h00). et dans le prédicat du if on va essayer de lire horaires[11]. A ce moment là, on tombera alors en erreur.

[koreimy]

Ok, pour celui est deja bon mais pour quoi du moment où row est nulle, il vient encore tester row1? pour tant si row est nulle "retour" va prendre la valeur 'true' et l'execution va s'arreter.

[deusyss]

Est-ce toi qui a écrit ce code? Si non, connaîtrais-tu l'auteur? Il serait le plus à même de t'expliquer précisément pourquoi il fait telle ou telle chose.

Ce que je comprend du code, on cherche d'abord si l'eleve est renseigné en base. Si non, on renvoie True, si oui on continue. On cherche alors à savoir l'établissement auquel l'eleve est rattaché.

J'avoue trouver ce code bizarre: Si l'eleve est dispo sur la plage horaire, n'existe pas en base, ou n'appartient à aucun établissement, alors on récupère True, False sinon. Personellement, j'aurai mis une variable entière à 3 états: 0: non libre, 1: dispo, -1: inconnu en base de donnée.

Pour résumer test tests avec None permettent juste de tester si on doit continuer ou non le code. De plus cela permet au passage de gérer des erreurs. En effet, si la seconde requête s'exécutait sans s'assurer de disposer de résultat à la requête précédente, le traitement tomberait en erreur.

[deusyss]

Si ton probleme est resolu, n'oublie pas la balise