[c++]wegschrijven vector naar bestand

[Beyond]

Hallo,

Ik heb een vector van objecten en die objecten probeer ik nu in een bestand weg te schrijven, maar hij schrijft maar 1 letter weg ipv de hele CString.

Dialoog:

C++:

void CDialoogAdresToevoegen::OnOK() 
{
    //Uitlezen textfield
    GetDlgItemText(IDC_TF_NAAM,m_Voornaam);
    GetDlgItemText(IDC_TF_ADRES,m_Adres);
    GetDlgItemText(IDC_TF_POSTCODE,m_Postcode);
    GetDlgItemText(IDC_TF_PLAATS,m_Plaats);

    //pGegevens is een pointer van het type CAdresgegevens
    CAdresgegevens * pGegevens;
    pGegevens = new CAdresgegevens(m_Voornaam,m_Adres,m_Postcode,m_Plaats);

    //Koppeling naar mainwindow
    CMainWindow * pFrame = (CMainWindow*)AfxGetMainWnd();
    //Doorgeven aan de hoofdclass
    pFrame->AddToAdresVector(pGegevens);
    
    
    CDialog::OnOK();
}

C++:

void CMainWindow::AddToAdresVector (CAdresgegevens * pAdresgegevens) 
{ 
    //adres->getAdres() is gelijk aan (*adres).getAdres()
    MessageBox(pAdresgegevens->getAdres());
    
    //Toevoegen aan vector ---> De inhoud van de pointer (*pAdresgegevens)
    adresVector.push_back(*pAdresgegevens);

    //Vector uitlezen kan zo: Dit gaat goed.
    MessageBox(adresVector.at(0).getPostcode());
      
}

Nu wil ik dus die vector doorlopen en naar een bestand wegschrijven:

C++:

void CMainWindow::OnMenuItemOpslaan()
{
    fstream iofile("output.txt", ios::in | ios::out |ios::ate);
    if (!iofile.fail()) { // Put whatever you need to generate new lines here
        
        for (int i = 1; i <= 3; i++ ) {
            CString postcode = adresVector.at(0).getPostcode();
            iofile << "Naam: " << *postcode << " at end" << endl;       
        }

    }
    else {
        MessageBox("Fout met openen uitvoerbestand!");
      
    }

}

Adresgegevens:

C++:

//////////////////////////////////////////////////////////////////////
// Construction/Destruction
//////////////////////////////////////////////////////////////////////

CAdresgegevens::CAdresgegevens(CString voornaam,CString adres, CString postcode, CString plaats )
    :_voornaam(voornaam),_adres(adres),_postcode(postcode), _plaats(plaats)
{

}

CString CAdresgegevens::getVoornaam()
{
    return _voornaam;
}

CString CAdresgegevens::getAdres()
{
    return _adres;
}

CString CAdresgegevens::getPostcode()
{
    return _postcode;
}

CString CAdresgegevens::getPlaats()
{
    return _plaats;
}


CAdresgegevens::~CAdresgegevens()
{

}

Nu schrijft hij dus alleen de eerste karakter van de postcode weg en niet de hele postcode.
Iemand een idee?

[farlane]

Ik denk dat je niet *Postcode moet gebruiken want dant maakt ie denk ik gebruik van de index operator [] die een char teruggeeft.

Probeer het eens met Postcode, die dan een const char* operator gebruikt van CString.

[Beyond]

Hij schrijft nu : 00422D7C weg en dat is het adres neem ik aan.

Geprobeerd:

C++:

CString postcode = adresVector.at(0).getPostcode();
iofile << "Naam: " << postcode << " at end" << endl;

i.p.v

C++:

CString postcode = adresVector.at(0).getPostcode();
iofile << "Naam: " << *postcode << " at end" << endl;

[ Voor 67% gewijzigd door Beyond op 20-06-2005 14:42 ]

[Beyond]

Thnx

C++:

const char* postcode = adresVector.at(0).getPostcode();

[farlane]

Beyond schreef op maandag 20 juni 2005 @ 14:46:
Thnx

Je kunt ook de << overloaden voor CString trouwens, met een global operator functie. Zie bv hier:

http://www.technology.nia...c1435/notes/overload.html

En dan nog iets, je kunt objecten beter als const ref meegeven als je ze alleen wilt gebruiken om te kopieren, zoals je in je constructor doet, dat scheelt een copy constructor aanroep die mogelijk veel overhead met zich meebrengt.

[ Voor 15% gewijzigd door farlane op 20-06-2005 15:23 ]

[MSalters]

Je hebt een memory leak.

C++:

    CAdresgegevens * pGegevens;
    pGegevens = new CAdresgegevens(m_Voornaam,m_Adres,m_Postcode,m_Plaats);

is overbodig. Je bedoelt gewoon

C++:

    CAdresgegevens Gegevens( m_Voornaam,m_Adres,m_Postcode,m_Plaats);

Vervolgens verander je de signature van AddToAdresVector naar CAdresgegevens const&.

[Beyond]

MSalters schreef op maandag 20 juni 2005 @ 15:35:
Vervolgens verander je de signature van AddToAdresVector naar CAdresgegevens const&.

Dit snap ik niet helemaal.
Hoe kan ik dat doen?

mijn h file:

C++:

#include <vector>

class CMyApp : public CWinApp
{

public:
    virtual BOOL InitInstance ();

};

class CMainWindow : public CFrameWnd
{


public:    
    CMainWindow ();
    void AddToAdresVector (CAdresgegevens * adres);
    
    std::vector<CAdresgegevens> adresVector;
    
    
private:
    
    
    
protected:  
    afx_msg void OnMenuItemInfoOver();
    afx_msg void OnMenuItemBestandToevoegen();
    afx_msg void OnMenuItemOpslaan();
    DECLARE_MESSAGE_MAP ()
};

en

C++:

// Adresgegevens.h: interface for the CAdresgegevens class.
//
//////////////////////////////////////////////////////////////////////

#if !defined(AFX_ADRESGEGEVENS_H__7AD0933A_8C28_481E_B693_994841570973__INCLUDED_)
#define AFX_ADRESGEGEVENS_H__7AD0933A_8C28_481E_B693_994841570973__INCLUDED_

#if _MSC_VER > 1000
#pragma once
#endif // _MSC_VER > 1000

class CAdresgegevens  
{
public: 
    CAdresgegevens(CString voornaam,CString adres, CString postcode, CString plaats);
    virtual ~CAdresgegevens();

    afx_msg CString getVoornaam();
    afx_msg CString getAdres();
    afx_msg CString getPostcode();
    afx_msg CString getPlaats();

private:
    CString _voornaam;
    CString _adres;
    CString _postcode;
    CString _plaats;
};

#endif // !defined(AFX_ADRESGEGEVENS_H__7AD0933A_8C28_481E_B693_994841570973__INCLUDED_)

[ Voor 90% gewijzigd door Beyond op 20-06-2005 16:32 ]

[farlane]

Beyond schreef op maandag 20 juni 2005 @ 16:21:
Dit snap ik niet helemaal.

Als rechtgeaarde C++ vindt MSalters (terecht) dat je niet met pointers/new/delete moet gaan pielen als het niet nodig is.

Je kunt de vector het memory management laten doen door gewoon stack objecten in je vector te plaatsten. Deze maakt er een kopie van en doet het memory management voor je. Het voorbeeld van MSalters maakt daar gebruik van.

[Beyond]

Ok maar wat moet ik in mijn code daar allemaal voor aanpassen?

als ik gebruik maak van:

C++:

CAdresgegevens Gegevens( m_Voornaam,m_Adres,m_Postcode,m_Plaats);

[ Voor 49% gewijzigd door Beyond op 20-06-2005 16:43 ]

[jos707]

Volgens mij kan je gewoon dit doen:

C++:

pFrame->AddToAdresVector(CAdresgegevens Gegevens( m_Voornaam,m_Adres,m_Postcode,m_Plaats));

[Verwijderd]

Heb je al geprobeerd om de null-terminated C string in die stream te gooien?

zo:

C++:

iofile << (LPCTSTR)(*postcode);

Ik weet uit m'n hoofd niet precies hoe die fstream werkt, maar ik dacht dat dit wel moet kunnen....

[Beyond]

Het werkt wel alleen ben ik benieuwd wat MSalters precies bedoelt en wat ik daarvoor allemaal moet aanpassen

[Verwijderd]

Beyond schreef op maandag 20 juni 2005 @ 17:27:
Het werkt wel alleen ben ik benieuwd wat MSalters precies bedoelt en wat ik daarvoor allemaal moet aanpassen

Hij zegt twee dingen:

1. als je 'new' t moet je ook ergens 'delete' en
   dat kun je doen door bijv. in de destuctor van (volgens mij) CMainWindow over de vector te loopen en de pointers te deleten
2. het eerste hoeft niet als je geen pointers opslaat in de vector maar gewoon kopieen. Maar de functie AddToAdresVector(CAdresGegevens*) moet dan veranderen naar: AddToAdresVector(const CAdresGegevens&)

[Beyond]

zal er naar kijken

[ Voor 119% gewijzigd door Beyond op 20-06-2005 22:23 ]