Массив указателей на символы перезаписывается даже в том случае, если оператор IF ложен - PullRequest
Новая
       17

Массив указателей на символы перезаписывается даже в том случае, если оператор IF ложен

0 голосов
/ 31 марта 2019

Я разбиваю строку по указанному разделителю и пытаюсь изменить структуру typedef в зависимости от первой строки.Я понял, что когда я выполняю код, структура typedef перезаписывается, даже если оператор if принимает значение false.Я ожидал, что он сохранит свое старое значение из предыдущей итерации цикла, но это не так.
Я также думал о создании локальных символьных переменных, которые содержат значения из str, чтобы я соответствующим образом обновил их и присвоил их значенияcoord typedef.Но тогда создается впечатление, что я создаю слишком много переменных.

Я хочу, чтобы typedef обновлял заголовок только тогда, когда строка начинается с определенной строки.В противном случае выведите то, что было доступно в заголовке. 

typedef struct {
    char* utc;
    char* lat;
    char* lat_dir;
    char* lon;
    char* lon_dir;
    char* speed_kn;
    char* heading;
} CoordinatesHandleTypeDef;

const char *str_gprmc[7] = {
    "$GPRMC,125812.50,A,5741.1245547,N,01158.9460229,E,10.324,207.1,270319,0.0,E,A*0F",
    "$GPRMC,130019.00,A,5741.5393572,N,01158.6608248,E,14.013,331.8,270319,0.0,E,A*0F",
    "$GPRMC,130019.50,A,5741.5414303,N,01158.6591608,E,15.498,331.8,270319,0.0,E,A*07",
    "$GPHDT,3.0979,T*01",
    "$GPRMC,130132.00,A,5741.6055487,N,01158.3862843,E,9.536,174.0,270319,0.0,E,A*35",
    "$GPRMC,130132.50,A,5741.6042334,N,01158.3862498,E,10.783,172.1,270319,0.0,E,A*00",
    "$GPHDT,357.8596,T*06"
};

CoordinatesHandleTypeDef coord = {0};

    // Loop through for every string that comes in. Imitate USART End of Line....then process the string

    // We gonna keep the heading parameter out of the loop to get updated only once the data is available
    // This is because according to the datasheet, its updated onchanged. i.e when there is a detection in heading mismatches
    char* headingPrev = NULL;
    uint8_t* str[8] = { NULL };
    uint8_t temp[50] = { NULL };

    for (size_t k = 0; k < ARRAY_SIZE(str_gprmc); k++)
    {
        size_t maxStorableTokens = 0;

        // store the string in a local variable to avoid access violation exceptions for read/write operations
        strcpy(temp, str_gprmc[k]);

        for (uint8_t *ptr_token = strtok(temp, ","); ptr_token; ptr_token = strtok(NULL, ","))
        {
            if (maxStorableTokens >= 0x08) break;

            str[maxStorableTokens++] = ptr_token;
        }

        // if the string at index[0] is $GPRMC
        if (strcmp(str[0], "$GPRMC") == 0)
        {
            coord.utc       = str[1];
            coord.lat       = str[3];
            coord.lat_dir   = str[4];
            coord.lon       = str[5];
            coord.lon_dir   = str[6];
            coord.speed_kn  = str[7];

            coord.heading   = headingPrev;
        }
        else if(strcmp(str[0], "$GPHDT") == 0) 
            coord.heading = headingPrev = str[1]; 

        // Print out the updated contents of cood after every loop
        printf("UTC : %s, \tLONG : %s, \tLONG_DIR : %s, \tLAT : %s, \tLAT_DIR : %s, \tSPEED(Kn) : %s, \tHEADING : %s\n", 
            coord.utc, coord.lat, coord.lat_dir, coord.lon, coord.lon_dir, coord.speed_kn, headingPrev);
    }

Printout of Results

1 Ответ

1 голос
/ 31 марта 2019

У вас есть 

char* headingPrev = NULL;
    uint8_t* str[8] = { NULL };

Здесь headingPrev и str являются указателями, они могут содержать только адреса, а не фактические строки (или массив символов).Таким образом, при попытке 

coord.heading = headingPrev = str[1];

этот оператор headingPrev указывает только на str [1].Это эквивалентно #define headingPrev &str[1], что явно не является вашим намерением.Вам необходим headingPrev для хранения значения str [1], а не его адреса.Итак, вам нужно выделить память для заголовка.Вы можете сделать char HeadingPrev[20] или использовать malloc / calloc для динамического выделения памяти, если размер str [1] не фиксирован.Затем вы должны скопировать str [1] в headingPrev, используя strcpy().Таким образом, окончательный код будет 

 if (strcmp(str[0], "$GPRMC") == 0)
        {
            coord.utc       = str[1];
            coord.lat       = str[3];
            coord.lat_dir   = str[4];
            coord.lon       = str[5];
            coord.lon_dir   = str[6];
            coord.speed_kn  = str[7];

            coord.heading   = headingPrev; //<---- This would still work since headingPrev would be same as &headingPrev[0] even after allocating memory
        }
        else if(strcmp(str[0], "$GPHDT") == 0) 
        {
           strcpy(headingPrev, str[1]);
            coord.heading = headingPrev; 
        }

Вы также можете поместить coord.heading = headingPrev; вне блока if-else, так как он появляется в блоках if и else.

...