[C] Free const * ?

[elgringo]

Ik heb een lijst met elementen waarin een constante naam zit (const char* ) en een reference naar het volgende sturcture (een linked list).

De naam wordt, nu even als test, door de gebruiker ingegeven. Later zal dit uit een bestand komen.
Zodra een element is aangemaakt verandert de naam NIET. Maar ik wil wel elementen kunnen verwijderen. Met free krijg ik fouten; logisch want hij slikt geen const *.

Om de naam in de elementen als niet const op te geven vindt in onlogisch aangezien deze na create nooit meer verandert.

Hoe kan ik zoiets het beste oplossen?

[wiho]

Volgens mij ontkom je hier niet aan een cast (const_cast<char*> in C++ of gewoon (char*) in C). De free functie geeft immers het geheugen vrij, waarna je geen garantie meer hebt dat de inhoud constant blijft, of zelfs maar beschikbaar. Dus een const cast is hier op z'n plek.

[MLM]

mits je de string aangemaakt hebt met malloc (of strdup), MOET je hem zelfs vrijgeven met free.
je kan je pointer gewoon casten naar non-const, mocht je compiler niet willen (maar ik dacht dat free() in C en delete in C++ beide ook pointers-naar-const konden verwerken zonder cast. staat je compiler wel in C mode ipv C++ mode?)

[Elijan9]

De gebruikelijke manier is om de naam terug te casten naar niet const bij de free, dus naar (char *), dus:

C:

free((char *)name);

wat ik er misschien wat leesbaarder uitziet en wat ik zelf veel tegenkom, is dan men een "UNCONST" macro definieert:

C:

#define UNCONST(ptr) (void *)(ptr)

...

free(UNCONST(name));

[ Voor 41% gewijzigd door Elijan9 op 18-03-2010 08:59 ]

[elgringo]

Dat casten naar non-const leek bij ook het beste.

De naam wordt met een malloc verkregen en moet dus vrijgegeven worden.

[Zoijar]

Je kan ook de lijst intern een char* laten gebruiken en de get() methode een const char* laten retourneren. Dat is ook meer in lijn met hoe je het gebruikt.

[elgringo]

Zoijar schreef op donderdag 18 maart 2010 @ 11:20:
Je kan ook de lijst intern een char* laten gebruiken en de get() methode een const char* laten retourneren. Dat is ook meer in lijn met hoe je het gebruikt.

Met C++ zou dit idd wel werken. Ik heb in C een struct gebruikt en die heeft geen private data of functies.

[Chip.]

elgringo schreef op donderdag 18 maart 2010 @ 15:06:
[...]


Met C++ zou dit idd wel werken. Ik heb in C een struct gebruikt en die heeft geen private data of functies.

Hoeft toch ook niet?

[.oisyn]

Het punt was meer dat een getter zoals Zoijar voorstelt dan wat onhandig werkt omdat je die niet als method kunt definieren

[Zoijar]

Wel via het "OO-C paradigm": een blackbox struct (forward declared pointer in de header, definitie alleen in de implementatie) die je als pointer meegeeft aan een get functie.

Dwz (heel minimaal):

C:

list.h:

#ifndef LIST_HPP_
#define LIST_HPP_

typedef struct _List* List;

List construct_List(); // ctor
void destruct_List(List self); // dtor
const char* getItem(List self); // getter

#endif

list.c:

#include "list.h"
#include <malloc.h>

// All members are effectively private
struct _List {
    char* head;
};

List construct_List() {
    List self = (List) malloc(sizeof(_List));
    self->head = 0;
    return self;
}

void destruct_List(List self) {
    free(self);
}

const char* getItem(List self) {
    return self->head;
}

main.c:
#include "list.h"

int main() {
    List list = construct_List();

    const char* item = getItem(list);

    destruct_List(list);

    return 0;
}

Een prima manier van programmeren.

[ Voor 87% gewijzigd door Zoijar op 18-03-2010 20:19 ]