C Преобразовать строку в выпуск Ints - PullRequest
Новая
       11

C Преобразовать строку в выпуск Ints

1 голос
/ 30 июля 2009

Я пытаюсь проанализировать некоторые входные данные во встроенной системе. Я ожидаю что-то вроде этого: 

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

Когда я конвертирую отдельные строки в целые, и atoi(), и strtol(), похоже, возвращают 0, если строка начинается с 8.

Вот мой код: 

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

UPDATE:

Я проверил inBuffer как до, так и после 'value [i] = atoi (pch);' и он идентичен и, кажется, был разделен на правильную точку. 

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(), все работает отлично. Так что с этим происходит?

Ответы [ 4 ]

1 голос
/ 02 августа 2009

Я только что попытался скомпилировать и запустить ваш пример кода на моей собственной системе. Вывод правильный (т. Е. «8» появляется там, где он должен быть в выходной строке), что указывает мне, что что-то еще происходит вне области кода, который вы нам предоставили.

Я собираюсь выйти на конечность и сказать, что одна из ваших переменных или функций попирает вашу входную строку или некоторую другую переменную или массив. SendLine и SetVariable - это места для поиска.

Но что более важно, вы не дали нам инструменты, которые помогут вам решить вашу проблему . Когда вы просите людей помочь вам отладить вашу программу, предоставляет простой тестовый пример с полным исходным кодом, который иллюстрирует проблему . В противном случае нам остается только догадываться, в чем заключается проблема, которая расстраивает нас и бесполезна для вас.

1 голос
/ 30 июля 2009

Не связано, но 

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

должно быть 

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

Может быть причиной проблемы, если другие части кода неправильно выполняют проверку переполнения и из-за этого перезаписывают часть вашей строки

1 голос
/ 30 июля 2009

ты не можешь написать свой собственный atoi? он длиной в десять строк, и тогда вы можете легко его отладить (и проверить, где проблема на самом деле)

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

и так далее, вам просто нужно сделать что-то вроде 

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

для каждой имеющейся у вас цифры

0 голосов
/ 30 июля 2009

atoi возвращает 0 для чего-то, что он не может отобразить как числовой - это всего лишь догадка, но вы пытались сбросить двоичное представление строки (или даже проверить, совпадают ли длины строки)?

...