Error de compilación, sugerencias?

[Miraii]

Disculpen pero tengo un pequeño problema con un programa que estoy realizando, el caso es crear un programa, vaya la redundancia que capture las matriculas y datos de un grupo de alumnos guardarlos y traerlos cuando se soliciten.
1.- podrían sugerirme una forma de guardar los datos para luego solicitarlos?
2.- Solo llevo la parte de capturar y todo eso solo que recibo errores a la hora de compilar podrían decirme en que estoy mal? .... les anexo el codigo que llevo hasta ahora:

Código C++:

//librerias
 
struct datos 
{
    char nombre[20];
    char apellidoP[20];
    char apellidoM[20];
    char materia[20];
    int matricula;
    int grupo;
};
int j=0;
datos alumno[15];
char menu;
bool salir=false;
 
 
void bienvenida()
{
    //solo algo que escirbi
}   
 
 
void captura()
{
    system("cls");
        for(int i=0; i<3; i++){
        j++;
        cout<<"\n\tCaptura de datos"<<"\nIntroduzca su nombre: ";
        _flushall();
        gets(alumno[j-1].nombre);
        cout<<"\nIntroduzca sus apellidos: ";
        _flushall();
        gets(alumno[j-1].apellidoP);
        cout<<"\
        cout<<"\nIntrduzca su edad: ";
        _flushall();
        cin>>(alumno[j-1].edad);
        }
}
 
void guardar()
{
    //si me recomiendan una forma facil de hacer esto seria perfecto
}
 
void desplegar()
{
    system("cls");
        cout<<"\nDespliegue de informacion";
        for(int i=0; i<3; i++)
        {
            cout<<"\nContacto "<<i+1<<"\nNombre: "<<alumno[i].nombre<<"\nApellidos: "<<alumno[i].apellido<<"\nEdad: "<<alumno[i].edad<<endl;
        }
        _getch();
}
 
void main(){
    do{
{
    system("cls");
    cout<<"\t\tMenu de opciones"<<"\na)Capturar datos"<<"\nb)Imprimir datos"<<"\nc)Salir\n";
    cout<<"\nOprima a para escojer la opcion a y asi sucesivamente\n";
    cin>>menu;
    switch(menu)
    {
    case 'a':
        bienvenida();
        break;
    case 'b':
        captura();
        break;
    case 'c':
        desplegar();
        break;
    case 'd':
        salir=true;
        break;
    default: cout<<"Opcion no valida";
        _getch();
    }
}
}
    while(!salir);
}
 

[cotolon]

Lo de guardar es con ficheros, creando un alumnos.txt por ejemplo.

Y el error supongo son las comillas " "
si te fijas deja casi todo el programa en rojo. Creo que te falta o te sobra una " en algún lado

[Fw190]

Faltan las comillas de cierre del cout de la línea 35, a partir de aquí las comillas quedan descompensadas y por eso tirará errores a puñados. Empieza por corregir eso.

Saludos,

[paula23andrea]

Para usar archivos o ficheros se usa la biblioteca <fstream.h> (flujo de archivos). En tu función guardar puedes incluir algo como esto.

Código C++:

void guardar()
{
 
ofstream sale("datos.txt");
 
for(int i=0; i<3; i++)
{
  sale<<"\nContacto "<<i+1<<"\nNombre: "<<alumno[i].nombre<<"\nApellidos: "<<alumno[i].apellido<<"\nEdad: "<<alumno[i].edad<<endl;
 
}
}
 

Y eso te crea el txt con los datos que envias.
Suerte!

[vosk]

Para guardar datos y recuperarlos de forma elegante puedes usar una base de datos, ya sea un standalone (gdbm o sqlite) o algo que funcione por red (p.ej. mysql, con servidor local o con acceso remoto que te permite generar consultas desde tu aplicacion de escritorio o desde una aplicacion web de pc o de dispositivo movil): te ahorra el trabajo de serializar las consultas y seguramente tendrá un mejor control de errores (y queda mas 'fashion' cuando creas un producto que trabaja en nube con acceso remoto a datos actualizados en linea y vendes la moto que quieras que una simple aplicacion que trabaja con un txt plano)

Saludos
vosk

[Miraii]

Gracias (algo tarde por cierto .-.) no se como se me pudo pasar lo de las comillas. Btw reescribí el código en su totalidad pero ahora tengo otros problemas me tira un error en un void.
Aquí el estado actual del código..

Código C++:

//librerias
 
struct estructura{
    char Nombre[30];
    char ApellidoP[30];
    char ApellidoM[30];
    double matri;
    double grupo;
 
    double Mate;
    double Historia;
    double Dibujo;
    double Relleno;
    double Progra;
    double Origami;
    double Masmate;
} Alumnos[30];
estructura auxiliar;
 
int cont=0;
char op;
bool byebee=false;
 
 
void save()
{
    ofstream archivo("Textito.txt",ios::out);
    int i=0;
    for(int i=0;i<cont;i++)
   {archivo<<Alumnos[i].matri<<"\n";
    archivo<<Alumnos[i].grupo<<"\n";
    archivo<<Alumnos[i].Historia<<"\n";
    archivo<<Alumnos[i].Dibujo<<"\n";
    archivo<<Alumnos[i].Relleno<<"\n";
    archivo<<Alumnos[i].Progra<<"\n";
    archivo<<Alumnos[i].Origami<<"\n";
    archivo<<Alumnos[i].Mate<<"\n";
    archivo<<Alumnos[i].Masmate<<"\n";
    archivo<<Alumnos[i].Nombre<<"\n";
    archivo<<Alumnos[i].ApellidoP<<"\n";
    archivo<<Alumnos[i].ApellidoM<<"\n";
    
    archivo<<cont<<"\n";
   } archivo.close();
}
 
void leer()
{
    if(cont<30){
    int i=0;
    ifstream archivo("Textito.txt");
    if(archivo.good()){
    while(!archivo.eof())
    {
        
        archivo>>Alumnos[i].Nombre;
        archivo>>Alumnos[i].ApellidoP;
        archivo>>Alumnos[i].ApellidoM;
        archivo>>Alumnos[i].matri;
        archivo>>Alumnos[i].grupo;
        archivo>>Alumnos[i].Historia;
        archivo>>Alumnos[i].Dibujo;
        archivo>>Alumnos[i].Relleno;
        archivo>>Alumnos[i].Progra;
        archivo>>Alumnos[i].Origami;
        archivo>>Alumnos[i].Mate;
        archivo>>Alumnos[i].Masmate;
        i++;
        
    }
    }
    else{
        cout<<"Accion imposible."<<endl;
    } cont=i-1;
    archivo.close();
    cont=i-1;
    }
}
 
int todo()
{
    system("cls");
    while (cont<30){
        cout<<"Elegiste la opcion de ingresar datos.\n"<<"Ingresa la matricula: ";
        cin>>Alumnos[cont].matri;
        cout<<"Ingresa el Nombre del Alumno: ";
        cin>>Alumnos[cont].Nombre;
        cout<<"\nIngresa el Apellido Paterno del Alumno: ";
        cin>>Alumnos[cont].ApellidoP;
        cout<<"\nIngresa el Apellido Materno del Alumno: ";
        cin>>Alumnos[cont].ApellidoM;
        cout<<"\nIngresa el Grupo del Alumno: ";
        cin>>Alumnos[cont].grupo;
        _flushall();
 
        cout<<"\nIngrese las calificaciones de las materias listadas:\n";
        cout<<"Historia D8: ";
        cin>>Alumnos[cont].Historia;
        cout<<"\nLucha I: Dibujo~: ";
        cin>>Alumnos[cont].Dibujo;
        cout<<"\nMateria de relleno(cofAti): ";
        cin>>Alumnos[cont].Relleno;
        cout<<"\nProgramación: ";
        cin>>Alumnos[cont].Progra;
        cout<<"\nOrigami: ";
        cin>>Alumnos[cont].Origami;
        cout<<"\nMatemáticas: ";
        cin>>Alumnos[cont].Mate;
        cout<<"\nMás matematicas Dx: ";
        cin>>Alumnos[cont].Masmate;
        _flushall();
        cont=cont+1;
        return(cont);
    }
}
 
void ordenar(){
    system("cls");
    int fuc=cont;
    for(int d=0;d<fuc;d++)
    {
        for(int f=0;f<fuc;f++)
        {
            if(f==(fuc-1))
            break;
            if(strcmp(Alumnos[f].matri,Alumnos[f+1].matri)>0)
            {
                auxiliar=Alumnos[f];
                Alumnos[f]=Alumnos[f+1];
                Alumnos[f+1]=auxiliar;
            }
        }
    }
    fuc--;
    _getch();
}
 
void lista()
{
    ordenar();
    cout<<"Lista de Alumnos\n";
    for(int i=0;i<cont;i++)
    {
        cout<<"Alumno: "<<Alumnos[i].Nombre<<endl;
        cout<<"Apellido Paterno: "<<Alumnos[i].ApellidoP<<endl;
        cout<<"Apellido Materno: "<<Alumnos[i].ApellidoM<<endl;
        cout<<"Matrícula: "<<Alumnos[i].matri<<endl;
        cout<<"Grupo: "<<Alumnos[i].grupo<<endl;
 
        cout<<"Calificaciones"<<endl;
        cout<<"Historia D8:: "<<Alumnos[i].Historia<<endl;
        cout<<"Dibujo~: "<<Alumnos[i].Dibujo<<endl;
        cout<<"Materia de relleno(cofAti): "<<Alumnos[i].Relleno<<endl;
        cout<<"Programación: "<<Alumnos[i].Progra<<endl;
        cout<<"origami: "<<Alumnos[i].Origami<<endl;
        cout<<"Matemáticas: "<<Alumnos[i].Mate<<endl;
        cout<<"Más Matemáticas Dx: "<<Alumnos[i].Masmate<<endl;
        
    }
    system("pause");
}
 
void promedio()
{
    double pd=0;
    system("cls");
    char src[10];
    cout<<"Escogio la opcion de Promedio General.\n Ingrese la matricula del estudiante: ";
    cin>>src;
    for(int i=0; i<=30; i=i)
    {
        if (strcmp(src,Alumnos[i].matri)==0)
        {
            cout<<"\n Los Datos del estudiante son: ";
            cout<<"\nNumero: "<<i+1;
            cout<<"\nMatricula :"<<Alumnos[i].matri;
            cout<<"\nNombre: "<<Alumnos[i].ApellidoP<<" "<<Alumnos[i].ApellidoM<<" "<<Alumnos[i].Nombre;
            cout<<"\nGrupo: "<<Alumnos[i].grupo<<"\n";
            cout<<"\nEl promedio general del alumno es: ";
            pd=Alumnos[i].Historia+Alumnos[i].Dibujo+Alumnos[i].Relleno+Alumnos[i].Progra+Alumnos[i].Origami+Alumnos[i].Mate+Alumnos[i].Masmate;
            pd=pd/7;
            cout<<pd;
            if(pd<=69)
                cout<<"\nEl Estudiante"<<Alumnos[i].Nombre<<"ha reprobado";
            else
                cout<<"\nEl Estudiante"<<Alumnos[i].Nombre<<"ha aprobado";
            break;
        }
        else
        {
            i++;
        }
        if (i>30)
        {
            cout<<"\nNo se encontraron coincidencias en el registro";
        }
    }
    system("pause");
}
 
void mod()
{
    int ento=0;
    system("cls");
    char modificar[30];
    cout<<"Ha decidido Modificar los Datos.\n"<<"Favor de Ingresar la matricula del estudiante: ";
    cin>>modificar;
    for(int i=0;i<30;i++){
        if (strcmp(modificar,Alumnos[i].matri)==0){
            cout<<"\nNombre: "<<Alumnos[i].Nombre;
            cout<<"\nApellidos: "<<Alumnos[i].ApellidoP<<" "<<Alumnos[i].ApellidoM;
            cout<<"\nGrupo: "<<Alumnos[i].grupo;
            cout<<"Calificaciones:\n"<<"\nHistoria: "<<Alumnos[i].Historia;
            cout<<"\nDibujo~: "<<Alumnos[i].Dibujo;
            cout<<"\nMateria de relleno(cofAti): "<<Alumnos[i].Relleno;
            cout<<"\nProgramación: "<<Alumnos[i].Progra;
            cout<<"\nOrigami: "<<Alumnos[i].Origami;
            cout<<"\nMatemáticas: "<<Alumnos[i].Mate;
            cout<<"\nMás matemáticas Dx: "<<Alumnos[i].Masmate;
            _flushall();
            cout<<"¿Seguro que desea editar?\n";
            cout<<"1.Si\t2.No";
            cin>>ento;
            switch(ento){
            case 1:{
                system("cls");
                cout<<"Elegiste editar\n";
                cout<<"Nuevo Nombre: ";
                cin>>Alumnos[i].Nombre;
                cout<<"\nNuevo Apellido Paterno: ";
                cin>>Alumnos[i].ApellidoP;
                cout<<"\nNuevo Apellido Materno: ";
                cin>>Alumnos[i].ApellidoM;
                cout<<"\nNueva matricula: ";
                cin>>Alumnos[i].matri;
                cout<<"\nNuevo Grupo: ";
                cin>>Alumnos[i].grupo;
                
                cout<<"Calificaciones\n";
                cout<<"Historia D8: ";
                cin>>Alumnos[cont].Historia;
                cout<<"\nLucha I: Dibujo~: ";
                cin>>Alumnos[cont].Dibujo;
                cout<<"\nMateria de relleno(cofAti): ";
                cin>>Alumnos[cont].Relleno;
                cout<<"\nProgramación: ";
                cin>>Alumnos[cont].Progra;
                cout<<"\nOrigami: ";
                cin>>Alumnos[cont].Origami;
                cout<<"\nMatemáticas: ";
                cin>>Alumnos[cont].Mate;
                cout<<"\nMás matematicas Dx: ";
                cin>>Alumnos[cont].Masmate;
                system("cls");
                break;
                   }
            case 2:{
                system("cls");
                cout<<"No se editaron los datos... ";
                ento=ento+1;
                break;
                   }
            default:{
                system("cls");
                cout<<"Las unicas opciones son 1 y 2";
                _getch();
                ento=0;
                    }
            }
        }
    }
    system("cls");
}
 
void eraser()
{
    system("cls");
    char izard[10];
    int eresante=0;
    cout<<"Ha ingresado al sistema de eliminación de datos.\n"<<"Por favor ingrese la matrícula: ";
    cin>>izard;
    for(int i=0;i<30;i++)
    {
        if(strcmp(izard,Alumnos[i].matri)==0){
            cout<<"\nNombre: "<<Alumnos[i].Nombre;
            cout<<"\nApellidos: "<<Alumnos[i].ApellidoP<<" "<<Alumnos[i].ApellidoM;
            cout<<"\nGrupo: "<<Alumnos[i].grupo;
            cout<<"Calificaciones:\n"<<"\nHistoria: "<<Alumnos[i].Historia;
            cout<<"\nDibujo~: "<<Alumnos[i].Dibujo;
            cout<<"\nMateria de relleno(cofAti): "<<Alumnos[i].Relleno;
            cout<<"\nProgramación: "<<Alumnos[i].Progra;
            cout<<"\nOrigami: "<<Alumnos[i].Origami;
            cout<<"\nMatemáticas: "<<Alumnos[i].Mate;
            cout<<"\nMás matemáticas Dx: "<<Alumnos[i].Masmate;
            _flushall();
 
            while(eresante<1){
                cout<<"\n¿Realmente desea borrar los datos de este alumno?\n";
                cout<<"1.Si\t2.No";
                cin>>eresante;
                switch(eresante){
                case 1:
                    {
                        system("cls");
                        strcpy(Alumnos[i].Nombre, " ");
                        strcpy(Alumnos[i].ApellidoP, " ");
                        strcpy(Alumnos[i].ApellidoM, " ");
                        strcpy(Alumnos[i].matri, " ");
                        Alumnos[i].grupo=0;
 
                        Alumnos[i].Historia=0;
                        Alumnos[i].Dibujo=0;
                        Alumnos[i].Relleno=0;
                        Alumnos[i].Progra=0;
                        Alumnos[i].Origami=0;
                        Alumnos[i].Mate=0;
                        Alumnos[i].Masmate=0;
 
                        cout<<"Datos Eliminados...";
                        system("pause");
                        break;
                    }
                case 2:
                    {
                        system("cls");
                        cout<<"No se eliminaron los datos...";
                        eresante=eresante+1;
                        break;
                    }
                default:
                    {
                        system("cls");
                        cout<<"Las unicas opciones son 1 y 2";
                        _getch();
                        eresante=0;
                    }
                }
            }
        }
    }
    system("cls");
}
 
//mi void main(no es problema esto)
 

principalmente me suelta problemas en el "ordenar y eraser"

[gonzo13]

Hola,

sé que ésta no es tu consulta, pero espero que te sirva de algo. No hace falta que uses variables de tipo double para las calificaciones. Con una de tipo float te sobra espacio. Te explico por qué.

El tipo float utiliza 4 bytes para almacenar (en formato de punto flotante) números en el intervalo +/- [ 1.2*10^(-38) , 3.4*10^(38) ]. El tipo double utiliza 8 bytes para almacenar (en formato IEEE de doble precisión) números en el intervalo +/- [2.3*10^(-308) , 1.7*10^(308) ].

Es decir, que tanto el tipo float como el tipo double te proporcionan una cantidad ingente de números posibles. Es cierto que el tipo double te da mucha más precisión que el tipo float, pero estás almacenando una calificación para la que no necesitarás más de 3 o 4 cifras significativas, precisión que el tipo float te asegura más que de sobra, y usando la mitad de memoria. Además, con el formato double necesitas el doble de tiempo para las operaciones, por lo que también estás desperdiciando tiempo de ejecución.

Un saludo,
gonzo

[vosk]

Estas enviando tipos double a la funcion strcmp cuando esta espera dos const char*, y por eso el compilador no acepta la llamada; este es el prototipo de strcmp

Código:

int strcmp ( const char * str1, const char * str2 );

"...principalmente me suelta problemas en el 'ordenar y eraser'..."
Te suelta problemas en todos los strcmp por no usar los tipos correctos.

Saludos
vosk

[dontexplain]

Hola seanmilecín (igual eres un compañero, si no eres ignóralo).

Si lo editas por Visual Studio te indica a cada momento si los tipos introducidos corresponden con los solicitados en el prototipo de la función.

Si el argumento ha de ser un puntero a un tipo tiene que ser un puntero a un tipo. Siempre los arrays a char son punteros per sé por lo que no hace falta indicar el puntero de forma explícita.

char a[] = "Ejemplo";

a es un puntero hacia un array de chars por lo que a es válido como argumento que solicite un puntero hacia este tipo.

const char * const p[2] = { "chars", "chars2"};

Eso sería un array de punteros hacia chars y cada una de las entidades literales de chars son en sí arrays de chars (es una excepción) y por tanto punteros porque todas las referencias a arrays son punteros en sí mismos salvo que no sean literales.

char a[1] = {0x0f};

En el caso anterior es un array no literal de chars, así que se indica tradicionalmente entre { y }.

Entonces strcmp admitiría una referencia directa a los arrays de char

Código C++:

char a[] = "asd";
char b[] = "asd";
char *d = a;
int c = strcmp(a,b);
 
int c = strcmp(a,d);
 

a y b son punteros hacia char, tal y como lo especifica el prototipo. d sería un puntero hacia char por lo que es equivalente.

Código HTML:

int strcmp ( const char * str1, const char * str2 );
 

Esto sucede porque const char *const p = arraychar es similar a const char *const p = &arraychar[0] por lo que se crea otro puntero en lugar de hacer un puntero de un puntero, como pareciera.

Un saludo.