如何更改字符串数组中的单个字符？

Question

有这个：

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

char *up(char *);

int main() {
    char initstr[20];
    printf("enter string\n");
    fgets(initstr, 20, stdin);

    char *str = up(initstr);
    printf("%s\n", str);
}

char *up(char *in) {
    char *ret;
    for (ret = in;
         *in != '\n'; 
         *(ret++) = toupper(*(in++))
        );
    return ret;
}

运行如下：

$./a.out
enter string
abc

#only new line from `printf("%s\n",str);`

从调试器

Hardware watchpoint 3: in

Old value = 0x7fffffffdc20 "abc\n"
New value = 0x7fffffffdc21 "bc\n"

Hardware watchpoint 2: ret

Old value = 0x7fffffffdc20 "abc\n"
New value = 0x7fffffffdc21 "bc\n"

Hardware watchpoint 3: in

Old value = 0x7fffffffdc21 "bc\n"
New value = 0x7fffffffdc22 "c\n"

Hardware watchpoint 2: ret

Old value = 0x7fffffffdc21 "bc\n"
New value = 0x7fffffffdc22 "c\n"
...

我可以看到两个变量都在减少，但我想逐个字符地更改ret 内联。但在最后（循环之后），ret 减少为空，程序只会输出\n。那么如何在循环头中实现这一点呢？

编辑： 感谢下面的回答，考虑到我必须返回指针的第一个地址，我可以通过这个实现 loop_head-only 函数：

char *up(char *in){
    char *ret;
    size_t size=strlen(in);
    for(ret=in;
         *in!='\n'; 
         *(ret++)=toupper(*(in++))
        );
    return (ret-size+1);
}

Answer 1

up 中的错误是您将 ret 一直递增到换行符 (\n) 并返回指向字符串中该字符的 ret。您应该返回一个指向初始字符的指针。

• 使用索引编写此函数更简单。
• 将所有逻辑打包到带有空主体的 for 子句中很难阅读并且容易出错。
• 另请注意，字符串可能不包含换行符，因此在空终止符处停止会更安全，toupper() 不会更改换行符。
• 最后，您不应将char 值传递给toupper()，因为此函数和<ctype.h> 中的所有函数仅针对unsigned char 类型的值和特殊的负值EOF 定义。在默认签名char 的平台上，字符串可能包含负的char 值，当传递给toupper() 时可能会导致未定义的行为。将它们转换为 (unsigned char) 以避免此问题。

这是修改后的版本：

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

char *up(char *s) {
    for (size_t i = 0; s[i] != '\0'; i++) {
         s[i] = toupper((unsigned char)s[i]);
    }
    return s;
}

int main() {
    char initstr[20];
    printf("enter string\n");
    if (fgets(initstr, sizeof initstr, stdin)) {
        char *str = up(initstr);
        printf("%s\n", str);
    }
    return 0;
}