C文字列を整数に変換する問題

StackOverflow https://stackoverflow.com/questions/1205945

  •  05-07-2019
  •  | 
  •  

質問

組み込みシステムで入力を解析しようとしています。 私はこのようなことを期待しています：

SET VARNAME=1,2,3,4,5,6,7,8,9,10\0

個別の文字列をintに変換するとき、文字列が8で始まる場合、 atoi（）と strtol（）の両方が0を返しているようです

ここに私のコードがあります：

char *pch, *name, *vars;
signed long value[256];
int i;

#ifdef UARTDEBUG
    char convert[100];
#endif
if(strncmp(inBuffer, "SET",3)==0)
{
    pch = strtok(inBuffer," ");
    pch = strtok(NULL," ");
    name = strtok(pch, "=");
    vars = strtok(NULL,"=");

    pch = strtok(vars,",");

    i = 0;
    while(pch != NULL)
    {
        value[i] = atoi(pch);
        #ifdef UARTDEBUG
            snprintf(convert, sizeof(convert), "Long:%d=String:\0", value[i]);
            strncat(convert, pch, 10);
            SendLine(convert);
        #endif
        i++;
        pch = strtok(NULL,",");

        // Check for overflow
        if(i > sizeof(value)-1)
        {
            return;
        }
    }    

    SetVariable(name, value, i);
}

渡す：

SET VAR=1,2,3,4,5,6,7,8,9,10\0

uartデバッグで以下を提供します：

Long:1=String:1                                                                
Long:2=String:2                                                                
Long:3=String:3                                                                
Long:4=String:4                                                                
Long:5=String:5                                                                
Long:6=String:6                                                                
Long:7=String:7                                                                
Long:0=String:8                                                                
Long:9=String:9                                                                
Long:10=String:10

更新：

'value [i] = atoi（pch）;'の前後でinBufferをチェックしました同一であり、適切なポイントに分割されているようです。

S  E  T     V  A  R     1     2     3     4     5     6     7     8     9  ,  1  0
53 45 54 00 56 41 52 00 31 00 32 00 33 00 34 00 35 00 36 00 37 00 38 00 39 2c 31 30 00 00 00 00

更新2：

現在のUARTDEBUGセクションの読み取り：

        #ifdef UARTDEBUG
            snprintf(convert, 20, "Long:%ld=String:%s", value[i], pch);
            SendLine(convert);
        #endif

snprintf（）の行をコメントアウトすると、すべてが完全に機能します。それで何が起こっているのですか？

役に立ちましたか？

解決

独自の atoi を作成してみませんか？ 10行の長さで、簡単にデバッグできます（そして問題が実際にどこにあるかを確認します）。

  • '0' = 0x30
  • '1' = 0x31

など、次のようなことを行う必要があります

string[x] - 0x30 * pow(10, n)

お持ちの数字ごとに

他のヒント

関係ありませんが、

if(i > sizeof(value)-1)
                {
                        return;
                }

あるべき

if(i == sizeof(value)/sizeof(value[0]) )
                {
                        return;
                }

他のコードがオーバーフローチェックを間違った方法で実行し、そのために文字列の一部が上書きされる場合、問題の原因になる可能性があります

自分のシステムでサンプルコードをコンパイルして実行しようとしました。出力は正しいです（つまり、出力文字列内の「8」が表示されます）。これは、提供したコードの範囲外で何かが行われていることを示しています。

手足に出て、変数または関数の1つが入力文字列または他の変数または配列を踏みつけていると言います。 SendLineとSetVariableは、見る場所です。

さらに重要なことは、問題の解決に役立つツールを提供していないことです。プログラムのデバッグの支援を依頼する場合は、問題を実証する完全なソースを含む簡単なテストケースを提供します。それ以外の場合は、問題が何であるかを推測する必要があります。これは私たちにとってイライラし、あなたにとって非生産的です。

atoiは、数値としてレンダリングできないものに対して0を返します-これは単なる手掛かりですが、文字列のバイナリ表現をダンプしようとしましたか（または、文字列の長さが一致することを確認しましたか）？

ライセンス： CC-BY-SA と 帰属
所属していません StackOverflow
scroll top