Seřazení dynamického pole struktur (studentů) – C / C++ – Fórum – Programujte.com

Dobrý večer, při spuštění mého kódu a zadání názvu souboru mi vyskočí chyba program ......exe přestal pracovat. Kód jsem si procházel několikrát a chybu nemohu najít. Překladač žádný error ani warning nevyhazuje.(používám CodeLite s TDM-GCC)

Jen kdyby náhodou, vstupní soubor vypadá nějak takhle:

Jarmila Adamkova 36
Jiri Lapcik 22
Milan Petru 12.7
Milena Novakova 37.5

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

FILE *vstup;

typedef struct 
{
	char * Jmeno;
	char * Prijmeni;
	float znamka;
	} Student;
	
	int porovnejZnamky(float a, float b){
		if (a<b) return -1;
		if (a>b) return 1;
		return 0;
		};
		
		
int porovnejStudenty(const void *n1,const void *n2){  
		if ((strcmp (((Student*)n1)->Prijmeni,((Student*)n2)->Prijmeni))==0) { 
				if ((strcmp (((Student*)n1)->Jmeno,((Student*)n2)->Jmeno))==0) {
					porovnejZnamky(((Student*)n1)->znamka,((Student*)n2)->znamka);
					}
					
				else return strcmp (((Student*)n1)->Jmeno,((Student*)n2)->Jmeno); 
			}
		else return strcmp (((Student*)n1)->Prijmeni,((Student*)n2)->Prijmeni); 
		};

Student *poleStudentu;

int main(int argc, char **argv)
{	
	
	char soubor[256];
	int p = 255;
	
	if (argc>1){
				
				vstup = fopen(argv[1], "r");
				if (vstup == NULL){
							printf("\nVstupni soubor nelze otevrit\n");
							return 1;
							}
				}
				
	else{
			printf("\nZadejte nazev vstupniho souboru: ");
			gets(soubor);
			vstup = fopen(soubor, "r");
			if (vstup == NULL){
							printf("\nVstupni soubor nelze otevrit\n");
							return 1;
							}
		}

	poleStudentu = (Student *) malloc(10 * sizeof(Student));
	if(poleStudentu == NULL) {
            printf("Nedostatek pameti!\n");
			getchar();
            exit(1);
        }

	while(1){
	for (int i = 0; i < p; i++){
	if (i%10==0){ 
	poleStudentu = (Student*)realloc(poleStudentu, (i+10) * sizeof(Student));}
		if(poleStudentu == NULL) {
            printf("Nedostatek pameti!\n");
			getchar();
            exit(1);
        }
	if(fscanf(vstup, "%s %s %f", poleStudentu[i].Jmeno , poleStudentu[i].Prijmeni , &poleStudentu[i].znamka) == EOF){
			p = 0;}
	
	}}

	int pocetPrvku=sizeof(poleStudentu)/sizeof(Student);
	
	qsort(poleStudentu, pocetPrvku, sizeof(Student), &porovnejStudenty);
	
		printf("Serazeni studenti: \n");
	for (int i = 0; i < pocetPrvku; i++){
	printf("\n%s %s %0.2f", poleStudentu[i].Jmeno, poleStudentu[i].Prijmeni, poleStudentu[i].znamka);}
		
		free(poleStudentu);
		getchar();
		return 0;
		
		
}

To byva vetsinou zpusobenou spatnou praci s pameti.

No po 10 minutach preformatovavani a dalsich 5 minutach WTF - mas tam peknou hromadu chyb:

1) ve strukture mas  char * Jmeno, char * Prijmeni - jsou to jen pointery ukazujici kdo vi kde. Bud musis alokovat pamet, nebo pouzit pole!

2)  sizeof u poleStudentu vrati 4 nebo 8 (podle pouzite architektury procesoru)

  // sizeof(Student*) / sizeof(Student)  nevesti nic dobreho. Pri trose stesti to bude jen 4/12 (proste nula)
  int pocetPrvku=sizeof(poleStudentu)/sizeof(Student);

a tak dale

#2 KIIV

Trochu jsem to pozměnil v téhle části, kde jsem myslel, že by mohl být zakopaný pes, ale bohužel.

Alespoň nějaký konkrétnější tip, v čem by to mohlo být ? 

poleStudentu = (Student *) calloc(100, sizeof(Student));
	if(poleStudentu == NULL) {
            printf("Nedostatek pameti!\n");
			getchar();
            exit(1);
        }

	while(1){
		for (int i = 0; i < p; i++){
			if(fscanf(vstup, "%s %s %f", poleStudentu[i].Jmeno , poleStudentu[i].Prijmeni , &poleStudentu[i].znamka) == EOF){
				p = 0;}
	
	}}

	int pocetPrvku=sizeof(poleStudentu)/sizeof(Student);
	
	poleStudentu = (Student*)realloc(poleStudentu, pocetPrvku * sizeof(Student));
		if(poleStudentu == NULL) {
            printf("Nedostatek pameti!\n");
			getchar();
            exit(1);
        }
	
	qsort(poleStudentu, pocetPrvku, sizeof(Student), &porovnejStudenty);

pro zacatek fscanf nacita na nejakou nahodnou adresu pameti... to je to, co jsem zminoval u te struktury

ale problemu tam bude daleko vic, si s tim chvili hraju a furt to ani zdaleka nefunguje

proc je tam vubec while (1) {  ???

 Trosku lepsi a mozna i funkcni:

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

FILE *vstup;

typedef struct {
  char    Jmeno[80]; // jen pointer neni dobry napad, musis alokovat pamet i pro tyhle retezce
  char Prijmeni[80]; // kdyz uz, tak aspon pole
  float  znamka;
} Student;
  
int porovnejZnamky(float a, float b) {
  if (a<b) return -1;
  if (a>b) return 1;
  return 0;
};

// male vylepseni (netestovano, mozna bude i fungovat):
int porovnejStudenty(void const * n1, void const * n2) {
  int tmp = strcmp(((Student const *)n1)->Prijmeni, ((Student const *)n2)->Prijmeni);
  if (tmp != 0) return tmp;
  tmp = strcmp(((Student const *)n1)->Jmeno, ((Student const *)n2)->Jmeno);
  if (tmp != 0) return tmp;
  return porovnejZnamky(((Student const *)n1)->znamka, ((Student const *)n2)->znamka);
};

Student *poleStudentu;

int main(int argc, char **argv)
{  

  char soubor[256];
  int p = 255;

  if (argc>1) {
    vstup = fopen(argv[1], "r");
    if (vstup == NULL) {
      printf("\nVstupni soubor nelze otevrit\n");
      return 1;
    }
    printf("otevren soubor %s\n", argv[1]);
  } else {
    printf("\nZadejte nazev vstupniho souboru: ");
    gets(soubor);
    vstup = fopen(soubor, "r");
    if (vstup == NULL) {
      printf("\nVstupni soubor nelze otevrit\n");
      return 1;
    }
    printf("otevren soubor %s\n", soubor);
  }

  poleStudentu = (Student *) malloc(1 * sizeof(Student));
  if(poleStudentu == NULL) {
    printf("Nedostatek pameti!\n");
    getchar();
    exit(1);
  }

  int pocetPrvku = 0;

  for (; pocetPrvku < p; ++pocetPrvku) {
    printf("pocet prvku: %d\n", pocetPrvku);
    if ((pocetPrvku % 10) == 0) {
      printf("realloc na %d\n", pocetPrvku+10);
      poleStudentu = (Student*)realloc(poleStudentu, (pocetPrvku+10) * sizeof(Student));
    }
    if (poleStudentu == NULL) {
      printf("Nedostatek pameti!\n");
      //getchar();
      exit(1);
    }
    int res = fscanf(
                vstup,
                "%s %s %f",
                poleStudentu[pocetPrvku].Jmeno,
                poleStudentu[pocetPrvku].Prijmeni,
                &poleStudentu[pocetPrvku].znamka
              );
    if (res != 3) {
      break;
    }
  }
  fclose(vstup);

  qsort(poleStudentu, pocetPrvku, sizeof(Student), &porovnejStudenty);
  
  printf("Serazeni studenti: \n");
  for (int i = 0; i < pocetPrvku; i++) {
    printf("%s %s %0.2f\n", poleStudentu[i].Jmeno, poleStudentu[i].Prijmeni, poleStudentu[i].znamka);
  }

  free(poleStudentu);
  //getchar();
  return 0;
}

#6 KIIV

Mnohokrát děkuji, vše funguje jak má. Nyní už vidím i ty absurdní chyby, které jsem udělal (např: úplné vynechání fclose a spoustu dalších věcí).

Jsem pouze samouk začátečník, každopádně děkuji za lekci.

#7 Horracio
zapomenute fclose vadi hlavne pri zapisovani. Nektery data zustavaji v bufferu a ten se bez fclose neulozi na disk. Jinak by to byl problem u neceho, co otevira soubory dokola, nejaka sluzba ci tak.