Editada, pois se o arquivo sendo lido não terminasse com uma quebra de linha ('\n'
), o último caractere seria eliminado. Colocado um teste para verificar se o último caractere da linha lida é '\n'
antes de substituir.
Como o user72726 explicou na resposta dele, a função fgets inclui o avanço de linha ao ler do arquivo ('\n'
). A princípio eu pensei em inserir o ponto-e-vírgula ao ler o arquivo, mas isto limita algum outro tratamento que você pode querer fazer.
Eu sugiro tirar este avanço de linha ao ler do arquivo
while (fgets(linha1,255,file)!=NULL) {
linha *l1 =(linha*) malloc(sizeof(linha));
if (linha1[strlen(linha1) - 1] == '\n') // testa se o último caractere é \n
linha1[strlen(linha1) - 1] = '\0'; // troca o \n pelo char null
l1->produtos =(char*) malloc(strlen(linha1)+1);
strcpy(l1->produtos,linha1);
l1->prev = tail;
tail = l1;
}
e depois adicionar o ponto-e-vírgula e o avanço de linha ao exibir na tela
while (current != NULL) {
printf("- %s;\n",current->produtos);
current = current->prev;
}
Funciona porque as funções que tratam strings param ao encontrar um caractere nulo - veja na ref. da strcpy do link abaixo, no final da primeira frase "including the terminating null character (and stopping at that point)".
Note que se fizer sizeof(linha1)
vai ficar diferente de strlen(linha1)
, o normal, se não me engano, é a diferença ser 1 mas aqui será 2 porque linha1
é um ponteiro que no final tem dois caracteres nulos ao invés de 1, que a strlen não conta.
O Parágrafo acima está incorreto porque sizeof(linha1)
no caso desse programa será sempre 255, pois é o tamanho alocado na declaração.
Uma coisa importante, embora fora do assunto da pergunta, é liberar toda a memória que você usa. Então, se fosse eu, para cada malloc eu colocaria um memfree no programa mesmo que o programa fosse fechar logo depois.
Referências (outras funções aparecem no menu à esquerda):
http://www.cplusplus.com/reference/cstring/strlen/
http://www.cplusplus.com/reference/cstring/strcpy/