[已解决]初学C语言指针遇到两个问题。

sunnyrubik

各位鱼友大家好，小弟刚刚跟着小甲鱼老师学习指针，这是网上的一个题目。

然后我写了以下代码

1. #include<stdio.h>
   
2. #include<stdlib.h>
   
3. #include<ctype.h>
   
4. void change(char *s);
   
5. void insert(char *m);
   
6. 
   
7. int main()
   
8. {
   
9.         char ch;
   
10.         int i=0;
    
11.         char str[100];
    
12.         while((ch=getchar())!='\n')
    
13.         {
    
14.                 str[i]=ch;
    
15.                 i++;
    
16.         }
    
17.         change(str);
    
18.         insert(str);
    
19.         printf("%s\n",str);
    
20.        
    
21.         system("pause");
    
22.         return 0;
    
23. }
    
24. void change(char *s)//明文中的字母转换成其后的第4个字母
    
25. {
    
26.         while(*s!='\0')
    
27.         {
    
28.                 while(*s)
    
29.     {
    
30.         if(*s >= 'A' && *s <= 'V')
    
31.             *s=*s+4;
    
32.         else if(*s >= 'a' && *s <= 'v')
    
33.             *s=*s+4;
    
34.         else if(*s='w')*s='a';
    
35.         else if(*s='W')*s='A';
    
36.         else if(*s='x')*s='b';
    
37.         else if(*s='X')*s='B';
    
38.         else if(*s='y')*s='c';
    
39.         else if(*s='Y')*s='C';
    
40.         else if(*s='Z')*s='D';
    
41.         else if(*s='z')*s='d';
    
42.         s++;
    
43.     }
    
44.         }
    
45. }
    
46. void insert(char *m)//将密文每两个字符之间插入一个空格
    
47. {
    
48.         while(*m!='\0')
    
49.         {
    
50.                 while(*m)
    
51.                 {
    
52.                         printf("%c",*m);
    
53.                         printf(" ");
    
54.                         m++;
    
55.                 }
    
56.         }
    
57. }

复制代码

但是运行的结果有一下两个问题
1.它会将所有的数字转化为a。这不是题目要求的
2.在空格时会出现问题，详见下图。
请大家帮忙看看是什么原因

最佳答案

月排行榜 / 总排行榜

丶忘却的年少o

2018-11-19 23:19:56

1. change函数你的else if判断的条件不是比较，是赋值，你将“==”写成了“=”，所以不管里面是什么都变了。
2. 就是change内的函数操作不当引起的。
我写一个我的代码，你可以参考下，优化下你的代码。（只是个人理解）

1. #include "stdio.h"
   
2. #include <string.h>
   
3. #include <stdlib.h>
   
4. 
   
5. /* 传进来的是指针和字符串的长度 */
   
6. void change(char *str, int len)
   
7. {       
   
8.         for ( int i = 0; i < len; i++ )
   
9.         {
   
10.                 // 外圈判断范围，内圈判断边界
    
11.                 if ( str[i] >= 'A' && str[i] <= 'Z' )
    
12.                 {
    
13.                         if ( str[i] <= 'V' )
    
14.                                 str[i] = str[i] + 4;
    
15.                         else
    
16.                                 str[i] = str[i] - 22;
    
17.                 }
    
18.                 else if ( str[i] >= 'a' && str[i] <= 'z' )
    
19.                 {
    
20.                         if (str[i] <= 'v')
    
21.                                 str[i] = str[i] + 4;
    
22.                         else
    
23.                                 str[i] = str[i] - 22;
    
24.                 }
    
25. 
    
26.         }
    
27. }
    
28. 
    
29. /* 我感觉是在这个函数内添加空格，不是打印，所以我这里
    
30.    传进指针，生成有空格的字符串副本再拷贝给自己 */
    
31. void insert(char *str, int len)
    
32. {
    
33.         char temp[40] = {0};                // 这个数组最好大一点，防止越界
    
34.         int i = 0;
    
35. 
    
36.         for ( ; i < len; i++ )
    
37.         {
    
38.                 temp[2*i] = str[i];
    
39.                 temp[2*i+1] = ' ';
    
40.         }
    
41. 
    
42.         strcpy_s(str, 40, temp);                // 把处理好的字符串给写回去
    
43. }
    
44. 
    
45. /* 函数加_s的是VS编译器的安全函数，加了操作字符串的长度
    
46.    不是VS编译器的可以不考虑，但是要自己注意边界 */
    
47. int main()
    
48. {
    
49.         char str[40] = {0};                // 直接就给初始化，让里面所有元素都为'\0'
    
50. 
    
51.         printf("请输入语句: ");
    
52.         scanf_s("%s", str, 20);       
    
53. 
    
54.         int len = strlen(str);
    
55.         change(str, len);
    
56.         insert(str, len);
    
57.         printf("%s\n",str);
    
58. 
    
59.         system("pause");
    
60. 
    
61.         return 0;
    
62. }
    
63. 
    

复制代码

跳转到最佳答案楼层

丶忘却的年少o

1. change函数你的else if判断的条件不是比较，是赋值，你将“==”写成了“=”，所以不管里面是什么都变了。
2. 就是change内的函数操作不当引起的。
我写一个我的代码，你可以参考下，优化下你的代码。（只是个人理解）

1. #include "stdio.h"
   
2. #include <string.h>
   
3. #include <stdlib.h>
   
4. 
   
5. /* 传进来的是指针和字符串的长度 */
   
6. void change(char *str, int len)
   
7. {       
   
8.         for ( int i = 0; i < len; i++ )
   
9.         {
   
10.                 // 外圈判断范围，内圈判断边界
    
11.                 if ( str[i] >= 'A' && str[i] <= 'Z' )
    
12.                 {
    
13.                         if ( str[i] <= 'V' )
    
14.                                 str[i] = str[i] + 4;
    
15.                         else
    
16.                                 str[i] = str[i] - 22;
    
17.                 }
    
18.                 else if ( str[i] >= 'a' && str[i] <= 'z' )
    
19.                 {
    
20.                         if (str[i] <= 'v')
    
21.                                 str[i] = str[i] + 4;
    
22.                         else
    
23.                                 str[i] = str[i] - 22;
    
24.                 }
    
25. 
    
26.         }
    
27. }
    
28. 
    
29. /* 我感觉是在这个函数内添加空格，不是打印，所以我这里
    
30.    传进指针，生成有空格的字符串副本再拷贝给自己 */
    
31. void insert(char *str, int len)
    
32. {
    
33.         char temp[40] = {0};                // 这个数组最好大一点，防止越界
    
34.         int i = 0;
    
35. 
    
36.         for ( ; i < len; i++ )
    
37.         {
    
38.                 temp[2*i] = str[i];
    
39.                 temp[2*i+1] = ' ';
    
40.         }
    
41. 
    
42.         strcpy_s(str, 40, temp);                // 把处理好的字符串给写回去
    
43. }
    
44. 
    
45. /* 函数加_s的是VS编译器的安全函数，加了操作字符串的长度
    
46.    不是VS编译器的可以不考虑，但是要自己注意边界 */
    
47. int main()
    
48. {
    
49.         char str[40] = {0};                // 直接就给初始化，让里面所有元素都为'\0'
    
50. 
    
51.         printf("请输入语句: ");
    
52.         scanf_s("%s", str, 20);       
    
53. 
    
54.         int len = strlen(str);
    
55.         change(str, len);
    
56.         insert(str, len);
    
57.         printf("%s\n",str);
    
58. 
    
59.         system("pause");
    
60. 
    
61.         return 0;
    
62. }
    
63. 
    

复制代码

sunnyrubik

丶忘却的年少o 发表于 2018-11-19 23:19
1. change函数你的else if判断的条件不是比较，是赋值，你将“==”写成了“=”，所以不管里面是什么都变了 ...

非常感谢您的解答

KAKUI丶

看了你的，然后扫了一眼楼上的代码，比较难看，然后自己再写了下，觉得比你们的都简单易懂整洁，代码如下
#include<stdio.h>
#include<string.h>

void change(char *s);
void insert(char *s);

int main()
{
        char str[40] = {0};
        char ch;
        printf("请输入密语：");
        scanf("%s",&str);
        change(str);
        insert(str);
}

void change(char *s)
{
        int i=0;
        while(s[i]!='\0')
        {
                if((s[i]>='a' && s[i]<='v') || (s[i]>='A' && s[i]<='V'))
                {
                        s[i]+=4;
                }
                else if((s[i]>='w' && s[i]<='z') || (s[i]>='W' && s[i]<='Z'))
                {
                        s[i]-=22;
                }
                i++;
        }
}

void insert(char *s)
{
        int i;
        for(i=0;i<=strlen(s);i++)
        {
                printf("%c ",s[i]);
        }
}

sunnyrubik

KAKUI丶 发表于 2018-11-20 17:28
看了你的，然后扫了一眼楼上的代码，比较难看，然后自己再写了下，觉得比你们的都简单易懂整洁，代码如下
...

谢谢大佬

KAKUI丶

KAKUI丶 发表于 2018-11-20 17:28
看了你的，然后扫了一眼楼上的代码，比较难看，然后自己再写了下，觉得比你们的都简单易懂整洁，代码如下
...

第一次用鱼C论坛的，不会发图片，验证过的，代码正常，只要长度不要超过40就行，另外字母溢出的需求没有写明白，我就当作是循环的一个字母表输出

KAKUI丶

卧槽，我就回帖慢那么一会，居然采纳别人的

sunnyrubik

KAKUI丶 发表于 2018-11-20 17:38
卧槽，我就回帖慢那么一会，居然采纳别人的

那个哥子是今天上午回复我的哈。另外谢谢你的解答哈（发代码是点那个尖括号哈）