为什么我的函数 encodeChar 不起作用？

Question

下面的代码应该允许用户输入两个字符串 s 和 t，一个结构数组和大小 数组作为参数，将 s 中的字符编码为 t，并通过引用调用将编码后的字符串 t 传递给调用者。 我不明白为什么我的函数 encodeChar 是错误的。我已经尝试过使用索引方法，它类似于正确的答案。但是当我输入

1
3
a
b
b
c
c
d
3
abcd
4

，输出为 abdd。我不明白我的错误在哪里。请让我解决我的误解。

#include <string.h>
typedef struct {
   char source;
   char code;
} Rule;
void createTable(Rule *table, int *size);
void printTable(Rule *table, int size);
void encodeChar(Rule *table, int size, char *s, char *t);
int main() 
{
   char s[80], t[80], dummychar, *p;
   int size, choice;
   Rule table[100];
     
   printf("Select one of the following options:\n");
   printf("1: createTable()\n");     
   printf("2: printTable()\n");
   printf("3: encodeChar()\n");      
   printf("4: exit()\n");    
   do {
      printf("Enter your choice: \n");
      scanf("%d", &choice);  
      switch (choice) {
         case 1:           
            printf("createTable(): \n");
            createTable(table, &size);           
            break;               
         case 2:           
            printf("printTable(): \n");
            printTable(table, size);           
            break;               
         case 3:       
            scanf("%c",&dummychar);
            printf("Source string: \n");
            fgets(s, 80, stdin);  
            if (p=strchr(s,'\n')) *p = '\0'; 
            encodeChar(table,size,s,t);
            printf("Encoded string: %s\n", t);   
            break;   
         default: 
            break;
      } 
   } while (choice < 4);
   return 0;
}
void printTable(Rule *table, int size)
{
   int i;

   for (i=0; i<size; i++)
   {
      printf("%d: %c->%c\n", i+1, table->source, table->code);
      table++;
   }
}
void createTable(Rule *table, int *size)
{
    /*edit*/
    /* Write your code here */
    /*edit*/
    /* Write your code here */
    int i;
    char dummy[80];
    printf("Enter number of rules: \n");
    scanf("%d", size); // dont careless here 
    for(i=0; i< *size; i++){
        printf("Enter rule %d: \n", i+1);
        fgets(dummy, 80,stdin); // why need to use this twice 
        printf("Enter source character: \n");
        scanf("%c", &table[i].source);    
        fgets(dummy,80,stdin);// why 
        printf("Enter code character: \n ");
        scanf("%c", &table[i].code);
    }
    /*end_edit*/
    /*end_edit*/
}
void encodeChar(Rule *table, int size, char *s, char *t) 
{
    /*edit*/
    /* Write your code here */// this is wrong
    int i, j;
    for(i = 0 ; s[i] != '\0'; i++){
        for(j = 0; j < size; j++){
            if(s[i] == table[j].source){
                *t = table[j].code;
            }
            else{
                *t = s[i];
            }
        }
        t++;
    }
    *t = '\0';

        /*edit*/
    /* this is correct 
    int i;
    while(*s!='\0'){
        for(i = 0; i < size; i++){

        if(*s != table[i].source)
            *t = *s;
        else
            *t = table[i].code;
       t++;
       s++;
        }
    }
    *t = '\0';
    
    */
}

Answer 1

这里一张纸和一支铅笔就足够了。

只需按照abcd 的代码即可。相关部分是

for(i = 0 ; s[i] != '\0'; i++){
    for(j = 0; j < size; j++){
        if(s[i] == table[j].source){
            *t = table[j].code;
        }
        else{
            *t = s[i];
        }
    }
    t++;
}
*t = '\0';

我们走吧：

• i 是 0，s[i] 是 a

  • j 为 0，table[j].source 为 a：*t 接收 b
  • j 是 1，table[j].source 是 b：*t 收到（返回）a！...

您的代码只能应用最后一条规则...

一个简单的解决方法是在应用规则时跳出循环以防止以下规则回滚更改：

for(i = 0 ; s[i] != '\0'; i++){
    for(j = 0; j < size; j++){
        if(s[i] == table[j].source){
            *t = table[j].code;
            break;      // do not examine other rules
        }
        else{
            *t = s[i];
        }
    }
    t++;
}
*t = '\0';

但是如果同一个字符被多个规则改变，这段代码会使用第一个，而正确的代码会使用最后一个，这样会更一致：

for(i = 0 ; s[i] != '\0'; i++){
    for(j = size-1; j >= 0; j--){  // examine rules in descending order
        ...

无论如何，这里仍有可能改进，因此您可以将其发布到 CodeReview 以获得最佳实践建议...