c - 嵌套的 if-else 语句给我带来了问题

Question

我编写的代码应该计算字符指针列表中 a、c、t 和 g 的数量。然后如果输入的字符不是 a、c、t 或 g，则应该将无效字母添加到字符列表 invalidBase 中。相反，当我输入数据 agtcpoop 时，它会打印出 Invalid Base: 但当它应该打印出 poop 时，字符不存在。请帮忙!这是代码:

void countBase(char *p)
{
   int aCount = 0, cCount = 0, tCount = 0, gCount = 0;
   char invalidBase[100];
   int i, j=0;
   while(*p != '\0')
{
  if(*p == 'A' || *p == 'a')
  {
     aCount++;
  }
  else if(*p == 'C' || *p == 'c')
  {
     cCount++;
  }
  else if(*p == 'T' || *p == 't')
  {
     tCount++;
  }
  else if(*p == 'G' || *p == 'g')
  {
     gCount++;
  }
  else
  {
     invalidBase[j] = *p;
  }
  j++;
  p++;
}

for(i = 0; invalidBase[i] != '\0'; i++)
{
   printf("Invalid Base: %c\n", invalidBase[i]);
}

printf(" A: %i\n C: %i\n T: %i\n G: %i\n", aCount, cCount, tCount, gCount);
}

Answer 1

Yoy 通过使用具有自动存储持续时间的未初始化变量的值调用了未定义的行为，这是不确定的。

改进:

• 仅当数据存储到 invalidBase 时才增加计数器 j。
• 使用j 确定要打印的长度。

改进的代码:

void countBase(char *p)
{
   int aCount = 0, cCount = 0, tCount = 0, gCount = 0;
   char invalidBase[100];
   int i, j=0;
   while(*p != '\0')
   {
      if(*p == 'A' || *p == 'a')
      {
         aCount++;
      }
      else if(*p == 'C' || *p == 'c')
      {
         cCount++;
      }
      else if(*p == 'T' || *p == 't')
      {
         tCount++;
      }
      else if(*p == 'G' || *p == 'g')
      {
         gCount++;
      }
      else
      {
         if(j < (int)(sizeof(invalidBase) / sizeof(*invalidBase))) /* avoid buffer overrun */
         {
            invalidBase[j] = *p;
            j++;
         }
      }
      p++;
   }

   for(i = 0; i < j; i++)
   {
      printf("Invalid Base: %c\n", invalidBase[i]);
   }

   printf(" A: %i\n C: %i\n T: %i\n G: %i\n", aCount, cCount, tCount, gCount);
}

请注意，您可以使用 switch 语句代替这些许多 if-else 语句。

void countBase(char *p)
{
   int aCount = 0, cCount = 0, tCount = 0, gCount = 0;
   char invalidBase[100];
   int i, j=0;
   while(*p != '\0')
   {
      switch(*p)
      {
      case 'A':
      case 'a':
         aCount++;
         break;
      case 'C':
      case 'c':
         cCount++;
         break;
      case 'T':
      case 't':
         tCount++;
         break;
      case 'G':
      case 'g':
         gCount++;
         break;
      default:
         if(j < (int)(sizeof(invalidBase) / sizeof(*invalidBase))) /* avoid buffer overrun */
         {
            invalidBase[j] = *p;
            j++;
         }
         break;
      }
      p++;
   }

   for(i = 0; i < j; i++)
   {
      printf("Invalid Base: %c\n", invalidBase[i]);
   }

   printf(" A: %i\n C: %i\n T: %i\n G: %i\n", aCount, cCount, tCount, gCount);
}