使用scanf for float **类型的分段错误

Question

我试图编写一个函数，要求用户输入矩阵。它提示的行数，列数，然后提示值矩阵中的每个元素：直到我进入一个价值i = 0然后按进入

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

void enterMatrix(float ** matrix, int nbLines, int nbColumns){ 
    for (int i = 0; i < nbLines * nbColumns; i++){ 
     printf("i = %d? ", i); 
     scanf("%f", matrix[i]); 
    } 
} 

int main(void){ 
    int nbLines, nbColumns; 
    printf("nbLines? "); scanf("%d", &nbLines); 
    printf("nbColumns? "); scanf("%d", &nbColumns); 
    float *matrix[nbL * nbCol]; 

    enterMatrix(matrix, nbLines, nbColumns); 
} 

一切工作正常，从而导致分段故障。

任何想法可能是错的？

Answer 1

您需要分配内存，因为您在编译时不知道变量nbLines和nbColumns将保持哪些值。

所以，你需要指针首先声明的矩阵：

float **matrix; 

然后开始分配内存，根据用户输入：

matrix = (float **)malloc(nbLines * sizeof(float *)); 

for (int i = 0; i < nbLines; i++) 
{ 
    matrix[i] = (float *)malloc(nbColums * sizeof(float)); 
} 

的分段错误发生becouse你不分配内存为您的矩阵，但只是一个[nbL * nbCol]指针float

Answer 2

您的问题是因为

float *matrice[nbL * nbCol]; 

定义了一个未初始化的指针数组（即，一个数组float *），而不是浮点数组。然后将其作为指向float的指针（即float **）传递给enterMatrix()。然后scanf()调用读取到matrix[i]这是一个未初始化的指针。结果是未定义的行为。

一个补丁修复将是main()改变matrice的定义

float matrice[nbL * nbCol]; 

，改变功能（我用评论来突出变化）

void enterMatrix(float *matrix, int nbLines, int nbColumns) /* note type of matrix */ 
{ 
    for (int i = 0; i < nbLines * nbColumns; i++) 
    { 
     printf("i = %d? ", i); 
     scanf("%f", &matrix[i]);  /* note what the second argument is */ 
    } 
} 

Answer 3

你不为你的数组分配足够的内存，因此你调用了Undefined Behavior，因为你超出了界限，导致了分段错误。

你可以宣布它像一个二维数组，像这样：

/* TODO: Check if allocation succeeded. (check for NULL pointer) */ 
float** matrix; 
matrix = malloc(nbLines * sizeof(float*)); 
for(int i = 0 ; i < N ; i++) 
    matrix[i] = malloc(nbColumns * sizeof(float)); 

我有一个动态分配here一个二维数组的其他方法。请注意：Do I cast the result of malloc?不！

也别忘了free()。

---

你可以模拟一个二维数组与一维数组，像这样：

void enterMatrix(float* matrix, int nbLines, int nbColumns){ 
    for (int i = 0; i < nbLines ; i++) { 
     for (int j = 0; j < nbColumns; j++) { 
     scanf("%f", matrix[i + nbColumns * j]); 
    } 
} 
float matrix[nbLines * nbColumns]; 

Answer 4

你要创建一个可变长度的数组，中，指针。

而其他的答案是完全有效的，如果你真的想一个二维数组，你只需要改变声明：

float matrix[nbLines][nbColumns]; 

申报花车的二维变长数组。

现在的难点是将这个VLA传递给一个函数并保留尺寸。

为此，您可以使用C99方式传递VLA（注意尺寸必须位于VLA本身之前）。参考：Passing a multidimensional variable length array to a function：

void enterMatrix(int nbLines, int nbColumns, float matrix[][nbColumns]){ 
    for (int i = 0; i < nbLines; i++){ 
     for (int j = 0; j < nbColumns; j++) 
    {    
     scanf("%f", &matrix[i][j]); 
    } 
    } 
} 

呼叫如下：

enterMatrix(nbLines, nbColumns, matrix);