strtok не примет: char * str - PullRequest
Новая
       13

strtok не примет: char * str

3 голосов
/ 27 марта 2010

strtok не будет работать правильно, если в качестве первого параметра используется char * str (не строка разделителей).

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

заранее спасибо

пример: 

//char* str ="- This, a sample string.";   // <---doesn't work
char str[] ="- This, a sample string.";   // <---works
char delims[] = " ";
char * pch;
printf ("Splitting string \"%s\" into tokens:\n",str);
pch = strtok (str,delims);
while (pch != NULL)
{
  printf ("%s\n",pch);
  pch = strtok (NULL, delims);
}
return 0;

Ответы [ 3 ]

6 голосов
/ 27 марта 2010

В первом случае вы передаете строковый литерал в strtok (). Поскольку strtok () изменяет эту строку, а строковые литералы не могут быть изменены по закону, вы в конечном итоге получаете неопределенное поведение. Во втором случае компилятор копирует строку в массив. Содержимое массива можно изменить, поэтому этот код в порядке.

2 голосов
/ 27 марта 2010

strtok изменяет свой 1-й аргумент.

В вашем случае 1 аргументом strtok является строка literal, которая не может быть изменена и, следовательно, strtok завершается неудачей. Но в случае 2 аргумент представляет собой модифицируемый массив char, который strtok модифицирует и разбивает на более мелкие строки.

0 голосов
/ 12 апреля 2015

здесь код, все аспекты должны быть расфасованы.

  • получение указателя на символ
  • использование strdup для использования указателя на символ в strtok_r
  • использование strtok_r для обеспечения безопасности потоков
  • бесплатный результат strdup, когда он был сделан, потому что он использует malloc внутри

дай мне подсказку, если я что-нибудь забуду 

#include <string.h>
#include <stdio.h>
#include <stdlib.h>

#define WHITE " \t\n" // white space, tab and newline for tokenizing arguments
#define MAXARGC 50 // max number of arguments in buf 

void handlecommand(int argc, char *argv[])
{
    // do some handle code, in this example, just print the arguments
    for(int i = 0; i < argc; i++)
        printf("argv[%d]='%s'\n", i, argv[i]);
}

void parsecommand(char * cmdstr)
{
    char *cmdstrdup = strdup(cmdstr);
    if(cmdstrdup == NULL)
      //insuficient memory, do some errorhandling. 
      return; 
    char *saveptr;
    char *ptr;
    char *argv[MAXARGC]; 
    int argc;

    if((ptr = strtok_r(cmdstrdup, WHITE, &saveptr)) == NULL)
    {
        printf("%s\n", "no args given");
        return;
    } 

    argv[argc = 0] = cmdstrdup;
    while(ptr != NULL) {
        ptr = strtok_r(NULL, WHITE, &saveptr);
        if(++argc >= MAXARGC-1) // -1 for room for NULL at the end
            break;
        argv[argc] = ptr;
    }

    // handle command before free
    handlecommand(argc, argv);

    // free cmdstrdup, cuz strdup does malloc inside
    free(cmdstrdup);
}

int main(int argc, char const *argv[])
{
    parsecommand("command arg1 arg2 arg3\targ4\narg5 arg6  arg7");
    return 0;
}

Результат 

argv[0]='command'
argv[1]='arg1'
argv[2]='arg2'
argv[3]='arg3'
argv[4]='arg4'
argv[5]='arg5'
argv[6]='arg6'
argv[7]='arg7'
...