ATOI機能がループ

Question

のためでは動作しません。私は、この式の出力を取得しようとしています：

44から10 + 11月5日

それは次のように正常に動作：

char str[] = "44 - 10 + 11/5"; 
int sum = 0 ; 
sum += atoi(str); // 0 + 44 
sum += atoi(str+3); // 44 + 10 
sum += atoi(str+5); // 54 + 11 
sum += atoi(str+7); // 65/5 = 13 
printf("%d/n",sum); // output = 13 

が、私はforループでatoi()機能を置けばそれは動作しません：

char str[] = "44 - 10 + 11/5"; 
int sum = 0; 
int i; 
sum += atoi(str); 
for (i = 0; i < 100; i++) { 
    if (!(str[i] >= 0 && str[i] <= 9)) { //if str[i] is not a number 
     sum += atoi(str + i); 
    } 
} 
printf("%d/n", sum); // output = 0 

Answer 1

数値のASCIIを含むchar値を実際の数値と比較しています。

if(!(str[i]>='0' && str[i]<='9')){ 

へ

if(!(str[i]>=0 && str[i]<=9)){ 

を変更し、残りのコードが正しいかどうかは確認しませんでしたが、確かに、これは問題の1つです。

Answer 2

if(!(str[i]>=0 && str[i]<=9)){ //if str[i] is not a number 

これは間違っています。 '0'と0は同じではありません。 9と9のどちらもありません。 '9'の値は、ASCIIチャートで '9'を表す整数になります。

論理の残りの部分は、例えば55があれば何が起きるのでしょうか、ifを55回、5回目は2回入力しますか？

Answer 3

これはちょっと奇妙なようです。目標は何ですか？

前のコメントに同意します。

なぜあなたは数字ではないのでatoi（）をやっていますか？

文字列の最後を通過します。 例のスペースCOUNTは

などとなります。 番号を見つけたらiを更新しないので、最初に44を44と数えます。

デバッガを学習し、コードをステップ実行し、データがどのように変化するかを確認したい場合は、こちらをご覧ください。

Answer 4

あなたのコードは以前のものと同じ計算をしないためです。最初のものでは、ポインタを次の番号に移動していて、それを番号に追加しているので、あなたのコードは次のようになります44 + 10 + 11 + 5。

2番目のケースでは、文字コードが0と9の間であるが、数字が48と57の間であることを確認しています - 文字 '0'と '9'。したがって、ifステートメントは常に間違っています。あなたの意図はおそらくこれでしたが、うまくいかないでしょう。

for(i = 0; i < 100 ; i++){ 
    if(!(str[i]>='0' && str[i]<='9')){ //if str[i] is not a number 
     sum += atoi(str+i); 
    } 
} 

また、これは、ここであなたは、次の番号の読み取りで、その結果、1によってポインタの間違ったシフトを行っているため、動作しません：44, 4, 10, 0, 11, 1, 5。 atoiプロパティのためです。you`dは、次のコードを記述する必要があります前にしていた正確に同じコードを実行するには、次のよう

int skip_non_diggits(int i, char *str) { 
    while (str[i] < '0' || str[i] > '9') { 
     if (str[i] == 0) 
      return -1; //end of string case 
     i++; 
    } 
    return i; 
} 

int skip_diggits(int i, char *str) { 
    while (str[i] >= '0' && str[i] <= '9') { 
     if (str[i] == 0) 
      return -1; //end of string case 
     i++; 
    } 
    return i; 
} 

をこれらの機能を使用することができる：

int i = 0, sum = 0; 
while (i != -1) { 
    sum += atoi(str+i); 
    i = skip_diggits(i, str); // positions i after the current number 
    i = skip_non_diggits(i, str); // positions i at the begin of the next number 
} 

ところで for (i = 0; i < 100; i++)を使用しないでくださいあなたがサイズを知らない弦を反復する。文字列の長さが100よりも短い場合は、メモリがデリフェアされている可能性があります。これはおそらく未定義の動作を生成するものではありません。代わりにstrlenメソッドをチェックするか、文字が\0と等しくないかどうかをチェックします。これはCで文字列の最後を示します。