Re: how to delete arrays to save memery space c++ - C / C++ / MFC Discussion Boards

Re: how to delete arrays to save memery space c++

David Crow3-May-10 9:04

David Crow

3-May-10 9:04

mrby123 wrote:
This was code I put in the beginning to show a two dimensional array

float* tim_var[1000];

But tim_var only has one dimension: 1000.

mrby123 wrote:
delete [] *tim_var;

Should be:

for (i = 0; i < 1000; i++)
    delete [] tim_var[i];

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"Man who follows car will be exhausted." - Confucius

Re: how to delete arrays to save memery space c++

mrby1233-May-10 9:14

mrby123

3-May-10 9:14

Thanks.

tim_var has two dimensions: tim_var[1000][6]

Your code below compiled OK:

for (i = 0; i < 1000; i++)
delete [] tim_var[i];

But the following code also compiled:

for (i = 0; i < 1000; i++)
delete tim_var[i];

Which one delete the two dimensional array tim_var[1000][6] ?

Please help

thanks

Re: how to delete arrays to save memery space c++

David Crow3-May-10 9:19

David Crow

3-May-10 9:19

mrby123 wrote:
tim_var has two dimensions: tim_var[1000][6]

No, tim_var has one dimension (1000), whereas tim_var[0...999] also has one dimension (6). The difference is subtle but important nonetheless.

mrby123 wrote:
But the following code also compiled:

for (i = 0; i < 1000; i++)
delete tim_var[i];

It will compile but will result in a memory leak.

mrby123 wrote:
Which one delete the two dimensional array tim_var[1000][6] ?

Neither, since that is a stack-based notation.

Re: how to delete arrays to save memery space c++

mrby1233-May-10 9:26

mrby123

3-May-10 9:26

Hi Cédric,

To delete the two dimensional array tim_var[1000][6], you please comment:

for (i=0;i<1000;i++)
{
delete tim_var[i];
}

or for (i=0;i<1000;i++)
{
delete[] tim_var[i];
}

Thanks

time_var[1000][6] was declared:

float* tim_var[1000];
for (i=0;i<1000;i++)
{
tim_var[i]=new float[6];
}
//
for(i=0;i<1000;i++) {
for(int j=0;j<6;j++)
{
tim_var[i][j]=0;
}
}
//

Re: how to delete arrays to save memery space c++

Rajesh R Subramanian3-May-10 8:13

Rajesh R Subramanian

3-May-10 8:13

There's no need to write such error-prone code (unless you're learning about how things work). Please use an STL container[^] if you're using C++.

“Follow your bliss.” – Joseph Campbell

Re: how to delete arrays to save memery space c++

mrby1233-May-10 8:19

mrby123

3-May-10 8:19

Thank you for your comments. Could you give me the code for STL container ? I am a learner.

Thanks for lighting me up.

Re: how to delete arrays to save memery space c++

Rajesh R Subramanian3-May-10 8:50

Rajesh R Subramanian

3-May-10 8:50

I already provided you a link. Click on each container class there and the page would contain an example at the bottom. You might want to search Google for more examples and tutorials.

“Follow your bliss.” – Joseph Campbell

Re: how to delete arrays to save memery space c++

CPallini3-May-10 8:18

CPallini

3-May-10 8:18

mrby123 wrote:
Should I put a loop of 1000 to delete ?

Yes.

mrby123 wrote:
I am not sure how to put code with a loop to delete.

As Nemanja suggested, you should call delete on every pointer that was assigned by means of new.
Smile | :)

If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler.
-- Alfonso the Wise, 13th Century King of Castile.

This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong.
-- Iain Clarke

[My articles]

Re: how to delete arrays to save memery space c++

mrby1233-May-10 8:35

mrby123

3-May-10 8:35

Should I put:

for (i=0;i<1000;i++)
{
delete tim_var[i];
}

or for (i=0;i<1000;i++)
{
delete *tim_var[i];
}

Please comment.

Thanks

Re: how to delete arrays to save memery space c++

Rajesh R Subramanian3-May-10 8:52

Rajesh R Subramanian

3-May-10 8:52

mrby123 wrote:
delete tim_var[i];

mrby123 wrote:
delete *tim_var[i];

Please try to understand the difference between the two. That'll help you do it on your own. Come up and ask if you have more doubts in the understanding process.

People here are helpful. But they try not to hand code to you in a plate, for your own benefit. Smile | :)

“Follow your bliss.” – Joseph Campbell

Re: how to delete arrays to save memery space c++

mrby1233-May-10 9:07

mrby123

3-May-10 9:07

Thanks. In fact,

delete *tim_var[i];

can not be compiled, but

delete tim_var[i];

I think the latter delete a two dimensional array declared and initialized by:

float* tim_var[1000];
for (i=0;i<1000;i++)
{
tim_var[i]=new float[6];
}

for(i=0;i<1000;i++) {
for(int j=0;j<6;j++)
{
tim_var[i][j]=0;
}
}

Re: how to delete arrays to save memery space c++

CPallini3-May-10 9:00

CPallini

3-May-10 9:00

consider

int * p;
p = new int[5];

the pointer assigned is p, hence you have to call

delete p;

in order to release memory.
Now try with your actual code.
Smile | :)

Re: how to delete arrays to save memery space c++

mrby1233-May-10 9:21

mrby123

3-May-10 9:21

Thanks CPallini.

Your above code works for one-dimensional array p[5].

But now I got tim_var[1000][6]

Re: how to delete arrays to save memery space c++

David Crow3-May-10 9:30

David Crow

3-May-10 9:30

mrby123 wrote:
But now I got tim_var[1000][6]

Have you tried:

float **tim_var;
  
tim_var = new float*[1000]; <<----------
                                       |
for (int x = 0; x < 1000; x++)         |
{                                      |
    tim_var[x] = new float[6]; <<----- |
                                     | | 
    for (int y = 0; y < 6; y++)      | |
        tim_var[x][y] = 0.0;         | |
}                                    | |
                                     | |
for (x = 0; x < 1000; x++)           | |
    delete [] tim_var[x]; <<---------- |
                                       |
delete [] tim_var; <<-------------------

Re: how to delete arrays to save memery space c++

mrby1233-May-10 9:54

mrby123

3-May-10 9:54

Thanks. Your code compiled OK.

Want do you think. I think it will work ?

Thanks

Re: how to delete arrays to save memery space c++

mrby1234-May-10 10:37

mrby123

4-May-10 10:37

My intention is to release all memories. The following code was suggested by you. I tried however, seems a problem. I got error message when I run the program: (delete statements got problem). It compiles OK.

"DAMAGE after normal block (#125) at 0x00E40610"

Would you please check ?

Re: how to delete arrays to save memery space c++

David Crow5-May-10 2:40

David Crow

5-May-10 2:40

mrby123 wrote:
Would you please check ?

Check what? The code snippet I provided you here is fine. If you have added to it or taken from it, that would be the reason for the error.

Re: how to delete arrays to save memery space c++

mrby1235-May-10 10:37

mrby123

5-May-10 10:37

Thanks. I believe your code is good. I got the problem that is how to declare, define, and call a function passing two dimensional array.

Please correct me. I am trying to delcare, define, and call a two dimensional array like:

declare in the header file:
void CCvib_procDlg::time(float*);

Define:
void CCvib_procDlg::time(float delms[][6])
{ ... code ... }

call:
time(&delms[0][0]);

When I tried to compile, I got:
error C2511: 'time' : overloaded member function 'void (float [][6])' not found in 'CC_vib_procDlg'

Please correct mm. Thanks

Re: how to delete arrays to save memery space c++

mrby1235-May-10 11:07

mrby123

5-May-10 11:07

This set does not compile, too.

Declare:
void CC_procDlg::time(float**);

define:
void CC_procDlg::time(float (*delms)[6])
{ code }

Call:
time(delms)

Please correct

Re: how to delete arrays to save memery space c++

CPallini3-May-10 10:33

CPallini

3-May-10 10:33

In your original post, you allocated an array of 1000 pointers on the stack.

float * tim_var[1000];

then, for every pointer, you allocated a memory buffer (6 floats) on the heap.
for (i=0;i<1000;i++)
{
tim_var[i] = new float[6];
}

Heap-allocated memory must be explicitely released, hence

// clean-up
for (i=0;i<1000;i++)
{
  delete tim_var[i];
}

On the other hand, you must not release stack-allocated memory.
Smile | :)

Re: how to delete arrays to save memery space c++

mrby1234-May-10 8:43

mrby123

4-May-10 8:43

My intention is to release all memories. The following code was suggested by a guru. I tried however, seems a problem. I got error message when I run the program: (delete statements got problem).

"DAMAGE after normal block (#125) at 0x00E40610"

////////////////////////////////////////////////////////////////////////////
float **tim_va;

tim_va = new float*[1000];

for (int ix = 0; ix < 1000; ix++)
{
tim_va[ix] = new float[6];

for (int jy = 0; jy < 6; y++)
tim_va[ix][jy] = 0.0;
}

for (ix = 0; ix < 1000; ix++)
delete [] tim_va[ix];

delete [] tim_va;
//////////////////////////////////////////////////////////////////////////////

Re: how to delete arrays to save memery space c++

CPallini4-May-10 9:05

CPallini

4-May-10 9:05

The code you're reporting doesn't compile. However if you modify it in order to make the compiler happy, it will run fine. Smile | :)

Re: how to delete arrays to save memery space c++

mrby1234-May-10 9:16

mrby123

4-May-10 9:16

Thanks. Would you please modify the code to make it work ?

Thanks

Re: how to delete arrays to save memery space c++

CPallini4-May-10 9:40

CPallini

4-May-10 9:40

float **tim_va;

tim_va = new float*[1000];

for (int ix = 0; ix < 1000; ix++)
{
    tim_va[ix] = new float[6];

    for (int jy = 0; jy < 6; jy++)
        tim_va[ix][jy] = 0.0;
}

for (int ix = 0; ix < 1000; ix++)
    delete [] tim_va[ix];

delete [] tim_va;

Re: how to delete arrays to save memery space c++

mrby1234-May-10 9:50

mrby123

4-May-10 9:50

Hi

Thanks. The code you put are the same as I reported before. I can not see any difference.

Please check. I know you have a very clear concept according to you post before.

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.