Pole reťazcov – C / C++ – Fórum – Programujte.com

void proc()
{

int i=0;
    char str[52];
    FILE *f;
    predaj = fopen("subor.txt", "r");
    char **udaje;
    if ((udaje = malloc(30 * sizeof(char *))) == NULL){
        printf("pamät fail");
        exit(1);
    }

        
    while ((fgets(str, 52, f)) != NULL){
        if ((udaje[i] = (char *)malloc(52)) == NULL){
            printf("pamät fail 2");
            exit(1);
        }               
        strcpy(udaje[i], str);
        i++;
    }

    for (i = 0; i < 30; i++) 
        free(udaje[i]);
    free(udaje);

}

po zavolaní tejto procedury mi to hadze error

Unhandled exception at 0x6B19FB53 (msvcr120d.dll) in PRPR PROJEKT 1.exe: 0xC0000005: Access violation reading location 0xCDCDCDCD.

prosím vás je tam niekde chyba? dakujem.

predaj = fopen("subor.txt", "r");, tam je samozrejme f miesto predaj.. to len drobna chybicka pri kopirovani

#2 Tomáš

chybu mas asi tu:

 FILE *f;
 predaj = fopen("subor.txt", "r");
/// f je neinicializvane

edit:

//miesto:
if ((udaje = malloc(30 * sizeof(char *))) == NULL);

//pouzi:
char* udaje[30];

//alebo:
char udaje[30][52];

inak 0xCDCDCDCD znamena ze je hodnta neinicializvana (plati len v debug mde u msvc kmpilatru)

edit2:

zabudes uzatvarat subor a zabudes uvalnvat pamet pri niektorych chybach

Před prací se souborem je dobré si ověřit, že je opravdu otevřený. Příčin kvůli kterým otevření souboru selže může být několik.

hu

FILE *f; predaj = fopen("subor.txt", "r"); /// f je neinicializvane
tam nemam chybu, to som komentoval ze len preklep, 
char *[30] nepomaha. to je to iste len inak napisane :)

toto robim projekt z viac procedur, vlastne ešte nemam vyriešiene to ze v jednej procedure si otvorim ten subor a budem si ho posuvat otvoreny, a az potom v maine ho zatvorim. (preto mi tu chyba to zatvorenie) a subor mi to otvara to viem lebo ked dam kontrolny vypis na vypisanie obsahu toho pola, udaje vypise a potom hned error. a len pri citani zo suboru sa myslim nemaju ako stratit udaje v tom subore, takze to nie je az taka chyba ked to windows za mna zatvori.

a problem pretrvava, aj ked som vyskušal všetky vaše hinty :(

sorry ze som nevyuzil  

tuto moznost

p.s. nevedomosť

 

kolik je řádků v souboru? opravdu 30?

hu

#6 Tomáš
Ked uz by si chcel pouzit tie zbytocne alokacie a chcel by si aj spravne dealokovat data pri chybach tak by si mohol pouzit nieco taketo:

void proc(){
    const int max_udaje_len = 30;
    const int max_data_size = 52;

    int udaje_len = 0;
    char str[max_data_size];
    char** udaje;
    FILE* file;

    if(!(file = fopen("data/subor.txt", "r"))){
        printf("subor fail");
        return;
    }

    if(!(udaje = (char**)malloc(max_udaje_len * sizeof(char*)))){
        printf("pamät fail");
        goto cleanup_file;
    }

    while(udaje_len < max_udaje_len && fgets(str, max_data_size, file)){
        if(!(udaje[udaje_len] = (char*)malloc(max_data_size))){
            printf("pamät fail");
            goto cleanup_malloc;
        }
        strcpy(udaje[udaje_len], str);
        ++udaje_len;
    }

    if(ferror(file)){
        printf("subor fail");
        goto cleanup_malloc;
    }

    for(int i = 0; i < udaje_len; ++i){
        printf("%d. %s\n", i, udaje[i]);
    }

cleanup_malloc:
    for(int i = 0; i < udaje_len; ++i){
        free(udaje[i]);
    }
    free(udaje);

cleanup_file:
    fclose(file);

}

uz som si spravil ucet, Tomáš=vrtkoo now :)

#8 vitamin
dík, funguje to. Kedže hovoris ze zbytocne alokacie? a ako by som to spravil bez nich? lebo bez nich mi to(povodna moja procedura) už vobec nefunguje, ani vypis.. (ale toto riešenie sa mi zide do dalšich procedur :) )

#9 vrtkoo
Cez staticke pole, potom ani nepotrebujes pomocnu premennu str, rovno budes zapisovat do toho pola.

void proc(){
    const int max_udaje_len = 30;
    const int max_data_size = 52;

    int udaje_len = 0;
    char udaje[max_udaje_len][max_data_size];
    FILE* file;

    if(!(file = fopen("data/subor.txt", "r"))){
        printf("subor fail");
        return;
    }

    while(udaje_len < max_udaje_len && fgets(udaje[udaje_len], max_data_size, file)){
        ++udaje_len;
    }

    if(ferror(file)){
        printf("subor fail");
        goto cleanup_file;
    }

    for(int i = 0; i < udaje_len; ++i){
        printf("%d. %s\n", i, udaje[i]);
    }

cleanup_file:
    fclose(file);
}

  #10 vitamin

 vdaka , toto je moja finalna verzia :) 

void proc()
{
	int i=0;
	
	FILE *subor;
	if ((subor = fopen("subor.txt", "r")) == NULL) {
		printf("neni otvoreny");	
		return;
	}
	char udaje[30][52];
		
	while ((fgets(udaje[i], 52, subor)) != NULL){
		i++;
	
	}
	for (i = 0; i < 30; i++){
		printf("%s", udaje[i]);
		
	}
	putchar('\n');
	
	fclose(subor);
}

#11 vrtkoo
ak bude v subore viac riadkov ako 30 tak si budes prepisovat data na stacku, preto tam treba mat este jednu podmienku:

while (i < 30 && (fgets(udaje[i], 52, subor)) != NULL){
	++i;
}
		

Radsej nepouzivaj magicke cisla.