使用指向字符的指针并编写用于分隔文本单词的函数

Question

我想写一个函数，它将把一个指向char的指针和一个int（扫描开始索引）作为输入，并返回它之后的第一个单词。这个代码有什么问题？没有字返回。创建了从何处包含分隔符之一的问题。涂鸦字符。

char *setWord(char string[], int *n) {
    char word[20] = "\0";
    string += *n;
    while (*string == ','||*string == '\t' || *string == '\n'
    || *string == ' '|| *string == '.' || *string == '!' || *string == '?') {
        string++;
       (*n)++;
    }
    int i = 0;
    while (*string != ',' && *string != '\t'
                     && *string != '\n' && *string != ' ' &&
                     *string != '.' && *string != '!' &&
                     *string != '?') {
        word[i] = *string;
        string++;
        (*n)++;
        i++;
    }
    char *words = word;


    return words;
}

Answer 1

您希望从函数返回一个C字符串（[指向]以null结尾的字符数组的指针）。问题是数组不是C中的第一类对象，所以只能传递和返回指针。任何自动数组都将在声明它的块的末尾达到其生命周期的终点。因此，如果您很好地将word构建成一个自动数组，并返回指向它的指针（当前代码所做的），那么当函数返回时只留下一个悬空的指针，数组就会被破坏。

有两种可能的方法，每种方法都有其优缺点：

>

它可以是递归的和线程安全的。简单地说，您必须记录调用者将负责释放内存块，以及它应该如何完成：如果您使用malloc，删除[]如果您使用new[]。

使用静态记忆

一个快速解决方法是写：

static char word[20];

静态数组将持续到程序结束，因此您可以在调用者中安全地使用它，不需要清理。但是在多线程程序中，静态数组将在所有线程之间共享，因此您必须确保只有一个线程在同一时间使用它，并且当您想要使用它时，另一个线程没有更改它。长话短说：如果该函数预计将在多线程程序中使用，请不要使用该函数。

Answer 2

如注释中所述，由于返回了指向局部变量（words）的指针，因此没有UB。最初，循环中还有n，而不是（n*），后者也是UB。您需要在代码中进行错误检查，但以下操作仍然有效，并且仍然坚持使用char*（无错误检查）：

#include <string>
#include <iostream>

char *setWord( char string[], int *n ) {
    char word[20] = { 0 };
    string += *n;
    while ( *string == ',' || *string == '\t' || *string == '\n' || *string == ' ' || *string == '.' || *string == '!' || *string == '?' ) {
        string++;
        ( *n )++;
    }
    int i = 0;
    while ( *string != ',' && *string != '\t' && *string != '\n' && *string != ' ' && *string != '.' && *string != '!' && *string != '?' ) {
        word[i++] += *string;
        string++;
        ( *n )++;
    }

    std::cout << word;
    char *words = new char[i + 1];
    strncpy( words, word, I + 1 );

    return words;
}

int main() {
    char str[] = "hello world!";
    int n = 5;
    char *word = setWord( str, &n );

    delete[] word;

    return 0;
}

使用std::字符串，可以进行（无错误检查）：

std::string setWord( std::string string, int *n ) { // copy on purpose
    int start = *n;
    *n = string.length();
    size_t end = string.find_last_of( ",\t\n .!?" );
    string = string.substr( string.find_first_not_of( ",\t\n .!?", start ) );
    if ( end != std::string::npos ) {
        string = string.substr( 0, string.find_last_of( ",\t\n .!?" ) ); // because it's not end
        *n = (int)end;
    }
    std::cout << string;
    return string;
}