C言語のstrtolを使った関数について

Question

C言語について質問です。
下記のような簡単な数式プログラムに入力文字が数値か判断する関数を組み込んだところ、アクセス違反になってしまいます。どこを修正したら良いでしょうか。
また、入力もscanf_s以外のものを使ったほうがいいでしょうか。
よろしくお願いします。

#include <stdio.h>

int main(void)
{
	int tanka = 0;
	int suryo = 0;
	int kingaku = 0;
	int zeikomi = 0;
	int a = 0;
	int b = 0;

	printf("単価=");
	scanf_s("%d",&tanka);

	a = check_arg(tanka);

	printf("数量=");
	scanf_s("%d",&suryo);

	b = check_arg(suryo);

	kingaku = a * b;
	zeikomi = kingaku * 108 / 100;

	printf("金額=%d、税込み=%d",kingaku,zeikomi);
	return 0;
}

/*check_arg.c
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

void check_arg(char* input_value)
{
int input_num = 0;
char *endptr = NULL;

input_num = strtol(input_value, &endptr, 10);
if (endptr != '\0')
{
	printf("整数を入力してください");
	return -1;
}

return 0;

}

Answer 1

あなたの書いたcheck_arg関数は、文字列を引数として渡して、それが数字からなるかどうかをチェックしていますが、メイン関数では、scanf_s("%d" で整数を入力してその整数をcheck_arg関数に渡すという意味不明の事をしています。アクセス違反は当然です。

あと、check_arg関数の戻り値は何だと思っていますか？

自分の書いたプログラムを、それぞれの文が何をしているのか、声に出しながら確認して下さい。

Comments

[nkt070408]

回答ありがとうございます。
整数を文字列と比較してしまってエラーになっていることはわかりました。さらにcheck_arg関数で数値かどうか判断して返すはずがそもそも戻り地がないためやっていることが無意味でめちゃくちゃだと反省しました。
早速修正します。

Answer 2

エラーになる理由は、char*型の引数に対して、int型を渡しているからです。
そしてcheck_arg関数で行いたいのは整数か否かだと思われますが、入力をint型で受け取っている以上、正数値が格納されていますからstrtolでのチェックはできません。
scanfでは文字列で受け取ってから、strtolで変換、エラーチェック、という流れが適切です。

Comments

[nkt070408]

回答ありがとうございます。
以前のプログラムではchar型の配列に文字列を格納して使っていたのでそれと同じように中身を考えずに使ってしまったのが元々の原因でした。
入力値や型についてもっと理解を深めます。

Answer 3

この状況では、check_argを用意しても無意味です。

scanf_s("%d")で読み取る以上、そもそも数値しか読み取れません。

そして、char*の引数に無理やりintを渡したためにエラーとなっています。

Comments

[nkt070408]

回答ありがとうございます。
型が違うもの同士を比較しているため、アクセス違反が起きているということに気づけなかった自分が情けないです。
数値しか読み取れないところもわかっていなかったので勉強しなおしてきます。