cambiar valor de return

[WinderJerter]

Hola que tal, estoy haciendo una clase para registrar empleados y para iniciar sesión, y lo que quiero hacer es que si el nombre de usuario y contraseña con correctos me muestre varias opciones, la verdad no se como hacerlo bien pero yo lo estoy intentado de esta manera pero no me funciona, les dejo el código abajo haber si me pueden ayudar a resolver mi problema.

Saludos y gracias de antemano : adios:

Código C++:

#include <iostream>
#include <fstream>
#include <string.h>
 
using namespace std;
 
class Empleado {
    public:
        void registrar();
        void iniciar_sesion( char *, char * );
        int estado() { return 0; }
        
    private:
        char usuario[ 22 ], contrasena[ 22 ];
} e;
 
void Empleado::registrar() {
    int bandera = 0;
    char comprobar_usuario[ 22 ];
    
    ifstream leer_empleados( "empleados.txt", ios::in | ios::app );
    
    if( leer_empleados.fail() ) {
        cout << "\nError al leer el archivo empleados.txt" << endl;
    } else {
        cout << "Usuario: "; cin >> comprobar_usuario;
        
        while( !leer_empleados.eof() ) {
            leer_empleados.read( (char*)&e, sizeof( e ) );
            
            if(leer_empleados.eof()) {
                break;
            }
            
            if( strcmp( usuario, comprobar_usuario ) == 0 ) {
                bandera = 1;
            }
        }
        leer_empleados.close();
        
        if( bandera == 1 ) {
            cout << "\nEl nombre de usuario ya esta en uso" << endl;
        } else {
            ofstream escribir_empleados( "empleados.txt", ios::binary | ios::app );
            
            strcpy( usuario, comprobar_usuario );
            cout << "Contraseña: "; cin >> contrasena;
            
            escribir_empleados.write( (char *) &e, sizeof( e ) );
            escribir_empleados.close();
        }
    }
}
 
void Empleado::iniciar_sesion( char comprobar_usuario[ 22 ], char comprobar_contrasena[ 22 ] ) {
    int bandera = 0, opcion;
    
    ifstream leer_empleados( "empleados.txt", ios::in );
    
    if( leer_empleados.fail() ) {
        cout << "\nError al leer el archivo empleados.txt" << endl;
    } else {
        while( !leer_empleados.eof() ) {
            leer_empleados.read( (char*)&e, sizeof( e ) );
            
            if(leer_empleados.eof()) {
                break;
            }
            
            if( strcmp( usuario, comprobar_usuario ) == 0 ) {
                if( strcmp( contrasena, comprobar_contrasena ) == 0 ) {
                    //CAMBIAR VALOR DEL RETURN A 1
 
                }
            }
        }
        leer_empleados.close();
    }
}
 
 
main() {
    char comprobar_usuario[ 22 ], comprobar_contrasena[ 22 ];
    
    cout << "NOMBRE DE USUARIO: "; cin >> comprobar_usuario;
    cout << "CONTRASEÑA: "; cin >> comprobar_contrasena;
    
    e.iniciar_sesion( comprobar_usuario, comprobar_contrasena );
 
 
    //Si el valor de return es 1 miestra opciones
    if( e.estado() == 1 ) {
        cout << "\nSe ha iniciado sesion correctamente" << endl;
                cout << "\nOpcion 1" << endl;
                cout << "Opcion 2" << endl;
 
    } else {
        cout << "\nError al iniciar sesion" << endl;
    }
    
    return 0;
}
 

[vosk]

Puedes usar otro flag de estado como el que tienes en 'registrar()', pero este será privado y ademas será el valor de retorno de la funcion 'estado()' (que puedes modificar el tipo para que retorne bool en vez de entero)

En la clase añades el flag privado:

Código C++:

bool flag_session_status;
 

Seguramente se iniciará el flag a 0, pero si quieres asegurarte añades el constructor de la clase y reinicias manualmente a 0:

Código C++:

Empleado::Empleado() {
    flag_session_status = 0;
}
 

Luego en el metodo 'iniciar_sesion()' si se consiguió iniciar la sesion asignas 1 al flag de estado, o en caso de error restauras a 0; y lo mismo para finalizar la sesion.

Finalmente modificas el metodo estado para que te retorne el flag de estado que será un indicador de la sesion:

Código C++:

bool Empleado::estado() {
    return flag_session_status;
}
 

Y para que quede redondo puedes añadir un entero a modo de ultimo error, de forma que si durante el inicio de sesion falla el nombre de usuario asignas (por ejemplo) 1 al ultimo error, si falla la contraseña asignas 2, si falla el acceso a la base de datos asignas 3; de esta forma ante cualquier error reseteas el flag de estado de la sesion a 0 y determinas un numero de error, luego en el main si el metodo 'estado()' te retorna 0 (es decir si no pudo iniciar la sesion) llamas al metodo 'get_last_error()' y recibes el numero de error, de forma que sabes lo que falló; junto con esto declaras una lista de errores posibles y una lista de descripciones textuales del error y asi compactas el manejo de errores:

Código C++:

enum SESSION_ERROR_INDEX {
    ERROR_SUCCESS,
    ERROR_MISSINGNAME,
    ERROR_WRONGNAME,
    ERROR_COUNT
};
 
static char *SESSION_ERROR_MESSAGES[ERROR_COUNT] = {
    "No error",
    "Missing name",
    "User name does not exist"
};
 
bool Empleado::iniciar_sesion(char *name) {
 
    //comprueba que recibe un nombre
    if(!name || strlen(name) <= 0) {
        flag_session_status = 0;
        last_error = ERROR_MISSINGNAME;
        return flag_session_status;
    }
    
    //evalua el nombre contra la base de datos
    if(strcmp(name, "qwe")) {
        flag_session_status = 0;
        last_error = ERROR_WRONGNAME;
        return flag_session_status;
    }
    
    //inicio correcto
    flag_session_status = 1;
    last_error = ERROR_SUCCESS;
    
    return flag_session_status;
}
 
 
bool Empleado::estado() {
    return flag_session_status;
}
 
 
int Empleado::get_last_error() {
    return last_error;
}
 
 
char *Empleado::get_last_error_message() {
    return SESSION_ERROR_MESSAGES[last_error];
}
 

Ahora el estado del inicio lo puedes comprobar directamente desde el metodo 'iniciar_sesion()' , y puedes dejar el metodo 'estado()' para siguientes comprovaciones:

Código C++:

if(e.iniciar_sesion("qwe")) {
    printf("%s", get_last_error_message());
}
 

*Nota: normalmente se usa el 0 como ejecucion sin error y >0 como indice de error.

Es solo una idea de implementacion para la clase, hay mas formas de hacerlo.

Saludos
vosk

[WinderJerter]

Hola que tal, gracias por responder. La verdad no tengo mucha experiencia en c y con lo que te entendí modifique así mi código, pero siempre me sale que inicie sesión correctamente. Si me puedieras ayudar a corregir esto te lo agradecería demasiado. Saludos

Código C++:

#include <iostream>
#include <fstream>
#include <string.h>
 
using namespace std;
 
class Empleado {
    public:
        Empleado();
        void registrar();
        void iniciar_sesion( char *, char * );
        bool estado();
        
    private:
        bool flag_session_status;
        char usuario[ 22 ], contrasena[ 22 ];
} e;
 
Empleado::Empleado() {
    flag_session_status = 0;
}
 
bool Empleado::estado() {
    return flag_session_status;
}
 
void Empleado::registrar() {
    int bandera = 0;
    char comprobar_usuario[ 22 ];
    
    ifstream leer_empleados( "empleados.txt", ios::in | ios::app );
    
    if( leer_empleados.fail() ) {
        cout << "\nError al leer el archivo empleados.txt" << endl;
    } else {
        cout << "Usuario: "; cin >> comprobar_usuario;
        
        while( !leer_empleados.eof() ) {
            leer_empleados.read( (char*)&e, sizeof( e ) );
            
            if(leer_empleados.eof()) {
                break;
            }
            
            if( strcmp( usuario, comprobar_usuario ) == 0 ) {
                bandera = 1;
            }
        }
        leer_empleados.close();
        
        if( bandera == 1 ) {
            cout << "\nEl nombre de usuario ya esta en uso" << endl;
        } else {
            ofstream escribir_empleados( "empleados.txt", ios::binary | ios::app );
            
            strcpy( usuario, comprobar_usuario );
            cout << "Contraseña: "; cin >> contrasena;
            
            escribir_empleados.write( (char *) &e, sizeof( e ) );
            escribir_empleados.close();
        }
    }
}
 
void Empleado::iniciar_sesion( char comprobar_usuario[ 22 ], char comprobar_contrasena[ 22 ] ) {
    int bandera = 0, opcion;
    
    ifstream leer_empleados( "empleados.txt", ios::in );
    
    if( leer_empleados.fail() ) {
        cout << "\nError al leer el archivo empleados.txt" << endl;
    } else {
        while( !leer_empleados.eof() ) {
            leer_empleados.read( (char*)&e, sizeof( e ) );
            
            if(leer_empleados.eof()) {
                break;
            }
            
            if( strcmp( usuario, comprobar_usuario ) == 0 ) {
                if( strcmp( contrasena, comprobar_contrasena ) == 0 ) {
                    
                    flag_session_status = 1;
                    
                }
            }
        }
        leer_empleados.close();
    }
}
 
 
main() {
    
    char comprobar_usuario[ 22 ], comprobar_contrasena[ 22 ];
    
    cout << "NOMBRE DE USUARIO: "; cin >> comprobar_usuario;
    cout << "CONTRASEÑA: "; cin >> comprobar_contrasena;
    
    e.iniciar_sesion( comprobar_usuario, comprobar_contrasena );
    
    if( e.estado() == 1 ) {
        cout << "\nSe ha iniciado sesion correctamente" << endl;
    } else {
        cout << "\nError al iniciar sesion" << endl;
        
    }
    
    return 0;
}