问题描述:
写一函数,使输入的一个字符串按反序存放,在主函数中输入并输出反序后的字符串(不包含空格)。
样例输入:
123456abcdef
样例输出:
fedcba654321
代码分析:
这段代码的主要功能是接受用户输入的字符串,然后反转该字符串并输出,但是在输出过程中会跳过空格符和换行符。
注意问题:
以下是应该注意的问题:
1.缺乏错误检查:
代码中如果没有对输入字符串长度进行检查,可能导致缓冲区溢出。在实际应用中,应该对输入的字符串长度进行限制,或者使用更安全的输入函数来处理。
2.循环条件错误:
在 for 循环中,多个条件之间应该是逻辑与运算,但是逗号操作符的使用可能导致逻辑错误。应该使用逻辑与运算符 &&。
3.换行符处理:
代码中没有处理输入字符串末尾的换行符。如果用户输入了换行符,反转后会在输出中产生额外的空格。
此处给出解决方案:
if (len > 0 && str[len - 1] == '\n') {
str[len - 1] = '\0';
}
4.魔法数值:
"魔法数值"(Magic Number)是指在代码中直接使用的、未经解释的硬编码数值。这些数值被称为"魔法",因为它们似乎出现在代码中,没有明确的说明或命名,使得人们很难理解它们的含义。这样的硬编码数字可能使代码变得难以理解、难以维护,并且容易引起错误。
代码实现:
#include<stdio.h>
#include<string.h> // 为了使用 strlen 函数
#define MAX_SIZE 200
void change(char x[])
{
size_t len = strlen(x);
char y[200];
for(int i=len-1,j=0;j<len&&i>=0;j++,i--)
{
y[j]=x[i];
if(y[j] != ' ' && y[j] != '\n')
printf("%c",y[j]);
}
}
int main()
{
char str[MAX_SIZE];
fgets(str, sizeof(str), stdin);
size_t len = strlen(str);
if (len > 0 && str[len - 1] == '\n') {
str[len - 1] = '\0';
}
change(str);
return 0;
}