c - 另一个C题

Question

我有一段代码如下所示

      #include <stdio.h>
      #include <stdlib.h>
      void Advance_String(char [2],int );
      int Atoi_val;
      int Count_22;
      int Is_Milestone(char [2],int P2);
      char String[2] = "0";
      main()
      {
         while(1)
         {

           if(Is_Milestone(String,21)==1)
           {
                 if(atoi(String)==22)
                 {
                     Count_22 = Count_22 + 1;
                 }
           }
           Atoi_val = atoi(String);
         Advance_String(S,Atoi_val); 
         }
       }
      int Is_Milestone(char P1[2],int P2)
      {
             int BoolInit;
             char *Ptr = P1;

             int value = atoi(Ptr);
             BoolInit = (value > P2);
             return BoolInit;
      }
     void Advance_String(char P1[2],int Value)
     {

             if(Value!=7)
             {
               P1[1] = P1[1]+1;
             }
             else
             {
               P1[1] = '0';
                  P1[0] = P1[0]+1 ;
             }
     }

现在我的问题是 Count_22 永远不会增加，因为 char 增量永远不会达到 21 或更高的值。有人能告诉我这种意外行为的原因吗？我这里的问题是找到 Count_22 的值。有什么问题吗代码？

感谢和问候，

麦迪

Answer 1

您的代码可能是我见过的最糟糕的 C 代码之一(没有冒犯，每个人都需要学习)。

它有语法错误(可能是复制/粘贴问题)、逻辑问题、无意义的混淆、不良做法(全局)、缓冲区溢出(atoi 用在没有地方存储终止零字节的 char 上)、未初始化的值(Count_22)，令人惊讶的命名约定(混合 CamelCase 和下划线，变量和函数以大写字母开头)，无限循环，没有标题，我忘记了一些。

更多，如果你想让任何人帮助你调试这段代码，你应该在列表中说出它应该做什么......

回答最初的问题:为什么 Count_22 永远不会递增？

因为 Is_Milestone 始终为假(有或没有 @Jay 更改)。 Is_Milestone 的意图似乎是将字符串“22”的十进制值与整数 21(或 1， bool 结果 21 == 1)进行比较，具体取决于版本。

这是合乎逻辑的，因为 Advance_String 行为。既因为 String 的初始值不正确(可能应该是 char String[3] = "00";)，又因为 Value != 7 测试。我猜你想要的是将数字与 7 进行比较，但 atoi 可以处理完整的字符串。在循环体中实现 Atoi_val = atoi(String+1); 的另一个小改动。话又说回来，你不会看到太多，因为循环永远不会停止，也不会打印任何东西。

如果这是第一次尝试某位老师给出的练习(例如“以 7 为基数编写两位数计数器”或类似内容)。您应该考虑根本不使用 atoi 并使用类似以下内容将字符数字转换为值:

digit_value = char_value - '0';

例子:

char seven_as_char = '7';
int seven_as_int = seven_as_char - '0';

如果您能解释您真正想要做什么，我们可能会向您展示一些简单的示例代码，而不是您试图调试的恐怖内容。

编辑

原来的代码真的更简单...

阅读 Ada 源代码后，我可以确认它确实是一个基于 Ascii 的八进制计数器。原始代码的质量本来就很差，这部分解释了生成的 C 代码的质量差。

一个可能的直接端口可能如下(但仍然需要认真清理才能看起来像 native C 代码......并且无论如何它都非常愚蠢，因为它打印一个常量):

#include <stdio.h>
#include <stdlib.h>

void Advance_String(char * P1)
{
     if((P1[1]-'0') != 7){
         P1[1]++;
     }
     else{
         P1[1] = '0';
         P1[0]++ ;
     }
}


int Is_Milestone(char * P1, int P2)
{
    return (atoi(P1) > P2);
}

main()
{
    int Count_11 = 0;
    int Count_22 = 0;
    int Count_33 = 0;
    int Count_44 = 0;
    char S[3] = "00";

    int cont = 1;

    while(cont)
    {
        if(Is_Milestone(S, 10)){
            if(atoi(S) == 11){
                Count_11 = Count_11 + 1;
            }
            if(Is_Milestone(S, 21)){
                if(atoi(S) == 22){
                    Count_22 = Count_22 + 1;
                }
                if(Is_Milestone(S, 32)){
                    if(atoi(S) == 33){
                        Count_33 = Count_33 + 1;
                    }
                    if(Is_Milestone(S, 43)){
                        if(atoi(S) == 44){
                            Count_44 = Count_44 + 1;
                        }
                        if (atoi(S) == 77){
                            cont = 0;
                        }
                    }
                }
            }
        }
        Advance_String(S);
    }
    printf("result = %d\n", Count_11 + Count_22 + Count_33 + Count_44);
}