Discussion :

[elite18000]

Bonjour a tous je bute sur un petit probleme.
J'essaye d'appeler une procedure stocke dans ma base de donnee mais Oracle me retourne une erreur !

Voici la fonction :

Code SQL :

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

1
2
3
4
5
6
7
8
9
10
create or replace procedure InsertAdherent(nom in VARCHAR2,pre in VARCHAR2, ecole in VARCHAR2,login in VARCHAR2, mdp in VARCHAR2, numR in VARCHAR2, Rue in VARCHAR2, codeP in VARCHAR2, Ville in VARCHAR2 ) is
idMax NUMBER(1);
 
Begin
	SELECT MAX(IDADHERENT) INTO idMax
	FROM VELUPSUD_ADHERENT;
 
	INSERT INTO VELUPSUD_ADHERENT
	VALUES(idMax,nom,pre,ecole,login,mdp,SYSDATE,SYSDATE,'MEMBRE',numR, Rue,codeP,Ville);
End;

Ensuite je tente de l'execute avec la commande SQL suivante trouvée sur internet et dans la doc d'oracle :

Code :

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

execute InsertAdherent('Frz','Clem','Orsay','test','coucou','4','rue jean','90000','Test2');

Oracle me retourne l'erreur suivante :

ORA-00900: instruction SQL non valide

Est ce que quelqu'un aurait une solution a mon probleme ? J'ai le meme probleme pour des fonctions mais je l'ai contourne avec "SELECT nom_fonction FROM DUAL".

Merci par avance !

[skuatamad]

2 points :
- Il faut toujours écrire la liste des colonnes que l'on insère, le fait qu'il soit possible de l'ommettre reste une mauvaise pratique (comme l'utilisation de *)

- idMax NUMBER(1), number(1) est vraiment petit de toute façon sélectionner le max (normalement la faute porte sur max+1) ne fonctionne pas en concurrence d'accès, il faut utiliser une séquence (tu peux te passer du trigger)

[Waldar]

En complément des informations communiquées par skuatamad, j'ajouterai qu'execute est une instruction SQL*Plus.

Pour appeler votre procédure de façon standard :

Code :

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

1
2
3
begin
    InsertAdherent('Frz','Clem','Orsay','test','coucou','4','rue jean','90000','Test2');
end;

[Mohamed.Houri]

Pour appeler votre procédure de façon standard :

Code :

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

1
2
3
begin
    InsertAdherent('Frz','Clem','Orsay','test','coucou','4','rue jean','90000','Test2');
end;

C'est vrai qu'utiliser des procédures stockées (SQL statique) nous enlève la charge de veiller à l'utilisation des "bind variables"; mais s'il fait tous ses appels tels que vous lui avez conseillé, bonjour alors le problème de la mémoire

http://hourim.wordpress.com/2011/06/...ing-parameter/

[elite18000]

merci pour toutes ces réponses je vais essayer immédiatement !

pour la fonction exécute, elle me permet simplement de faire des test du bon fonctionnement de la procédure. En revanche je n'arrive pas a l'appelle depuis mon programme JAVA, avec le JDBC, et callableStatement..

Merci !

[skuatamad]

En revanche je n'arrive pas a l'appelle depuis mon programme JAVA, avec le JDBC, et callableStatement..

C'est là qu'intervient call

Code :

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

call InsertAdherent(?,?,?,?,?,?,?,?,?);

A combiner avec les méthodes PREPAREDSTATMENT et BIND, que je ne connais pas en java, pour en savoir plus n'hésite pas à poster aussi sur le forum java.

En attendant regarde les méthodes java utiliseé par alberto dell'Era et le follow up de tom kyte pour des exemple de syntaxes (notamment .setInt pour bind un integer à priori) :
http://asktom.oracle.com/pls/asktom/...48426602526853