入力した値とメモリ領域の先頭アドレスを画面表示する方法

Question

前提・実現したいこと

以下のような動作をするプログラムを作成したいのですが、4で入力した値を画面表示する方法がわかりません。
どのようにしたら良いでしょうか、教えていただけると嬉しいです。

1 整数型ポインタ変数pを定義する.
2 以下の3~6を無限ループで繰り返す.
3 整数型変数を格納するためのメモリ領域を malloc()を用いて作成し，先頭アドレスをポイ
ンタ変数 p に格納する.
4 キーボードから整数値を入力し，3で作成したメモリ領域に値を格納する.負の値が入力
された場合には，メモリ領域を解放した後でループを抜けて終了する.
5 4で入力した値と，メモリ領域の先頭アドレスを画面表示する.
6 3で作成したメモリ領域を開放する.

該当のソースコード

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

int main(void){

  int *p;

  while(1){
    p = (int *)malloc(sizeof(int));

    printf("数値を入力して下さい->");
    scanf("%d",p);

    if(*p<0){
    break;
    }
  }
  printf("%d,%p\n",*p,p);

  free(p);
  return 0;
}

現在の実行結果

数値を入力して下さい->3
数値を入力して下さい->4
数値を入力して下さい->5
数値を入力して下さい->-1
0,0x601070

Comments

[dodox86]

> 4で入力した値を画面表示する方法がわかりません。

質問としては上記ですが、現状で、6.が満たせていませんね。負の値が入力されたときにしかfree()されていません。

[choke]

6が満たせていなかったことに気付いていませんでした。ご指摘ありがとうございます。

Answer 1

free(p)した後で、printf("%d,%p\n",*p,p);のところで*pを参照しているのでエラーです。

Comments

[choke]

コメントありがとうございます。4で負の値が入力された場合はメモリ領域を解放した後でループを抜けて終了する.との指示があるのですが、この場合printfでの指定を*p以外に変更すれば条件は満たせるでしょうか..?

[otn]

解放したら、参照できないのでprintf出来ません。
そもそもこのプログラムは入力したデータを全部捨てているので、
データを全部捨てた後で何をprintfしたいのか、よく考える必要があります。

[choke]

確かにそうですね。今一度考え直して見たいと思います。ご丁寧にありがとうございました。

Answer 2

そのprintfの文を１行上に持っていけばどうでしょう（上のカッコの中）

Comments

[choke]

コメントありがとうございます。教えていただきました方法ですと
数値を入力して下さい->1
1,0x601010
数値を入力して下さい->2
2,0x601030
数値を入力して下さい->-1
という形になるのですがこれを
数値を入力して下さい->1
数値を入力して下さい->2
数値を入力して下さい->-1
1,0x601010
2,0x601030
という形にすることは可能でしょうか..?

[y_waiwai]

可能ですけど、あなたの質問文はそうなってませんよ