C - 矩阵乘法分割故障

时间:2012-03-29 14:01:00

标签: c segmentation-fault dynamic-memory-allocation matrix-multiplication

运行此代码时出现分段错误。谁知道为什么?感谢。

#include <stdio.h>

int main()
{
    double **m1, **m2, **mr;
    int m1_rows, m1_cols, m2_rows, m2_cols, mr_rows, mr_cols;
    int i, j, k;

    printf("Enter number of rows for matrix 1: ");
    scanf("%d", &m1_rows);

    printf("Enter number of columns for matrix 1: ");
    scanf("%d", &m1_cols);

    printf("Enter number of rows for matrix 2: ");
    scanf("%d", &m2_rows);

    printf("Enter number of columns for matrix 2: ");
    scanf("%d", &m2_cols);

    //allocate memory for matrix 1 m1
    m1 = (double **) calloc(m1_rows, sizeof(double *));
    for (i = 0; i < m1_rows; i++) {
        m1[i] = (double *) calloc(m1_cols, sizeof(double));
    }

    //allocate memory for matrix 2 m2
    m2 = (double **) calloc(m2_rows, sizeof(double *));
    for (i = 0; i < m2_rows; i++) {
        m2[i] = (double *) calloc(m2_cols, sizeof(double));
    }

    //allocate memory for sum matrix mr
    mr = (double **) calloc(mr_rows, sizeof(double *));
    for (i = 0; i < mr_rows; i++) {
        mr[i] = (double *) calloc(mr_cols, sizeof(double));
    }

    //assign  mr_rows and mr_cols
    mr_rows = m1_rows;
    mr_cols = m2_cols;

    //initialize product matrix
    for (i = 0; i < m1_rows; i++) {
        for (j = 0; j < m2_cols; j++) {
            mr[i][j] = 0;
        }
    }

    //perform matrix multiplication
    for (i = 0; i < m1_rows; i++) {
        for (j = 0; j < m2_cols; j++) {
            mr[i][j] = 0;
            for (k = 0; k < m1_cols; k++) {
                mr[i][j] += m1[i][k] * m2[k][j];
            }
        }
    }

    //print result
    for (i = 0; i < mr_rows; i++) {
        for (j = 0; j < mr_cols; j++) {
            printf("%f\t", mr[i][j]);
        }
    }

    //free memory m1
    for (i = 0; i < m1_rows; i++); {
        free(m1[i]);
    }
    free(m1);

    //free memory m2
    for (i = 0; i < m2_rows; i++); {
        free(m2[i]);
    }
    free(m2);

    //free memory mr
    for (i = 0; i < mr_rows; i++); {
        free(mr[i]);
    }
    free(mr);

    return 0;
}

我使用valgrind valgrind --tool=memcheck a.out了解有关分段错误的更多信息,但结果是超过30000个错误,因此它没有打印出来。

2 个答案:

答案 0 :(得分:2)

您没有分配mr_rowsmr_cols。他们需要像这样设置:

mr_rows = m1_rows;
mr_cols = m2_cols;

这条线不好:

mr[i][j] += m1[i][k] * m2[k][j];

这将访问超出范围的元素,尤其是因为k未初始化。你需要在三个嵌套for循环中的那行代码。实际上,您也可以将零化代码推入此中。

for(i=0; i<m1_rows; i++){
    for(j=0; j<m2_cols; j++){
        mr[i][j] = 0;
        for(k=0; k<m1_cols; k++){
            mr[i][j] += m1[i][k]*m2[k][j];
        }
    }
}

此外,所有内存释放循环都是错误的。而不是

for(i=0; i<m1_rows; i++);{
    free(m1[i]);
}
free(m1);

应该阅读

for(i=0; i<m1_rows; i++){
    free(m1[i]);
}
free(m1);

那个流浪的分号正在杀死你。

您还需要检查m1中的列数是否等于m2中的行数,即测试m1_cols == m2_rows

最后一点。你在这里可怕地复制你的代码。没有三个相同的for循环来分配矩阵,三个相同的for循环来解除分配。将这些操作解压缩为辅助函数,可以从main调用。

这就是我能找到的一切!

答案 1 :(得分:1)

您没有在任何地方为mr_rows and mr_cols分配任何值。因此,他们将拥有垃圾值,您可以使用它们通过callin calloc().

分配内存