Discussion :

[aldente]

Rebjr !

Voilà, je veux donc éxécuter un bloc pl/sql qui me permet d'appeler une procédure qui va inscrire l'utilisateur donnée à un client dans une table et tester s'ils st pas déjà présents dans la table.
J'emploie donc la procédure dans une boucle

Code :

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

1
2
3
4
5
6
7
8
9
10
 
BEGIN
FOR rec in (select nom,prenom,mdp from table_user
                where not exists (select 'x' from table_inscrits t2 ,table_clients t3
                                                 where t1.iduser=t2.iduser
                                                 and t1.idclient=t3.idclient))
LOOP
mon_package.maprocedure(rec.nom,rec.prenom,rec.mdp,client);
END LOOP;
END;

Mais voilà, message d'erreur sur ma procédure "L'extraction exacte ramène plus que le nombre de lignes demandé".
Prtant la requête dans ma procédure fait une comparaison entre mes paramètres (IN) avec les données de mes tables, et fais l'insertion qd c'est le cas, donc ne renvoie qu'une seule ligne qd il y a correspondance car une seule ligne par utilisateur inscrit à un client donné.

Où pourrais-je me tromper ?

[Loyd1974]

L'erreur doit provenir peut-être de ta procédure, parce que là, d'un rapide coup d'oeil, je ne vois pas ce qui cloche.

Pourrais-tu donner plus de détail sur l'algo de ta procédure ?

Edit: Je parle bien sûr de la procédure que tu as représenté par :

Code :

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

 mon_package.maprocedure(rec.nom,rec.prenom,rec.mdp,client);

[aldente]

Là voilà !

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
 
PROCEDURE ma_proc(name,fname,mdpwd,client)
--variables
BEGIN
--test client&utilisateur existent
select id_utl,nom,prenom,mdp,id_cli,nomcli INTO iduser,idcli...
   from table_user,table_clients
   where name=nom
   and   fname=prenom
   and   mdpwd=mdp
   and  client=nom_client;         <---modification le 30/08 à 20h56
--test si utilisateur inscrit
select actif into a 
from table_inscrit
 where id_cli=idcli
 and id_utl=iduser
  if a=0 THen
   raise
  end if;
EXCEPTION
 when no_data_found then
    if iduser is null THEN
      raise..
    else
      insert...
   end if;
END;

[McM]

cette requete doit ramener plus d'une ligne.

Code :

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

1
2
3
4
5
6
select id_utl,nom,prenom,mdp,id_cli,nomcli 
INTO iduser,idcli...
   from table_user,table_inscrit,table_clients
   where name=nom
   and   fname=prenom
   and   mdpwd=mdp

ou

Code :

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

1
2
3
4
select actif into a 
from table_inscrit
 where id_cli=idcli
 and id_utl=iduser

Tu peux gérer le WHEN TOO_MANY_ROWS

[aldente]

Le fait est qu'elle va comparer ma chaîne de caractère et récupérer l'id de l'utilisateur et du client concerné. Comment se pourrait-il qu'il renvoie plus d'une ligne vu qu'il n'y a qu'une inscription possible ? étant donné qu'un utilisateur ne peut pas être membre plus d'une fois à un même client.

edit:non j'ai bien vérifié. qd je rentre les bons paramètres, mes ids concernés sur la première requête sont prises, puis dans la 2eme je fais une comparaison avec ces premiers pour gérer s'ils sont déjà inscrits. J'ai bien une ligne de retournée.

[Phil_67]

Bonjour,

Tu pourrais pour trouver d'où vient ton erreur ajouter une variable number
'numinst' et lui affecter des valeurs incrémentée avant et après chacun de tes select puis gérer l'exception TOO_MANY_ROWS et retourner en paramètre OUT de ta procédure le numéro d'instruction et pourquoi pas le concaténer avec le name et le client afin d'identifier quel enregistrement te pose problème.

[McM]

Code :

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

1
2
3
4
5
6
select id_utl,nom,prenom,mdp,id_cli,nomcli 
INTO iduser,idcli...
   from table_user,table_inscrit,table_clients
   where name=nom
   and   fname=prenom
   and   mdpwd=mdp

1 : met des alias.
2 : Où sont les jointures avec ces tables table_inscrit,table_clients
3 : Attention aux paramètres du même nom qu'une colonne (pour éviter ça toujours avoir un nommage particulier pour les paramètres genre p_nomparam

[aldente]

Oula la grosse bêtise ça se voit que j'étais fatigué :p.
Pour cette requête,McM, voici sa correction, la table table_inscrit n'y est pas je ne l'utilise pas dans la première requête, uniquement dans la 2eme.
Pour les alias, vu que je ne récupère qu'un champ(enfin 2 j'oublie le nom ^^) de la table_client je n'ai pas jugé nécessaire de le mettre.

Code :

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

1
2
3
4
5
6
7
8
 
select id_utl,nom,prenom,mdp,id_cli,nomcli 
INTO iduser,idcli...
   from table_user,table_clients
   where name=nom
   and   fname=prenom
   and   mdpwd=mdp
   and   client=nom_client

Voilou, Phil jte dis ça demain .
Merci de votre attention.

[McM]

Table_client: le champ nom_client(qui n'est pas la pk qui doit être idclient) est-il unique ?

[aldente]

Oui il est unique.
Je ne comprends pas que l'extraction puisse tjs rammener plus de ligne.
Vu que ma première requête fait une comparaison entre les paramètres rentrés et ce qui est présent dans la table, et ma 2ème requête qui va bien vérifier avec les clauses where, les id sélectionnés par la première requête.

[McM]

ajoute une trace

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
PROCEDURE ma_proc(name,fname,mdpwd,client)
--variables
BEGIN
--test client&utilisateur existent
select id_utl,nom,prenom,mdp,id_cli,nomcli INTO iduser,idcli...
   from table_user,table_clients
   where name=nom
   and   fname=prenom
   and   mdpwd=mdp
   and  client=nom_client;         <---modification le 30/08 à 20h56
--test si utilisateur inscrit
select actif into a 
from table_inscrit
 where id_cli=idcli
 and id_utl=iduser
  if a=0 THen
   raise
  end if;
EXCEPTION
 when no_data_found then
    if iduser is null THEN
      raise..
    else
      insert...
   end if;
WHEN TOO_MANY_ROWS
THEN
 DBMS_OUTPUT.PUT_LINE('Toomany ' || name ||' '||fname ||' '||mdpwd ||' '||client);
END;

[Loyd1974]

En toute logique si une des 2 requêtes suivantes ramène au moins une ligne, tu auras identifié ton problème

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
-- recherche de doublon dans la première partie
   SELECT nom, prenom, mdpwd, client, count(*)
   FROM table_user,table_clients
   GROUP BY nom, prenom, mdpwd, client
   HAVING count(*) > 1;
 
-- recherche de doublon dans la 2nd partie
   SELECT id_cli, id_utl, count(*)
   FROM table_inscrit
   GROUP BY id_cli, id_utl
   HAVING count(*) > 1;

Sinon, cela veut dire que ton erreur ne provient pas de là.

La dernière fois que j'ai eu ce genre de mésaventure, c'était la faute d'un trigger : regarde si tu n'as pas un trigger BEFORE INSERT dans ta table cible, et si oui, vérifie qu'il n'y ait pas de loup à l'intérieur

[aldente]

He bien merci à vous 2 pour ces solutions .

Dans un premier temps, j'ai testé ta première requête, et effectivement j'avais un doublon (incroyable lol).-->donc merci Loyd1974
Puis dans un deuxième temps, la gestion du "too_many_rows", a pu m'afficher le doublon concerné et l'insertion s'est bien passée-->donc merci McM.

Sympa pour le coup de main

[McM]

alors

[aldente]

alors

En effet voilà !