Error al actualizar las tablas

[Nathan]

Hola Gente
les comento el problema que tengo, que al actualizar la tabla 'detalle_factura' lo guarda con 'null'.

Código SQL [-]
procedure TDatos.DetFactura(id, id2, obs: string;  idC: integer);
begin
  Actualizar.SQL.Clear;
  with Actualizar.SQL  do
  begin
    Add('UPDATE DetFactura SET ');
      Add('Id =:valor,');
      Actualizar.Parameters.ParamByName('valor').Value:=id;
      Add('IdFact =:valor,');
      Actualizar.Parameters.ParamByName('valor').Value:=id2;
      Add('Observaciones =:valor2');
      Actualizar.Parameters.ParamByName('valor2').Value:=obs;
    Add('WHERE IdDetFact =:valorX');
    Actualizar.Parameters.ParamByName('valorX').Value:=idC;
  end;// fin with
  Actualizar.ExecSQL;
end;

lo raro es que no genera ningun error en la compilacion y ejecucion , pero lo guarda con null y cuando lo realizo por la consola de mysql no da error, alguien me podria decir que es lo que estoy haciendo mal ?.

pd.- estoy usando en la conexcion Ado y MySql.


De antemano gracias.

[[Lepe]]

Supongo que será errores de copiar y pegar, pero tienes 3 parámetros que se llaman "valor".

Lo normal es hacerlo así:

Código Delphi [-]
procedure TDatos.DetFactura(id, id2, obs: string;  idC: integer);
begin
  Actualizar.SQL.Clear;
  with Actualizar.SQL  do
  begin
    Add('UPDATE DetFactura SET Id =: prId, IdFact =: prIdFac, Observaciones =: prObservaciones');
    Add('WHERE IdDetFact =: prIdDetFact');
      Actualizar.Parameters.ParamByName('prId').Value:=id;
      Actualizar.Parameters.ParamByName('prIdFac').Value:=id2;
      Actualizar.Parameters.ParamByName('prObservaciones').Value:=obs;
    Actualizar.Parameters.ParamByName('prIdDetFact').Value:=idC;
  end;// fin with
  Actualizar.ExecSQL;
end;

Ya que al dar valores a cada parámetro se tiene que hacer un parser de la cadena SQL para averiguar los parámetros, así que solo lo hacemos una sola vez.

[Nathan]

Gracias Lepe por tu respuesta, ya resolvi el problema que me da verguenza decirlo , pero bueno me di cuenta que estaba utilizando parametro 'valor' para id y id2 es ahi donde se generaba el error.




Saludos.

[[Lepe]]

Como suele decirse, 4 ojos ven más que 2 , y nunca está demás ver como lo hacen otra gente, siempre se toman ideas.

Saludos y me alegro que se solucionara rápido.