Code:int i, j; C = malloc(ROWS * sizeof(float **)); for (i = 0; i < ROWS; ++i) { C[i] = malloc(ROWS * sizeof(float *)); for (j = 0; j < ROWS; ++j) { C[i][j] = malloc(6); } }when im trying to run i got errorCode:void free_data(float ***data) { int i, j; for (i = 0; i < COLS; i++) { for (j = 0; j < ROWS; j++) { free((data)[j][i]); } free((*data)[i]); } free(data); }
can you help me please?
Notice that when you allocate, you're only looping over ROWS, both in the outer and inner loops.
My guess is that you want the outer loop to loop over ROWS and the inner loop to loop over COLS (i.e., row-major). If so, change both your allocation and deallocation code to loop in the same way.
Note that this means that if you allocate for C[i], you will deallocate for data[i], not (*data)[i].
Also, malloc(6) is probably wrong: you likely want malloc(float) instead. Unless your want to model a case where a float does not exist (hence represented by a null pointer), this would be an unnecessarily expensive approach: it would be cheaper just to allocate COLS number of floats per row. Later, after getting this to work, you could try your hand at an even cheaper approach: one allocation for the entire 2D array of floats as a single large flat dynamic array, and another for an array of pointers so that the array of floats can be used as a 2D array.
Besides these, you should check that malloc did not return a null pointer before using what it returns.
Last edited by laserlight; 05-30-2020 at 03:34 AM.
Look up a C++ Reference and learn How To Ask Questions The Smart WayOriginally Posted by Bjarne Stroustrup (2000-10-14)
is there a " like " button to put when people give feedback? i love jow everyone takes the time to help each other.
This appears to have been superseded by Problem of allocation memory, so I shall close this thread.
Last edited by laserlight; 06-01-2020 at 05:54 AM.
Look up a C++ Reference and learn How To Ask Questions The Smart Way
