Archief - [PROG][C] Pointer decl. in main, alloc. in functie, niet returnen. foute opgave?

• Er word in main een int pointer gedeclareerd (lijn 34)
• Deze wordt dan gebruikt in de functie maakparade die aangeroepen word op lijn 53
• in maakparade (lijn 117) word deze pointer dan gealloceerd (hij krijgt dus een ander adres in de functie)
• er worden bewerkingen uitgevoerd op de gealloceerde ruimte (lijn 122-127)
• Dan word een integer waarde gereturned (deze is nodig in een andere functie)

Tyfius zei:

Dat is nu net het punt van pointers. Je alloceert met de declaratie op lijn 34 geen geheugen, je vertelt alleen dat er moet rekening gehouden worden met een pointer naar een geheugenlocatie. Momenteel bevat die dus nog niets. Een goeie manier van programmeren is ook de pointer bij de declaratie op NULL te zetten.

Op lijn 117 alloceer je vervolgens een blok geheugen en zorg je ervoor dat die parade pointer naar die locatie verwijst. Doordat je aan de maakparade() functie een verwijzing naar een pointer variabele doorgeeft en geen kopie van een variabele pas je dus rechtstreeks de originele waarde aan (die in dit geval nog niet bestaat of NULL is).

Meer informatie over pointers in C kan je hier vinden.

Klik om te vergroten...

Tyfius zei:

Je moet die toch niet returnen? Die pointer is een verwijzing, die blijft bestaan. Je verandert alleen de waarde.

Het is hetzelfde principe als in het volgende voorbeeld (basis pointer lessen):

Code:

void foo(int *bar) {
  bar += 2;
}

int bar = 0;
foo(bar);

De variabele bar is nu dus 2, zonder dat die gereturned werd.

In jouw oefening is alleen de operatie anders.

Klik om te vergroten...

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

void doeiets(int *p){
	p=(int*)malloc(2*sizeof(int));
	*p=5;
	*(p+1)=8;
	//hetgene in p zit (nieuw adres van malloc word niet gereturned, net als in die examenoef)

}
void main(void){
	int *ipointer=NULL;
	doeiets(ipointer);
	printf("%d %d", *ipointer, *(ipointer+1));//fout, ipointer heeft nog geen ruimte gealloceerd gekregen
}

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

int * doeiets(int *p){
	p=(int*)malloc(2*sizeof(int));
	*p=5;
	*(p+1)=8;
	//hetgene in p zit (nieuw adres van malloc word niet gereturned, net als in die examenoef)
	return p;
}
void main(void){
	int *ipointer=NULL;
	ipointer=doeiets(ipointer);
	printf("%d %d", *ipointer, *(ipointer+1));//fout, ipointer heeft nog geen ruimte gealloceerd gekregen
}

Tyfius zei:

Eens korter gekeken en de oefening gecompileerd.

In de main functie staat 1 fout (geeft een warning normaal):

Code:

aantal = maakparade(haven, "groen", &parade, aantalboten); // fout

aantal = maakparade(haven, "groen", parade, aantalboten); // goed, geen & gebruiken

Hetzelfde geldt voor de andere maakparade() oproep. Als dit zo al in de main() stond die jij gekregen hebt zitten ze met een probleem.

In toonparade() doe je hetvolgende:

Code:

printf("%05d ",*(parade+i));

Hier mag je die * niet gebruiken. Correct is dus:

Code:

printf("%05d ",(parade+i));

Klik om te vergroten...

Tyfius zei:

Als die main() extact is zoals jij ze hebt gekregen, dan zit er een fout in.

Daarnaast ben ik niet helemaal zeker of de uitwerking van de andere oefeningen volledig correct is. Het lijkt mij allemaal een beetje onlogisch soms, is het mogelijk de opgave ook eens te posten? Ik snap bijvoorbeeld niet echt het nut van die maakparade() functie zoals ze in jouw oefening staat, nog van de toonparade() functie.

Klik om te vergroten...