我找不到任何会导致此结果的错误

Question

我是c++的新手，想测试一下我到底学了多少，所以我做了这个简单的cRaZyTeXt生成器。但是有一个奇怪的bug，我找不到任何方法来解决。

代码在这里:

#include <iostream>
#include <string>
#include <algorithm>
#include <windows.h>
char convertToUppercase (char x)
{
int asciiCode {static_cast<int>(x) - 32};
char y {static_cast<char>(asciiCode)};
return y;
}
char convertToLowercase (char x)
{
int asciiCode {static_cast<int>(x) + 32};
char y {static_cast<char>(asciiCode)};
return y;
}
void toClipboard(const std::string &s){
OpenClipboard(0);
EmptyClipboard();
HGLOBAL hg=GlobalAlloc(GMEM_MOVEABLE,s.size() + 1);
if (!hg){
CloseClipboard();
return;
}
memcpy(GlobalLock(hg),s.c_str(),s.size() + 1);
GlobalUnlock(hg);
SetClipboardData(CF_TEXT,hg);
CloseClipboard();
GlobalFree(hg);
}
int main()
{
std::cout << "Enter the text you want to convert into cRaZy TeXt: " << 'n';
std::string userInput {};
std::getline(std::cin >> std::ws, userInput);
char userInputArray [userInput.size()];
std::copy(userInput.begin(), userInput.end(), userInputArray);
char outputArray [userInput.size()];
for (int i = 0; i <= userInput.size(); ++i)
{
int x {static_cast<int>(userInputArray[i])};
if (i % 2 == 0)
{
if (x <= 90 && x >= 65)
outputArray[i] = convertToLowercase(userInputArray[i]);
else
outputArray[i] = userInputArray[i];
}
else
{
if (x <= 122 && x >= 97)
outputArray[i] = convertToUppercase(userInputArray[i]);
else
outputArray[i] = userInputArray[i];
}
}
std::cout << outputArray << 'n';
toClipboard(outputArray);
system("pause");
return 0;
}

当我输入Hello, world!时，它可以输出我想要的hElLo, WoRlD!。证明

但是当我尝试my name is sean.时，它的输出看起来像这样:

mY NaMe iS SeAn.@y name is sean.@%�

更奇怪的是，my name is ma sean.和my name is sean ma.都工作得很好。

我叫马尚恩。

我的名字是Sean ma。

我在发布和调试配置中尝试了以上四种输入，结果都是一样的。

请详细说明这个问题，并使解释对初学者更友好。

任何帮助都是感激的。提前谢谢你。

Answer 1

对于初学者来说，可变长度数组的声明，例如:

char userInputArray [userInput.size()];

不是标准的c++特性。

不需要使用辅助数组来执行任务。您可以更改类型为std::string的原始对象userInput。

可变长度数组

char outputArray [userInput.size()];

不包含结束零字符''的空格，以使存储的字符序列成为字符串。

因此输出

std::cout << outputArray << 'n';

调用未定义行为。

for循环

for (int i = 0; i <= userInput.size(); ++i)

导致访问超出声明的可变长度数组的内存，因为索引的有效范围是[ 0, userInput.size() )。

此外，使用幻数(例如65或90)也是一个坏主意。这使得代码不可读。

如果我理解正确的话，您需要的是一个像下面演示程序中所示的函数。

#include <iostream>
#include <string>
#include <cctype>
std::string & cRaZyTeXt_generator( std::string &s )
{
int upper_case = 1;
for (auto &c : s)
{
if ( std::isalpha( static_cast< unsigned char >( c ) ) )
{
if ( ( upper_case ^= 1 ) )
{
c = std::toupper( static_cast< unsigned char >( c ) );
}
else
{
c = std::tolower( static_cast< unsigned char >( c ) );
}
}
}
return s;
}
int main()
{
std::string s( "Hello, World!" );
std::cout << s << 'n';
std::cout << cRaZyTeXt_generator( s ) << 'n';
}

程序输出为

Hello, World!
hElLo, WoRlD!