如何将 cstring 作为函数参数/参数传递

Question

我有一个小程序可以打印出单词的每个字母的大写形式，但是当我编译它时我得到错误有符号/无符号不匹配，因为我在这个程序中将 cstring 作为普通 string 传递.如何正确传递它以便我仍然可以使用text.length()？这是我得到的错误“Tester.cpp(22,23): warning C4018: 'for (int i = 0; i < text.length(); i++)

#include <iostream>
using namespace std;

string capitalizeFirstLetter(string text);

int main() {
    char sentence[100];
    for ( ; ; )
    {
        cin.getline(sentence, 100);
        if (sentence != "0")
            capitalizeFirstLetter(sentence);
    }

    return 0;
}

string capitalizeFirstLetter(string text) {

    for (int i = 0; i < text.length(); i++)
    {
        if (i == 0)
        {
            text[i] = toupper(text[i]);
        }
        if (text[i] == ' ')
        {
            ++i;
            text[i] = toupper(text[i]);
        }
    }
    cout << text;

    return text;
}

Answer 1

由于您将 cstring 作为普通 string 传递给函数，因此未生成错误，但这是由于您尝试在语句中使用 != 运算符比较 c 样式字符串

if (sentence != "0")
    capitalizeFirstLetter(sentence);

为此尝试使用strcmp()

Answer 2

您的代码有问题：

1. if (sentence != "0") : 非法比较。如果您想中断将0 作为输入，请尝试使用strcmp（包括<cstring>）作为if (strcmp(sentence, "0")。 （请注意，当两个字符串相等时，strcmp 返回0。）或者干脆做if (!(sentence[0] == '0' and sentence[1] == 0))。此外，这个条件应该伴随else break;，以防止for循环永远运行。

2. for (int i = 0; i < text.length(); i++) ：由于有符号和无符号类型之间的比较而产生警告。将 i 的数据类型更改为 string::size_type 以防止出现警告。

3. <string>（std::string）和<cctype>（std::toupper）不包括在内。

4. 感谢@john 指出这一点。如果字符串的最后一个字符是空格，则您的代码具有未定义的行为。在使用text[i]之前添加检查i是否仍小于text.length()。

5. 另一种错误情况是0 后面有空格。将 getline 移动到 for 的条件以解决此问题。现在不需要输入 0 来终止程序。此外，我建议为此使用while 循环而不是for。

6. 您可能还需要打印换行符来分隔句子。此外，我更喜欢使用从capitalizeFirstLetter 返回的字符串在main() 函数中打印修改后的句子。

7. 简短（初级）代码无关紧要，但要避免养成将using namespace std; 放在您编写的每个代码顶部的习惯。 Refer this.

固定代码：

#include <cctype>
#include <cstring>
#include <iostream>
#include <string>
using namespace std;

string capitalizeFirstLetter(string text);

int main() {
  char sentence[100];
  while (cin.getline(sentence, 100))
    cout << capitalizeFirstLetter(sentence) << '\n';
}

string capitalizeFirstLetter(string text) {
  for (string::size_type i = 0; i < text.length(); i++) {
    if (i == 0)
      text[i] = toupper(text[i]);
    if (text[i] == ' ')
      if (++i < text.length())
        text[i] = toupper(text[i]);
  }
  return text;
}

Sample Run：

输入：

hello world
foo bar

输出：

Hello World
Foo Bar

---

我的版本（需要 C++20）：

#include <cctype>
#include <iostream>
#include <string>

auto capitalizeFirstLetter(std::string text) {
  for (bool newWord = true; auto &&i : text) {
    i = newWord ? std::toupper(i) : i;
    newWord = std::isspace(i);
  }
  return text;
}

int main() {
  std::string sentence;
  while (std::getline(std::cin, sentence))
    std::cout << capitalizeFirstLetter(sentence) << std::endl;
}

Sample Run

Answer 3

这里有几件事困扰着我。

首先，不要使用using namespace std，在这种情况下“可以”，但不要习惯它，它会引起相当大的麻烦。 见Why is “using namespace std;” considered bad practice?

接下来就是在这里使用std::string而不是cstrings，它更容易编写和阅读，并且不会产生任何可测量的性能损失或其他东西。而且这种方式更难产生错误。 所以只需使用

std::string sentence;

和

getline(std::cin, sentence);

为什么要在转换字符串的函数中处理输出？只需让主打印转换后的字符串。 所以你的主要可能看起来像这样：

int main() {
    std::string sentence;
    
    while(true)
    {
        getline(std::cin, sentence);
        auto capitalized = capitalizeFirstLetter(sentence);
        std::cout << capitalized;
    }

    return 0;
}

PS：您收到的“错误”是一个警告，因为您将int i 与text.length() 进行比较，size_t 又名unsigned int 或unsigned long int。

Answer 4

将sentence 作为字符串传递的最简单方法是将其括在一个花括号集合中，为参数std::string text 提供直接初始化 例如......

    for ( ; ; )
    {
        std::cin.getline(sentence, 100);
        if (*sentence)
            capitalizeFirstLetter({sentence});
    }

这允许字符串sentence 用作Direct initialization 以在您的capitalizeFirstLetter() 函数中初始化std::string text：

std::string capitalizeFirstLetter (std::string text) {

    for (size_t i = 0; i < text.length(); i++)
    {
        if (i == 0)
        {
            text[i] = toupper(text[i]);
        }
        if (text[i] == ' ')
        {
            ++i;
            text[i] = toupper(text[i]);
        }
    }
    std::cout << text;

    return text;
}

阅读Why is “using namespace std;” considered bad practice?后，您的完整代码将是：

#include <iostream>

std::string capitalizeFirstLetter (std::string text) {

    for (size_t i = 0; i < text.length(); i++)
    {
        if (i == 0)
        {
            text[i] = toupper(text[i]);
        }
        if (text[i] == ' ')
        {
            ++i;
            text[i] = toupper(text[i]);
        }
    }
    std::cout << text;

    return text;
}

int main (void) {
    
    char sentence[100];
    
    for ( ; ; )
    {
        std::cin.getline(sentence, 100);
        if (*sentence)
            capitalizeFirstLetter({sentence});
    }

    return 0;
}

（注意：取消引用sentence提供了第一个字符，然后确认该字符不是nul-terminating字符（ASCII 0））

更好的 CapitalizeFirstLetter()

一种更简单的大写方法是包含<cctype> 和int 来保存最后读取的字符。然后逻辑简单地循环遍历每个字符，如果第一个字符是字母字符，则将其大写，否则仅当当前字符为字母字符且最后一个字符为空格时才将字母大写，例如

std::string capitalizeFirstLetter (std::string text)
{
    int last = 0
    
    for (auto& c : text)
    {
        if (isalpha(c))
        {
            if (!i || isspace (last))
                c = toupper(c);
        }
        last = c;
    }
    std::cout << text;

    return text;
}

（注意：上面使用了基于范围的for循环）

任何一种方式都可以。