在c中制作将小写字符串转换为大写字符串的功能

Question

我仍然是制作函数和 C 编程的初学者。 我正在尝试制作一个将其转换为大写的函数，但似乎我在指针处搞砸了（？）

#include <stdio.h>

void mytoupper(char *s[]) {
    int i = 0;
    while (s[i] != '\0') {
        if (s[i] >= 'a' && s[i] <= 'z') {
            s[i] = s[i] - 32;
        }
        i++;
    }
    return s;
}

int main(void) {
    char s[32];
    printf("Insert string:");
    printf("%s", s);
    printf("%s", mytoupper(s[32]));
    return 0;
}

Answer 1

您的代码中存在多个问题：

• mytoupper 的定义不正确：它应该采用 char *s 参数而不是 char *s[]，并返回 char *。
• 将字符从小写更改为大写不应使用硬编码值32，该值仅适用于 ASCII，请使用更通用的方法s[i] = s[i] - 'a' + 'A';
• 要读取字符串，使用scanf("%31s", s);而不是printf("%s", s);，强烈建议测试scanf()的返回值
• printf("%s", mytoupper(s[32])); 中的参数不正确：你应该只写printf("%s", mytoupper(s));

这是一个更正的版本：

#include <stdio.h>

char *mytoupper(char *s) {
    int i = 0;
    while (s[i] != '\0') {
        if (s[i] >= 'a' && s[i] <= 'z') {
            s[i] = s[i] - 'a' + 'A';
        }
        i++;
    }
    return s;
}

int main(void) {
    char s[32];
    printf("Insert string:");
    if (scanf("%31s", s) == 1) {
        printf("%s\n", mytoupper(s));
    }
    return 0;
}

Answer 2

在您的代码中有一些地方无法按预期工作。

1. 您没有正确获取用户输入，而是尝试打印“空”字符串。

   printf("%s",s);
   

   您可以将其更改为：

   scanf("%31s",s);
   

2. 您在函数mytoupper 中接受了一个字符指针数组，但这不是必需的。您可以改为简单地传递一个字符数组。

   void mytoupper(char s[]) 
   

3. 您正在尝试使用void 函数的返回值。您可以调用函数然后打印字符串，或者让函数返回字符串。

   如果您想更改字符串然后单独打印，您需要将mytoupper 更改为不再返回任何内容并将其保留为void 返回类型。

   mytoupper(s);
   printf("%s", s);
   

   或者把函数改成：

   char* mytoupper(char s[]) {
   

   然后打印字符串：

   printf("%s", mytoupper(s));