Discussion :

[ZIED]

Bonjour
Je veux créer une procédure stocké à laquelle je veux passer en paramètre 2 variables Le nom d'un champ et sa valeur

J'ai écrit ce code pour tester mais ça ne marche pas :
Code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
CREATE OR REPLACE FUNCTION __searchtest(character, character)
  RETURNS SETOF record AS
$BODY$
select prod_version from vprod_details
where $1=$2;
 
$BODY$
  LANGUAGE 'sql' VOLATILE
  COST 100
  ROWS 1000;
ALTER FUNCTION __searchtest(character, character) OWNER TO postgres;

avec $1 le nom du champ et $2 est la valeur du champ.
Est-ce possible?
Si oui comment faire?

[kain_tn]

Bonjour.

Tu peux le faire mais tu dois passer par l'instruction EXECUTE:
http://www.postgresql.org/docs/8.3/i...-EXECUTING-DYN

[ZIED]

J'ai essayé cette écriture

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
CREATE OR REPLACE FUNCTION __searchtest2(character, character)
  RETURNS bigint AS
$BODY$
PREPARE  test(character,character) as select prod_version from vprod_details
where $1=$2;
 
EXECUTE test('prod_ref','ZA01820C'); 
 
$BODY$
  LANGUAGE 'sql' VOLATILE
  COST 100;
ALTER FUNCTION __searchtest(character, character) OWNER TO postgres;

J'ai l'erreur suivante

ERROR: return type mismatch in function declared to return bigint
DÉTAIL : Function's final statement must be a SELECT.
CONTEXTE : SQL function "__searchtest2"

[estofilo]

Le EXECUTE dont kain_tn parle est celui du langage pl/pgsql, et non du langage SQL, tu ne pourras pas l'utiliser si tu te cantonnes au langage SQL.

[kain_tn]

Le EXECUTE dont kain_tn parle est celui du langage pl/pgsql, et non du langage SQL, tu ne pourras pas l'utiliser si tu te cantonnes au langage SQL.

Tout à fait.

En fait, la documentation officielle de PostgreSQL recommande d'utiliser le pl/pgSQL plutôt que le SQL pour les procédures stockées pour des questions de performances.

Tu dois d'abord ajouter ce langage à ta base:

Code :

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

CREATE LANGUAGE plpgsql;

Ensuite, ta procédure que tu peux réécrire ainsi:

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 OR REPLACE FUNCTION public.proc_searchtest2
(
    _key character varying,
	_value character varying,
	OUT prod_version bigint
)
RETURNS record
AS $$
DECLARE __query text DEFAULT null;
BEGIN
 
	__query := 'SELECT prod_version FROM vprod_details WHERE "' || _key || '" = \'' || _value || '\'';
	EXECUTE __query INTO prod_version;
 
	RETURN;
 
END $$ LANGUAGE plpgsql;

[ZIED]

merci c'est résolu!