c++ 在使用 tolower 时使用 c 字符串格式化错误

Question

嗨，所以我尝试将 cstring 设为小写，但是当我在最后打印 cstring 时，我得到了一个奇怪的格式框，其中一些字母应该是。有人有什么想法吗？

#include <string>
#include <iostream>
#include <string.h>

using namespace std;

int main () 
{ 
    int i=0; 
    char* str="TEST"; 
    char c; 
    char* cstr = new char[strlen(str) + 1];
    while (str[i]) 
    { 
        c = str[i]; 
        c = tolower(c);
        strcat(cstr, &c); 
        i++; 
    } 

    cout << cstr << endl; 
    return 0; 
}

Answer 1

问题是您错误地调用了strcat。第二个参数不是以 null 结尾的字符串。

您根本不需要致电strcat。直接写入输出字符串即可：

试试：

  while (str[i])
  {
    c = str[i];
    c = tolower(c);
    cstr[i] = c;
        i++;
  }
  cstr[i] = 0;

或者，等效地：

while(str[i])
{
  cstr[i] = tolower(str[i]);
  i++;
}
cstr[i] = 0;

Answer 2

strcat 需要一个 null-terminated char*，因此通过提供本地 char 的地址，您正在调用 undefined behavior。

另外，new char[std::strlen(str) + 1] 不会将数组初始化为0s，这意味着cstr 也不会被正确地以空值终止；将() 添加到new[] 会导致数组被初始化。

试试这个：

#include <cstddef>
#include <cctype>
#include <cstring>
#include <ostream>
#include <iostream>

int main()
{
    char const* str = "TEST";
    char c[2] = { };
    char* cstr = new char[std::strlen(str) + 1]();
    std::size_t i = 0;
    while (str[i])
    {
        c[0] = static_cast<char>(std::tolower(str[i++]));
        std::strcat(cstr, c);
    }
    std::cout << cstr << std::endl;
    delete [] cstr;
}

Answer 3

strcat 的第二个参数应该是一个以空字符结尾的字符串，而不是单个字符的地址。 strcat 不适合这种用途。

int main ()
{
    const char* str="TEST";
    char* cstr = new char[strlen(str) + 1];
    cstr[strlen(str)] = 0;
    for (int i = 0; str[i]; ++i) {
        cstr[i] = tolower(str[i]);
    }
    cout << cstr << endl;
}

Answer 4

#include <cstddef>
#include <cctype>
#include <cstring>
#include <ostream>
#include <iostream>

#include <string>

int main()
{
    std::string str = "TEST";
    std::string cstr;

    for (std::string::const_iterator it = str.begin(); it!= str.end(); ++it)
        cstr.push_back(tolower(*it));

    std::cout << cstr << std::endl;
}

甚至更短：

#include <algorithm>
#include <iterator>

...

    std::transform(str.begin(), str.end(), std::back_inserter(cstr), tolower);