1

因此，凭借我在 C 方面的初级经验，我一直在尝试编写一个将十六进制输入转换为带有数组的十进制的代码。我相信通过查看我的代码，您会更清楚地了解它，但它不能正常工作。我不断收到比预期大得多的金额。

#include <stdio.h>

int main()
{


    int i, k, j, N, power, result;

    char array[50];

    result = 0;

    printf("how many charecters are there in your hexadecimal input?: \n");
    scanf("%d", &N);

    for(k=0; k<=N-1; k++)
    {
        printf("What is the %d . value of your hexadecimal input?: \n", k+1);
        scanf("%s", &array[k]);
    }


    for(i=0; i<=N-1; i++)
    {
        power = 1;
        for(j=0; j<=i; j++)
        {
        
            power = power *16;
        }
    
        if((array[i] >= 0) && (array[i] <= 9))
        {
            result = result + array[i] * power;
        
        }
    
        else
        {
            result = result + (array[i] - 'A' + 10) * power;
        }
    }   


    printf("your result is %d", result);

    return 0;
    

}
4

2 回答 2

2

您的代码过于复杂和错误，但想法是正确的。

你要这个：

#include <stdio.h>
#include <string.h>

int main()
{
  char array[50];
  scanf("%49s", array);            // just use a single scanf with %s
                                   // "49s" will avoid buffer overflow of array

  int length = strlen(array);      // get length of string (and use meaningful
                                   // variable name length instead of N)    
  int result = 0;
  for (int i = 0; i < length; i++)
  {
    int nibble;                    // nibble is the hexadécimal "digit" from 0 to 15

    if ((array[i] >= '0') && (array[i] <= '9'))
    {
      nibble = array[i] - '0';
    }
    else
    {
      nibble = array[i] - 'A' + 10;
    }

    result *= 16;                  // no need to maintain power, just multiply by 16
                                   // on each run
    result += nibble;              // ... and add the nibble
  }

  printf("your result is %d", result);
  return 0;
}

仍有改进的余地：

  • 你应该接受小写
  • 没有针对“G”、“H”等无效字符的测试。
  • 您应该将转换代码放在一个函数中，例如
    int HexStringToDecimal(const char *hexstring)
  • 当然还有很多我忘记的东西。

为您练习：转换该代码，使其转换为十进制字符串而不是十六进制 字符串。提示：代码会更简单。

边注：

避免这样的结构：

for(k=0; k<=N-1; k++)

而是使用这个更易读的等价物：

for(k=0; k<N; k++)
于 2020-12-01T12:52:12.597 回答
1

只是稍微编辑了你的代码，而且输入应该是从左到右 而不是从右到左，我认为你已经编码了。因此，对于 C6，首先是 C，然后是 6。

#include <stdio.h>

int main()
{


    int i, k, j, N, power, result;

    char array[50];

    result = 0;

    printf("how many charecters are there in your hexadecimal input?: \n");
    scanf("%d", &N);

    for(k=0; k<=N-1; k++)
    {
        printf("What is the %d . value of your hexadecimal input?: \n", k+1);
        scanf(" %c", &array[k]);
    }


    for(i=0; i<=N-1; i++)
    {
        power = 1;
        
        for(j=0; j<=N-2-i; j++)
        {
            power = power*16;
        }
        
        if((array[i] >= '0') && (array[i] <= '9'))
        {
            result = result + (array[i] - '0') * power;
        
        }
    
        else
        {
            result = result + (array[i] - 'A' + 10) * power;
        }
    }   


    printf("your result is %d", result);

    return 0;
    
}

外卖

  1. 您的输入效率低下。并且在 scanf 中也是 %c 而不是 %s。
  2. if((array[i] >= 0) && (array[i] <= 9))，假设 i=1 和 array[1] = 7。现在，您正在比较 7 的 ascii 值，即 55，而不是 7。
  3. 在 if 的语句中同样的事情，正在使用 array[i] 的 ascii 值，所以我相应地减去了。
于 2020-12-01T15:04:31.923 回答