C / C++ / MFC

25-Dec-02 23:05

Hi
I'm a VC beginner.I'm attempting to make use of the CArray template but i'm at a lost now.(i used CArray as the size of my array is not fixed)I declared a CArray object in my document class and wants to create a function so that other classes might access the array.

Below is an abstract of my program.

in CDocument header file, where m_array1 and GetRawDataArray are declared as public...
----------------

struct SRawData
{
double x;
double y;
};

CArray<srawdata, srawdata=""> m_array1;
CArray GetRawDataArray();

--------------------------------
in the CDocument .cpp file...
------------------------------

CArray CRainAttDoc::GetRawDataArray()
{
return m_array1;
}

this gives me the error message:
(a)error C2955: 'CArray' : use of class template requires template argument list
(b)error C2553: no legal conversion of return value to return type 'class CArray *'

Does anyone know what is happening and what i can do?
Thanks.

KaЯl25-Dec-02 23:24

25-Dec-02 23:24

Try this:

in your header file, declare outside of the doc class declaration

typedef CArray <SRawData *, SRawData *> CSRawDataArray;

In the document class definition:

CSRawDataArray m_myDataArray;

Now you may use m_myDataArray like

SRawData *pNewData = new SRawData;
m_myDataArray.Add(pNewData);

while(m_myDataArray.GetCount())
delete m_myDataArray.RemoveHead()

HTH,

K.

<font size=0 color=#C0CFC0>
<hr color=#A0AFA0 width = 60% align=left><i>
One small village of indomitable geeks still holds out against the invaders. And life is not easy for the managers legionaries who garrison the fortified camps of Microsoftum, Javum, Ceplumplum and Vebasum</i></font>

raner26-Dec-02 7:00

26-Dec-02 7:00

haa ok

i said thks for your suggestion.gave me quite some insight really.But i still dunno how to create a function to return the array in some way so that other classes may access it.u have any suggestion on that?

KaЯl26-Dec-02 7:26

26-Dec-02 7:26

There are several solutions:

* You may add public member functions to your document to manipulate the array from the outside of the document object
* You may directly "lend" the array to other classes like this:

CSRawDataArray &CMyDocument::GetArray()<br />
{<br />
     return m_myDataArray;<br />
}

Beware that this method returns a reference on the array, i.e. the "real" object, not a copy.

void COtherClass::AddData(CMyDocument *pDoc, CSRawDataArray *pData)<br />
{<br />
     CSRawDataArray &myArray = pDoc->GetArray();<br />
<br />
     myArray.Add(pData);<br />
}

HTH,

K.

One small village of indomitable geeks still holds out against the invaders. And life is not easy for the managers legionaries who garrison the fortified camps of Microsoftum, Javum, Ceplumplum and Vebasum

raner26-Dec-02 22:21

26-Dec-02 22:21

Hi..i've tried the following but it's still not working.Do you know wat i shld do?

-------------------------------------------------------------------
typedef CArray <srawdata *,="" srawdata="" *=""> CSRawDataArray;//outside class declaration

//in doc header
CSRawDataArray m_array1;
CSRawDataArray& GetRawDataArray() const;

//in doc source file
//doing 2 things here.(1)load data into array from text file (2)define get function

void CRainAttDoc::Serialize(CArchive& ar)
{
CString strOneLine;

while(ar.ReadString(strOneLine))
{
SRawData *pData = new SRawData;
sscanf(strOneLine,"%g%g\n",pData->x,pData->y);
m_array1.Add(pData);
}
}

CSRawDataArray &CRainAttDoc::GetRawDataArray() const
{
return m_array1;
}
---------------------------------------------------------------------
This gives me the following error:
error C2440: 'return' : cannot convert from 'const class CArray<struct srawdata="" *,struct="" *="">' to 'class CArray<struct srawdata="" *,struct="" *=""> &'

When i try to change the object declaration to this:

const CSRawDataArray m_array1;

error C2440 remains and on top of that, the following message for RemoveAll() and Add()

'RemoveAll' : cannot convert 'this' pointer from 'const class CArray<struct srawdata="" *,struct="" *="">' to 'class CArray<struct srawdata="" *,struct="" *=""> &'

I was trying to declare the function const so that other classes cant alter the value as suggested by Roman Fadeyev but it just cant work.I'm pulling my hair out!..haa thks alot for your help.

KaЯl26-Dec-02 22:36

26-Dec-02 22:36

I don't understand why you're using the keyword "const". Have you tried without it ?

If you want to be sure the others classes don't mess up with the array, you may declare it as protected in your document class and add public methods (to your document class), to be sure it's the only class which will manipulate directly the array.

raner26-Dec-02 22:49

26-Dec-02 22:49

oh yah you are right, i totally forgot i can declare it as protected.

yes, it can be compiled now.however when trying to open the text file after executing the program, the serialize() function fails and returns a what appear to be an assertion failure.

KaЯl26-Dec-02 23:20

26-Dec-02 23:20

I suppose your Serialize method is like

void CRainAttDoc::Serialize(CArchive& ar)<br />
{<br />
CString strOneLine; <br />
<br />
while(ar.ReadString(strOneLine)) <br />
{ <br />
SRawData *pData = new SRawData;<br />
sscanf(strOneLine,"%g%g\n",pData->x,pData->y); <br />
m_array1.Add(pData); <br />
} <br />
}

First of all, you should test if you are writing or reading, using ar.IsStoring() (Serialization works both way). Also instead using a "while" statement, I would suggest you to write explicitly at the beginning of the archive the number of items to read
It would look like this

void CRainAttDoc::Serialize(CArchive& ar)<br />
{<br />
     if(ar.IsStoring()){<br />
          // writing the data<br />
          int iSize = m_array1.GetSize();<br />
          ar << iSize;<br />
          for(int i = 0; i w iSize; i++){<br />
               // write each element here in the archive<br />
               ...<br />
          }<br />
     }else{<br />
          // reading the data<br />
          m_array1.RemoveAll();<br />
<br />
          int iSize;<br />
          ar >> iSize;<br />
          m_Array1.SetSize(iSize);<br />
          for(int i = 0; i < iSize; i++){<br />
               // read one element here and add it to the array<br />
               ...<br />
          }<br />
     }<br />
}

I'm not absolutly sure of the syntax, I write it from memory Smile | :)

HTH,

K.

raner27-Dec-02 5:06

27-Dec-02 5:06

Thks alot.

To read each element and add it to the array, can i use the code below?

SRawData *pData = new SRawData;
sscanf(strOneLine,"%g%g\n",pData->x,pData->y);
m_array1.Add(pData);

KaЯl27-Dec-02 5:13

27-Dec-02 5:13

Why not using directly the archive ?

When loading:

SRawData *pData = new SRawData;
ar >> pData->x;
ar >> pData->y;
m_array1.Add(pData);

When writing:

SRawData *pData = m_array1.GetAt(i);
ar << pData->x;
ar << pData->y;
HTH,

K.

raner27-Dec-02 6:22

27-Dec-02 6:22

Because i'm not familiar with CArchive and have never tried using them before Poke tongue | ;-P

..Anyway i've tried the following.I think this has the right syntax,i've checked my books.However, it returns a "Failed to open document" dialog so i wrote the try-catch exception handling.

And it turns out that SetSize()is indeed creating problem. And the "Testing" Message Box appears a number of times before "Unexpected file format" is displayed.The document i was opening is a text file with 14 rows, 2 columns of values.

if(ar.IsStoring())
{ // writing the data
int iSize = m_array1.GetSize();
ar << iSize;
for(int i = 0; i < iSize; i++)
{
SRawData *pData = m_array1.GetAt(i);
ar << pData->x;
ar << pData->y;
}

}
else
{
m_array1.RemoveAll();
int iSize=13;
ar >> iSize;
try{
m_array1.SetSize(iSize);
}
catch(CMemoryException *e)
{
AfxMessageBox("Cant set size");
e->Delete();
}

for (int i = 0; i < iSize; i++)
{
AfxMessageBox("Testing");
SRawData *pData = new SRawData;
ar >> pData->x;
ar >> pData->y;
m_array1.Add(pData);
}
}

any idea wat is wrong now?Really appreciate your patience;)

KaЯl27-Dec-02 12:47

27-Dec-02 12:47

Check your mail Smile | :)

Roman Fadeyev26-Dec-02 0:05

Roman Fadeyev

26-Dec-02 0:05

Firstly, i advise you to read at least a couple pages of any book about templates. It will help you to control a situation. As for your question, you should type:

CArray < SRawData, SRawData& > m_array1;

Notice that your function

CArray CRainAttDoc::GetRawDataArray()

will copy whole array since you return it by value, not by ref. It is better way to use

const CArray<......>  & CRainAttDoc::GetRawDataArray() const

raner26-Dec-02 6:38