Discussion :

[NewJS]

Bonjour à tous,
j'ai une erreur de code sous Oracle que voici. En fait je crée une fonction dans un package, là voici. C'est une requête en PL/SQL.

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
function p1 ( p1_id in number) return varchar2
    is
    retour varchar2(80);
    begin
 
    retour := null;
 
    begin
    SELECT paym.payment_method_code
    INTO retour
    FROM ap_suppliers fou,
    ap_supplier_sites_all fous,
    iby_external_payees_all payee,
    iby_ext_party_pmt_mthds paym
    WHERE fou.party_id = payee.payee_party_id
    AND payee.ext_payee_id = paym.ext_pmt_party_id
    AND fous.vendor_site_id = p1_id
    AND fou.vendor_id = fous.vendor_id
    AND fous.vendor_site_id = payee.supplier_site_id
    AND paym.primary_flag = 'Y';
    exception
    when no_data_found then null;
    end;
    return retour;
    end;

Ensuite, j'essaye d'insérer cette fonction dans une table pour me retourner une valeur.

Code :

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

1
2
3
4
5
6
7
8
9
insert into table_test (
    org_id,
    champ_test
    )
    select
    fous.org_id, --ORG_id
    package1.p1 (fous.vendor_site_id)
    from ap_supplier_sites_all fous
    where ( fous.vendor_site_id) in (select DISTINCT vendor_site_id_attribute1 from table_test2_v);

Mais en retour j'ai une erreur. Pourtant j'ai bien 2 champs dans mon INSERT. Pourquoi il met cette erreur 'Too many values' ? Quelqu'un a une idée ?

Code :

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

1
2
3
4
insert into table_test (
    *
    ERROR at line 1:
    ORA-00913: too many values

[vanagreg]

Bonjour,

Es tu sûr que ta fonction ne retourne qu'au plus une ligne?

[NewJS]

Bonjour

Non, je ne suis pas sûr et je soupçonne qu'elle retourne plusieurs lignes.
Mais comment le savoir ? Sur SQLDeveloper par exemple

[SQLpro]

Votre erreur ne serait-elle pas ici :

Code :

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

in (select DISTINCT vendor_site_id_attribute1 from table_test2_v)

Combien de valeurs sont retournées par cette sous-requête ?
Optez pour un jointure !

A +

[vanagreg]

Il faudrait joindre la requête de la fonction à la requête servant à l'insertion Déjà, pour savoir si c'est bien le problème, ajoute "or too_many_rows" à l'exception de ta fonction:

Code :

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

when no_data_found or too_many_rows then null;

[NewJS]

Il faudrait joindre la requête de la fonction à la requête servant à l'insertion Déjà, pour savoir si c'est bien le problème, ajoute "or too_many_rows" à l'exception de ta fonction:

Code :

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

when no_data_found or too_many_rows then null;

Alors que je rajoute le too_many_rows, j'ai le message Package created No errors
En revanche, la console ne me rend pas la main, donc pour quitter je fais un CTRL C et j'ai ce message

[vanagreg]

Tu dois annuler quand tu fais la requête d'insertion?

[NewJS]

Non, c'est juste depuis que j'ai ajouté le too_many_values

[NewJS]

Le problème vient clairement de cette fonction là, car j'ai testé une autre fonction (avec une autre requête à l'intérieur) dans mon INSERT et ça fonctionnait

[NewJS]

Si je fais un ROWNUM <= 1 dans le WHERE de mon fonction, ça va me ramener qu'une ligne, non ?

[vanagreg]

Oui, mais "au hasard". Si peut importe la ligne retournée si plusieurs occurrences sont trouvées, alors tu peux utiliser rownum = 1 en effet.

[NewJS]

OK, j'essaye ça dans un 1er temps pour voir le résultat.
Mais je pense que je devrais trouver autre chose pour avoir une cohérence des données.

[JeitEmgie]

ORA-00913 or ORA-00947 complains about returned number of columns, not rows

[NewJS]

Merci
Donc je retourne trop de colonnes ? Ce n'est pas le nombre de lignes le souci ?
Effectivement, mon ROWNUM <= 1 n'a rien changé pour l'instant, toujours le même souci

[NewJS]

Votre erreur ne serait-elle pas ici :

Code :

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

in (select DISTINCT vendor_site_id_attribute1 from table_test2_v)

Combien de valeurs sont retournées par cette sous-requête ?
Optez pour un jointure !

A +

Alors effectivement, je pense que la piste est plutôt à creuser ici.

Code :

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

select DISTINCT vendor_site_id_attribute1 from table_test2_v

me retourne 25 lignes.

[vanagreg]

Non ce n'est pas grave si la sous-requête ramène plusieurs lignes. Au fait le distinct ne sert à rien dans une sous-requête IN, dans ce cas il vaut mieux faire un EXISTS.
Il faut y aller par étape. D'abord, enlever l'appel à la fonction. Est-ce que l'instruction suivante fonctionne:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
insert into table_test (
    org_id,
    champ_test
    )
    select
    fous.org_id, --ORG_id
    'Test', --package1.p1 (fous.vendor_site_id)
    from ap_supplier_sites_all fous
    where exists (select 1 from table_test2_v v
                        where v.vendor_site_id_attribute1  = fous.vendor_site_id
                       );

[JeitEmgie]

Alors effectivement, je pense que la piste est plutôt à creuser ici.

Code :

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

select DISTINCT vendor_site_id_attribute1 from table_test2_v

me retourne 25 lignes.

ce n'est pas le nombre de lignes le problème.

[NewJS]

Non ce n'est pas grave si la sous-requête ramène plusieurs lignes. Au fait le distinct ne sert à rien dans une sous-requête IN, dans ce cas il vaut mieux faire un EXISTS.
Il faut y aller par étape. D'abord, enlever l'appel à la fonction. Est-ce que l'instruction suivante fonctionne:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
insert into table_test (
    org_id,
    champ_test
    )
    select
    fous.org_id, --ORG_id
    'Test', --package1.p1 (fous.vendor_site_id)
    from ap_supplier_sites_all fous
    where exists (select 1 from table_test2_v v
                        where v.vendor_site_id_attribute1  = fous.vendor_site_id
                       );

Oui tout à fait. Cela fonctionne, j'ai testé d'autres fonctions (que j'avais crée dans mon package) et le INSERT se passe sans problèmes.
C'est vraiment cette fonction en particulier qui me ramène l'erreur Erreur ORA-00913: too many values au moment de l'INSERT.

Quand je l'a crée dans mon package et que je compile, c'est crée avec succès et sans erreur. C'est vraiment au moment d'insérer ce champ dans ma table qu'il y a une erreur.

[SQLpro]

Votre fonction utilise la même table (ap_supplier_sites_all fous) que la requête. Il est probable que vous avez alors une récursivité induite....

A +

[NewJS]

Intéressant. Mais quand j'ai crée d'autres fonctions, celles ci utilisaient aussi la même table, et il n'y avait pas d'erreur au moment de l'INSERT.