ayuda programa con archivos binarios

[nahuel]

hola, este es mi primer post en este foro, queria pedirles ayuda con este codigo ya que lo ejecute paso a paso muchas veces y no logro encontrarle el error, el problema es que siempre me guarda el primer registro, aunque cambie los datos siempre me guarda el primer registro que he cargado,por ejemplo cargo el primer registro con el nombre de "juan" completo los otros datos y cuando agrego otro registro distinto, por ejemplo "pedro" en el archivo se guardan 2 registros "juan" con todos los datos del primer registro cargado. quisas el problema este en que la variable de registro sea global?, desde ya muchas gracias y pido disculpas por no haber leido las recomendaciones de estilo antes de escribir en el foro :)
aca les dejo el codigo:





procedure TForm4.Button2Click(Sender: TObject); //boton guardar

var
cerrar:boolean;
v1,v2,v3,v4,v5,v6,v7,v8,v9:integer;
Comp:TValueRelationship;
fechaactual:TDateTime;
j:integer;

begin
cerrar:=True;
fechaactual:=date;
try
if Edit1.text<>'' then
reg.Nombre:=Edit1.Text;

if Maskedit1.text<>' / / ' then
begin
reg.CLU:=StrToDate(Maskedit1.Text);
mostrar(v1,Comp,fechaactual,reg.CLU,label12);
end;
if Maskedit2.text<>' / / ' then
begin
reg.Port:=StrToDate(Maskedit2.Text);
mostrar(v2,Comp,fechaactual,reg.Port,label13);
end;
if Maskedit3.text<>' / / ' then
begin
reg.Med:=StrToDate(Maskedit3.Text);
mostrar(v3,Comp,fechaactual,reg.Med,label14);
end;
if Maskedit4.text<>' / / ' then
begin
reg.Psi:=StrToDate(Maskedit4.Text);
mostrar(v4,Comp,fechaactual,reg.Psi,label15);
end;
if Maskedit5.text<>' / / ' then
begin
reg.Reinc:=StrToDate(Maskedit5.Text);
mostrar(v5,Comp,fechaactual,reg.Reinc,label16);
end;
if Maskedit6.text<>' / / ' then
begin
reg.Mor:=StrToDate(Maskedit6.Text);
mostrar(v6,Comp,fechaactual,reg.Mor,label17);
end;
if Maskedit7.text<>' / / ' then
begin
reg.AntProv:=StrToDate(Maskedit7.Text);
mostrar(v7,Comp,fechaactual,reg.AntProv,label18);
end;
if Maskedit8.text<>' / / ' then
begin
reg.CredProv:=StrToDate(Maskedit8.Text);
mostrar(v8,Comp,fechaactual,reg.CredProv,label19);
end;
if Maskedit9.text<>' / / ' then
begin
reg.CredCap:=StrToDate(Maskedit9.Text);
mostrar(v9,Comp,fechaactual,reg.CredCap,label20);
end;
except
showmessage('fecha/s invalida/s');
cerrar:=false;
end;

if cerrar then
begin
j:=posicion(reg.Nombre);
if j=-1 then
begin
seek(arch,filesize(arch));
write(arch,reg);
label21.Caption:=intToStr(filesize(arch));//////////////////////debug
label22.Caption:=intToStr(filepos(arch));///////////////////////////
end
else
begin
showmessage('encontrado!');
seek(arch,j);
write(arch,reg);
end;


end;

end;

[__cadetill]

Ya que eres nuevo, un par de recomendaciones

Si vas a publicar tanto código, y aunque fuese poco, te recomiendo el uso de la etiqueta "[ code ] aquí mi código [ /code ] (sin los espacios en blanco). Así es prácticamente imposible de seguir el código.

Mira de editar tu mensaje y pon las etiquetas, a ver si lo podemos seguir algo mejor y podemos ayudarte

También te aconsejaría la lectura de la guía de estilos (ver firma)

Por otra parte, ¡¡¡bienvenido a los foros!!!

[nahuel]

muchas gracias por la bienvenida !

ya edite mi mensaje para tratar de hacerlo mas compresible, gracias por los consejos

[Julià T.]

Tal y como veo yo como funciona:

j:=posicion(reg.Nombre);
if j=-1 then
begin
seek(arch,filesize(arch));
write(arch,reg);
//si j=-1 se añade 1 registro al final del fichero
end
else
begin
seek(arch,j);
write(arch,reg);
//si j<>-1 se cambia el registro j por el nnuevo
end;


En ningun caso se añaden 2 registros como indicas.

[nahuel]

muchas gracias a todos por haberse molestado en tratar de ayudarme, les comento que ya encontre cual era el problema, como decis vos julia T , en el procedimiento de guardar no habia ningun error, el problema estaba en el procedimiento de posicion que era el que se fijaba si el nombre ya existia o no, y de acuerdo a eso sobreescribia el existente o lo agregaba al final , respectivamente, como la variable para leer los registros era del tipo global, siempre me cambiaba el nuevo registro con el primero que leia para verificar si existia o no, gracias a los watches me pude dar cuenta.


function posicion(n:string40):integer;
var
res:boolean;
begin
res:=false;
seek(arch,0);
while not eof(arch) and not res do
begin
read(arch,reg);// aca estaba el problema
res:=AnsiUpperCase(reg.Nombre)=AnsiUpperCase(n);
end;
if res then
posicion:=filepos(arch)-1
else
posicion:=-1;
end;


entonces lo que hice fue poner una variable local al procedimiento que sea la que lea los registros y listo.