救助，，我的c语言计数和删除好像有些问题

我是个汉子

/***********************************************
题目：主函数main()中数字字符串s为测试数据，程序主要先对数
      字字符串s进行降序排序，将结果存入在数组b中，然后对
      数组b中数据进行压缩，不同数字保存在数组b中，数字出
      现次数保存在数组times中，计算各个数字加权和（即数字
      本身乘以该数字出现次数），结果保存在数组c中。

编写程序：
      1. 编写函数void  SortAndChange(char s[],int n, int b[])，
         对字符串s中n个字符进行降序排列,并将n个字符转换为
         数字，保存在数组b中。
   
      2. 编写函数int Calculcate(int b[],int n，int times[],int c[])，
         对数组b中数字进行压缩，不同数字保存在数组b中，不
         同数字出现次数存入数组times中，并计算各个数字加权
         和，结果存入数组c中,函数返回不同数字的个数。
  
      测试数据： 13334444555582298760
      运行结果：b[0]=9 and times[0]=1, the weight sum is 9
                b[1]=8 and times[1]=2, the weight sum is 16
                ...
                b[9]=0 and times[9]=1, the weight sum is 0
*************************************/

1. #include<stdio.h>
   
2. #include<stdlib.h>
   
3. #include<conio.h>
   
4. #include<ctype.h>
   
5. #include<string.h>
   
6. #define N 50
   
7. 
   
8. void SortAndChange(char s[],int n,int b[])
   
9. {
   
10. /**********Program**********/
    
11. int i,j;
    
12. char t;
    
13. for(i=0; i<n-1; i++)
    
14. {
    
15.     for(j=i+1; j<n; j++)
    
16.     {
    
17.         if(s[i]<s[j])
    
18.         {
    
19.             t=s[i];
    
20.             s[i]=s[j];
    
21.             s[j]=t;
    
22.         }
    
23.     }
    
24. }
    
25. for(i=0; i<n; i++)
    
26.     b[i]=s[i]-'0';
    
27. /**********  End  **********/
    
28. }
    
29. int Calculcate(int b[],int n,int times[],int c[])
    
30. {
    
31. /**********Program**********/ //98876555544443332210
    
32. int i,j,k=0,num,count;
    
33. for(i=0; i<n; i++)
    
34. {
    
35.     count=1;
    
36.     for(j=i; j<n; j++)
    
37.     {
    
38.         if(b[j]==b[j+1])
    
39.             count++;
    
40.         else
    
41.         {
    
42.             times[k]=count;
    
43.             b[k]=b[j];
    
44.             i=j;
    
45.             c[k]=b[k]*times[k];
    
46.             k++;
    
47.             break;
    
48.         }   
    
49.     }        
    
50. }
    
51. return k;
    
52. /**********  End  **********/
    
53. }
    
54. 
    
55. int main()
    
56. {
    
57.     char s[N]="13334444555582298760";
    
58.     int b[N]={0};
    
59.     int c[10]={0};
    
60.     int times[10]={0};
    
61.     int num;
    
62.     int i,n;
    
63.             
    
64.     FILE *fp;
    
65.     if((fp=fopen("DATA.TXT","w"))==NULL)
    
66.     {
    
67.         printf("File open error\n");
    
68.         exit(0);
    
69.     }
    
70.         
    
71.     n=strlen(s);
    
72.     SortAndChange(s,n,b);
    
73.     num=Calculcate(b,n,times,c);
    
74. 
    
75.     for(i=0;i<num;i++)
    
76.     {
    
77.         printf("b[%d] = %d and times[%d] =%d, the weight sum is %d\n",i,b[i],i,times[i],c[i]);
    
78.         fprintf(fp,"b[%d]= %d and times[%d] =%d, the weight sum is %d\n",i,b[i],i,times[i],c[i]);
    
79.     }
    
80.    
    
81.     fclose(fp);
    
82.    
    
83.     return 0;
    
84. }
    

复制代码

我的输出：
b[0] = 9 and times[0] =1, the weight sum is 9
b[1] = 8 and times[1] =2, the weight sum is 16
b[2] = 7 and times[2] =1, the weight sum is 7
b[3] = 6 and times[3] =1, the weight sum is 6
b[4] = 5 and times[4] =4, the weight sum is 20
b[5] = 4 and times[5] =4, the weight sum is 16
b[6] = 3 and times[6] =3, the weight sum is 9
b[7] = 2 and times[7] =2, the weight sum is 4
b[8] = 1 and times[8] =1, the weight sum is 1

Croper

1. if(b[j]==b[j+1])
   
2.             count++;
   
3.         else
   
4.         {
   
5.             times[k]=count;
   
6.             b[k]=b[j];
   
7.             i=j;
   
8.             c[k]=b[k]*times[k];
   
9.             k++;
   
10.             break;
    
11.         }   

复制代码

这一段至少有两个问题
1.既然j<n当j=n-1时 b[j+1]溢出，这是不安全的代码
2.考虑下边界情况。你这样写只有b[j]!=b[j+1]时结算，如果最后b[n-1]的值刚好与b[n]这个内存位置储存的值相同，那么最后一次结算就直接被你吃掉了
这就是输出中为什么没有0的原因

我是个汉子

Croper 发表于 2019-2-25 20:29
这一段至少有两个问题
1.既然j

1. for(i=0; i<n; i++)
   
2. {
   
3.         count=1;
   
4.         for(j=i; j<n-1; j++)
   
5.         {
   
6.                 if(b[j]==b[j+1])
   
7.                         count++;
   
8.                 else
   
9.                 {
   
10.                         times[k++]=count;
    
11.                         b[k]=b[j];
    
12.                         i=j;
    
13.                         c[k]=b[k]*times[k];
    
14.                         break;
    
15.                 }       
    
16.         }

复制代码

这样修改也不行的

Croper

什么叫“这样修改也不行的”？
那是你自己的代码啊- -，我又没给你改