带有可变函数参数的分段错误

Question

我的程序编译没有错误，但是当我运行它时，在输入一个半径等值之后，它会以分段错误退出。

我已经构建了一个可变数量的参数，但我怀疑它可能与我在“shape_area”函数的操作过程中调用每个参数的方式有关。

任何人都可以帮助解释我在这里做错了吗？

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

double shape_area(double shapetype, ...); 

int main(void) 
{ 
    int shapetype; 
    double radius, side, length, width, area; 

    printf("\n\nPlease enter the type of shape you wish to get the area of:\n"); 
    printf("| 1-Circle | 2-Square | 3-Rectangle |\n"); 
    scanf("%d", &shapetype); 

    // Circle 
    if(shapetype == 1) 
    { 
     printf("\nPlease enter the radius of your circle: "); 
     scanf("%lf", &radius); 
     area = shape_area(shapetype, radius); 
    } 

    // Square 
    else if(shapetype == 2) 
    { 
     printf("\nPlease enter the side length of your square: "); 
     scanf("%lf", &side); 
     area = shape_area(shapetype, side); 
    } 

    // Rectangle 
    else if(shapetype == 3) 
    { 
     printf("\nPlease enter the side length of your square: "); 
     scanf("%lf", &length); 
     printf("\nPlease enter the side length of your square: "); 
     scanf("%lf", &width); 
     area = shape_area(shapetype, length, width); 
    } 

    else 
    { 
     printf("\n\nInvalid Input!\n"); 
     return (0); 
    } 

    printf("\n\nArea of Shape: %lf\n\n", area); 

    return (0); 

} 

double shape_area(double shapetype, ...) 
{ 
    va_list args; 

    double temparea; 
    double radius; 
    double side; 
    double length; 
    double width; 

    radius = va_arg (args, double); 
    side = radius; 
    length = radius; 
    width = va_arg (args, double); 

    if(shapetype == 1) 
    { 
     temparea = M_PI*radius*radius; 
    } 

    if(shapetype == 2) 
    { 
     temparea = side*side; 
    } 

    if(shapetype == 3) 
    { 
     temparea = length*width; 
    } 

    va_end (args); 
    return temparea; 
} 

Answer 1

需要初始化使用va_start

double shape_area(double shapetype, ...) 
{ 
va_list args; 
va_start(args,shapetype); 
. 
. 

Answer 2

warzon正确地指出，你需要使用的va_start中的args列表，但这里是如何弄清楚发生了什么事情在未来：

# The -g is the important part here, -O0 will help too if you don't care about optimization. 
$ gcc -Wall -std=gnu99 -O0 -Wextra -Werror -g -foptimize-sibling-calls -o shape shape.c 
$ gdb ./shape 
... 
(gdb) b shape_area # Set a breakpoint. 
Breakpoint 1 at 0x80485b8: file shape.c, line 63. 
(gdb) run 
Please enter the type of shape you wish to get the area of: 
| 1-Circle | 2-Square | 3-Rectangle | 
1 
Please enter the radius of your circle: 3 
63   radius = va_arg (args, double); 
(gdb) next # Run the line that assigns the radius. 
65   side = radius; 
(gdb) p radius 
$1 = 0 

您将在此之前触发段错误，或者为您的va_arg查看错误的值。这导致va_args的手册页，其中指出：

参数ap是由va_start（）初始化的va_list ap。

由于您忘记了调用va_start（），所以应该导致aha时刻。一般来说，如果你得到一个段错误，首先要做的就是启动一个调试器。这很可能会指出你对这个问题的看法。

Answer 3

在我看来，你所做的错误是将所有不同的公式变成一个函数。当然，@warzon是正确的，在这种情况下你需要va_start，但我认为这种方法没有优势，除非，可以想象，这个问题需要你使用它。 shape_area中的大部分共享代码涉及va_list机制的开销！

如果你想周围的信息传递有关的一些类型之一的形状（你不能使用继承，因为C不是OO），你会更好地创建一个struct或struct个union 。在你的程序中，不过你也可以制作circle_area，square_area,rectangle_area并且调用合适的。这也避免了需要记录该说法！