CArray, what am i doing wrong?

Question

0.00/5 (No votes)

See more:

Declaration in header file:

CArray<int[1][1],int[1][1]> ptArray;

Source file

void cDlgGrid::OnClick(NMHDR *pNotifyStruct, LRESULT* pResult)
{
        LPNMITEMACTIVATE pNMIA = reinterpret_cast<lpnmitemactivate>(pNotifyStruct);
        int row = pNMIA->iItem;
	int col = pNMIA->iSubItem;

        ptArray.SetSize(1,1);
	int SelectedCell[1][1];
        SelectedCell[0][0] = row;
	SelectedCell[0][1] = col;
        ptArray.Add(SelectedCell);  --> I am getting an error here

</lpnmitemactivate>

error:

C2664: 'CArray<type,arg_type>::Add' : cannot convert parameter 1 from 
'int (*)[1][1]' to 'int [][1]'
        with
        [
            TYPE=int [1][1],
            ARG_TYPE=int [1][1]
        ]
        Types pointed to are unrelated; conversion requires reinterpret_cast, C-style cast or function-style cast

What should i do.

Posted 1-Aug-11 5:31am

amarasat

Add a Solution

2 solutions

Solution 2

The problem is: if there is more than one possible cast you will get always this compiler error. You have to cast explicit the parameter. Therfore its the better solution to use struct types with only one possible cast, the compiler can implicit cast the type.
btw:

C#

int SelectedCell[1][1]; // this is exact one int
    SelectedCell[0][0] = row;
SelectedCell[0][1] = col; // this will possible throw a runtime error

Regards.

Posted 1-Aug-11 8:02am

mbue

Comments

amarasat 1-Aug-11 14:10pm

Struct helped me in solving my error.

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

TRK3 · Accepted Answer · 2011-08-01T06:30:00

There are a number of things wrong here.

To fix the compiler error you'd need to do this:

ptArray.Add(*SelectedCell);

or change the CArray declaration to:

CArray<int[1][1],&int[1][1]> ptArray;

But that's the least of your problems.

You've declared SelectedCell as a 2 dimensional array of 1 row and 1 column. Which means it contains only 1 integer.

When you assign to SelectedCell[0][1] you are assigning to a location outside the bounds of the array. The results of that are undefined.

It seems to me that you actually want to do something like this:

struct SPoint { int row; int col; };

CArray<SPoint,&SPoint> ptArray;


//...

SPoint SelectedCell;
SelectedCell.row = row;
SelectedCell.col = col;

ptArray.Add(SelectedCell);