Skip to main content
3 de 3
Removendo ruídos na resposta

Boa noite, seu código há alguns erros, então o texto vai ser um pouco longo...

Protótipos das funções

Começo pelos protótipos de suas funções:

int    menu           ();
void   ler            (FILE*);
int    gravar         (FILE*);
int    escrever_dados (FILE*);

Tire os espaços entre os parênteses e o nome delas, também diminua o espaço entre o tipo de retorno com o nome, assim fica mais organizado:

int menu(void);
void ler(FILE*);
int gravar(FILE*);
int escrever_dados(FILE*);

Outro problema é nos argumentos delas, repare que você esqueceu de colocar a variável, você apenas colocou o tipo. O protótipo precisa ficar assim:

int menu(void);
void ler(FILE *file);
int gravar(FILE *file);
int escrever_dados(FILE *file);

Agora os protótipos podem usar um ponteiro para arquivos, antes não podiam por que não estavam declarados.

Declaração de ponteiro para aquivo

Nessa parte do código você não precisa usar o malloc:

FILE *file = (FILE*)malloc(sizeof(file));

Quando você faz FILE *file; o ponteiro para arquivo já é criado. Criar um ponteiro para o arquivo usando a malloc() só seria útil caso o ponteiro estivesse encapsulado em outro arquivo, mas como esse não é o caso então apenas faça isso:

FILE *file = NULL;

Deixar o ponteiro igual a NULL não é obrigatório, mas isso pode evitar você a usar ponteiros apontando para lugares desconhecidos na memória... Então recomendo você sempre igualar a NULL ponteiros não usado no momento.

Variável select

A variável select é meio que inútil no seu código, pois o retorno da função menu() pode ser usada sem problema no switch:

switch(select = menu()) // Isso dá no mesmo
switch(menu()) // que isso

fopen()

Não é necessariamente um erro, mas o fopen() retorna NULL quando falha. Mas acho que seu código ficará mais elegante com:

if(file != NULL)
/* Você ainda pode otimizar espaço ao fazer isso */
if((file = fopen(arquivo,"r")) == NULL) return /* numero com erro */; /* E aqui pode encerrar a função/programa em caso de erro, pode usar um return para sair da função com erro */

/* Aqui será ativado em caso de sucesso */

Esquecimento de fechar o arquivo

No final da função ler() você esquece o fclose(nome_do_arquivo) para fechar o arquivo. Lembre-se que em C os dados não vão diretamente para o arquivo de disco e sim para a stream, o fclose é para avisar para o programa que todas as alteração naquele arquivo já terminaram e ele pode fechar e descarregar os dados lá, caso não faça isso então há possibilidades de perder os dados... Você também esquece o fclose na função gravar()... Exemplo de como ficaria o fclose() em seu código:

fclose(file);

fflush(stdin)

fflush() têm o comportamento indefinido quando o argumento é stdin, ou seja, pode funcionar, pode não funcionar, ou pode ocorrer outras coias... Mesmo que funcione no seu PC eu recomendo não usar.

Uma solução para resolver esse problema que alguns recomendam é o setbuf(stdin, NULL), mas esse comando não funciona em todos os computadores. Não vou entrar em muitos detalhes, pois questões de como limpar o buffer já foram discutidas muito, dê uma pesquisada depois. A solução que eu te apresento é a seguinte:

void limpar_buffer(void)
{
    char lixo;
    do
    {
        lixo = getchar();
    }while(lixo != '\n');
}

Escrevi ela de forma bem didática para você entender, mas é possível compactar essa função. Use ela sempre que você tiver certeza que há lixo no buffer.

Leitura do teclado com gets ou scanf("%s")

Essas função não servem para ler strings do teclado, pois elas não têm necessariamente um limite de caractere para ser lido, ou seja, se o usuário escrever algo com 50 caracteres e a string tiver apenas 20 de tamanho então 30 caracteres serão lidos e colocados em algum lugar na memória (provavelmente nas posições seguintes de seu vetor, isto é, v[21], v[22], ..., v[30], considere que a partir de v[21] não são partes de seu vetor). Esse problema é chamado de overflow, dê uma pesquisada depois.

Para resolver o problema a maioria recomenda o uso do fgets(variavel_que_ficara_a_string, tamanho_da_string, stdin). Exemplo de uso:

char nome[20];
fgets(nome, 20, stdin);

Dessa forma você coloca um limite de 20 caracteres para serem lidos

Problemas do fgets com a função de limpar buffer que eu recomendei

Eu particulamente não recomendo o uso do fgets, por alguns motivos, o principal e que de fato vem ao caso é que ele tem um problema com o buffer.

Bem, quando o fgets ler menos caracteres ou a quantidade esperada então ele não deixa lixo no buffer e isto é ótimo, mas se o usuário digitar uma quantidade maior então os caracteres que não entrarem no vetor/string ficarão como lixo de memória. Teste o código abaixo para ver:

char nome[5], n[5];
fgets(nome, 5, stdin);
fgets(n, 5, stdin);
printf("%s \n", nome);
printf("%s \n", n);

No código acima digite nomes com menos de 5 caracteres e observe a saída, depois digite um nome com 6 ou mais caracteres e perceba que o lixo que ficará será atribuído ao segundo fgets.

A primeira solução para resolver isso é usar a função de limpar_buffer() que eu recomendei, mas com isso teremos um problema. O problema é que a função lmipar_buffer() irá abrir a entrada do teclado caso o buffer estiver limpo e isso iria causa um "pause" no programa, e isso não é o que queremos. Pelo fato de não ser possível prever se o fgets deixará lixo no buffer então não podemos usar o limar_buffer junto com o fgets.

Uma solução é criar a própria função para ler uma string:

void ler_string(char *nome, int tamanho)
{
    char letra, i = 0;
    do
    {
        letra = getchar();
        if(letra != '\n' && i < tamanho - 1)
        {
            nome[i] = letra;
            i++;
        }
    }while(letra != '\n');
    nome[i] = '\0';
}

A função acima pode ser compactada, mas escrevi assim para ficar mais fácil o entendimento dela. Essa função nunca deixará lixo no buffer independente do que o usuário digitar.

Problema em usar o getchar para pausar o programa

Muitos usam o getchar() como uma alternativa para o system("pause"), é uma ótima alternativa caso queira dar um pause para evitar o fechamento da janela quando o programa acaba, mas se usado no meio do programa o getchar() pode deixar um lixo no buffer e trazer um comportamento não esperado. Exemplo:

printf("Pressione enter para continuar... \n");
getchar();
printf("Digite um caractere: ");
scanf("%c", &n);

No programa acima se o usuário digitar enter então tudo bem, mas caso digite algo como aofhdskajfe aperte enter então o a será "excluído" e o resto ficará como lixo no buffer, isto é, o scanf iria ler o "o" e isso não é o que queremos. fora que o lixo pode ser lido em uma string sem a gente esperar. Uma solução simples para isso é usar a função limpar_buffer() para pausar a tela, ela sempre pausa a tela quando não há lixo no buffer (por isso não se dá bem com o fgets como mencionei). O código acima ficaria assim:

printf("Pressione enter para continuar... \n");
limpar_buffer();
printf("Digite um caractere: ");
scanf("%c", &n);

Esses são os principais problemas que encontrei no seu código, pode haver mais, não sei. Faça as alterações e veja. Eu não testei com as alterações que mencionei pois usei muito tempo analisando seu código e escrevendo essa resposta e por isso estou meio cansado. Mas espero ter ajudado em algo. Caso ainda dê algum problema então avise.