warning strtok

@stratovarius · Регистрация: 12.07.2009

Пишу в 10 студии:

C++

1	char* bla = strtok("bla bla bla", " ");

Выдаёт варнинг: This function or variable may be unsafe. Consider using strtok_s instead.... И пры выполнении программы на этой строке выкидывает

@Chelioss · 13.10.2011, 00:06

Строка "bla bla bla" является константной. А функция strtok изменяет строку. Получается, что strtok пытается изменить константную строку. Поэтому и ошибка.
Варнинг говорит, что эта функция может быть опасной. Действительно, она изменяет строку, которую получает первым параметром.

@stratovarius · 13.10.2011, 00:13 **[ТС]**

если объявить

C++

1 2	char* bla = "bla bla bla"; char* bla2 = strtok(bla, " ");

без слова const, то тот же варнинг

@Chelioss · 13.10.2011, 00:19

Еще раз повторю: варнинг не имеет отношение к этой ошибке. Варнинг просто есть и все. Просто предупреждение для программиста.
А то что вы и так и так написали, то разницы нет. Вы и в том и в том случае пишите "bla bla bla".
Во втором случае используете указатель на эту строку. Но если указатель

C++

char *bla

, т.е. не

C++

1	const char *bla

, то это не значит, что данные можно изменять.
Дело не в указателе bla, а в строку "bla bla bla"

@stratovarius · 13.10.2011, 00:33 **[ТС]**

у меня есть класс, в конструкторе я делаю

C++

BufferCreator::BufferCreator(char* str)
{
    for (char* rule = strtok(str, ";"); rule != NULL; rule = strtok(NULL, ";"))
    {
        strcat(res, rule);
    }
 
}

и сразу выбивает ошибку, внутрь фора даже не заходит... как исправить?

@Chelioss · 13.10.2011, 00:35

Указатель на что указывает? Покажите то место, где вы создаете строку и вызываете этот метод.
И еще, зачем вы каждый раз в цикле вызываете strtok(str, ";"); ?
Достаточно только 1 раз вызвать, а потом вызывать strtok(NULL, ";");.
В итоге так должно быть:

C++

BufferCreator::BufferCreator(char* str)
{
        for (char* rule = strtok(str, ";"); rule != NULL; rule = strtok(NULL, ";"))
        {
                strcat(res, rule);
        }
 
}

@aeshes · 13.10.2011, 00:43

Это же у вас конструктор, и вы туда наверняка передаете строку в кавычках

C++

1	BufferCreator a("aa;bb");

так?
А внутри функции пытаетесь изменить эту строку "aa;bb", но так делать нельзя
попробуйте что-то типа

C++

BufferCreator::BufferCreator(char* str)
{
    char *temp=new char[strlen(str)+1];
    strcpy(temp, str);
 
        for (char* rule = strtok(str, ";"); rule != NULL; rule = strtok(NULL, ";"))
        {
                strcat(res, rule);
        }
 
}

@Chelioss · 13.10.2011, 00:45

Так правильнее

C++

BufferCreator::BufferCreator(char* str)
{
        char *temp=new char[strlen(str)+1];
        strcpy(temp, str);
 
        for (char* rule = strtok(temp, ";"); rule != NULL; rule = strtok(NULL, ";"))
        {
                strcat(res, rule);
        }
        delete [] temp;
 
}

@stratovarius · 13.10.2011, 00:46 **[ТС]**

точно) просто я думал, что если параметр конструктора char*, то даже если передадим строку, он не будет константным... спасибо!

@aeshes · 13.10.2011, 00:57

в спешке забыла про delete. точно(

@Thinker · 13.10.2011, 09:28

Добавлю, что strtok заменяет символы разделители на символ '\0', а если объявить строку типа
char *s = "abc...",
то компилятор может ее поместить в область памяти read only (только для чтения) и изменять символы будет нельзя, поэтому запись strtok(s, " ") ошибочна.

@Chelioss 185 / 185 / 21 Регистрация: 08.01.2011 Сообщений: 1,139
	13.10.2011, 00:06	2
	Строка "bla bla bla" является константной. А функция strtok изменяет строку. Получается, что strtok пытается изменить константную строку. Поэтому и ошибка. Варнинг говорит, что эта функция может быть опасной. Действительно, она изменяет строку, которую получает первым параметром. 1

@stratovarius 14 / 14 / 3 Регистрация: 12.07.2009 Сообщений: 114
	13.10.2011, 00:46 [ТС]	9
	точно) просто я думал, что если параметр конструктора char*, то даже если передадим строку, он не будет константным... спасибо! 0

@aeshes 446 / 209 / 21 Регистрация: 07.10.2011 Сообщений: 462
	13.10.2011, 00:57	10
	в спешке забыла про delete. точно( 0

@Thinker 4254 / 2228 / 203 Регистрация: 26.08.2011 Сообщений: 3,802 Записей в блоге: 5
	13.10.2011, 09:28	11
	Добавлю, что strtok заменяет символы разделители на символ '\0', а если объявить строку типа char *s = "abc...", то компилятор может ее поместить в область памяти read only (только для чтения) и изменять символы будет нельзя, поэтому запись strtok(s, " ") ошибочна. 0