上层到下部不起作用，帮助我的代码出了什么问题

Question

main.cpp

#include <iostream>
#include "Module2.h"
int main()
{
std::cout<<"This is a test of Module2.h"<<std::endl;
std::cout<<UCase("This is a test of UCase")<<std::endl;
std::cout<<LCase("This is a test of LCase")<<std::endl;
system("pause");
return 0;
}

模块2.h

#include <iostream>
#include "Module2.h"
int main()
{
std::cout<<"This is a test of Module2.h"<<std::endl;
std::cout<<UCase("This is a test of UCase")<<std::endl;
std::cout<<LCase("This is a test of LCase")<<std::endl;
system("pause");
return 0;
}

模块2.cpp

///////////////////////////////////////////////////
//Module : Module2.cpp
//
//Purpose : Shows the usage of modular functions
///////////////////////////////////////////////////
#include "Module2.h"
///////////////////////////////////////////////////
//UCase()
char *UCase(char *str)
{
//convert each char in  the string to uppercase
//
int len = strlen(str);
for ( int i ; i < len ; i++)
{
    std::cout<<"In UCase"<<std::endl;
     str[i]=toupper(str[i]);
}
return str;
}
///////////////////////////////////////////////////
//LCase()
char *LCase(char *str)
{
//convert each char in  the string to uppercase
//
int len = strlen(str);
for ( int i ; i < len ; i++)
{
    std::cout<<"In LCase"<<std::endl;
    str[i]=tolower(str[i]);
}  
return str;
}

当我运行它时，没有警告或错误。但是，它不上下弦。我以为我的 for 循环是错误的，但它似乎是对的。我的代码有什么问题。

Answer 1

主要问题是您正在尝试修改字符串文字，例如 "This is a test of UCase" .这是未定义的行为。您需要将文本复制到可以修改的char数组中。

另请注意，出于充分的理由，已弃用并禁止将char*绑定到字符串文本。这应该发出警告：

UCase("This is a test of UCase") // not good: binding char* to literal

您的代码还有其他问题：具有未初始化变量的循环中的未定义行为 （UB），

for ( int i ; i < len ; i++) // using uninitialized i: UB

您还应该查看toupper和tolower文档。他们都接受int，但对他们的价值观有一些限制。您必须确保不会传递导致未定义行为的值，请记住char可以签名。例如，在调用toupper之前，我需要转换为无符号字符吗？

Answer 2

像这样的循环具有未定义的行为：

for ( int i ; i < len ; i++)

原因是您没有以值 0 开始i。
你不知道i的价值始于什么！
可能是-10，可能是824。

如果要初始化值，则必须对其进行初始化。
我建议：

for (int i=0; i < len; i++)

Answer 3

char *UCase( char *str)
{
char ch;
int i=0;
while(str[i])
{
    ch=str[i];
    putchar(toupper(ch));
//putchar : The value is internally converted to an unsigned char when written.
    i++;
}
}
///////////////////////////////////////////////////
//LCase()
char *LCase(char *str)
{
char ch;
int i=0;
while(str[i])
{
    ch=str[i];
    putchar(tolower(ch));
    i++;
}
}

我终于写了这个.虽然，我还是不太明白，但是我学会了

#include <ctype.h>
int tolower( int ch );

意味着下到上每次只能更改一个字符，所以我不能

tolower ("This is a test of LCase");

像这样的代码.