Opinión sobre un código

[Carmelo Cash]

Hola Foro:

Este código borra un registro de un RXMemoryData.
Cuando hago in click en el dbgrid, cargo la variable ID_INI y luego para borrar hago una busqueda secuencial desde el primer registro hasta el último. Si por una de esas no está OK queda en False y no hago nada.
Esto anda bién, pero me parece mucha cosa para borrar un registro.

Qué opinión les merece?. Desde ya muchas gracias por su opinión.

Código Delphi [-]
 
  with RxMDCompleto do begin
    OK:=False;
    First;
    while not(eof) do begin
      if fieldByName('ID_INI').AsInteger=ID_INI then begin
        OK:=True;
        Break;
      end;
      next;
    end;    // while

    if OK=True then begin
      Delete;
    end;
  end;

[[eduarcol]]

pero ese dataset del cual quieres borrar el registro esta conectado al grid?

[Carmelo Cash]

Si, eduarcol, y lo borra correctamente.

[*Caro*]

Hola Carmelo Cash, tu RXMemoryData, es el que esta enlazado a tu DbGrid, o es es otro DataSet el cual enlazas a tu DBGrid, porque si es el primero no necesitas recorrer, haces directamente el Delete, pero si es otro entonces, para hubicarte en el registro que quieres borrar puedes utilizar el Locate. Supongo que la RXMemoryData tiene el Locate.

Saluditos

[[maeyanes]]

Hola...

Podrías usar el método Locate:

Código Delphi [-]
with RxMDCompleto do
  if Locate('ID_INI', ID_INI, []) then
    Delete

Saludos...

[[eduarcol]]

Empezado por Carmelo Cash

Si, eduarcol, y lo borra correctamente.

entonces como te explica caro te bastaria con un delete

[Carmelo Cash]

El método Locate con el ejemplo, me parece la solución que estaba buscando.

Gracias Caro, Maeyanes y Eduarcol.

[Neftali [Germán.Estévez]]

Empezado por Carmelo Cash

Si, eduarcol, y lo borra correctamente.

Empezado por Carmelo Cash

El método Locate con el ejemplo, me parece la solución que estaba buscando.

Funcionar, funcionará, pero yo creo que estás haciendo más trabajo de la cuenta (como ya te han dicho antes), porque el registro que estás buscando es el registro sobre el que estás posicionado. Entonces ¿para qué buscar lo que ya tienes?

Locate funcionará (siempre te va a encontrar el registro actual), pero no es necesario.

[Carmelo Cash]

Entendido.
Gracias.