Discussion :

[Jeannotc]

Bonjour,
Je bute sur cette requête
with ADOQuery1 do begin
SQL.Clear;
SQL.Add('UPDATE landry SET');
SQL.Add('AUTEURS=:CAuteur1, OUVRAGES=:CTitre1, LIEU=:CEtage1 WHERE');
SQL.Add('AUTEURS=:CAuteur, OUVRAGES=:CTitre, LIEU=:CEtage ');
ADOQuery1.Prepared:=True;
ExecSQL;
end;
Je précise :
Les variables CAuteur1, CTitre1 et CEtage1 contiennent les nouvelles valeurs
Les variables CAuteur, CTitre et CEtage les anciennes
J'ai constamment l'erreur me diant que Objet Parameter defini de manière incorrect Des informations incohérentes ou incomplètes ont été fournies !
Un petit coup de main serait le bienvenu !
merci

[Nono40]

Fait voir le code de mise à jour des paramètres stp...

[Jeannotc]

voilà :
Parameters.ParamByName('CAuteur').Value := CAuteur1;
Parameters.ParamByName('CTitre').Value := CTitre1;
Parameters.ParamByName('CEtage').Value := CEtage1;

je pense que c'est ça !

ces lignes sont mises avant le ADOQuery.Prepared
et merci pour cette réponse rapide !

[cmen76]

Il pourrait s'agir d'un problème dans la construction implicite des paramètres (genre type ftUnknown).
As-tu fait l'essai de mettre le texte SQL à la conception dans l'ADOQuery, et de consulter ses 'Parameters' ?
Voir aussi la propriété ParamCheck...

[Jeannotc]

salut,
Dans la construction implicite des paramètres les trois sont de type ftString.
Quand à la propriété ParmaCheck elle est à true !
Non j'ai pas essayé de mettre le texte SQL à la conception car je fais plusieurs requêtes différentes avec le même TADOQuery.

La première pour trouver l'enregistrement donc avec un select et là ca fonctionne parfaitement, puis après les modifs, je cherche donc à modifier l'enregistrement.

Mais je vais peut-être essayer en mettant un deuxième TADOQuery !
je vais voir et vous tiens au courant
merci pour la réponse !

[Jeannotc]

re salut,
bon après avoir créé un 2ème TADOQuery puis à la conception mis le teste suivant dans le SQL :

Code :

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

1
2
3
 
UPDATE landry SET AUTEURS=:titre1, OUVRAGES=:titre, LIEU=:titre2
WHERE OUVRAGES=:titre4

puis changé la boucle de la manière suivante :

Code :

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

1
2
3
4
5
6
7
8
9
 
 with ADOQuery2 do begin
             Parameters.ParamByName('titre1').Value:=CAuteur1;
             Parameters.ParamByName('titre').Value:=CTitre1;
             Parameters.ParamByName('titre2').Value:=CEtage1;
             Parameters.ParamByName('titre4').Value:=CTitre;
             ADOQuery2.Prepared:=True;
             ExecSQL;
             end;

ca a l'air de fonctionner !
Mais ne risquerais-je pas un problème en cas de deux ouvrages qui auraient le même titre ?
CVest un programme destiné à retrouver des livres rangés dans une bibliothèque !

En tous cas un grand merci pour votre aide

[cmen76]

Effectivement la mise à jour se fera sur tous les livres portant le titre passé en clause where de l'update.
En principe pour éviter cela, on crée une clé primaire assurant l'unicité d'enregistrement dans la base.

Bon courage.

[Jeannotc]

Oui, je me doutais bien de la chose !
C'est la raison pour laquelle je voulais prendre en compte le nom de l'auteur et le nom de l'ouvrage !
Mais je vais voir pour recréer la base avec une clé primaire!
Dans ce cas, je mettrais la clé dans la close WHERE !

enfin merci beaucoup, mais je comprends pas pourquoi la 1ère méthode ne voulait pas fonctionner !

[cmen76]

rien ne t'empêche non plus d'établir une clause where telle que

Code :

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

1
2
 
...WHERE OUVRAGES=:titre4 AND AUTEURS=:auteur

En fait, ce qu'il faut retenir, c'est qu'en affectant dynamiquement le texte SQL à un objet Query, on remet en cause les paramètres qui avaient pu être "parsés" (analysés) dans le mode conception.
Dès lors, ces paramètres potentiellement recréés ne seront peut-être pas directement utilisables.

N'oublies pas d'utiliser le tag résolu si tu estimes que c'est le cas.

[Jeannotc]

oui, je vais essayer cette solution !

[Jeannotc]

Salut,
Oui, tout marche comme sur des roulettes
encore une fois milles mercis