Discussion :

[pinocchio]

Bonjour,
Je dois effectuer une procédure de mise à jour de données en fonction de 4 paramètres qui seront remplis ou non.
Mon souci est que je pense utiliser une méthode loin d'être optimisé mais je ne trouve pas la solution.
Je passe 4 paramètres :
- nCo, nET et nAG qui permettent de sélectionner les enregistrements à mettre à jour
- nDa qui correspond à la date que je mettrai à jour.
Ma procédure

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
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
CREATE OR REPLACE  PROCEDURE "MODIFIER_INSCRIPTION"
     (
	nCo        in varchar2,
	nDa        in varchar2, -- sera toujours renseigné.
	nET	in varchar2,
	nAG	in varchar2
) IS
BEGIN
	if(nAG is null) then
		if(nCo is null) then
			if(nET is null) then
-- Raise_application_error(-20000,'Critère de sélection absent');
			else
			     update avec le test sur nET
			end if;
		else
			if(nEt is null) then
			     update avec le test sur nCo
			else
			     update avec le test sur nET et nCo
			end if;
		end if;
	else
		if(nCo is null) then
			if(nET is null) then
			     update avec le test sur nAG
			else
			     update avec le test sur nAG et nET
			end if;
		else
			if(nEt is null) then
			     update avec le test sur nAG et nCo
			else
			     update avec le test sur nAG, nET et nCo
			end if;
		end if;
	end if;

Je pense que je fais bien trop de if else end if.
Pouvez-vous m'indiquer l'instruction qui pourrait m'aider s'il vous plait?
Cordialement
Pinocchio

[McM]

Tu peux rajouter des AND dans les IF.. mais bon, ça reste des updates séparés pour tous les tests

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
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
CREATE OR REPLACE  PROCEDURE "MODIFIER_INSCRIPTION"
     (
	nCo        IN varchar2, 
  nDa        IN varchar2, -- sera toujours renseigné.
	nET	IN varchar2, nAG	IN varchar2
) 
IS
BEGIN
 
	IF nAG IS NULL AND nCo IS NULL AND nET IS NULL
  THEN
		RAISE_APPLICATION_ERROR(-20000,'Critère de sélection absent');
  END IF;
 
  IF nAG IS NULL AND nCo IS NULL AND nET IS NOT NULL
  THEN
  	UPDATE avec le TEST sur nET
  ELSIF nAG IS NULL AND nCo IS NOT NULL AND nET IS NOT NULL
  THEN
  	UPDATE avec le TEST sur nET et nCo
  ELSIF nAG IS NULL AND nCo IS NOT NULL AND nET IS NULL
  THEN
  	UPDATE avec le TEST sur nCo
	ELSIF nAG IS NOT NULL AND nCo IS NULL AND nET IS NULL
  THEN
  	UPDATE avec le TEST sur nAG
  ELSIF nAG IS NOT NULL AND nCo IS NULL AND nET IS NOT NULL
  THEN
  	UPDATE avec le TEST sur nAG et nET
  ELSIF nAG IS NOT NULL AND nCo IS NOT NULL AND nET IS NULL
  THEN
  	UPDATE avec le TEST sur nAG et nCo
  ELSE --nAG, nCo, nET NOT NULL 
  THEN
  	UPDATE avec le TEST sur nAG, nET et nCo
  END IF;
 
END;

Le mieux est bien sur de faire un seul update avec prise en compte des valeurs nulles des paramètres.

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
19
 
IF nAG IS NULL AND nCo IS NULL AND nET IS NULL
  THEN
		RAISE_APPLICATION_ERROR(-20000,'Critère de sélection absent');
  END IF;
 
 -- Si les colx sont not null
 UPDATE XXX
 SET ...
 WHERE col1 = NVL(nAG, col1)
 AND col2 = NVL(nCo, col2)
 AND col3 = NVL(nET, col3)
 
 -- Si les colx peuvent être NULL
 UPDATE XXX
 SET ...
 WHERE (col1 = nAG OR nAG IS NULL)
 AND (col2 = nCo OR nCo IS NULL)
 AND (col3 = nET OR nET IS NULL)

[pinocchio]

Merci beaucoup, j'avais pas pensé à la seconde solution.