C言語 reallocの使い方

Question

int *sortindex = calloc(ndata, sizeof(int));の領域を拡張したい。

int *s = calloc(max + 1, sizeof(*sortindex));----------方法1

int *s = realloc(sortindex, sizeof(sortindex)(max + 1));------方法2

<質問>
方法2を方法1と同じ領域範囲の確保をしたいのですが、うまくいきません。方法をご教授お願いします。

試したこと
int *m;
int *s = realloc(sortindex, sizeof(sortindex)(max + 1));
m = s;

結果は方法1と同じになりませんでした。
<追記>

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define swap(type, x, y) \
    do                   \
    {                    \
        type t;          \
        t = x;           \
        x = y;           \
        y = t;           \
    } while (0)
/*--- 会員データ ---*/
typedef struct
{
    int no;        /* 番号 */
    char name[20]; /* 氏名 */
} Member;
/*--- 会員の番号の昇順比較関数 ---*/
int AscendingMemberNoCmp(const Member *x, const Member *y)
{
    return x->no < y->no ? -1 : x->no > y->no ? 1 : 0;
}
/*--- 会員の番号の降順比較関数 ---*/
int DescendingMemberNoCmp(const Member *x, const Member *y)
{
    return x->no < y->no ? 1 : x->no > y->no ? -1 : 0;
}
/*--- 会員の氏名の昇順比較関数 ---*/
int AscendingMemberNameCmp(const Member *x, const Member *y)
{
    return strcmp(x->name, y->name);
}
/*--- 会員の氏名の降順比較関数 ---*/
int DescendingMemberNameCmp(const Member *x, const Member *y)
{
    return strcmp(y->name, x->name);
}
/*--- 会員データ（番号と氏名）の表示（改行あり）---*/
void PrintLnMember(const Member *x)
{
    printf("%d %s\n", x->no, x->name);
}
/*--- 全データの表示 ---*/
void Print(const Member *data, int *m, int n)
{ //     Print(data, sortindex, ndata);
    int i;
    for (i = 0; i < n; i++)
        PrintLnMember(data + i);
}

void Number_compare(Member *a, int *m, int n, int compare(const Member *y, const Member *z))
{ //heapsort(data, sortindex, ndata, AscendingMemberNoCmp
    int i;
    int max = 9;
    int *s = calloc(max + 1, sizeof(*m));
    for (i = 0; i <= max; i++)
        s[i] = 0;
    /*度数分布表作成*/ //ok
    for (i = 0; i < n; i++)
        s[a[i].no]++;
    /*累積分布表作成*/ //ok
    for (i = 1; i <= max; i++)
    {
        s[i] += s[i - 1];
    }
    /*目的配列の作成*/
    Member *b = calloc(n + 1, sizeof(Member));
    for (i = n - 1; i >= 0; i--)
        b[--s[(a[i].no)]] = a[i];
    /*配列のコピー*/
    for (i = 0; i < n; i++)
        a[i] = b[i];
    if (compare == DescendingMemberNoCmp)
    {
        for (i = 0; i < (n / 2); i++)
            swap(Member, a[n - i - 1], a[i]);
    }
    /*解放*/
    free(b);
    b = NULL;
}

void Name_compare(Member *a, int *m, int n, int compare(const Member *y, const Member *z))
{ //heapsort(data, sortindex, ndata, AscendingMemberNoCmp
    int i;
    int max = 118; //uの10進数
    Member *b = calloc(max, sizeof(Member));
    int *s = calloc(max + 1, sizeof(*m));
    for (i = 0; i <= max; i++)
        s[i] = 0;
    /*度数分布表作成*/ //ok
    for (i = 0; i < n; i++)
        s[*(a[i].name)]++;
    /*累積分布表作成*/
    for (i = 1; i <= max; i++)
        s[i] += s[i - 1];
    for (i = n - 1; i >= 0; i--)
        b[--s[*(a[i].name)]] = a[i];
    /*配列のコピー*/
    for (i = 0; i < n; i++)
    {
        a[i] = b[i];
    }
    /*降順の場合、昇順で並べた配列を逆順にする*/
    if (compare == DescendingMemberNameCmp)
    {
        for (i = 0; i < (n / 2); i++)
            swap(Member, a[n - i - 1], a[i]);
    }
    /*解放*/
    free(b);
    free(s);
    b = NULL;
    s = NULL;
}
/*--- ヒープソート ---*/
void heapsort(Member *a, int *m, int n, int compare(const Member *y, const Member *z))
{ // heapsort(data, sortindex, ndata, AscendingMemberNoCmp);
    //ndata == 8
    if (compare == AscendingMemberNoCmp)
        Number_compare(a, m, n, AscendingMemberNoCmp);
    if (compare == DescendingMemberNoCmp)
        Number_compare(a, m, n, DescendingMemberNoCmp);
    if (compare == AscendingMemberNameCmp)
        Name_compare(a, m, n, AscendingMemberNameCmp);
    if (compare == DescendingMemberNameCmp)
        Name_compare(a, m, n, DescendingMemberNameCmp);
}
/*--- メニュー ---*/
typedef enum
{
    TERMINATE,
    ASCEND_NO,
    ASCEND_NAME,
    DESCEND_NO,
    DESCEND_NAME,
    PRINT_ALL
} Menu;
/*--- メニュー選択 ---*/
Menu SelectMenu(void)
{
    int i, ch;
    char *mstring[] = {
        "番号で昇順ソート", "名前で昇順ソート",
        "番号で降順ソート", "名前で降順ソート",
        "データを表示"};

    do
    {
        for (i = TERMINATE; i < PRINT_ALL; i++)
        {
            printf("(%2d) %-22.22s ", i + 1, mstring[i]);
            if ((i % 3) == 2)
                putchar('\n');
        }
        printf("( 0) 終了 ：");
        scanf("%d", &ch);
    } while (ch < TERMINATE || ch > PRINT_ALL);

    return (Menu)ch;
}
/*--- メイン ---*/
int main(void)
{
    Menu menu;
    Member data[] = {
        {1, "takahashi"}, {3, "konishi"}, {5, "ueda"}, {7, "sato"}, {9, "niimi"}, {8, "okonishi"}, {2, "motoike"}, {4, "agemi"}};
    int ndata = sizeof(data) / sizeof(data[0]);
    int i, *sortindex = calloc(ndata, sizeof(int));
    //int sortindex[8]
    for (i = 0; i < ndata; i++)
        sortindex[i] = i; /* data[]の添字を管理する配列*/
    do
    {
        switch (menu = SelectMenu())
        {
        case ASCEND_NO: /* 番号で昇順にソート */
            heapsort(data, sortindex, ndata, AscendingMemberNoCmp);
            break;
        case ASCEND_NAME: /* 名前で昇順にソート */
            heapsort(data, sortindex, ndata, AscendingMemberNameCmp);
            break;
        case DESCEND_NO: /* 番号で降順にソート */
            heapsort(data, sortindex, ndata, DescendingMemberNoCmp);
            break;
        case DESCEND_NAME: /* 名前で降順にソート */
            heapsort(data, sortindex, ndata, DescendingMemberNameCmp);
            break;
        case PRINT_ALL: /* 全データを表示 */
            Print(data, sortindex, ndata);
            break;
        }
    } while (menu != TERMINATE);
    free(sortindex);
    sortindex = NULL;
    return 0;
}

Name_compare関数の int *s = calloc(max + 1, sizeof(*m));をreallocで再現できないのかと思い
int *s = realloc(m,sizeof(m)(max + 1))
という領域確保がいいと思ったのですができませんでした。

Comments

[退会済みユーザー]

同じにならない、というのはどういう意味でしょう？
ちなみに、reallocで拡張された分は、callocと違い0で初期化されませんが。

[junnnnchan]

追記しました。
reallocを初めて使うので正しく理解できていません...
インターネットで調べたところ、mallocをreallocで拡張するというデータばかりだったので...

[y_waiwai]

うまく行かないとはどうなる？
問題がみえません

[SHOMI]

calloc:指定サイズのメモリを確保し0クリアする
realloc:確保済み領域のサイズを変更する(必要であれば再確保を行い、内容をコピーする)
用途が違いますが、一体何をしたいのでしょうか。
reallocだけではメモリ確保できず、先にmallocやcallocで確保しておく必要がありますよ。

[junnnnchan]

自分の説明不足でした。すいません。
main voidのほうで一回callocで領域を確保しています。Name_compareでmainで確保した領域よりも多くの領域が必要なので自分は関数内でもう一度callocで確保しなおしています。

callocで確保しなおしているのを、reallocで確保しなおしたいと思ってます。

[y_waiwai]

で、うまくいかないとはなにを見てうまくいかないと判断してる？

なにが問題なのかがわからない

Answer 1

reallocはmalloc+memmove+freeと、ほぼ等価です。

int *s = realloc(sortindex, sizeof(*sortindex)*(max + 1));

が成功しsがNULL以外だった場合、元のsortindexはfreeされている可能性があります。