Discussion :

[Raptor92]

Bonjour à Tous,

Avec le code ci-dessous, je cherche à envoyer une alerte mail si la quantité en stock est > à 0.

Mais je dois faire une erreur car cela envoi une alerte dans tous les cas.

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
SQLChangeConnexion(gnConn3)
sReqêteAlerteStock est une chaîne="SELECT SUM(STOC_QUANTITE) FROM STOCKS WHERE STOC_QUANTITE <> 0 AND STOC_EMPLACEMENT ='R-COR-PIK' "
SI EnModeTest() ALORS Trace(sReqêteAlerteStock)
SI SQLExec(sReqêteAlerteStock,"ReqAlerteStock") ALORS
	SQLTable("ReqAlerteStock",TABLE_ALERTE_STOCK)
 
	FIN
	POUR TOUTE LIGNE DE TABLE_ALERTE_STOCK
	SI TABLE_ALERTE_STOCK.COL_STOC_QTE..Valeur <> 0 ALORS
	Alerte_Stocks_LOTUS()
		SINON SI TABLE_ALERTE_STOCK.COL_STOC_QTE..Valeur = 0 ALORS
 
		FIN	
	FIN
SQLFerme("ReqAlerteStock")
SQLChangeConnexion(gnConn3)

Pouvez vous me donner votre avis.

Très bonne journée

[vmolines]

Bonjour,

Focalisez vous sur les données de votre requête. Le résultat attendu (mail de notification) est une finalité qui ne sera atteinte que si les données de votre requête sont bonnes.

Tout ça pour dire que quand vous présentez un problème de requête, il est préférable d'indiquer les tables et colonnes qui entrent en jeu, le résultat attendu, la requête utilisée et le résultat obtenu.

Le code qui exploite le résultat est un autre problème qui sera traité que si cette partie est bonne.

Et au final ce ne sera peut être pas un problème de Windev

[mogwai162]

Salut.

J'ai pas bien compris le test a l'intérieur de la boucle. normalement tu aurais du tout mettre au niveau de ta requête. Celle ci ne devrait te lister que ce que tu veux envoyer comme informations.

[sphynxounet]

A quel moment tu déclenches ta requête ?

Je saisi pas trop l'enchainement de 2 SI : tu as SI <> 0 et après le SINON tu mets un autre SI = 0. Dans la théorie si c'est pas <> 0 ça doit être =0.

Par contre tu dis que ça doit envoyer un mail si c'est > 0. Les valeurs peut-être négatives ?

[Raptor92]

Bonjour,

Focalisez vous sur les données de votre requête. Le résultat attendu (mail de notification) est une finalité qui ne sera atteinte que si les données de votre requête sont bonnes.

Tout ça pour dire que quand vous présentez un problème de requête, il est préférable d'indiquer les tables et colonnes qui entrent en jeu, le résultat attendu, la requête utilisée et le résultat obtenu.

Le code qui exploite le résultat est un autre problème qui sera traité que si cette partie est bonne.

Et au final ce ne sera peut être pas un problème de Windev

Bonjour Vmolines,

Après réflexion et 2 cafés, voici comment j'ai résolu mon problème.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
SQLChangeConnexion(gnConn3)
sReqêteAlerteStock est une chaîne="SELECT SUM(STOC_QUANTITE) FROM STOCKS WHERE STOC_EMPLACEMENT ='R-COR-PIK' "
SI EnModeTest() ALORS Trace(sReqêteAlerteStock)
SI SQLExec(sReqêteAlerteStock, "ReqAlerteStock") ALORS
	SI SQLAvance("ReqAlerteStock")=0 ALORS
		SI Val(SQLLitCol("ReqAlerteStock",1))<>0 ALORS
			Alerte_Stocks_LOTUS()
		FIN
	FIN
FIN
SQLFerme("ReqAlerteStock")
SQLChangeConnexion(gnConn3)

Merci pour vos commentaires qui m'ont permis de réfléchir autrement
Très bonne journée à Tous

[mogwai162]

Oui ça parait très bien sauf que moi j'aurais écrit :

Code :

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

1
2
3
4
5
6
7
sReqêteAlerteStock est une chaîne="SELECT SUM(STOC_QUANTITE) FROM STOCKS WHERE STOC_EMPLACEMENT ='R-COR-PIK' AND SUM(STOC_QUANTITE) <> 0"
SI EnModeTest() ALORS Trace(sReqêteAlerteStock)
SI SQLExec(sReqêteAlerteStock, "ReqAlerteStock") ALORS
	SI SQLAvance("ReqAlerteStock")=0 ALORS
		Alerte_Stocks_LOTUS()
	FIN
FIN

[Raptor92]

Bonjour Mogway

Avant tout merci pour vos remarques très pertinentes.
En effet, cela me parait plus simple

Bonne après midi