операция на «Я» может быть определено

Question

У меня есть этот код, чтобы взять строку формы bla_2 и отделить его:

void separate(char* str, char* word, int* n) { 
    int i = 0; 
    while(str[i] != '_') { 
     word[i] = str[i++]; 
    } 
    *n = str[++i] - '0'; 
} 

я получил:

warning: operation on ‘i’ may be undefined [-Wsequence-point] 

Но я только меняется i через ++ оператор, я ничего не приписываю.

Итак, почему UB, если это так? Если нет, как избавиться от предупреждения?

Обратите внимание, что на мой взгляд, вопрос this обрабатывает другую проблему.

Answer 1

word[i] = str[i++]; есть проблема.

Это выбор компилятора, если i в word[i] доступ до или после того, как i увеличивается на единицу в str[i++];

Выполните i++ после назначения

word[i] = str[i]; 
i++; 

---

Далее while(str[i] != '_') вероятно должен быть

while(str[i] != '_' && str[i] != '\0') 

для предотвращения переполнения буфера.

Answer 2

Причина в том, что обе стороны присваивания зависят от значения i, однако вы изменяете i в заявлении и нет точки последовательности между двумя видами использования i. Согласно стандарту C это приводит к неопределенному поведению.

Чтобы исправить это предупреждение, измените строку в вопросе

word[i] = str[i]; 
i++; 

Кстати, вы могли бы использовать указатели вместо явных индексов, которые, вероятно, сделать код немного более читаемым:

void separate(char* str, char* word, int* n) { 
    while(*str != '_') { 
     *word++ = *str++; 
    } 

    *n = *++str - '0'; 
}