Question

我是C语言的新手，我不太明白为什么这个程序会导致失败。我觉得这是方法isTrue的结果。该方法的目的是确保输入的字符串是一个实际的整数，但它似乎不起作用。我不确定为什么。由于某种原因，我得到了一些奇怪的值。

/* Program: binToDec.c
   Author: Sayan Chatterjee
   Date created:1/25/17
   Description:
     The goal of the programme is to convert each command-line string
     that represents the binary number into its decimal equivalent 
     using the binary expansion method.
*/

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

/*Methods*/
int strLen(char *str);
int isTrue(char *str);
int signedBinToDec(char *str);

/*Main Method*/
int main(int argc, char **argv)
{
  /*Declaration of Variables*/
  int binNum = 0;
  int decNum = 0;
  int i;

  /*Introduction to the Programme*/
  printf("Welcome to Binary Converter\n");
  printf("Now converting binary numbers\n");
  printf("...\n");

  /*Check to see any parameters*/
  if(argc > 1)
  {
    for(i = 1; i < argc; i++)
    {
      if(isTrue(argv[i] == 1))
      {
        if(strLen(argv[i] <= 8))
        {
          binNum = atoi(argv[i]);
          decNum = signedBinToDec(binNum);
        }
        else
        {
          printf("You did not enter a 8-bit binary\n\a");
          break;
        }
        break;
      }
      else
      {
        printf("You did not enter a proper binary number\n\a");
        break;
      }
    } 

    /*Printing of the Result*/
    printf("Conversion as follows: \n");
    printf("%d\n", decNum);

  }
  else
  {
    printf("You did not enter any parameters. \a\n");
  }
}

int strLen(char *str)
{
  int len = 0;
  while(str[len] != '\0')
  {
    len++;
  }
  return len;
}

int isTrue(char *str)
{
  int index = 0;
  if(str >= '0' && str <= '9')
  {
    return 1;
  }
  else
  {
    return 0;
  }
}


int signedBinToDec(char *str)
{
  int i;
  int len = strLen(str);
  int powOf2 = 1;
  int sum = 0;

  for(i = len-1; i >= 0; i--)
  {
    if(i == 0)
    {
      powOf2 = powOf2 * -1;
    }
    sum = (str[i]*powOf2) + sum;
    powOf2 = powOf2 * 2;
  }
  return sum;
}

Answer 1

if声明

  if( isTrue(argv[i] == 1) )

非常错误。由于两个案例，它很糟糕

argv[i] == 1是指针和int之间的比较，这是非法的。这会导致约束违规。根据{{1}}，章节§6.5.9，等于运算符


以下其中一项应成立：

- 两个操作数都有算术类型;

- 两个操作数都是指向兼容类型的限定或非限定版本的指针;

- 一个操作数是指向对象类型的指针，另一个是指向限定符号或指针的指针   不合格的版本;或

- 一个操作数是指针，另一个是空指针常量。
比较结果再次使用C11值作为函数参数，而函数应该接受int。 char *和int不兼容。

看来，你打算写

char *

因为您需要比较if ( isTrue(argv[i]) == 1 )来电的返回值。

isTrue和其他人也一样。

那就是说，还有其他问题。

strLen(argv[i] <= 8)仅检查索引isTrue()中的值，对于传递的参数，您需要某种循环来检查整个字符串。
已有众所周知的库函数，如isDigit()，它可以很好地完成工作，尝试使用它们。

带C

1 个答案: