用代数表示法分配上三角矩阵的空间

时间:2015-04-25 11:21:10

标签: c matrix malloc free algebra

我正在研究涉及矩阵分解的项目(用C语言编写),我需要一些帮助。 我的目标是为上三角矩阵分配内存,我想通过代数行和列表示法(即{1,2,...,n}中的i,j而不是{0,1,...,n-1}中的i,j)来访问它。 例如,在5x5矩阵中,如果我输入matrix[3][4],我应该能够访问[3] [4]元素。

我的非代数索引上三角矩阵的代码如下所示:

double** malloc_sup_matrix (int n)
{
    double**    L;
    int         i;

    L = (double**)malloc((n)*sizeof(double*));

    if(L == NULL)
        printerror("allocating space for the matrix (rows).");        

    for(i = 0;  i < n;  i++)
    {
        L[i] = (double*)malloc((n-i)*sizeof(double));
        if(L[i] == NULL)
            printerror("allocating space for the matrix (cols).");
        L[i]-=i;
    }
    return L;
}

我的代数索引代码一(我没有检查分配的空间是否为空,我会在我停止搞乱时这样做):

    int**   m;
    int     i, n;

    n = 10;

    m = (int**)malloc((n+1)*sizeof(int*));
    for(i = 0; i < n; i++)
    {
        m[i] = (int*)calloc((n+1)-(i),sizeof(int));
        m[i] -= i;
    }

    m--;
    for(i = 0; i < n; i++)
    {
        m[i]--;
    }

它的工作方式与我想要的方式相同,我在释放我使用过的空间时遇到了问题。这就是我这样做的方式:

    for(i = 1; i <= n; i++)
    {
        m[i]++;
    }    

    for(i = 0; i < n; i++)
    {
        m[i] += (i);
        free(m[i]);
    }
    m++;
    free(m);

你们有什么建议吗?非常感谢你提前^^。

1 个答案:

答案 0 :(得分:0)

这一行存在问题:

m--;
for(i = 0; i < n; i++)
{
    m[i]--;
}

你正在递减m,但是然后继续从0开始索引......我想你最终可能会弄乱堆结构。

我设法让您的代码valgrind没有错误,如下所示:

#include <stdio.h>
#include <string.h>
#include <stdlib.h>


int main(int argc, char *argv[])
{
    int**   m;
    int     i, j, n;

    n = 10;

    m = (int**)malloc((n+1)*sizeof(int*));
    for(i = 0; i < n; i++)
    {
        m[i] = (int*)calloc((n+1)-(i), sizeof(int));
        m[i] -= i;
    }

    for(i = 0; i < n; i++)
    {
        m[i]--;
    }

    m--;

    /* Access it like m[1][1] ... m[n][n], m[i][j] (with i <= j) */

    /*
    for (i = 1; i <= n; i++) {
        for (j = i; j <= n; j++) {
            m[i][j] = i+j;
        }
    }
    */

    m++;

    for(i = 0; i < n; i++)
    {
        m[i]++;
    }

    for(i = 0; i < n; i++)
    {
        m[i] += (i);
        free(m[i]);
    }

    free(m);

    return 0;
}