[C] Ik zie de fout niet

[a3konijn]

Ik heb de volgende code:

code:

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

void ToonInvoer(char **ppc, int totaal);

void main()
{
char **ppc, *pc, woord[100];
int teller = 0;
ppc = (char**) malloc(sizeof(pc));
printf("Geef een aantal woorden, stop = stoppen\n");
printf("Geef een woord : ");
scanf("%s", woord);

while(strcmp(woord,"stop")!=0)
{
//Zolang ingevoerde woord geen "stop" is:

//dynamisch geheugen toewijzen aan pc
//lengte woord + stringafsluiter
pc = (char*) malloc(strlen(woord)+1);
//kopieren van woord naar pc
strcpy(pc,woord);
//grootte van geheugen benodigd voor pcc gelijk aan ...
ppc = (char**) realloc(ppc, sizeof(ppc)+sizeof(pc));
//pcc naar pc laten wijzen
ppc[teller] = pc;
printf("Geef nog een woord : ");
scanf("%s",woord);
teller++;
}
printf("\nAantal ingevoerde woorden : %d\n",teller);
printf("Ingevoerde woorden: ");
ToonInvoer(ppc,teller);
}

void ToonInvoer(char** ppc,int totaal)
{
int i;
for(i=0;i<totaal;i++)
{
printf("Woord %d : %s\n",i, *ppc);
ppc++;
}
}

Een gebruiker wordt net zolang om strings gevraagd totdat hij stop invoert. Het ingevoerde woord wordt in woord opgeslagen. Vervolgens wordt er geheugen gereserveerd ter grootte van het woord en wordt de string in woord gekopieerd naar de character pointer pc. ppc wordt gebruikt voor de opslag van de adressen van de beginletters van iedere string.

Wat gaat er goed:
Opslag.

Wat gaat er mis:
Het weer terugzetten op het scherm van de ingevoerde woorden. Gaat alleen goed bij een aantal van 3 of minder. Wat gaat er hier mis?? Ligt het aan realloc???

[Verwijderd]

Kan je die code ff goed formatten?

[D2k]

Op dinsdag 26 februari 2002 14:50 schreef XKB het volgende:
Kan je die code ff goed formatten?

code:

#include <string.h>
#include <malloc.h>

void ToonInvoer(char **ppc, int totaal);

void main()
{
    char **ppc, *pc, woord[100];
    int teller = 0;
    ppc = (char**) malloc(sizeof(pc));
    printf("Geef een aantal woorden, stop = stoppen\n");
    printf("Geef een woord : ");
    scanf("%s", woord);
    
    while(strcmp(woord,"stop")!=0)
    {
        //Zolang ingevoerde woord geen "stop" is:
        
        //dynamisch geheugen toewijzen aan pc
        //lengte woord + stringafsluiter
        pc = (char*) malloc(strlen(woord)+1);
        //kopieren van woord naar pc
        strcpy(pc,woord);
        //grootte van geheugen benodigd voor pcc gelijk aan ...
        ppc = (char**) realloc(ppc, sizeof(ppc)+sizeof(pc));
        //pcc naar pc laten wijzen
        ppc[teller] = pc;
        printf("Geef nog een woord : ");
        scanf("%s",woord);
        teller++;
    }
    printf("\nAantal ingevoerde woorden : %d\n",teller);
    printf("Ingevoerde woorden: ");
    ToonInvoer(ppc,teller);
}

void ToonInvoer(char** ppc,int totaal)
{
    int i;
    for(i=0;i<totaal;i++)
    {
        printf("Woord %d : %s\n",i, *ppc);
        ppc++;
    }
}

graag gedaan

[Verwijderd]

ik denk dat het idd aan de realloc ligt. Als ik me niet vergis geef je daardoor het geheugen waarin je een eerder woord had opgeslagen weer vrij.

[.oisyn]

code:

ppc = (char**) malloc(sizeof(pc));

...

ppc = (char**) realloc(ppc, sizeof(ppc)+sizeof(pc));

dat gaat niet werken

sizeof (pc) = sizeof (char * ) (want pc is een char * ), en dat is gelijk aan 4 (een pointer is 4 bytes)
sizeof (ppc) is ook 4, aangezien ppc ook een pointer is (naar een pointer naar char). Met sizeof kun je dus niet de grootte van de gereserveerde array opvragen. Dat is ook logisch als je bedenkt dat ppc eigenlijk helemaal geen array is, het is een pointer naar het 1e element van de array.

Je zult dus het aantal woorden bij moeten houden, en dan voor ppc een geheugenblok reserveren van aantal_woorden * sizeof (char * ) bytes

.edit: hmmm ja idd best wel gaar, dat * en ) een wordt

[a3konijn]

Gelukkig dat het aan de realloc lag. Vermoedde ik het i.i.g. goed.

Op dinsdag 26 februari 2002 15:10 schreef OiSyN het volgende:

code:

ppc = (char**) malloc(sizeof(pc));

...

ppc = (char**) realloc(ppc, sizeof(ppc)+sizeof(pc));

dat gaat niet werken

sizeof (pc) = sizeof (char (want pc is een char , en dat is gelijk aan 4 (een pointer is 4 bytes)
sizeof (ppc) is ook 4, aangezien ppc ook een pointer is (naar een pointer naar char). Met sizeof kun je dus niet de grootte van de gereserveerde array opvragen. Dat is ook logisch als je bedenkt dat ppc eigenlijk helemaal geen array is, het is een pointer naar het 1e element van de array.

Je zult dus het aantal woorden bij moeten houden, en dan voor ppc een geheugenblok reserveren van aantal_woorden * sizeof (char bytes

Duidelijk

Bedankt allemaal!

(Leuk effect trouwens die * ) == )
Blijkt trouwens uit dat een zowel met * ) als * D gemaakt kan worden.

[a3konijn]

Getest en het werkt goed

[.oisyn]

oh overigens, er is ook een functie strdup, die je string dupliceert en dus automatisch een nieuw stuk geheugen aanroept...

oftewel, dit:

code:

char * str2 = (char *) malloc (sizeof (str) + 1);
strcpy (str2, str);

kan geschreven worden als

code:

char * str2 = strdup (str);

[farlane]

en verderop in je code

code:

free(str2);