stackの使い方

Question

数値を入力して下さい（正：格納、負：取出）-> 56
num=1 [56 ]
数値を入力して下さい（正：格納、負：取出）-> 83
num=2 [56 83 ]
数値を入力して下さい（正：格納、負：取出）-> 27
num=3 [56 83 27 ]
このように表示したいのですが、実行結果がnum1=[56] num2=[83] num3=[27]というようになってしまいます。どうすれば、良いですか？
stackの配列の使い方について教えてください。
----------------------------------------------------------c言語
コード
#include <stdio.h>
#define N 10

int func(int);

int main()
{
int n,r;

while(1) {
printf("数値を入力して下さい（正：格納、負：取出）-> ");
scanf("%d",&n);
if (n==0) break;
r=func(n);/* スタックに格納または取出*/
func(0); /* スタックを表示 /
if (r>0) printf("取り出したデータ: %d\n",r); / 取得データの表示 */
else if (r==-1) printf("エラー（データがありません）\n");
else if (r==-2) printf("エラー（オーバーフロー）\n");
}
return 0;
}

/* 引数：：正の整数：スタックに格納、負の整数：スタックから取出
０：スタックの中身を表示
戻り値：スタックから取り出した値。取出時スタックが空なら-1,
格納時スタックがいっぱいのとき-2、表示の際はいつも0 を返す*/
int func(int x)
{
static int stack[N]; /* スタック本体 /
static int num=0; / データ数 */
int i;
if(x>0){
//stack[num]=x;
num++;
stack[num]=x;
for(i=0;i<num;i++){

 }  printf("num=%d [%d]\n",num,stack[num]);

}

else if(x<0){
num--;
x=stack[num];
return x;
}
}

Comments

[fuzzball]

コードは ``` で囲って下さい。

Answer 1

未実装の部分はこれから作ると仮定して、格納時の処理のみ修正します。

配列の添字は0から始まります

num++; 
stack[num]=x;

これだと、最初に格納されるのが[1]になってしまいますので、コメントアウトしている方が正解ということになります。

stack[num]=x;
num++; 

出力(その1)

配列の中身を出力するところがstack[num]となっており、これでは全て最後に追加した値だけ出力されてしまいます。stack[i]（添字にループカウンタiを指定）が正解です。

出力(その2)

繰り返さない部分（データ数や括弧）はforループの外に出しましょう。

printf("ヘッダ");
for (...) {
    printf("中身");
}
printf("フッタ");

まとめ

if(x > 0) { 
    stack[num] = x; 
    num++;
    //header 
    printf("num=%d [", num);
    //values
    for(i = 0; i < num; i++) {
        printf("%d", stack[i]); /*おまけ開始*/
        if (i == num-1) {
            /*最後なら閉じ括弧(footer)*/
            printf("]\n");
        } else {
            /*途中はスペース(separator)*/
            printf(" ");
        }                       /*おまけ終了*/
    }
    return 0;/*とりあえず*/
}

おまけ

中身の出力部分（コメントのおまけ開始〜おまけ終了まで）は、三項演算子を使って一行で書くことも出来ます。

printf("%d%s", stack[i], (i == num-1)? "]\n": " ");

Comments

[ikuo-biyori]

お返事ありがとうございます。わかりました。

[ttyp03]

格納（引数の値が正）のときに表示までするんですか？
func(0)の意味がなくないですか？
なんだかなぁ。

[fuzzball]

表示がおかしいという質問なので、それに対しての回答です。回答の冒頭に「未実装の部分はこれから作ると仮定して」と断っていますが。

[ttyp03]

あ、すみません。
fuzzballさんよりかは質問者さん向けのコメントでした。

[ikuo-biyori]

>ttyp03さん、すいません。問題をよく見ていませんでした。func(0)はstackを表示する処理を行うものでした。おかげさまで、解決しました。ありがとうございました。

Answer 2

func関数のコメントに書かれていることが実装されていませんね。
戻り値を返さないルートも存在しています。
またA.Ichiさんの回答にあるように、格納開始が[1]から始まってしまっています。
func関数を見直したコードを貼っておきます（未検証）

int func(int x) 
{ 
	static int stack[N]; /* スタック本体 */ 
	static int num=0;  /* データ数 */ 
	int i; 
	int r = 0;

	// 格納
	if( x > 0){ 
		if(num >= N){
			r = -2;		// オーバーフロー
		} else {
			stack[num]=x; 
			num++; 
		}

	// 取出
	} else if(x < 0){ 
		if(num == 0){
			r = -1;	// データなし
		} else {
			num--; 
			r = stack[num]; 
		}

	// 表示
	} else {
		printf("num=%d [", num);
		for(i = 0; i < num; i++){
			printf("%d ", stack[i]);
		}
		printf("]\n");
	}

	return r; 
}

Answer 3

表示の部分だけです、下記の部分を変更してみました。スタック機能全体としては見れていません、stack配列が1から始まっている様です？

    if(x>0){
        //stack[num]=x;
        num++;
        stack[num]=x;
        printf("num=%d [",num);
        for(i=1;i<=num;i++){
            printf("%d ",stack[i]);
        }
        printf("]\n");

    }