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!

[Kornich182]

Bonjour.
Tout d'abord désolé de réouvrir un sujet clos, mais je voudrais utiliser le même genre de procédure.

Mon problème étant que lors de l'exécution de ma procédure, le nom du champ n'est pas reconnu(du fait que pour exécuter la procédure le premier paramètre est entre simples quotes)...

Merci de votre aide, je désespère!