[C] Programma crasha se aggiungo più di un elemento al DB

[stichtom]

Ciao a tutti, sto facendo un esercizio per la gestione di un registro scolastico. L'ho finito e all'inizio sembrava funzionare correttamente. Poi mi sono accorto che se aggiungo più di uno studente al database, il programma crasha. In particolare ho notato che il problema è collegato in qualche modo al salvataggio del nome (forse qualcosa con la realloc), infatti se lavoro solo con i numeri delle matricole non c'è nessun problema.


Qualcuno potrebbe darmi una mano? Il codice completo (in particolare le funzioni add e print sono quelle interessate):


#include <stdio.h>
#include <stdlib.h>
#include <limits.h>


#ifdef _WIN32
#define CLEAR "cls"
#else // Negli altri OS
#define CLEAR "clear"
#endif


#define CLEAR_BUFFER do { c = getchar(); } while (c != '\n' && c != EOF); // Macro per pulire il buffer in input
#define MAX 100 // Definisco il numero massimo di caratteri per il nome, cognome e indirizzo
#define TRUE 1
#define FALSE 0


typedef struct {
int day;
int month;
int year;
} DATE_T;


typedef struct {
int code;
char *name;
char *surname;
char *address;
DATE_T date;
int *exams;
} STUDENT_T;


STUDENT_T *student;
char courses[][15] = {"Programmazione", "Inglese", "Analisi"}; // SISTEMARE POI
int myIndex = 0; // Contatore che conta il numero degli studenti


int checkValue(int value, int check, int min, int max);
void checkAllocation(STUDENT_T *student);
void menu();
void add(STUDENT_T *student);
void printDatabase(STUDENT_T *student);
char* readString();
void exitmenu();




int main()
{


student = (STUDENT_T*) malloc(sizeof(STUDENT_T));
checkAllocation(student);


menu();


free(student);


return 0;
}


void menu()
{
int menu, check;


system(CLEAR);


printf("\t\t ---- MENU GESTIONE SEGRETERIA ---- \n\n");
printf("1 - Inserisci un nuovo studente\n");
printf("2 - Stampa il registro\n");
printf("3 - Elimina uno studente\n");
printf("4 - Ricerca uno studente\n");
printf("5 - Ricerca studenti per esami da sostenere\n");
printf("6 - Esci dal programma\n\n");
printf("Scegli un opzione dal menu: ");
check = scanf("%d", &menu);
menu = checkValue(menu, check, 1, 6);


switch (menu) {
case 1 :
add(student);
break;
case 2 :
printDatabase(student);
break;
case 3 :
//cancel(student);
break;
case 4 :
//search(student);
break;
case 5 :
//searchSubject(student);
break;
case 6 :
exit(EXIT_SUCCESS);
default :
printf("Errore, scelta non valida");
exit(EXIT_FAILURE);
}


return;
}




int checkValue(int value, int check, int min, int max)
{
int c;


CLEAR_BUFFER
while (value < min || value > max || check != 1) {
printf("Errore, devi inserire un valore valido compreso tra %d e %d: ", min, max);
check = scanf("%d", &value);
CLEAR_BUFFER
}


return value;
}


void checkAllocation(STUDENT_T *student)
{
if (student == NULL) {
printf("Errore durante l'allocazione dinamica della memoria");
exit(EXIT_FAILURE);
}


return;
}


void exitmenu()
{
int option, check;


printf("\n\nPremi 0 per tornare al main o 1 per uscire: ");
check = scanf("%d", &option);
option = checkValue(option, check, 0, 1);


if (option == 0)
menu();
else if (option == 1)
exit(EXIT_SUCCESS);
}


void add(STUDENT_T *student)
{


char c;
int check, j;
int i = myIndex;
/*enum subjects {Programmazione, Inglese, Analisi, Algoritmi, Architettura_Elaboratori, Algebra};
enum subjects courses;*/
system(CLEAR);


student = (STUDENT_T*) realloc(student, (myIndex + 1) * sizeof(STUDENT_T));
checkAllocation(student);


printf("I: %d, INDEX = %d", i, myIndex);
printf("\t\t ---- MENU GESTIONE SEGRETERIA ---- \n\n");
printf("Inserisci il numero di matricola: ");
check = scanf("%d", &student[i].code);
student[i].code = checkValue(student[i].code, check, 0, INT_MAX);
for (j = 0; j < myIndex; j++)
if (student[i].code == student[j].code) {
printf("Il numero di matricola inserito e\' gia presente nel database");
exitmenu();
}


printf("Inserisci il nome: ");
for (j = 0; j < MAX && (c = getchar()) != '\n'; j++) {
student[i].name = (char*)realloc(student[i].name , (j+1) * sizeof(char));
if (student[i].name == NULL) {
printf("Errore durante l'allocazione dinamica della memoria");
exit(EXIT_FAILURE);
}
student[i].name[j] = c;
}
student[i].name[j] = '\0';


// altri input


myIndex++;


exitmenu();


return;


}


void printDatabase(STUDENT_T *student)
{
int i, j;


system(CLEAR);


printf("\t\t ---- VISUALIZZA CONTENUTO DATABASE ---- \n\n");
printf("INDEX: %d\n\n", myIndex);
if (myIndex == 0)
printf("Non ci sono elementi da visualizzare nel database");
else {
for (i = 0; i < myIndex; i++) {
printf("INDEX: %d, i: %d\n\n", myIndex, i);
printf("Numero matricola: %d\n", student[i].code);
printf("\tNome e Cognome: %s\n", student[i].name/*, student[i].surname*/);
// Altre stampe


}


}


exitmenu();


return;


}

[sottovento]

Ci sono un po' di cose che hanno bisogno di una messa a punto.
Il tuo crash e' probabilmente questo:


student[i].name = (char*)realloc(student[i].name , (j+1) * sizeof(char));


Vai a reallocare qualcosa che non e' stato precedentemente allocato.

Io sistemerei anche la dichiarazione di questa funzione:


void add(STUDENT_T *student)


visto che al suo interno reallochi student, sarebbe meglio fare in modo che il nuovo indirizzo possa essere riportato al chiamante, e non perso dopo l'esecuzione della funzione

[wingman87]

Potrebbe essere effettivamente colpa della realloc. In particolare non mi sembra che dopo aver allocato uno student ne hai inizializzato i campi prima di utilizzarli. Ad esempio passi student[i].name alla realloc ma in student[i].name cosa c'è inizialmente? Se c'è NULL la realloc si comporta bene, se c'è un altro valore no.

Ho visto ora che ha risposto anche sottovento, sottoscrivo anche le sue osservazioni :)

[stichtom]

Grazie mille, in effetti l'errore era quello che passavo il puntatore alla funzione ma poi una volta che essa finiva tutte le modifiche andavano perse.

Per quanto riguarda la vostra seconda osservazione mi viene un dubbio: ma se il puntatore non è stato inizializzato con una malloc ma direttamente con una realloc, questa non dovrebbe comportarsi esattamente come una malloc e quindi non dare problemi?

[sottovento]

Per quanto riguarda la vostra seconda osservazione mi viene un dubbio: ma se il puntatore non è stato inizializzato con una malloc ma direttamente con una realloc, questa non dovrebbe comportarsi esattamente come una malloc e quindi non dare problemi?
Certo, se passi un puntatore alla memoria precedente pari a NULL; ma tu lo passi indeterminato, che quindi punta ad una locazione casuale (e quasi sicuramente invalida) nella memoria.

[stichtom]

Certo, se passi un puntatore alla memoria precedente pari a NULL; ma tu lo passi indeterminato, che quindi punta ad una locazione casuale (e quasi sicuramente invalida) nella memoria.

Giusto, non so per quale motivo ma mi ero fissato che i puntatori si inizializzassero direttamente a NULL