Discussion :

[James85]

Bonjour à tous.

Je suis sous MSDE avec SQLExec.

J'ai la proc. stock. suivante :

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
CREATE PROCEDURE maproc
AS
BEGIN
	SELECT * into #mytempdb
	FROM Table1 AS A, Table2 AS B, Table3 AS C, Table4 as D
	WHERE A.Indice>0.05
	and A.Num_val1 > A.Num_Val2
	and C.NbOcc>1
	and A.Num_Val1=B.Num_val
	and A.Num_val2=C.Num_Val
	and A.Num_Val1=D.Num_Val1
	and A.Num_Val2=D.Num_Val2
 
	Select top 500 * from #mytempdb order by NbOcc1 DESC,Indice DESC
END

Quand je l'execute cette proc avec SQLExec, ça marche, mais quant je l'appelle en ASP, il n'ouvre pas le RecordSet :

Code :

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

1
2
3
4
5
6
7
 
Set RS=Connexion.execute("exec maproc")
If not(RS.EOF and RS.BOF) then
	response.write "Ca marche"
else
	response.write "Ca marche PAS"
end if

Le message d'erreur ASP :

Code :

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

1
2
3
4
ADODB.Recordset error '800a0e78' 
Cette opération n'est pas autorisée si l'objet est fermé. 
 
test.asp, line 43

La ligne 43 c'est :

Code :

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

If not(RS.EOF and RS.BOF) then

Où est le pb ?

Merci

James

[denisys]

Tu peut afficher le code complet de ta Page ASP ??????

[seedorf]

Je sais pas si ca peut venir de là ou pas mais, t'es sur de ta condition????not(RS.EOF and RS.BOF) ????????


Merci

[James85]

Oui, c'est ce que j'utilise pour vérifier qu'il y a bien un résultat à ma requête avant d'appeler ce résultat avec des RS(0) qui entrainent des erreurs si le RS est vide.

Merci

[James85]

Je ne peux pas afficher tout mon code, mais je peux dire que lorsque je modifie ma ps 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
CREATE PROCEDURE maproc 
AS 
BEGIN 
   SELECT TOP 500 * into #mytempdb 
   FROM Table1 AS A, Table2 AS B, Table3 AS C, Table4 as D 
   WHERE A.Indice>0.05 
   and A.Num_val1 > A.Num_Val2 
   and C.NbOcc>1 
   and A.Num_Val1=B.Num_val 
   and A.Num_val2=C.Num_Val 
   and A.Num_Val1=D.Num_Val1 
   and A.Num_Val2=D.Num_Val2 
   order by NbOcc1 DESC,Indice DESC 
END

Ca marche. Le reste de la page ne semble donc pas en cause.

Il semblerait qu'il y ait un pb lorsque j'enchaine les deux requêtes. Faut-il fetcher ma table #mytempdb ?

Merci

James

[Gandalf Le Blanc]

Slt,

Il faut que tu rajoutes un SET NOCOUNT ON dans ta procedure :

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
CREATE PROCEDURE maproc 
AS 
BEGIN 
 
   SET NOCOUNT ON
 
   SELECT * into #mytempdb 
   FROM Table1 AS A, Table2 AS B, Table3 AS C, Table4 as D 
   WHERE A.Indice>0.05 
   and A.Num_val1 > A.Num_Val2 
   and C.NbOcc>1 
   and A.Num_Val1=B.Num_val 
   and A.Num_val2=C.Num_Val 
   and A.Num_Val1=D.Num_Val1 
   and A.Num_Val2=D.Num_Val2 
 
   Select top 500 * from #mytempdb order by NbOcc1 DESC,Indice DESC 
END

a+

[SQLpro]

Je ne voit vraiment pas l'intérêt d'une SP dans ce cas :
1) tu ne passe aucun paramètre
2) il n'y a qu'une seule requêteet donc pas de transaction à faire !
3) fait tes jointures dans la ckause FROM et pas dans le WHERE !
4) pour lancer une procédure qui renvoi un jeu de résultat il faut l'ouvrir avec une métode OPEN et non l'exécuter avec une méthode EXEC. L'algo est : OPEN, boucle de lecture(FETCH), CLOSE

donc :

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
SELECT TOP 500 * 
 
INTO   #mytempdb
 
FROM   Table1 AS A
       INNER JOIN Table2 AS B
             ON A.Num_Val1 = B.Num_val
       INNER JOIN Table3 AS C
             ON A.Num_val2 = C.Num_Val
       INNER JOIN Table4 as D
             ON A.Num_Val1=D.Num_Val1
                AND A.Num_Val2=D.Num_Val2
 
WHERE  A.Indice>0.05
  and  A.Num_val1 > A.Num_Val2
  and  C.NbOcc>1

A +

[denisys]

Affiche la partie déclaration

Code :

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

Dim

Ainsi que la partie ouverture de connexion .

[James85]

Merci Gandalf, le SET NOCOUNT ON a résolu mon pb.

J'utilise une proc car je souhaite manipuler seulement les premiers enregistrements (le top 800 sur plus de 20000 enregistrements), et je ne vois donc pas comment faire autrement.

Par ailleurs, les inner join on s'y fait bien, mais cela ne semble pas changer grand chose aux perfs ... Ni à l'analyse de la requête...

Merci à tous,

James