C言語 配列要素数の動的確保

Question

前提・実現したいこと

・実現したいこと
配列の要素数をループの回数にしたいことが目的とした数字を２つ入力してもらい、ファイルとターミナルに表示するプログラムの出力ミスの原因の理解
おそらく動的確保の方法が間違っているとは思いますが、考えてもわからなかったため質問させていただきます。

発生している問題・エラーメッセージ

数字1と2を入力形式（1,2）として入力してください（両方0入力で終了）: 1,1
数字1と2を入力形式（1,2）として入力してください（両方0入力で終了）: 1,1
数字1と2を入力形式（1,2）として入力してください（両方0入力で終了）: 1,1
数字1と2を入力形式（1,2）として入力してください（両方0入力で終了）: 0,0
x = 1, y = 1
x = 1, y = 1
x = 1, y = 1513857330
result.csv ファイルへの出力を終えました

エラーではありませんが、出力時にyの最後の値がおかしい値になっています。
また、4回以上入力をすると出力ファイルが開けないという結果になります。

該当のソースコード

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

int main(void){
    int *ipx,*ipy;                          //int型ポインタ
    int i,count;                            //カウンタ

    FILE *fp;                               //ファイルポインタ
    char *fname = "resultaaa.csv";             //出力ファイル名

    /* メモリ領域の確保 */
    ipx = (int *)malloc(sizeof(int));
    ipy = (int *)malloc(sizeof(int));

    /* 配列要素の格納、表示 */
    i = 0;
    while(1){
        printf("数字1と2を入力形式（1,2）として入力してください（両方0入力で終了）: ");
        scanf("%d,%d",&ipx[i],&ipy[i]);
        //終了条件
        if(ipx[i] == 0 && ipy[i] == 0)  break;
        i++;
    }

    /* ファイルオープン処理 */
    fp = fopen(fname,"w");
    if(fp == NULL){
        printf("%sファイルが開けませんでした。\n",fname);
        return -1;
    }

    /* ファイル、ターミナル出力 */
    count = 0;
    while(count < i){
        printf("x = %d, y = %d\n",ipx[count],ipy[count]);
        fprintf(fp,"%d,%d\n",ipx[count],ipy[count]);
        count++;
    }

    /* ファイルクローズ */
    fclose(fp);
    printf("%s ファイルへの出力を終えました。\n",fname);

    /* メモリ解放 */
    free(ipx);
    free(ipy);

    return 0;
}

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[episteme]

自己参照構造体の質問してるやん。コレ使えばいいのに。
https://teratail.com/questions/267119

Answer 1

ipx = (int *)malloc(sizeof(int));

ipy = (int *)malloc(sizeof(int));

では、int一個分の領域しか確保できていません。幾つ入力するか入力して、その分を確保しましょう。
動的に確保するならrealloc()等で、領域を増やすことも出来ます。（reallocはポインタが変わる可能性が有るので古いポインタを上書きしないように）
→確保したメモリ領域のサイズを変更する
ちなみに↑で解説している“if ((ptr = (int *)realloc(ptr, sizeof(int) * 15)) == NULL)”では、同じポインタ(ptr)を使っていますが、reallocに渡すptrと復帰値を受け取るptrは別物にして下さい。で、違った場合は古いptrを開放(free)して新しい方を使いましょう。

Comments

[退会済みユーザー]

回答ありがとうございます。こだわっているわけではありませんが、配列要素数を最初に決めない（ループの回数）ことにはこだわっております。realloc()という考え方は知らなかったため、実装をしてみようと思います。

[angel_p_57]

> で、違った場合は古いptrを開放(free)して新しい方を使いましょう。

古い方を呼び出し側でfree()するという決まりはなかったように思います。逆に変にfree()すると二重解放のおそれがあります。
https://linuxjm.osdn.jp/html/LDP_man-pages/man3/malloc.3.html より
> realloc() は、(略) ptr が指す領域が移動されていた場合は free(ptr) が実行される。
free() は realloc()内部で自動的に行われるように読めますね。

[退会済みユーザー]

ミスです。

[退会済みユーザー]

angel_p_57さんの参照しているサイトで自動で行われるように書いてあったため、記述しなかったところ正常に動きました。ご指摘ありがとうございます。
cateyeさんのご指摘のrealloc()をうまく使用し、複雑にはなってしまいましたが実現したいことが実現できました。本当にありがとうございました。

[cateye]

誤記（勘違い）が有りました。訂正します。

Answer 2

    /* メモリ領域の確保 */
    ipx = (int *)malloc(sizeof(int));
    ipy = (int *)malloc(sizeof(int));

それぞれint 1個分しか確保してない。
n組入力したいなら n個分確保しなさい。

Comments

[退会済みユーザー]

回答ありがとうございます。n組入力したいのですがipx = (int *)malloc(n*sizeof(int))としてその段階ではnの数が決まらない場合（ここではループの回数をnとしたいのですが）はこの定義の仕方でよいのでしょうか？

[episteme]

ダメに決まってます。
reallocするか、さもなくば
1 新たに(少し大きめの)領域を確保 2.旧領域から新領域へコピー 3.旧領域を廃棄

[退会済みユーザー]

簡潔にご指摘してくださりありがとうございます。解決することができました。