有没有更好的方法来为 C++ 编写这个？ [关闭]答案

【问题标题】：Is there a better way to write this for C++? [closed]有没有更好的方法来为 C++ 编写这个？ [关闭]
【发布时间】：2020-03-18 05:46:39
【问题描述】：

目标是通过提取由空格分隔的单词，从输入的字符串中创建子字符串。

子字符串本身必须是变量。

听起来很简单，但难点在于您只能使用 strcpy、strcmp、strlen、strcat、strncpy、strncmp、strnlen 和 strncat。

例子：

输入：

"John 40 Hitman"

司机：

...

cout << word1 << endl 
     << word2 << endl 
     << word3 << endl;

输出：

John
40
Hitman

这是我的代码

#include <iostream>
#include <cstring>
#include <stdio.h>

int main(){

const char *string = "Name Age Job";
char name[10];
char age[10];
char job[10];
int length = strlen(string);
int temp = 0;
bool flag = false;

for(int i = 0; i < length + 1; i++){
  if(isspace(string[i]) && !flag){
    strncpy(name, string, i);
    name[i] = '\0';
    temp = i;
    flag = !flag;
    cout << name << endl;
    continue;
  }
  if(isspace(string[i])){
    strncpy(age, string + temp + 1, length - i - 1);
    age[temp - i] = '\0';
    temp = i;
    cout << age << endl;
    continue;
  }
  if(string[i] == '\0'){
    strncpy(job, string + temp + 1, length);
    job[temp - i] = '\0';
    cout << job << endl;
  }
}

它可以工作，但它必须使用标志布尔值，字符串不是动态的，只适用于有 2 个空格的字符串，并且有很多重复的代码。总的来说真的很笨拙，但我花了大约两个小时，我不知道如何改进它。

如果您想知道，这确实是一个家庭作业问题，但它是一个入门课程，我的教授只想要一个只有 3 个单词的硬编码字符串的正确输出。但是，我想学习如何改进它，并且非常感谢任何帮助。谢谢。

【问题讨论】：

codereview.stackexchange.com
如果仅使用 C 风格的 str 函数是真的要求，那么您的类是 C 编程类而不是 C++ 类。对不起，但这就是现实。通过将 char[] 更改为 std::string 来对其进行 C++ 化。
@acraig5075 这是一个很好的实践，可以真正理解字符串是如何工作的。 C++ 程序员必须知道这一点，即使他们只使用 std::string。
关于代码审查 - 是的，这是获得有关完整工作代码的一般反馈的最佳站点。不过，只要代码足够本地化，这并不是题外话。
我投票结束这个问题，因为它是代码审查请求。

标签： c++ string substring c-strings

【解决方案1】：

您需要做的就是用'\0'（字符串结尾）替换空格' '，从而从原始字符串创建3个子字符串。以下程序会执行此操作并将字符串转储到cout，但您也可以将指针保存在数组中（例如char* substring[3]）。

int main(){
    char string[] = "Name Age Job";
    char* temp = string;
    for(char* it = string; *it; ++it ) {
        if (*it == ' ') {
            *it = '\0';
            std::cout << temp << std::endl;
            temp= it + 1;
        }
    }
    std::cout << temp << std::endl;
}

【讨论】：

输入字符串在问题中是不可变的。
那么你最终会在同一个数组上迭代两次，这很慢。
性能是手动推出此代码的唯一原因。然后你必须非常擅长了解目标系统。即使是我的版本也太天真了，不能成为库质量的代码，尽管我希望它在低端系统上表现良好。
@Lundin 您似乎错过了这里的主要内容：如果您需要复制/第二次迭代并且如果您首先需要优化代码，那么优化已经足够容易了。

【解决方案2】：

仅使用 C 函数执行此操作的正确方法是使用 strtok，尽管它会就地切碎字符串。

关于您的代码，有很多不必要的分支和检查。你应该避免continue，这是一个几乎 100% 确定的循环需要改进的迹象。每当您发现自己需要它时，通常都有更好的方法。

您还应该避免使用strncpy，因为正如您所注意到的，跟踪它何时终止以及何时不终止是很痛苦的。这是一个危险的功能，比memcpy慢得多，可以在这里使用。

这是一个基于使用 2 个指针的简化版本，一个始终指向下一个空格，一个始终指向下一个有效字的开头。

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

const char* find_next_space (const char* str)
{
  while(*str != '\0' && !isspace(*str))
    str++;
  return str;
}

int main (void)
{
  const char str[] = "hello world how are you";
  char substr[5][10];


  const char* word_start = str;
  for(size_t i = 0; i<5; i++)
  {
    if(*word_start == '\0')
      break;

    const char* next_space = find_next_space(word_start); 

    size_t length = (size_t)(next_space-word_start);
    memcpy(substr[i], word_start, length); 
    substr[i][length] = '\0';
    puts(substr[i]);

    word_start = next_space+1;
  }
}

此代码通过不检查字符串是否适合并且不分配内存来简化事情。真正的生产质量 C 代码不会使用char [5][10]，而是使用指针数组char* [5]，其中每个指针都设置为指向动态分配的内存。

【讨论】：

非常感谢！也许我会想办法自己做指针数组的事情
@user12414777 - 指针只是一个普通变量，它保存其他东西的地址作为它的值——而不是像int a = 5; 这样的立即值。指针将保存5 存储在内存中的地址。例如int *b;是一个指向int的指针，b = &a;将5存储在内存中的地址分配为b持有的值（例如b现在指向to a) -- 5 驻留在内存中。

【解决方案3】：

我同意@acraig5075，因为您的代码比 C++ 更多。如果您正在考虑使用 STL 字符串在 C++ 中编写此代码，那么执行此操作的一种方法如下。

#include <iostream>
#include <string>
#include <sstream>
#include <vector>


std::vector<std::string> split_string( const std::string &srcString, const char &delimiterKeywrd )
{
    /**
     *  Splits a string according to a keyword delimiter and returns a vector with the 
     *  segments of the split string.
     *  
     *  Args:
     *      @param  srcString:          The required string to split.
     *      @param  delimiterKeywrd:    The delimiter keyword that the string will be splitted to
     *      @return segmentsVectr:      A vector holding the segments of the srcString that was splitted. 
     *
    */

    std::stringstream inputStr;
    inputStr.str( srcString );
    std::string segment;

    std::vector<std::string> segmentsVectr;

    while( std::getline( inputStr, segment, delimiterKeywrd ) )
    {
        segmentsVectr.push_back( segment );
    }

    return segmentsVectr;
}


int main() {
    std::string inputStr{"John 40 Hitman"};

    std::vector<std::string> result;
    char delimiterKeywrd = ' '; 

    result = split_string( inputStr, delimiterKeywrd );

    //  Iterate through the vector and print items on a different line.
    for ( const std::string &item : result )
    {
        std::cout << item << std::endl;
    }

    return 0;
}

如果您不熟悉，请查看 cppreference 中的 std::string 和示例。同样在这里我使用std::stringstream，这使得使用std::getline变得容易。

如果 std::stringstream 不是您的首选方式，您可以随时检查this 的实现，从另一个关于按字符拆分字符串的问题中查看。

希望这会有所帮助。

【讨论】：