Dietel 的“C：如何编程”一书中的这个例子有什么问题？ [关闭]

Question

我正在学习 Dietel 的 C：如何编程。在指针章节中，有这个示例代码：

#include <stdio.h>
#include <ctype.h>

void convertToUppercase(char *sPtr);

int main( void )
{
    char string[] = "cHaRaCters and $32.98";
    printf( "The string before conversion is: %s", string );
    convertToUppercase( string );
    printf( "\nThe string after conversion is: %s\n", string );
}

void convertToUppercase(char *sPtr)
{
    while(*sPtr != '\0') {
        *sPtr = toupper(*sPtr);
        ++*sPtr;
    }
}

当我编译它时，没有问题。但是当我运行它时，什么也没有发生。我无法找出问题所在！

Answer 1

您需要增加指针而不是值将++*sPtr 更改为++sPtr

#include <stdio.h>
#include <ctype.h>

void convertToUppercase(char *sPtr);

int main( void )
{
    char string[] = "cHaRaCters and $32.98";
    printf( "The string before conversion is: %s", string );
    convertToUppercase( string );
    printf( "\nThe string after conversion is: %s\n", string );
}

void convertToUppercase(char *sPtr)
{
    while(*sPtr != '\0') {
        *sPtr = toupper(*sPtr);
        ++sPtr;
    }
}

Answer 2

函数有错别字

void convertToUppercase(char *sPtr)
{
    while(*sPtr != '\0') {
        *sPtr = toupper(*sPtr);
        ++*sPtr;
        ^^^^^^^^
    }
}

必须有

        ++sPtr;
        ^^^^^^^

这个表达式++*sPtr 递增指针sPtr 指向的字符，而这个表达式++sPtr 递增指针本身。

尽管如此，该代码有几个缺点。

C 标准中有一个字符串函数约定，它们应该返回指向目标字符串的指针。所以最好像这样声明函数

char * convertToUppercase(char *sPtr);

函数toupper 依赖于它的参数可以表示为无符号字符。

来自 C 标准（7.4 字符处理）

1 标头声明了几个有用的函数 分类和映射字符。198) 在所有情况下，论点都是 一个 int，其值应表示为无符号字符 or 应该等于宏 EOF 的值。 如果参数有任何 其他值，行为未定义。

这样写会更正确

        *sPtr = toupper( ( unsigned char )*sPtr);

在这种情况下，for 循环将比 while 循环更合适。

考虑到所有这些，该功能可以通过以下方式实现

char * convertToUppercase(char *sPtr)
{
    for ( char *p = sPtr; *p != '\0'; ++p ) {
        *p = toupper( ( unsigned char )*p );
    }

    return sPtr;
}

并像这样称呼

printf( "\nThe string after conversion is: %s\n", convertToUppercase( string ) );