[debutant][procedure stockee][plpgsql][java] aucun retour !?

Version imprimable

Bonjour,

J'apprends à créer des procédures stockées sous postgres et je bloque dès le début.

Ma procédure a pour but de valider la bonne connexion d'un utilisateur en fonction du login saisi et de son mot de passe. Voici son code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
DECLARE
  ident integer;
BEGIN
  SELECT INTO ident id FROM admin.users WHERE login = login AND password = password;
 
  IF ident IS NULL THEN
    -- l'utilisateur n'existe pas
    RETURN -1;
  ELSE
    UPDATE admin.users SET connection = connection+1 WHERE id = ident;
    RETURN ident;	
  END IF;
END;

et voici mon appel en java :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
    CallableStatement stUser = null;
    stUser = conn.prepareCall("{? = call admin.isvalidlogon(?, ?)}");
 
    stUser.registerOutParameter(1, Types.INTEGER);
    stUser.setString(2, "tata");
    stUser.setString(3, "toto");
 
    stUser.executeUpdate();
 
    if (stUser.getInt(1) > -1) {
      System.out.println("Connexion autorisée");
    }

et voici l'exception levée :

Code:

1
2
3
4
5
6
 
org.postgresql.util.PSQLException: Un CallableStatement a été exécuté mais n''a rien retourné.
	at org.postgresql.jdbc2.AbstractJdbc2Statement.executeWithFlags(AbstractJdbc2Statement.java:340)
	at org.postgresql.jdbc2.AbstractJdbc2Statement.executeUpdate(AbstractJdbc2Statement.java:282)
	at fr.ggl.utils.database.sp.ProcedureStockee.logon(ProcedureStockee.java:48)
	at fr.ggl.utils.database.sp.ProcedureStockee.main(ProcedureStockee.java:19)

13/09/2005, 12h48
GrandFather

Bonjour,

je ne suis pas sûr que ta chaine d'appel à la procédure stockée soit totalement compatible avec le driver JDBC pour PostgreSQL. Essaie plutôt une requête du style :

Code:

SELECT admin.password('toto', '12345') AS logon
13/09/2005, 14h01
Invité

ok j'ai trouvé mon erreur : il faut lancer execute() et no pas executeUpdate().

seulement maintenant le problème viens de ma procédure, elle me retourne toujours le même id : 1. Quoique je mette en login et mot de passe.

:?: :!:
13/09/2005, 15h13
hpalpha

euh c'est plutot logique :
tu as :

Code:

WHERE login = login AND password = password

donc tu compare un champ avec le meme champ,
replace par tes arguments

Code:

WHERE login = $1 AND password = $2

oui c clair j'ais fait le changement :oops: grosse boulette

voici mon nouveau code de procédure :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 DECLARE ident integer; BEGIN SELECT INTO ident id FROM admin.users WHERE name = login AND pass = password; IF ident IS NULL THEN -- l'utilisateur n'existe pas RETURN -1; ELSE UPDATE admin.users SET connection = connection+1 WHERE id = ident; RETURN ident; END IF; END;
en java :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 CallableStatement stUser = null; stUser = conn.prepareCall("{ ? = call admin.isvalid( ?, ? ) }"); stUser.registerOutParameter(1, Types.OTHER); stUser.setString(2, "tete"); stUser.setString(3, "tete"); stUser.execute(); System.out.println("identifiant : " + stUser.getObject(1)); stUser.close();
Le problème maintenant c'est que mon update ne fait rien, certainement dû au fait que fais un execute() et non pas un executeUpdate().
Mais un executeUpdate() n'est utilisable que si on ne retourne pas de valeur (si j'ai bien compris et c'était pas première erreur).

Alors comment faire :?:

13/09/2005, 16h20
hpalpha

peux tu nous mettre le code complet de ta fonction (avec le create function ....) en plpgsql ?

OK :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
CREATE OR REPLACE FUNCTION admin.isvalid("varchar", "varchar")
  RETURNS "unknown" AS
$BODY$DECLARE
  ident integer;
BEGIN
  SELECT INTO ident id FROM admin.users WHERE name = login AND pass = password;
 
  IF ident IS NULL THEN
    -- l'utilisateur n'existe pas
    RETURN -1;
  ELSE
    UPDATE admin.users SET connection = connection+1 WHERE id = ident;
    RETURN ident;	
  END IF;
 
END;$BODY$
  LANGUAGE 'plpgsql' VOLATILE;
ALTER FUNCTION admin.isvalid("varchar", "varchar") OWNER TO cristal;
GRANT EXECUTE ON FUNCTION admin.isvalid("varchar", "varchar") TO public;
GRANT EXECUTE ON FUNCTION admin.isvalid("varchar", "varchar") TO cristal;

pourquoi ?

bah, c'est bien ce que je pensais tu ne declare pas login et password

essaye sans declaration, juste les $1 et $2

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
CREATE OR REPLACE FUNCTION admin.isvalid(text,text) RETURNS integer AS
$BODY$
DECLARE
  ident integer;
BEGIN
  SELECT INTO ident id FROM admin.users WHERE name = $1 AND pass = $2;
 
  IF ident IS NULL THEN
    -- l'utilisateur n'existe pas
    RETURN -1;
  ELSE
    UPDATE admin.users SET connection = connection+1 WHERE id = ident;
    RETURN ident;   
  END IF;
 
END;
$BODY$
LANGUAGE 'plpgsql' VOLATILE;

ou avec declaration

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
CREATE OR REPLACE FUNCTION admin.isvalid(text,text) RETURNS integer AS
$BODY$
DECLARE
  ident integer;
  login ALIAS FOR $1;
  password ALIAS FOR $2;
BEGIN
  SELECT INTO ident id FROM admin.users WHERE name = login AND pass = password;
 
  IF ident IS NULL THEN
    -- l'utilisateur n'existe pas
    RETURN -1;
  ELSE
    UPDATE admin.users SET connection = connection+1 WHERE id = ident;
    RETURN ident;   
  END IF;
 
END;
$BODY$
LANGUAGE 'plpgsql' VOLATILE;

tu retourne un type unknown !!! j'ai mis integer

13/09/2005, 17h32
Invité

Mon SELECT fonctionne bien, c'est mon UPDATE qui ne fais rien.

Pour l'histoire je l'ai crée avec phpPgAdmin, mais il ne marque pas l'en-tete de la proc (CREATE...) dans le code SQL qu'il affiche (il la marque dans arguments si tu l'as), je l'ai donc récupéré avec pgAdmin3 qui lui marque tout mais pas correctement, et je le constate grâce à ta remarque.

C'est un peu compliqué je sais mais quand je l'ai crée avec phpPgAdmin j'avais mis en argument : "login" character varying, "password" character varying et en valeur de sortie, comme je fais des tests et que ma valeur à changée au fil du temps, j'avais mis au début "unknown" (disponible dans la liste, je ne sais pas vraiment à quoi ce correspond d'ailleur).

En fait pgAdmin ne le marque pas dans l'en-tete mais les variables sont bien déclarées car mon select fonctionne.

Mais comment faire pour que mon update soit pris en compte :?:

pour être plus clair voici l'image phPgAdmin :
http://jtodo.tuxfamily.org/wp-images/sp-phpPgAdmin.jpg