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;

}

Module2.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;

}

Module2.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可以签名。请参阅示例Do I need to cast to unsigned char before calling toupper?

Answer 2

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

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

原因是您没有以值i开始0 您不知道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 toupper每次只能改变一个角色，所以我不能

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

像这样的代码。

toupper tolower不工作，帮助我的代码出了什么问题

3 个答案: