0

我有三个文件：

某事.h

typedef struct {
    int size_t;
    char *c;
} p;

p ** createMatrix(int r, int c);

void printMatrix(p **matrix, const int r, const int c);

某事.c

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

p **
createMatrix(int r, int c)
{
    int rowsize=r*2+1;
    int colsize=c*2+1;

    /* memory alloc of rows */
    p **matrix=malloc(rowsize*sizeof(p *));
    int k;
    for(k=0; k<rowsize; k++)
    {
        /* memory alloc of columns */
        matrix[k]=malloc(colsize*sizeof(p));
    } 
    int i, j;
    for(i=0; i<rowsize; i++)
    {
        for(j=0; j<colsize; j++)
        {
            /* columns is between letters */
            if(j%2!=0)
            {   
                matrix[i][j].size_t=7;
                matrix[i][j].c=malloc(8*sizeof(char));
                strcpy(matrix[i][j].c,"       ");
            }
            else if(i%2==0 && j%2==0)
            {
                matrix[i][j].size_t=1;
                matrix[i][j].c=malloc(sizeof(char));
                *(matrix[i][j].c)='a';
            }
            else
            {
                matrix[i][j].size_t=1;
                matrix[i][j].c=malloc(sizeof(char));
                *(matrix[i][j].c)=' ';
            }
        }
    }       
    return matrix;
}

void
printMatrix(p **matrix, const int r, const int c)
{
    int rowsize=r*2+1;
    int colsize=c*2+1;
    printf("\n");
    int i, j;
    for(i=0; i<rowsize; i++)
    {
        printf("\t");
        for(j=0; j<colsize; j++)
        {
            if(matrix[i][j].size_t==1)
                printf("%c", *matrix[i][j].c);
            else
                printf("%s", matrix[i][j].c);

        }
        printf("\n");
    }
    printf("\n");
 }

主程序

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

void
printMenufunction(p **matrix, int rows, int cols)
{
    int rowsize=rows*2+1;
    int colsize=cols*2+1;
    /* memory alloc of rows */
    matrix=malloc(rowsize*sizeof(p *));
    int i;
    for(i=0; i<rowsize; i++)
    /* memory alloc of columns */
        matrix[i]=malloc(colsize*sizeof(p));
    matrix=createMatrix(rows, cols);
}

int
main(void)
{
    int rows, cols;
    p **matrix;
    char stringtemp[3];
    printf("ask for row and col in form (5x5):\n");
    scanf("%s", stringtemp);
    sscanf(stringtemp, "%dx%d", &rows, &cols);
    printMenufunction(matrix, rows, cols);
    printMatrix(matrix, rows, cols);
    return 0;
}

这个代码示例是我的 prog 的简化版本，它给了我一个分段错误。我已经用 gdb 对其进行了调试，所以我知道它在这些行中的某个地方是故障所在。我将不胜感激，谢谢。

更新：我无法调试此版本。我有更多代码的旧编告诉我它在这个区域，但至少我不知道在哪里？所以这就是为什么我不能发布导致分段错误的确切行。如果我知道我想我可以自己解决这个问题。我希望有人能比我看得更清楚。。

4

2 回答 2

1

由于 C 是一种按值传递的语言，对函数参数的任何赋值都只会修改参数的本地副本，因此当printMenufunction返回时，matrix仍然没有初始化。

您需要通过引用将指针传递给矩阵：

printMenufunction(&matrix, rows, cols);
printMatrix(&matrix, rows, cols);

并在函数内部取消引用。例如：

//                       v Pointer to the double-pointer
void printMenufunction(p ***matrix, int rows, int cols)
{
    int rowsize=rows*2+1;
    int colsize=cols*2+1;
    /* memory alloc of rows */
//  v Dereferencing the pointer
    *matrix=malloc(rowsize*sizeof(p *));
   // ...
}
于 2012-11-07T20:35:56.427 回答
1

首先，告诉我们gdb指出的哪一行有问题，而不是仅仅说“它在这些行中的某个地方”，这对我们很有帮助。gdb 应该能够准确地告诉您是哪一行触发了错误。

话虽如此，这里有一个候选人：

char *stringtemp;
printf("ask for row and col in form (5x5):\n");
scanf("%s", stringtemp);

stringtemp没有被分配到任何有意义的地方；您所拥有的只是一个初始值不确定的指针。所以这是一个潜在的段错误。stringtemp为指向的缓冲区分配一个缓冲区，或者将其分配为一个数组char：

char stringtemp[SOME_SIZE]; // where SOME_SIZE is as big as you need.

你为什么要分配矩阵printMenuFunction？！你为什么不使用你的createMatrix功能？！

编辑

我讨厌变老；我完全错过了createMatrix被调用的printMenuFunction那个（这意味着malloc调用printMenuFunction是多余的，需要消除）。

重写你printMenuFunction的如下：

void printMenuFunction(p ***matrix, int rows, int cols)
{
  *matrix = createMatrix(rows, cols);
}

请注意，在该函数中，如果您需要访问 的元素matrix，则需要在应用下标之前取消引用它，例如

(*matrix)[i][j].size = ...;

重写你main的如下：

int
main(void)
{
    int rows, cols;
    p **matrix;
    char stringtemp[3]; // 3 is not big enough to hold a string like "5x5";
                        // that requires an array size of *at least* 4.  
                        // "10x10" would require 6 characters.  You need
                        // an extra character for the 0 terminator.

    printf("ask for row and col in form (5x5):\n");
    scanf("%s", stringtemp);
    sscanf(stringtemp, "%dx%d", &rows, &cols);
    printMenufunction(&matrix, rows, cols); // note the & in front of matrix
    printMatrix(matrix, rows, cols);
    return 0;
}
于 2012-11-07T20:42:58.533 回答