Discussion :

[guigui64]

Bonsoir,

Je viens vers vous pour vous faire part d'une erreur pénible : j'ai tout une colonne qui se rempli d'une même fausse valeur. Comme vous, je pensais que ça venait d'un problème de type.
Seul hic: toutes les variables appelées ici ont le même type : c'est-à-dire number.
Pouvez-vous m'aider?

Le code est chargé de calculer l'écart entre le nombre de réponse type à un questionnaire de hasard parmi mille répondants et le nombre de réponse réelle des répondants. Celui qui a le plus faible écart est premier.
Voici mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
create or replace
PROCEDURE CALCULER_ECART (P_NB_REP_TYPE IN number) AS
cursor cur is select * from bons_bons;
V_ecart number;
BEGIN
  for bon in cur loop
    V_ecart :=  P_NB_REP_TYPE - bon.rep2;
  end loop;
 
  update bons_bons set ecart =V_ecart;
END CALCULER_ECART;

[Pozzo]

Bonjour,

C'est votre algorithme qui est erroné.

Je suppose que le nombre de réponse réel est dans le champ rep2 et que le nombre de réponse type est dans P_NB_REP_TYPE.

Vous affectez successivement chaque écart à la même variable v_ecart.
En sortie de boucle loop v_ecart contient l'écart pour la dernière ligne de bons_bons.
Ensuite vous mettez à jour TOUTE la table bons_bons avec cette valeur puisque qu'il n'y a pas de clause where à votre update. Donc tout le monde a l'écart du dernier.

Si vous voulez utiliser du PlSql et une boucle il faut mettre à jour la table bons_bons à chaque tour de boucle à l'intérieur du loop, en utilisant une clé unique de la table bons_bons si elle en a une. En a-t-elle une ?

Mais pour votre problème vous n'avez pas besoin d'une boucle.
Un simple update sur la table suffit.

Pozzo

[McM]

Si vous voulez utiliser du PlSql et une boucle il faut mettre à jour la table bons_bons à chaque tour de boucle à l'intérieur du loop, en utilisant une clé unique de la table bons_bons si elle en a une. En a-t-elle une ?

Tu n'es pas obligé d'avoir une PK pour faire l'update, avec current of

Code :

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

1
2
3
4
5
6
7
8
DECLARE
Cursor C is SELECT * FROM BONS FOR UPDATE;
BEGIN
FOR r IN C LOOP
UPDATE BONS SET col1 = col2
WHERE current of c;
END LOOP;
END;