デバックエラーが消せません・・・

Question

###プログラミング初心者です。
C言語で自作関数を用いて5桁の数字を入力して逆数を表示するプログラムを作っています。
（戻り値有りの自作関数を作成中です）
プログラムを作ってデバックをし、数字を入力したあとデバックエラーが出てしまいます。

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

エラーメッセージ
Debug Error!と出てきてしまいます

###該当のソースコード
#include<stdio.h>
int inverse(int a) {
int x;
int p, q[4];

for (p = 0; p<5; p++) {

	q[p] = a % 10;
	a /= 10;
}
x = 10000 * q[0] + 1000 * q[1] + 100 * q[2] + 10 * q[3] + q[4];
return x;

}

int main(void) {
int num;
printf("正の整数を入力：");
scanf("%d", &num);
printf("%dを逆から読むと%d", num, inverse(num));

return 0;

}

###試したこと
自作関数部にprintfをしてみたりmain関数の%d部分を変えてみたりしました

###補足情報(言語/FW/ツール等のバージョンなど)
C言語
戻り値なしの場合はうまくできたのですが、戻り値なし、仮引数を受け取らない場合エラーが出てきてしまいます
上のプログラム自体綺麗ではないとは思いますがどうかこの方法でどこが間違っているのか返答お願いします

Answer 1

直接的な回答は既に出ていますので、少し違った観点で。

この程度の簡単なバッファオーバーフローなどの実装ミスは、OSSの静的解析ツール CppCheck で検出可能です。

以下、掲載ソースをCppCheckでチェックした結果です。

[src.cpp:7]: (error) Array 'q[4]' accessed at index 4, which is out of bounds.
[src.cpp:10]: (error) Array 'q[4]' accessed at index 4, which is out of bounds.

Answer 2

int p, q[4];

qの配列を、q[4]と宣言していますので
q[0]～q[3]までとなります。

for (p = 0; p<5; p++) {
  q[p] = a % 10;    // <= p=4 の場合 q[4]
  a /= 10;
} 

で宣言以上のエリアへ値を代入しているため、
「Debug Error!」が出ていると思われます。

Comments

[KSehino]

q[4]ではq[0]~q[3]までしかできないのですね！
勉強になりました。詳しく教えていただきありがとうございました！

Answer 3

とりあえずC言語としての間違いは既出です。が、それを「逆数」というのは違うと思います。

Answer 4

int p, q[4];

5桁の数値を一桁ずつ配列に入れたいのですよね。q[4]で定義すると、q[0]～q[3]の4個しか確保されません。