Má execução deste code, help!!

1. Má execução deste code, help!!

theThux
(usa Debian)

Enviado em 05/01/2018 - 12:27h

Boa tarde galera, criei esse code para adicionar produtos em um .txt
o problema é que na execução "trava " por algum motivo, não consigo ver qual é
alguém poderia me ajudar?

typedef struct st_produto{
   int codigo;
   string descricao;
   float preco;
} produto;

bool existe(int codigo){ // funcao bool para verificar se o codigo digitado ja existe no banco de dados
	int codigo_lido;
	FILE *arquivo; // ponteiro do arquivo
	arquivo = fopen("/sdcard/produtos.txt", "r");
	while(!feof(arquivo)){ 
		fscanf(arquivo, "%i\n", &codigo_lido);
		if(codigo_lido == codigo){
			fclose(arquivo);
    		return false;     
		}else{
			fclose(arquivo);
			return true;
		}
	}
	fclose(arquivo);
    return false;
} // existe

int cadastrar_produto() // função de cadastro de produtos no banco de dados
{
	ofstream arquivo; // cria um objeto de fluxo de output
	arquivo.open ("produtos.txt", ofstream::app); // abre o arquivo para testar se ele pode ser aberto
	if (arquivo.is_open()){ // "se o arquivo estiver aberto, o codigo continua"
		int i = 1;
		arquivo.close(); // fecha o arquivo, pois inicialmente fora feito apenas um teste
		while(i != 0){
			
			arquivo.open ("produtos.txt", ofstream::app); // abre o arquivo para gravar dados nele
			system("cls");
			cout << "\t\tBem vindo ao sistema de cadastro de produtos da papelaria RCGA\n\n";
			produto novo;
			
			/**
				No trecho de codigo abaixo grava-se os dado na estrutura
			**/
			
  		    cout << "Codigo: ";
 		    scanf("%i",&novo.codigo);
 		    cin.ignore();

 	        cout << "Descrição: ";
			getline(cin, novo.descricao);
            cout << "Preço: ";
 		    scanf("%f",&novo.preco);  
 		   //cout << "\nCódigo: " << novo.codigo << "\nDescrição: " << novo.descricao << "\nPreço: " << novo.preco << endl;
 		   if(existe(novo.codigo)){ // if para verificar a partir da funcao, se o codigo digitado existe ou nao
        		arquivo << novo.codigo << " " << novo.descricao << " " << novo.preco << "\n"; // envia os dados agrupados ao arquivo
            	arquivo.close(); // fecha o arquivo
        		cout << "\n\t\tProduto cadastrado!\n\nPara sair pressione [0] Continuar [1]" << endl;
   	    	 cin >> i; // variável que controla o loop
		    }else{
        		cout << "\n\t\tO codigo deste produto ja existe!\n\n" << endl;
        		return 0;
      	  } //if else
      
		}//while
	}// if
	return 0;
} // cadastrar_produto();

0 0

Quote

2. Re: Má execução deste code, help!!

removido
(usa Nenhuma)

Enviado em 05/01/2018 - 12:46h

Olá, eu vou mostrar os pontos que achei "suspeitos" do seu codigo:


while(i != 0){
_c_compiler: Objeto While, não e possivel quebrar.

system("cls");
_c_compiler: Objeto "cls" in system.bash não está disponivel, use: clear

scanf (cout << ) objects:
_c_compiler: Objeto: scanf is a c lang library, and not a c++

while(!feof(arquivo)){ 
_c_compiler: Objeto While, não e possivel quebrar.

(com.c-compiler.process[1]): Missing a class (init) in file.
(com.c-compiler.process[2]): Missing a #include files to read.

Erros: While inquebraveis, Include libs não existe, Classe init não existe. Funções do linux que não existe.

Tente fazer os whiles quebraveis, pois eles ficaram repetindo, a classe init não existe. O Cls não existe no linux.

Frases que fazem sentido.
Flutuações Quânticas podem Existir, assim como o Decaimento do Vácuo.
O Futuro não importa, o que importa e da para frente. HAHAHA

0 0

Quote

3. Re: Má execução deste code, help!!

theThux
(usa Debian)

Enviado em 05/01/2018 - 13:08h

AndrewUser escreveu:

Olá, eu vou mostrar os pontos que achei "suspeitos" do seu codigo:


while(i != 0){
_c_compiler: Objeto While, não e possivel quebrar.

system("cls");
_c_compiler: Objeto "cls" in system.bash não está disponivel, use: clear

scanf (cout << ) objects:
_c_compiler: Objeto: scanf is a c lang library, and not a c++

while(!feof(arquivo)){ 
_c_compiler: Objeto While, não e possivel quebrar.

(com.c-compiler.process[1]): Missing a class (init) in file.
(com.c-compiler.process[2]): Missing a #include files to read.

Erros: While inquebraveis, Include libs não existe, Classe init não existe. Funções do linux que não existe.

Tente fazer os whiles quebraveis, pois eles ficaram repetindo, a classe init não existe. O Cls não existe no linux.

Frases que fazem sentido.
Flutuações Quânticas podem Existir, assim como o Decaimento do Vácuo.
O Futuro não importa, o que importa e da para frente. HAHAHA

Sim, eu sei que não existe, porém estou no Windows. até gostaria de usar o sleep do Linux mas ...

então, como assim quebraveis? sempre com um termino?

não faço ideia do que estou errando...-

0 0

Quote

4. Re: Má execução deste code, help!!

removido
(usa Nenhuma)

Enviado em 05/01/2018 - 14:29h

theThux escreveu:

AndrewUser escreveu:

Olá, eu vou mostrar os pontos que achei "suspeitos" do seu codigo:


while(i != 0){
_c_compiler: Objeto While, não e possivel quebrar.

system("cls");
_c_compiler: Objeto "cls" in system.bash não está disponivel, use: clear

scanf (cout << ) objects:
_c_compiler: Objeto: scanf is a c lang library, and not a c++

while(!feof(arquivo)){ 
_c_compiler: Objeto While, não e possivel quebrar.

(com.c-compiler.process[1]): Missing a class (init) in file.
(com.c-compiler.process[2]): Missing a #include files to read.

Erros: While inquebraveis, Include libs não existe, Classe init não existe. Funções do linux que não existe.

Tente fazer os whiles quebraveis, pois eles ficaram repetindo, a classe init não existe. O Cls não existe no linux.

Frases que fazem sentido.
Flutuações Quânticas podem Existir, assim como o Decaimento do Vácuo.
O Futuro não importa, o que importa e da para frente. HAHAHA

Sim, eu sei que não existe, porém estou no Windows. até gostaria de usar o sleep do Linux mas ...

então, como assim quebraveis? sempre com um termino?

não faço ideia do que estou errando...-

Tipo assim:


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

using namespace std;

int main() {
     int main2 = 1;
     string instr;
     while(main2<1000) {
          cout << "Bem Vindo ao Shell de comandos. Coloque Sim ou Sair" << endl;
          cout << "Comando: ";
          cin >> instr;
          if(instr=="sair") {
                break;
          }
          /* Code Here */
       }
}

E preciso sempre fechar o while. Além disso, pode mostrar todo o codigo, com a função init?

Frases que fazem sentido.
Flutuações Quânticas podem Existir, assim como o Decaimento do Vácuo.
O Futuro não importa, o que importa e da para frente. HAHAHA

0 0

Quote

5. Re: Má execução deste code, help!!

paulo1205
(usa Ubuntu)

Enviado em 08/01/2018 - 00:37h

theThux escreveu:

Boa tarde galera, criei esse code para adicionar produtos em um .txt
o problema é que na execução "trava " por algum motivo, não consigo ver qual é
alguém poderia me ajudar?

typedef struct st_produto{
   int codigo;
   string descricao;
   float preco;
} produto;

bool existe(int codigo){ // funcao bool para verificar se o codigo digitado ja existe no banco de dados
	int codigo_lido;
	FILE *arquivo; // ponteiro do arquivo
	arquivo = fopen("/sdcard/produtos.txt", "r");
	while(!feof(arquivo)){ 
		fscanf(arquivo, "%i\n", &codigo_lido);
		if(codigo_lido == codigo){
			fclose(arquivo);
    		return false;     
		}else{
			fclose(arquivo);
			return true;
		}

Em ambos os casos, você sai do laço de repetição, sem efetivamente fazer repetição alguma. Isso tem toda a cara de não estar certo.

Adicionalmente, parece-me que o nome da função está equivocado, pois ela retorna verdadeiro (true) quando o registro não existe, e falso (false) quando ele existe.

	}
	fclose(arquivo);
    return false;
} // existe

int cadastrar_produto() // função de cadastro de produtos no banco de dados
{
	ofstream arquivo; // cria um objeto de fluxo de output
	arquivo.open ("produtos.txt", ofstream::app); // abre o arquivo para testar se ele pode ser aberto
	if (arquivo.is_open()){ // "se o arquivo estiver aberto, o codigo continua"

Existem vários meios de testar se a abertura do arquivo funcionou, mas eu acho que esse que você escolheu não é um dos melhores, nem um dos mais comuns.

Além disso, testar se pode ser aberto, para logo em seguida o fechar, e quase imediatamente abri-lo novamente acaba sendo desperdício. Acho que você poderia repensar essa decisão.

Além disso, o nome do arquivo aqui não é exatamente o mesmo usado na função existe(). É isso mesmo?

		int i = 1;

Essa variável não passa de um indicador para garantir que o laço de repetição com while será executado pelo menos uma vez. Isso indica que você deveria, então, usar do/while em lugar de while, e a variável de controle (que poderia ser do tipo bool) poderia ser definida apenas dentro do laço, não antes dele.

		arquivo.close(); // fecha o arquivo, pois inicialmente fora feito apenas um teste
		while(i != 0){
			
			arquivo.open ("produtos.txt", ofstream::app); // abre o arquivo para gravar dados nele
			system("cls");

Por causa da imprevisibilidade (o compilador não tem como identificar se o que está dentro das aspas está correto ou não), não-portabilidade (ficar amarrado a um sistema específico, e até a uma configuração particular dentro desse sistema) e do custo envolvido (criar processo, executar nesse processo novo um ou mais comandos externos, tratar a terminação desses comandos e cuidar de encerrar os processos), eu geralmente não gosto quando se chama um programa externo, ainda mais quando é apenas para limpar a tela, e especialmente quando isso acontece dentro de um laço de repetição.

Recomendo fortemente que você use uma solução mais elegante. Uma delas, acredite, é simplesmente não limpar a tela.

			cout << "\t\tBem vindo ao sistema de cadastro de produtos da papelaria RCGA\n\n";
			produto novo;
			
			/**
				No trecho de codigo abaixo grava-se os dado na estrutura
			**/
			
  		    cout << "Codigo: ";
 		    scanf("%i",&novo.codigo);
 		    cin.ignore();

Essa mistura de operações em std::cin e stdin geralmente pode e deve ser ser evitada, até porque é teoricamente possível que eles estejam dissociados ou não-sincronizados.

Além disso, conviria que você verificasse se as operações de entrada estão funcionando a contento. Em lugar disso, você está supondo que todas elas são sempre bem sucedidas.

 	        cout << "Descrição: ";
			getline(cin, novo.descricao);
            cout << "Preço: ";
 		    scanf("%f",&novo.preco);  
 		   //cout << "\nCódigo: " << novo.codigo << "\nDescrição: " << novo.descricao << "\nPreço: " << novo.preco << endl;
 		   if(existe(novo.codigo)){ // if para verificar a partir da funcao, se o codigo digitado existe ou nao

Como vimos acima, a função existe() tem o problema de não verificar nada além do primeiro registro. Cuidado com isso.

        		arquivo << novo.codigo << " " << novo.descricao << " " << novo.preco << "\n"; // envia os dados agrupados ao arquivo
            	arquivo.close(); // fecha o arquivo
        		cout << "\n\t\tProduto cadastrado!\n\nPara sair pressione [0] Continuar [1]" << endl;
   	    	 cin >> i; // variável que controla o loop
		    }else{
        		cout << "\n\t\tO codigo deste produto ja existe!\n\n" << endl;
        		return 0;
      	  } //if else
      
		}//while
	}// if
	return 0;
} // cadastrar_produto();