una entrar unos datoss y comparar con txt..

[snitch]

hola escribo haber si me pueden solucionar lo siguiente ya que soy muy novato en programación

Seguramente estarán hartos de novatos que piden el mismo programa. Estoy intentando hacer un programa en el que hay dos opciones una entrar y la otra registro ( cmo un login). En la opcion de registro pide el user y el pass y lo guarda en un txt. Hasta aqui funciona.

Lo que quiero hacer ahora es que en el menu de entrar yo pueda introducir el user y el pass y compare lo que he entrado con lo guardado en el text y en el caso que coincidan validar.

les dejo el código de lo q seria el menu y de la aplicacion del login ( lo he hecho en dos programas)

utilizo dev c++ y c++

menu

#include <iostream>
#include <fstream>


using namespace std;


int main(){
system("color 1E");
menu:
cout<<"1-Entrar"<<endl;
cout<<"2-Registrarse"<<endl;
int opt;
cin>>opt;
system("cls");


switch(opt){

case 1:{ system("jk");}

case 2:{

char usuario[10];
char pass[10];
registro:
ofstream fs("reg.txt", ios::app);
cout<<"Introduce nombre de usuario:";
cin>>usuario;
cout<<endl;
cout<<"Introduce el pass:";
cin>>pass;
fs<<usuario<<"\n";
fs<<pass;
fs.close();

ifstream fe("reg.txt");

system("cls");
cout<<"Usuario:"<<usuario<<endl;
fe.getline(usuario,10);
cout<<"\n";
cout<<"Pass:"<<pass<<endl;
fe.getline(pass,10);
cout<<"\n";
cout<<"Datos correctos?s/n"<<endl;
string resp;
cin>>resp;
if(resp=="s"){system("cls"); goto menu;}
if(resp=="n") {goto registro;}
else{exit(0);}
system("cls");


}
}
}






jk (programa de login)

#include <iostream>
#include<fstream>
using namespace std;

int main()
{

login:
char usuario[10],pass[10] ;
cout<<"User:";
cin.getline(usuario,10);
cout<<"\n";
cout<<"Pass:";
cin.getline(pass,10);

cout<<"Comprovando"<<endl;

ifstream fe("reg.txt");
fe.getline(usuario,10);
fe.getline(pass,10);
system("cls");

if(cin.getline(usuario,10)==fe.getline(usuario,10) and cin.getline(pass,10)==fe.getline(pass,10)){
cout<<"Login correcto"<<usuario<<endl;
system("toperativ");}
else{cout<<"Login incorrecto"<<endl;
system("pause");
system("cls");
goto login;}

}



el codigo de registar funciona, lo que no consigo lograr es lo de entar los datos en el login y que los compare con el txt y si son correctos valide


muchas gracias
PD: perdón por tal idea de nob

[CapacProg]

Código C++:

//jk (programa de login)
 
#include <iostream>
#include<fstream>
using namespace std;
 
int main()
{
 
login:
char usuario[10],pass[10] ;
cout<<"User:";
cin.getline(usuario,10);
cout<<"\n";
cout<<"Pass:";
cin.getline(pass,10);
 
cout<<"Comprovando"<<endl;
 
ifstream fe("reg.txt");
fe.getline(usuario,10);
fe.getline(pass,10);
system("cls");
 
if(cin.getline(usuario,10)==fe.getline(usuario,10) and cin.getline(pass,10)==fe.getline(pass,10)){
cout<<"Login correcto"<<usuario<<endl;
system("toperativ");}
else{cout<<"Login incorrecto"<<endl;
system("pause");
system("cls");
goto login;}
 

En este código:

Primero: Veo que no usas la librería "string.h" para manejar las cadenas, como en la línea 25 para hacer la comparacion con strcmp o con strncmp, ya que en la forma que lo haces no va a funcionar por no ser el método de comparar cadenas, eso serviría en números o en caracteres, pero no en cadenas.

Segundo: Las variables que usas para el ingreso de datos del usuario y pass los sobreescribes con lo que lees del archivo, así que no estas haciendo nada, coloca cada dato en variables diferentes.

Tercero: No estas recorriendo todo tu archivo buscando al usuario y pass correspondiente, ya que en el modo que abres tu archivo se supone que vas a tener más de un usuario (ios::app) y tu sólo lees la primera y segunda linea.

Espero te sirva.

[snitch]

gracias, intentaré ir paso x paso haber si corrijo, porqe no se x dónde empezar

[sam90]

Primero que todo ya que estas usando C++ usaria string y no cadenas de caracteres. Los strings son mas faciles de usar.

string user, pass;
string user2,pass;

Luego, si usas cin.getline siempre vas a leer del teclado nunca del archiv oqeu abiste.

getline(fe,user2);
getline(fe,pass2);

Luego para comprar string simplemente haces:

if (user == user2 && pass == pass2)

Lo que si como dijo el compañero capacpro si solo haces un getline obtendras los datos de un solo usuario deberias hacer un bucle para ir leyendo usuario por usuario.

[snitch]

bueno de momento preubo con un solo usuario, luego más adelante cuando vaya probando y viendo cmo va lo cambio

gracias

[snitch]

bien ya lo he cambiado y ahora creo que lo de antes esta correjido( miren a ver si es correcto) con strings pero para un solo usuario( cuando consiga uno ya intentaré el bucle).

Ahora creo que el problema esta en como lee el archivo txt porque si abro el txt normal me aparece desde el inicio del documento primera linea blanco,segunda user y tercera pass y necesito que el programa guarde el user en la primera y el pass en la segunda para que funcione.Lo hice manualmente escribiendo en el txt en primera linea y segunda y funcionó. ( si estoy totalmente equivocado corrijanme)

codigo del menu:

//Menú de programa
#include <iostream>
#include <fstream>


using namespace std;


int main(){
system("color 1E");
menu:
cout<<"1-Entrar"<<endl;
cout<<"2-Registrarse"<<endl;
int opt;
cin>>opt;
system("cls");


switch(opt){

case 1:{ system("jk");}

case 2:{

string usuario;
string pass;
registro:
ofstream fs("reg.txt", ios::app);
cout<<"Introduce nombre de usuario:";
cin>>usuario;
cout<<endl;
cout<<"Introduce el pass:";
cin>>pass;
fs<<usuario<<"\n";
fs<<pass;
fs.close();

ifstream fe("reg.txt");

system("cls");
cout<<"Usuario:"<<usuario<<endl;
getline(fe,usuario);
cout<<"\n";
cout<<"Pass:"<<pass<<endl;
getline(fe,pass);
cout<<"\n";
cout<<"Datos correctos?s/n"<<endl;
string resp;
cin>>resp;
if(resp=="s"){system("cls"); goto menu;}
if(resp=="n") {goto registro;}
else{exit(0);}
system("cls");


}
}
}





codigo del login

#include<fstream>
#include<string>
using namespace std;

int main()
{
system("color 1E");
login:
string user,pass;
string user2,pass2;
cout<<"User:";
cin>>user;
cout<<"\n";
cout<<"Pass:";
cin>>pass;

cout<<"Comprovando"<<endl;

ifstream fe("reg.txt", ios::in);
getline(fe,user2);
getline(fe,pass2);
system("cls");

if(user==user2 && pass==pass2){
cout<<"Login correcto"<<user<<endl;
system("toperativ");}
else{cout<<"Login incorrecto"<<endl;
system("pause");
system("cls");
goto login;}

}

[CapacProg]

Después de

Código C++:

cin>>pass;
 

utiliza

Código C++:

cin.ignore()
 

para descartar el Enter que das al confirmar el ingreso de la cadena, lo mismo deberias usar despues de leer una cadena. Ese puede ser tu error.

[snitch]

el cin.ignore no lo soluciona, pero gracias

Creo que el error( a parte de la organización) esta en que no lee todo el archivo que solo me lee el primer usuario y pass que inserto y los siguientes no.

En el txt las entradas se van guardando(user,pass) así x ejemplo:
juan
lele
pepe
123

si introduzco en user juan y pass lele funciona, pero en los succesivos no. he decidido rehacer el codigo haber si alguien encuentra como arreglarlo


codigo menu
/

Código C++:

/Menu
#include<iostream>
#include<fstream>
#include<string>
 
using namespace std;
struct registro{
       
       string user;
       string pass;
       }persona;
       
struct login{
       string luser;
       string lpass;
       }lpersona;
       
       
using namespace std;
int main(){
    
    system("color 1E");
    menu:
    cout<<"1-Entrar"<<endl;
    cout<<"2-Registrarse"<<endl;
    cout<<"3-Salir"<<endl;
     int a;
     cin>>a;
     system("cls");
     switch(a){
               case 1:{login:
                       cout<<"\tLogin"<<endl;
                       cout<<"\nUser:";
                       cin>>lpersona.luser; 
                       cout<<"\nPass:";
                       cin>>lpersona.lpass; 
                       ifstream fe("db.txt" ,ios::in);
                       getline(fe,persona.user);
                       getline(fe,persona.pass);
                       
                       while(!fe.eof()){
                                        if(lpersona.luser==persona.user && lpersona.lpass==persona.pass){system("cls");
                                                                       cout<<"\nLogin correcto"<<endl;
                                                                       system("pause");
                                                                       system("tactivo");
                                                                       exit(0);
                                                                       }
                                        if(lpersona.luser!=persona.user && lpersona.lpass!=persona.pass){system("cls");
                                                                       cout<<"\nLogin incorrecto"<<endl;
                                                                       system("pause");
                                                                       system("cls");
                                                                       goto login;
                                                                       }
                                        }                             
                       
                                
                       
               
               }
               case 2:{registro:
                       ofstream fs("db.txt", ios::app);
                       cout<<"Introduce el user que quieras:";
                       cin>>persona.user; fs<<persona.user<<"\n";
                       cout<<"\nIntroduce el pas que quieras:";
                       cin>>persona.pass; fs<<persona.pass<<"\n";
                       fs.close();
                       system("cls");
                       ifstream fe("db.txt");
                       getline(fe,persona.user);
                       cout<<"User:"<<persona.user<<endl;
                       getline(fe,persona.pass);
                       cout<<"Pass:"<<persona.pass<<endl;
                       cout<<"\nDatos correctos?s/n"<<endl;
                       string resp;
                       cin>>resp;
                       if(resp=="s"){cout<<"Guardando datos"<<endl;
                       system("cls");
                       goto menu; break;}
                       if(resp=="n"){system("cls"); goto registro; break;}
                       else{return(0);}
                       
 
                    }
               case 3:{exit(0);}
               }
               
}
 

tactivo ( lugar donde dirije una vez logeado correctamente, no tiene errores)

Código C++:

//Terminal activo
#include <iostream>
 
using namespace std;
 
int main(){
    
    system("cls");
    system("color 1E");
    cout<<"\nBienvenido al terminal"<<endl;
    cout<<"\nTerminal en pruebas,proxiamente funcional.."<<endl;
    system("pause");
    exit(0);
    
}
 

[sam90]

Tu problema esta que siempre comparas con el primero ya que solo lees una vez y despues volves a cargar el fichero si falla. Mira esto:

Código C++:

Deberias leer las lineas dentro del While.
 
case 1:{login:
                       cout<<"\tLogin"<<endl;
                       cout<<"\nUser:";
                       cin>>lpersona.luser; 
                       cout<<"\nPass:";
                       cin>>lpersona.lpass; 
                       ifstream fe("db.txt" ,ios::in);
                      
                       
                       while(!fe.eof()){
                                        if(lpersona.luser==persona.user && lpersona.lpass==persona.pass){system("cls");
                                                                       cout<<"\nLogin correcto"<<endl;
                                                                       system("pause");
                                                                       system("tactivo");
                                                                       exit(0);
                                                                       }
                                        getline(fe,persona.user);
                                        getline(fe,persona.pass);
                                        }           
                       cout<<"\nLogin incorrecto"<<endl;
 

[snitch]

perfecto ahora ya funciono muchas gracias!
ahora tengo que cambiar la parte del registro para que cuando confirme los datos muestre los entrados bien.