return konverterer min char

Gæt: du returnerer adressen på en lokal variabel - og det kan man ikke

char *f()
{
  char a[10];
  ...
  return a;
}

duer ikke

char a[10];

char *f()
{
  ...
  return a;
}

er OK

char *f(char a[10])
{
  ...
  return a;
}

er bedre

Hvordan får jeg så det til at virke med min kode?:

const char *cLoader::LoadXML(char *xmlfil, char *xmlpar, char *xmlchi)
{
    TiXmlDocument doc;
    doc.LoadFile(xmlfil);
    TiXmlNode* xmlnode = NULL;
    TiXmlElement *xmlelem = NULL;
    TiXmlAttribute *xmlattr = NULL;
    const char *xmlretu;
   
    xmlnode = doc.FirstChild( xmlpar );
    xmlelem = xmlnode->FirstChildElement( xmlchi );
    xmlattr = xmlelem->FirstAttribute();
    xmlretu = xmlattr->Value();
   
    return xmlretu;
}

måske:

const char *cLoader::LoadXML(char *xmlfil, char *xmlpar, char *xmlchi, char *retbuf)
{
    TiXmlDocument doc;
    doc.LoadFile(xmlfil);
    TiXmlNode* xmlnode = NULL;
    TiXmlElement *xmlelem = NULL;
    TiXmlAttribute *xmlattr = NULL;
    const char *xmlretu;
 
    xmlnode = doc.FirstChild( xmlpar );
    xmlelem = xmlnode->FirstChildElement( xmlchi );
    xmlattr = xmlelem->FirstAttribute();
    xmlretu = xmlattr->Value();
    strcpy(retbuf, xmlretu);
    return retbuf;
}

#include <string>

std::string cLoader::LoadXML(char *xmlfil, char *xmlpar, char *xmlchi)
{
    TiXmlDocument doc;
    doc.LoadFile(xmlfil);
    TiXmlNode* xmlnode = NULL;
    TiXmlElement *xmlelem = NULL;
    TiXmlAttribute *xmlattr = NULL;

    xmlnode = doc.FirstChild( xmlpar );
    xmlelem = xmlnode->FirstChildElement( xmlchi );
    xmlattr = xmlelem->FirstAttribute();
    return xmlattr->Value();
}

forhindrer der doc's destructor i at deallokere det memory ?

Arne, jeg forstår ikke din sidste kommentar.

arne_v << Hva ska jeg så smide ind i "char *retbuf" når jeg kalder LoadXML ?

bertel>

xmlretu er en pointer - der er ikke noget problem i at returnere den.

Problemet er hvad den peger på.

Så vidt jeg kan gennemskue så peger xmlattr->Value() på noget som gemmer sig et
eller andet sted i doc.

doc's destructor vil blive kaldt - og jeg formoder at den deallokerer alt
memory inklusive det som xmlattr->Value() eller xmlretu peger på.

c-skau>

det er det som xmlretu peger på jeg mener at du skal kopiere (strcpy)

Når man returnerer en std::string returneres et kopi af det xmlattr->Value() peger på.
Denne std::string vil normalt blive slettet "af sig selv" efter brug.
Der burde dermed ikke kunne opstå problemer.

sorry - jeg havde ikke set at du havde ændret retur typen fra char* til string - så
får du jo også kopieret data inden de forsvinder

Undskyld den pludselige forsvinden, men jeg har siddet og kæmpet med det her i en rum tid nu.
Jeg har lidt problemer med at få string metoden til at virke da "#include <string>" desværre kolidere med noget i den 3D motor (Irrlicht) jeg bruger til mit projekt.
Jeg har også prøvet med Arnes metode, men jeg kan ikke rigtig få den til at virke heller.. måske er det fordi jeg ikke helt kan finde ud af hva jeg ska smide ind i den ene ende under "char *retbuf"..?

char retbuf[100];

[ret 100 til whatever]

Ah.. nu forstår jeg =)
Du må meget undskylde min uforståenhed, men jeg er forholdsvis ny til C++ og er van til VB hvor man slet ikke behøver at bekymre sig om den slags!
Men efter lidt prøven frem og tilbage så lykkedes det mig at få det til at virke :D
Det ska du vist have lidt point for så arne_v hvis du lige smidder et svar?

svar