1

マトリックスの最初の要素を入力した後、「セグメンテーション エラー (コア ダンプ)」が表示されます。物理的にメモリにない何かにアクセスしようとすると、セグメンテーション違反が発生することはわかっていますが、ここでこのエラーが発生する理由がわかりません。

ポインタの使い方を学んでいるので、わざとポインタを使っています。

#include<stdio.h>
#include<stdlib.h>
void main()
{
    int i, j, m, n;
    int **p, **q, **res;
    p = (int**) malloc(10 * sizeof(int));
    q = (int**) malloc(10 * sizeof(int));
    res = (int**) malloc(10 * sizeof(int));
    printf("Enter the number of rows and columns:");
    scanf("%d %d", &m, &n);
    printf("Enter the elements of the matrix\n");
    for(i=0;i<m;i++) 
    {
        for(j=0;j<n;j++) 
        {
            scanf("%d", &(*(*(p+i)+j)));
        }
    }

    for(i=0;i<m;i++) 
    {
        for(j=0;j<n;j++)
        {
            printf("%d      ", (*(*(p+i)+j)));
        }
        printf("\n");
    }
}
4

3 に答える 3

4

これは、 orp内のデータに実際にメモリを割り当てていないためです。10 個の整数にサイズを割り当てますが、最初に 10 個の整数ポインターに割り当ててから、その中のデータに割り当てる必要があります。qres

このように：

/* Get `m` and `n`... */

p = malloc(m * sizeof(int *));
for (i = 0; i < m; i++)
    p[i] = malloc(n * sizeof(int));

もちろん、これは他の人に対しても行う必要があります。

また、配列と同じ構文を使用してこれらにアクセスできることをご存知ですか? ポインター演算またはポインター逆参照の必要はありません。シンプルなだけでp[i][j]うまくいきます。

于 2013-08-02T12:53:58.123 に答える
1

コード内のさまざまなエラー。
変更へのコメントをインライン化しました。と を削除qしresました。

コードには 2 つのバリアントがあり、1 つは size のメモリの単一の「ブロック」をm * n使用し、もう 1 つは size のメモリの 1 つのブロックを使用して sizeのメモリの他のブロックへのポインタをm保持します。mmn

m * nm 行ごとに n が一定の場合に便利なサイズのメモリの単一「ブロック」を使用する

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

void main()
{
    int i,j,m,n;

    /* Changed to *p. Instead of an array of arrays 
       you'll use a single block of memory */
    int *p;

    printf("Enter the number of rows and columns:");
    scanf("%d %d",&m,&n);

    /* m rows and n columns = m * n "cells" */
    p = (int*) malloc(m * n * sizeof(int));

    printf("Enter the elements of the matrix\n");

    for (i=0;i<m;i++)
    {
        for (j=0;j<n;j++)
        {
            /* the element at row i and column j is the (i * m) + j 
               element of the block of memory, so p + (i*m) + j .
               It's already an address to memory, so you don't need the & */
            scanf("%d", (p + (i*m) + j));
        }
    }

    for (i=0;i<m;i++)
    {
        for (j=0;j<n;j++)
        {
            /* same as before, but this time you have 
               to dereference the address */
            printf("%d      ", *(p + (i*m) + j));
        }

        printf("\n");
    }
}

mmmnm 行ごとに n が可変の場合 (「ギザギザ」配列) に役立つサイズのメモリの他のブロックへのポインターを保持するために、サイズのメモリの 1 つのブロックを使用する

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

void main()
{
    int i,j,m,n;
    int **p;

    printf("Enter the number of rows and columns:");
    scanf("%d %d",&m,&n);

    /* We will have m "rows", each element a ptr to a full row of n columns 
       Note that each element has size == sizeof(int*) because it's a ptr
       to an array of int */
    p = (int**) malloc(m * sizeof(int*));

    printf("Enter the elements of the matrix\n");

    for (i=0;i<m;i++)
    {
        /* For each row we have to malloc the space for the 
           columns (elements) */
        *(p + i) = (int*)malloc(n * sizeof(int));

        for (j=0;j<n;j++)
        {
            /* Compare the reference to the one before, note
               that we first dereference *(p + i) to get the
               ptr of the i row and then to this ptr we add
               the column index */
            scanf("%d", *(p + i) + j);
        }
    }

    for (i=0;i<m;i++)
    {
        for (j=0;j<n;j++)
        {
            /* Note the double dereferencing, first to the address 
               of the row of data
               and then to the exact column */
            printf("%d      ", *(*(p + i) + j));
        }

        printf("\n");
    }
}
于 2013-08-02T13:02:45.747 に答える