Cleaner Mise en œuvre de strtok

Question

Juste pour le plaisir, j'écris un programme qui prendra une chaîne entrée par l'utilisateur (ou peut-être même un document texte) et brouiller les mots dans la chaîne.

J'essaie d'utiliser la fonction strtok pour séparer chaque mot de la chaîne. En ce moment, je me sens comme ma mise en œuvre actuelle de strtok est bâclée:

int main(int argc, char *argv[]) 
{ 
    char *string, *word; 

    if(!(string = getstr()))  //function I wrote to retrieve a string 
    { 
     fputs("Error.\n", stderr); 
     exit(1); 
    } 
    char array[strlen(string) + 1];  //declare an array sized to the length of the string 
    strcpy(array, string);   //copy the string into the array 
    free(string); 
    if(word = strtok(array, " ")) 
    { 
     //later I'll just write each word into a matrix, not important right now. 
     while(word = strtok(NULL, " ")) 
     { 
     //later I'll just write each word into a matrix, not important right now. 
     } 
    } 
    return 0; 
} 

Je me sens comme il doit y avoir un moyen plus propre de la mise en œuvre strtok sans déclarer un tableau à mi-chemin à travers le programme. Cela ne me semble pas correct. Est-ce que l'utilisation du strtok est la bonne façon de procéder? Je préfère ne pas utiliser un tableau de taille fixe, comme j'aime tout être dynamique, c'est pourquoi je commence à douter en utilisant strtok est la bonne façon de procéder.

Answer 1

Si votre chaîne est malloced comme suggéré par votre libre. Ensuite, vous n'avez pas besoin de le copier dans un nouveau tampon (qui est btw 1 caractère trop court). Utilisez le tampon qui vous a été fourni.

Il vous suffit de le dupliquer s'il vous a été fourni par un const char *, c'est-à-dire que vous n'êtes pas autorisé à modifier le contenu du tampon.

Il est également préférable d'utiliser strtok_r car le strtok normal n'est pas réentrant.

Answer 2

Vous pouvez utiliser scanf() au lieu de getstr() et strtok()

char word[100]; 
while(scanf(" %s",word)!=EOF) { 
    // use the word string here 
} 

l'utilisateur doit arrêter chracters d'entrée avec

• EOF = CTRL + D (pour Linux)

• EOF = CTRL +Z (pour Windows)