スタックのエラーに関して

Question

配列を利用してスタックを作成し、実装するコードが書きたいのですが、以下のコードだとうまく行きません。
エラーで'push'の戻り値があっていないと言われているのはわかったので、
void push(int value);　の部分を　int push(void) に変換したら解決されると思ったのですが、そうすると今度は
stack_queue.c:26:18: error: too many arguments to function call, expected 0, have 1 push(data);
と言われてしまいました。

どのように訂正したらうまくいくのかを教えていただきたいです。よろしくお願い致します。

#include <stdio.h>

#define SIZE 5
int stack[SIZE];
int sp;

void push(int value);
int pop(void);

int main(void) {
    sp = 0;
    int resp, data;

    while(1) {
        printf("1:push 2:pop 0:end  :");
        scanf("%d",&resp);

        if(!resp) break;

        switch (resp) {
        case 1:
            printf("push:");    
            scanf("%d",&data);
            push(data);
            break;
        case 2:
            pop();
            break;
        }
    printf("sp=%d\n",sp);
    }
    printf("\n");
    for (int i=0; i<sp; i++) {
        printf("stack[%d]=%d \n",i,stack[i]);
    }
    return 0;
}

void push(int value) {
    if (sp >= SIZE) {
        printf("スタックが満杯で入りませんでした\n");
        return 0;
    } else {
        return stack[--sp];
    }
}

int pop(void) {
    if (sp<=0) {
        printf("スタックが空で取り出せませんでした\n");
        return 0;
    } else {
        return stack[--sp];
    }
}

stack_queue.c:44:9: error: void function 'push' should not return a value [-Wreturn-type]
        return 0;
        ^      ~
stack_queue.c:46:9: error: void function 'push' should not return a value [-Wreturn-type]
        return stack[--sp];
        ^      ~~~~~~~~~~~
2 errors generated.

Answer 1

void push(int value);　の部分を　int push(void) に変換したら解決されると思ったのですが

そういう小手先の話ではなくて、もうちょっと上の視点で考えたほうが良いです。

push という関数の役割は何で、
それにはどのような機能が必要で、
それにはどのような入出力が必要なのか？

って点を一度整理してみてから、関数の形(戻り値や引数の型等)を考えれば、迷う事は少なくなるかと思います。

Comments

[Red_Bull]

push関数の役割をきちんと理解できていませんでした。

Answer 2

void push(int value)

void なんだから値を返してはいけません。

Comments

[Red_Bull]

ですよね...どのように訂正したらいいのかがわかりません。
教えていただけますか？

[episteme]

単に return; って書けばいい。
# マチガイは他にもあるけど...