🚀 go-pugleaf

"Skender" <Skender@GeenSpamAUB.VrijZijn.be> wrote in message
news:NhDBc.162226$LL2.8273027@phobos.telenet-ops.be...

> > naam was aangemaakt via new char[], en moet dus wel via delete[]
> > verwijderd worden.
>
> Wanneer moet ik eigenlijk een string deleten?
Je moet niets. Je mag hem deleten zodra je hem niet meer nodig hebt.

> In deze functie bijvoorbeeld
> worden twee strings aangemaakt, maar ik kan ze niet deleten, want ik heb
er
> geen pointer naar.
>
> bool Gemeente::naamBevat(char* gezocht){
>   return strstr(getNaamKlein(),strlwr(gezocht)) != 0;
> }
>
> Moet ik dat dan vervangen door dit?
>
> bool Gemeente::naamBevat(char* gezocht){
>   char* naamKlein = getNaamKlein();

>   char* gezochtKlein = strlwr(gezocht);
>   bool res = strstr(naamKlein,gezochtKlein) != 0;
>   delete [] naamKlein;

Alleen als getNaamKlein() een new[] heeft uitgevoerd


>   delete [] gezochtKlein;
strlwr() is geen standaard C++ functie. De eerste strlwr die ik op google
zie geeft dezelfde waarde terug als die je erin stopt, maw: hij verandert de
string die je doorgaf aan de functie. Als dat ook jouw strlwr() is, dan heb
je bij deze niet alleen 'gezochtKlein' verwijderd maar ook 'gezocht'. Ik
denk niet dat dat je bedoeling was, want een tweede aanroep van naamBevat
voor een ander object, maar met dezelfde string pointer, zou dan crashen.

In dat geval had de compiler trouwens de fout zelf ook opgespoord als je hem
had aangegeven dat het niet je bedoeling was 'gezocht' zelf te wijzigen
(zoals waarschijnlijk je intentie was:)

bool Gemeente::naamBevat(char const * gezocht)

Je zou het ook met std::string kunnen oplossen, op voorwaarde dat
getNaamKlein ook een std::string teruggeeft:

using namespace std;
bool Gemeente::naamBevat(string const &gezocht){
  string naamKlein = getNaamKlein();
  string gezochtKlein = gezocht;
  transform(gezochtKlein.begin(), gezochtKlein.eind(), gezochtKlein.begin(),
tolower);
  return naamKlein.find(gezochtKlein) != string::npos;
}

Dan ben je helemaal van de deletes af. string regelt dat allemaal zelf.
Nadeel is dat de code enigszins onnatuurlijk kan overkomen, als je nog niet
echt gewend bent aan de C++ uitbreidingen op de C libraries.