中置記法の問題なのですが…

中置記法でかかれたもの後置記法に直してから計算するプログラムをつくったのですが、atoi関数のところで、うまく数字が渡されず、0となってします。これらの原因と解決方法、また解決したプログラムを教えてください。

#include<stdio.h>
#include<string.h>
#include<stdlib.h>
#include"Item1.h"
#include"Stack1.h"
#include"Item2.h"
#include"Stack2.h"

int main(int argc, char *argv[])
{
char *a = argv[1];
int i;
int N = strlen(a);
char r;
int s;

STACKinit1(N);STACKinit2(N);
for(i=0;i<N;i++)
{
if(a[i] == ')')
{
r = STACKpop1();
if(r == '+')
{
STACKpush2(STACKpop2() + STACKpop2());
}
else if(r == '*')
{
STACKpush2(STACKpop2() * STACKpop2());
}
}
if(a[i] == '+' || a[i] == '*')
{
STACKpush1(a[i]);
}

if ((a[i] >= '0') && (a[i] <= '9'))
{
s = atoi("a[i]");
STACKpush2(s);
}

}
printf("%d \n",s);
printf("%d \n",STACKpop2());
return 0;
}

No.1 回答者： honor 回答日時： 2012/11/16 07:15

>s = atoi("a[i]");

"a[i]"という文字列に数字は入ってないので変換不能なのでは。

この回答へのお礼

知識不足ですいません。
どのように直せばいいのでしょうか…

お礼日時：2012/11/16 08:05

No.2 回答者： maru_yoshi_ 回答日時： 2012/11/16 17:07

>>>s = atoi("a[i]");

>>"a[i]"という文字列に数字は入ってないので変換不能なのでは。
>
>知識不足ですいません。
>どのように直せばいいのでしょうか…

この部分に単純に回答すれば、
s = atoi(&a[i]);
でいいように思うが、
for(i=0;i<N;i++)
となっているので2ケタ以上の数値を演算する場合、うまくいかない。

No.3 回答者： tsuduki123 回答日時： 2012/11/16 21:01

atoi()の結果がおかしいのは ＃1の指摘通りではないかと思いますが

個人的には
atoi()でなくて、strtol()を使って'\0'まで処理するようにループを変更した方が
よいかと思います。

あと、この手のやつは、最初に字句解析(この場合、数字と演算子を分解)してから
計算を始めた方がきれいにいけますよ。

準備するスタックの量は演算子の数*2確保しておけば足ります。

No.4 回答者： kabe64 回答日時： 2012/11/25 11:40

皆さんの回答の通りですが一番簡単なのは

s = atoi("a[i]");

を

s = (int)(a[i] - '0');

に変更すれば終わると思いますよ。せっかくifで判定してるし。