Заменить символ ошибки сегментации языка c strchr

Question

Мне нужно заменить каждое вхождение '&' to ',' в строке C.

Я сделал это, и это работает

Код 1:

char *val, *querydup;
.
.
.
val=strchr(querydup,'&');
while(val != NULL) {
*val=',';
val=strchr(querydup,'&');
}

Для того, чтобы быть "элегантным", я попробовал следующее, но это приводит к ошибке сегмента, к точке, где даже мой указатель курсора поврежден !. странно, я запускаю Linux внутри VMware VM.

Код 2:

while(val=strchr(querydup,'&') != NULL) {
*val=',';
}

Так что может быть не так? ..

Считаете ли вы код 1 "элегантным"? ...

Привет.

Answer 1

Ваш главный вопрос получен ответ от emboss .

Я отвечаю на часть "элегантность":)

Вы проверяете одни и те же символы снова и снова.

Предположим, querydup имеет "&&&&&&&&&&". После первых нескольких изменений он имеет ",,,,,,&&&&", и все равно вы все равно проверяете с самого начала.

Попробуйте повторно использовать val для "элегантности":)

val = strchr(val, '&');

Answer 2

Это приоритет оператора , попробуйте

while((val=strchr(querydup,'&')) != NULL) {
    *val=',';
}

Answer 3

Возможно, более простым решением было бы на самом деле сделать это вручную:

char *ptr = querydup;
while (*ptr) {
    if (*ptr == '&') {
        *ptr = ',';
    }
    ptr++;
}

Это больше строк кода, но он проходит через строку только один раз, где повторные исполнения strchr () пройдутстрока несколько раз.

Answer 4

Вы присваиваете val логическое значение.Ваш код эквивалентен:

while(val = (strchr(querydup,'&') != NULL))

Измените его на:

while((val = strchr(querydup,'&')) != NULL)