二进制转换代码分段错误

Question

我是 C 编程的新手（只用了 2 周）。我无法弄清楚为什么我的代码会引发分段错误。如果我将 long int num 设置为一个静态数字，我就能让程序工作。但是我需要程序能够接受来自命令行的用户输入（不是在程序运行后）

示例： ./binary 7

应该输出 7的二进制数是：111

我尝试过使用 strcpy(num, argv[0]) 但编译时也会抛出错误。

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

void decToBinary(long int num)   // Function Definition
{
    long int remainder[50];
    int i=0;
    int length=0;
    printf("The binary number for %d is: ",num);
    while(num > 0)
     {
             remainder[i]=num%2;    // does the mod function
             num=num/2;             // Divides original number by 2
             i++;                   // Increases count for the upcoming for-loop
             length++;              // Increases length display digits
     }
    for(i=length-1;i>=0;i--)        // Prints out the binary number in order (ignoring the previous 0's)
    {
            printf("%ld",remainder[i]);
    }
    printf("\n");                   // Adds a new line after the binary number     (formatting)
}
//================================================================================================
int main(char argc, char* argv[])
{
    long int num;                           //HOW DO I TAKE ARGV[0] AND MAKE IT A USEABLE VARIABLE???

    printf("Enter the decimal number:  ");  //TEMPORARY until problem above is solved
    scanf("%ld",&num);                      //TEMPORARY until problem above is solved

    decToBinary(*num);                      // Calling decToBinary function
    return 0;                               // Program terminated successfully
}

Answer 1

提示1：如果你想在decToBinary()函数中改变num的值，你必须在其中传递它的地址。所以你应该这样称呼它：

decToBinary(&num);

那么你的函数原型应该是这样的：

void decToBinary(long int *num)

所以你也必须正确地修改函数的主体。我猜这就是导致分段错误的原因。

---

提示2：argc 代表参数计数，因此它的类型不是char，而是int。

---

正如 @Kerrek SB 在他的 cmets 中指出的那样，scanf(3) 是一个返回值类型为 int 的函数。检查返回值并处理任何可能发生的错误被认为是明智的。

---

argv[0] 是可执行文件的名称。如果您想使用命令行中的第一个参数，它存储在argv[1] 中，并且它的类型为char *。所以如果你想用它作为一个数字，你可以使用stdlib.h的these函数之一。

Answer 2

类似的东西（不处理一些错误情况）

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

void decToBinary(long int num)   // Function Definition
{
    long int remainder[50];
    int i=0;
    int length=0;
    printf("The binary number for %ld is: ", num);
    while(num > 0)
     {
             remainder[i]=num%2;    // does the mod function
             num=num/2;             // Divides original number by 2
             i++;                   // Increases count for the upcoming for-loop
             length++;              // Increases length display digits
     }
    for(i=length-1;i>=0;i--)        // Prints out the binary number in order (ignoring the previous 0's)
    {
            printf("%ld",remainder[i]);
    }
    printf("\n");                   // Adds a new line after the binary number     (formatting)
}
//================================================================================================
int main(int argc, char* argv[])
{
    if (argc > 1)
    {
        long int mylong = atol(argv[1]);
        decToBinary(mylong);                      // Calling decToBinary function
    } else
    {
        int errno = 0;
        long int num;                           
        int res;

        printf("Enter the decimal number:  ");  //TEMPORARY until problem above is solved
        res = scanf("%ld",&num);                      //TEMPORARY until problem above is solved

        if (res != EOF)
        {
            decToBinary(num);                      // Calling decToBinary function
        } else
        if (errno != 0)
        {
            perror("scanf");
        } else
        {
            printf("No number found in input\n");
        }
    }

    return 0;                               // Program terminated successfully
}

** 我在 Ubuntu 14.04 上构建它 **

请注意 decToBinary(*num) 是错误的。您只能在指针上使用取消引用运算符，而不是 int 或任何非指针类型。

指针在内存中保存一个地址，当您取消引用它时，您是在告诉您的程序从该地址读取。如果您取消引用例如 10 的 int 值，则您是在告诉程序读取地址 10 处的值。这是一个非常低的地址值，将超出您的程序允许访问的内存范围，因此它将是分段错误。

首先你必须注意编译器警告。如果适用，在构建可执行文件时使用 -Werror 和可能的 -Wextra 标志。

例如 gcc -g -Werror -Wextra myfile.c -o myfile

这是学习使用调试器查找段错误的好机会，如果您对 C 编程的基本知识很认真的话。请注意上面的 -g 标志，它将调试符号构建到您的可执行文件中（没有符号，调试器将不知道函数名称）。然后，您可以使用 gdb（或其他适用的调试器）并启动您的可执行文件，然后运行它并获取回溯以告诉您何时发生段错误。

Answer 3

一种将argv[1] 参数提取到其组成位并显示这些位的简单方法...

// following eliminates any nonnumeric char from input
// by stopping at first nonnumeric char
long int value = atol(argv[1]);  
int position = sizeof( long int ) -1;

// following loop could be modified/prefixed with a loop to skip leading 0's
for( ; position >= 0; position-- )
{
    printf( "%c, ", ('0' + ( value >> position) & 1) );
}

printf( "\n" );