文字列

質問

・数字文字列を数値化する関数AtoS()を制作する。

書式：short AtoS(char *pStr, int *pRetCode);
引数：char *pStr; 文字列の先頭アドレス
　　　int *pRetCode; 動作の正否を返す
戻り値:pStrを数値化した値
処理: pStrで与えられた文字列をshort型に変換する。
呼び出し側の書式は以下の通りです。
void main(void)
{
　short val;
int code;
val = AtoS("1234", & code);
printf("%d\n",val);
val = AtoS("-789", & code);
printf("%d\n", val);
}

です。

自分自身で、正の整数はできました。見てください。そして、負の整数や、「int *pRetCode」の使い方をおしえてください。

#include <stdio.h>
short AtoS(cahr *pStr, int *pRetCode);

void main(void)
{
short val;
int code;

val = AtoS("1234", & code);
printf("%d\n", val);
val = AtoS("-789", & code);
printf("%d\n", val);
}

short AtoS("char *pStr, int *pRetCode)
{
short suu;
suu = 0;
while("\n" != *pStr) {
suu = *pStr - '0' + suu * 10;
pStr++;
　　}
return(suu);
}

までしかできません。どなたか教えてください。

回答No.1

toysmith · ベストアンサー

まずは指摘から
while("\n" != *pStr) {
この判定は全くダメです。
・判定条件は*pStrが'0'～'9'の間にあることにすべき
・""は文字列のアドレスです。判定は文字と比べること

符号についてですが、変換ループに入る前に符号を判定しておく方が簡単です。
int sign = +1;
if (*pStr == '-') {
sign = -1 ;
pStr++ ;
} else if (*pStr == '+') {
pStr++ ;
}
pStrは符号を除く数字列の先頭を指すはずなので、taka1192さんが考えた方法で絶対値を10進数化してください（ループ終了条件の判定を変えることをお忘れなく）。

ループ終了後に*pStrが'\0'なら変換はオッケーです。
'\0'以外なら入力文字列が数字列でなかったわけだからエラーを返すことが出来ます。
int *pRetCodeはこういう場合にエラーを返すためにあるのではないでしょうか？

最後にsuuにsignを掛ければ変換終了します。