文字列をasciiコード表の10進数に変換する

Question

初心者です。
C言語で入力した文字列をasciiコード表の10進数に変換するといったプログラムを作っていたのですが、
入力した文字列のあとに不要なものがついて出力されます。
おそらくfgets関数が原因だとは思ったのですが、改善方法をお願いします。

以下ソースコード

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

#define NUMBER 32

int ToDec(char pre);

int main(void){
	char pre[NUMBER];
	int i;
	
	fgets(pre, sizeof(pre), stdin);
	
	for(i=0; i<=sizeof(pre); i++){
		printf("%d ", ToDec(pre[i]));
	}
	printf("\n");
	
	return 0;
}

int ToDec(char p){
	int i, dec;
	dec = (int)p;
	
	return dec;
}

Answer 1

こんにちは。

fgets(pre, sizeof(pre), stdin);は、入力された文字列を最大preのバイト数(32バイト)まで読み取ります。
例えば、abc[Enter]と入力すると、preには下記のように入ります。
pre[0]='a';
pre[1]='b';
pre[2]='c';
pre[3]='\n';
pre[4]=0;
これ以降は不定値。

この状態でfor(i=0; i<=sizeof(pre); i++)にてループを回すと33回ループを周りますね。
i=5以降については不定値を処理しますので、わけの分からない値が出力されます。
更に、preの要素数は32個ですから、pre[0]～pre[31]までしか存在しません。i=32の時は不正な位置をアクセスしています。

ループの終了判定の工夫が必要です。'\n'と一致した場合もループを抜けると良いです。
例えばfor(i=0; (i<sizeof(pre)) && (pre[i]!='\n'); i++)としてみて下さい。

Answer 2

入力された文字列の末尾にってことでいいのかな？
fgets関数は改行まで読み取ってしまいます。
つまり

program

と入力すると配列には

program¥n

となります。なので改行を消せば求めてる答えがでるかと。

追記
forの部分見てなかった。それだとpre[]分回るから入力値以上は不定なるます

Comments

[d_red]

Chironianさんの解答をみて気づきました。
気を付けないと.....