Error to delete the first binary code entry

Question

I found this code on the Internet only it has an error: when I register one user and try to delete it shows that it has been removed, but when I open again the program it is still there.

This error occurs only when trying to delete the first registration. Ex:

• when I register 3 users and delete the last two, they are removed
• if I register 3 users and delete the 3, shows that they were deleted, but when I open the program again, they are still there

That is the code:

//O programa abaixo eh uma agenda simples, 
//com opcoes de cadastrar e exibir contatos.
//Os dados da agenda ficam armazenados em um 
//arquivo "agenda.dat" e sao lidos no inicio
//do programa e atualizados na saida.

//O arquivo "agenda.dat" contem no inicio
//o numero de registros e na sequencia os
//dados dos registros, tudo em binario.

//O objetivo do programa eh ilustrar o uso
//de alocacao dinamica de memoria e de
//arquivos binarios.


#include <stdio.h>
#include <stdlib.h>
#include <string.h>

//Cada contato possui um nome e telefone.
struct Contato{
  char nome[100];
  int  telefone;
};

int main(){
  //Ponteiro para estrutura. Vai ser o nosso vetor de 
  //estruturas dinamicas com os dados da agenda.
  struct Contato *agenda;
  int cap; //Capacidade do vetor (tamanho real do vetor).
  int n;   //Numero de contatos cadastrados (tamanho usado).
  int opc; //Opcao do menu selecionada.
  int i;
  FILE *fp;

  //Abrindo arquivo em modo de leitura ("r" = read).
  fp = fopen("agenda.dat","r");

  //Se retorna NULL eh pq nao encontrou o arquivo.
  //Isso acontece na primeira vez que rodamos o programa.
  if(fp==NULL){
    n = 0;    //Base de dados inicialmente vazia.
    cap = 20; //Fixamos uma capacidade inicial (ex: 20).

    //Alocammos o vetor de estruturas de forma
    //dinamica conforme a capacidade.
    agenda = (struct Contato *)malloc(sizeof(struct Contato)*cap);
  }
  else{ //Arquivo encontrado, logo procedemos com a leitura.

    //Le o numero de registros.
    fread(&n, sizeof(int), 1, fp);

    //A capacidade deve ser maior ou igual a "n" (ex: n*2).
    cap = n*2; 

    //Alocammos o vetor de estruturas de forma
    //dinamica conforme a capacidade.
    agenda = (struct Contato *)malloc(sizeof(struct Contato)*cap);

    //Leitura dos dados dos registros do arquivo para o vetor.
    fread(agenda, sizeof(struct Contato), n, fp);

    //Fecha arquivo apos leitura.
    fclose(fp);
  }

  do{
    //Exibe o menu de opcoes.
    printf("*** Menu ***\n");
    printf("1) Cadastrar contato.\n");
    printf("2) Exibir contatos.\n");
    printf("3) Apagar.\n");
    printf("4) Sair.\n");
    //Le a opcao selecionada.
    scanf("%d",&opc);

    if(opc==1){
      //Inserir novo contato no final do vetor.

      if(n==cap){
    //Capacidade esgotada, devemos aumentar o 
    //vetor usando "realloc" (ex: dobrar capacidade).
         cap *= 2;
         agenda = realloc(agenda, sizeof(struct Contato)*cap);
      }
      //Leitura dos dados.
      printf("Digite o nome: ");
      scanf(" %[^\n]",agenda[n].nome);
      printf("Digite o telefone: ");
      scanf("%d",&agenda[n].telefone);
      n++; //Incrementa numero de registros cadastrados.
    }
    else if(opc==2){
      //Exibe todos contatos cadastrados.
      for(i=0; i<n; i++){
        printf("*** Contato %d ***\n",i+1);
        printf("Nome: %s\n",agenda[i].nome);
        printf("Telefone: %d\n",agenda[i].telefone);
      }
    }
    else if(opc==3){
      //Apaga um contato
      char nome[100];
      int j;
      scanf("%s",&nome);
      for(i=0; i<n; i++){
             if (strcmp(agenda[i].nome, nome)==0){
                for(j=i+1; j<n; j++){
                         strcpy(agenda[j-1].nome,agenda[j].nome);
                         agenda[j-1].telefone=agenda[j].telefone;
                 }
               n--;
               i=n;
              }
      }
    }
    //Enquanto nao for opcao de saida continua mostrando menu.
  }while(opc!=4); 

  if(n>0){
    //Se existe algum contato cadastrado 
    //entao grava para o disco.

    //Abre arquivo em modo de gravacao ("w" = write).
    fp = fopen("agenda.dat","w");

    //Grava o numero de contatos no inicio do arquivo.
    fwrite(&n, sizeof(int), 1, fp);

    //Grava os dados do vetor no arquivo.
    fwrite(agenda, sizeof(struct Contato), n, fp);


    //Fecha arquivo apos a gravacao.
    fclose(fp);
  }

  //Libera a memoria alocada do vetor.
  free(agenda);

  return 0;
}

Answer 1

Analyze the code where your program saves the data, ie at the end. Before saving, there is a condition if (n > 0). Imagine that you register 10 users, quit your program (such as n == 10, and n > 0, your saved program), when returning, your database is as you left, with 10 users, but now you decide to remove all and soon after leaving your program; as n == 0 now, since you have removed all, the condition n > 0 is false, i.e., your program does not execute the code responsible for saving.

Remove this condition and the bugs cited by you will be fixed.

...
}while(opc!=4); 

//Se existe algum contato cadastrado 
//entao grava para o disco.    
//Abre arquivo em modo de gravacao ("w" = write).
fp = fopen("agenda.dat","w");      
//Grava o numero de contatos no inicio do arquivo.
fwrite(&n, sizeof(int), 1, fp);    
//Grava os dados do vetor no arquivo.
fwrite(agenda, sizeof(struct Contato), n, fp);     
//Fecha arquivo apos a gravacao.
fclose(fp);
//Libera a memoria alocada do vetor.
free(agenda);
return 0;
...