C Repetição da última linha do printf

Ezequiel André · 13 de janeiro de 2018

Boa noite pessoal. Estou engatinhando em um pequeno projeto em C e não estou com o código fonte todo pronto, estou criando em partes. E já na primeira parte, a que deveria mostrar a lista de itens de um arquivo, a última linha fica repetindo. Estou usando o "fscanf" para ler do arquivo. Se alguém puder me ajudar a resolver, ou me explicar onde estou errando, agradeço. Segue o código fonte, lembrando que as outras funções eu ainda não implantei.Segue também imagem das telas de execução, só para pré-visualização

#include<stdio.h>
#include<stdlib.h>
#include<stdbool.h>
#include<windows.h>
void gotoxy(int x, int y){
  COORD coord;
  coord.X = x;
  coord.Y = y;
  SetConsoleCursorPosition(GetStdHandle(STD_OUTPUT_HANDLE), coord);
}
typedef struct{
	int cod;
	char nome[15];
	float quant;
	float valor;
	float v_total;
	float v_venda;
	float lucro;
}Dados;
int head(){
	for(int i = 0 ; i < 6 ; i++){
		for(int j = 0 ; j < 50 ; j++){
			if(i==0){
				if(j==0){
					printf("%c",201);
				}
				if(j==49){
					printf("%c\n",187);
				}
				else{
					printf("%c",205);
				}
			}
			if(i==5){
				if(j==0){
					printf("%c",200);
				}
				if(j==49){
					printf("%c\n",188);
				}
				else{
					printf("%c",205);
				}
			}
						
		}if(i>1&&i<5){
				printf("%c%50c\n",186,186);
			}
	}
	gotoxy(2,2);
	printf("\tSISTEMA DE GERENCIAMENTO DE ESTOQUE\n\n\n");
	
}
int body(){
	for(int i = 0 ; i < 17 ; i++){
		for(int j = 0 ; j < 50 ; j++){
			if(i==0){
				if(j==0){
					printf("%c",201);
				}
				if(j==49){
					printf("%c\n",187);
				}
				else{
					printf("%c",205);
				}
			}
			if(i==16){
				if(j==0){
					printf("%c",200);
				}
				if(j==49){
					printf("%c\n",188);
				}
				else{
					printf("%c",205);
				}
			}
						
		}if(i>1&&i<16){
				printf("%c%50c\n",186,186);
			}
	}
}
int menu(int *op){
	head();
	body();
	gotoxy(2,6);
	printf("Digite a opcao desejada:");
	int j=1;
	printf("\n\n");
	
	for(int i = 9 ; i < 16 ; i++){
		
		if(i==9){
			gotoxy(2,i);
			printf("%d- Mostrar relatorio do estoque",j);
			j++;
		}
		if(i==10){
			gotoxy(2,i);
			printf("%d- Salvar relatorio do estoque",j);
			j++;
		}
		if(i==11){
			gotoxy(2,i);
			printf("%d- Inserir item no estoque",j);
			j++;
		}
		if(i==12){
			gotoxy(2,i);
			printf("%d- Localizar item no estoque",j);
			j++;
		}
		if(i==13){
			gotoxy(2,i);
			printf("%d- Excluir item do estoque",j);
			j++;
		}
		if(i==14){
			gotoxy(2,i);
			printf("%d- Ajuda",j);
			j++;
		}
		if(i==15){
			gotoxy(2,i);
			printf("%d- Sair",j);
			j++;
		}	
	}
	gotoxy(27,6);
	scanf("%d",&*op);
	system("cls");
	//return *op;
}
int main(){
	Dados dados;
	FILE *txt;
	int opcao;
	
	do{
		menu(&opcao);
		system("cls");
		switch(opcao){
			case 1:{
				head();
				body();
				
				if((txt=fopen("estoque.txt","r"))==NULL){
					printf("Não foi possivel aabrir o arquivo");
				}
				else{
					int i=8;
					gotoxy(1,7);
					printf("COD  NOME     QUANT VALOR VALOR_V VALOR_T  LUCRO");
					while(!feof(txt)){
						fscanf(txt," %d %s %f %f %f %f %f",&dados.cod,&dados.nome,&dados.quant,&dados.valor,&dados.v_venda,&dados.v_total,&dados.lucro);
						++i;
						gotoxy(2,i);
						printf("%d  %s\t%.2f%5.2f%6.2f%9.2f%8.2f",dados.cod,dados.nome,dados.quant,dados.valor,dados.v_venda,dados.v_total,dados.lucro);
					}
				}
					fclose(txt);
					gotoxy(2,6);
					printf("Para sair digite 7:");
					gotoxy(22,6);
					scanf("%d",&opcao);
				
			}
			case 2:{
				printf("tchau");
				break;
			}
		}
	}while(opcao!=7);
	
	gotoxy(1,20);
	return 0;
}

vangodp · 13 de janeiro de 2018

O problema é a causa do mal uso de feof devido a um falho de lógica. Muito comumente programadores, na sua amplia maioria novatos, fazem um mal uso do feof, achando que ele tem o comportamento de "averiguar se existe mais dados por ler", quando deveria saber, que o verdadeiro comportamento do feof é: "Que aconteceu com a ultima leitura?".
Suponhamos que seu programa leu todas as linhas do arquivo menos a ultima.

fscanf faz a leitura da ultima linha e não teve problemas, porém já não há mais dados que ler.
i incrementou.
gotoxy foi para (2, i).
printf imprimiu os dados corretamente.
(ponto critico!) feof retorna false pois teve uma leitura exitosa, não encontrou o fim do arquivo.
(ERRO)Repete o laço uma vez mais, pois como falei, feof retornou false, porém agora o fscanf falha, pois não há mais dados que ler pois agora sim foi atingido o EOF que marca o fim do arquivo.
Continuam os erros, i incrementa, e não deveria pois não foram lidos novos dados.
gotoxy foi para (2, i), lembre que o i já não é o mesmo, pois incrementou em 1 e mudou a posição.
printf imprime os mesmos dados que da anterior vez pois desta ultima não foram gravados novos dados, porém os dados antigos continuam na estrutura, e por isso se repete.

Pense nisso.... nesse seu while você pergunta por feof(Atingiu o fim do arquivo?), logo no inicio, antes de ler nada. Como o programa vai saber si atingiu o fim do arquivo se você ainda não leu nada? Entendeu? O feof averigua se a ultima leitura foi exitosa, e não se a seguinte leitura vai ser exitosa, esse é o problema.

Como arrumar isso?
Em vez de usar feof, você pode usar o próprio valor de retorno de fscanf para saber se a leitura foi correta, pois fscanf retorna os seguintes valores conforme os seguintes casos:

Conforme o valor de retorn de fscanf descrito em http://www.cplusplus.com/reference/cstdio/feof/

Citação

Com sucesso, a função retorna o número de itens da lista de argumentos preenchidos com sucesso. Essa contagem pode coincidir com o número esperado de itens ou ser menor (inclusive zero) devido a uma falha correspondente, um erro de leitura ou ter atingido o final do arquivo(EOF).

Se um erro de leitura ocorrer ou o final do arquivo for atingido durante a leitura, o indicador apropriado é configurado (feof ou ferror). E, se qualquer um acontece antes que qualquer dado possa ser lido com sucesso, EOF é retornado.

Se um erro de codificação ocorrer interpretando caracteres amplos, a função define errno para EILSEQ.

fscanf, scanf, printf, etc, são funçoes, funções podem retornar valores, e essas em particular retornam, é conveniente você estudar o valor de retorno dessas funções.

para nosso caso bastaria com mudar essa parte:

while(!feof(txt)){
	fscanf(txt," %d %s %f %f %f %f %f",&dados.cod,&dados.nome,&dados.quant,&dados.valor,&dados.v_venda,&dados.v_total,&dados.lucro);
	++i;
	gotoxy(2,i);
	printf("%d  %s\t%.2f%5.2f%6.2f%9.2f%8.2f",dados.cod,dados.nome,dados.quant,dados.valor,dados.v_venda,dados.v_total,dados.lucro);printf
}

Por essa outra:

while(	fscanf(txt," %d %s %f %f %f %f %f",&dados.cod,&dados.nome,&dados.quant,&dados.valor,&dados.v_venda,&dados.v_total,&dados.lucro) != EOF ){
	++i;
	gotoxy(2,i);
	printf("%d  %s\t%.2f%5.2f%6.2f%9.2f%8.2f",dados.cod,dados.nome,dados.quant,dados.valor,dados.v_venda,dados.v_total,dados.lucro);printf
}

Com isso a leitura deveria ser feita de forma correta.

Deixo avisado que não provei seu código, ignoro se tem mais erros, me limito a lhe explicar somente o tema da consulta. Caso tiver mais erros exponha em um novo tema, ou incluso neste mesmo post. Faça a prova e me conte se funcionou.

Ezequiel André · 13 de janeiro de 2018

Funcionou perfeitamente. Eu já havia tentado antes colocar o fscanf dentro do whille , mas da seguinte forma:

"while((fscanf(txt,"%d %s%f............., &dados.cod,&.......)!=NULL), mas também não funcionou.

Agora está tudo certo, Muito obrigado e espero que este post ajude a quem estiver com esta duvida também, por que pesquisei em vários lugares e não achei uma explicação tão boa.

Muito obrigado

vangodp · 13 de janeiro de 2018

De nada.
Lembre que funções retornam valores, e que se aprender sobre esses valores, você será um melhor programador. Na pagina cplusplus.com, por exemplo(existem outras como cppreference e etc), é uma pagina de referencia e explica o funcionamento de cada função.

É importante destacar que uma função é algo parecido a isso aqui:

Neste caso a função é a maquina de moer carne. Esta maquina(função) aceita carne em pedaços grandes, ou pequenos, ou sei lá >_<(Dados: ints, chars, structs, etc), e pela saída ela retorna(return) a carne moída. Isso pode ser a analogia de uma função perfeitamente, porém não todas as funções fazem a mesma coisa, nem todas as maquinas por suposto. No seguinte caso você pode ver outro exemplo:

Neste claro exemplo a "função" aceita os mesmo tipos de dados, porém internamente trabalha de forma diferente, e por suposto retorna outro tipo de dado, que é a lingüiça neste caso, e não carne moída como no caso anterior.

O que eu quero lhe ressaltar, é que você não precisa saber como a função trabalha por dentro, para poder usa-la de forma efetiva somente se precisa saber 3 coisas.

Que tipo de dados aceita.
O que é o que faz(para que serve).
E o que é o que retorna(Em caso de retornar algo claro.)

Com saber esses 3 dados anteriores sobre qualquer função, você será capaz de usa-la de forma efetiva, e é justamente isso para que servem as paginas de referencia, como cplusplus.com por exemplo, entre outras coisas. Essas paginas de referencia lhe ajudam a saber essas 3 coisas sobre qualquer função da biblioteca padrão, e mais.