c - C 中存储的数据被损坏

Question

我正在学习自己用 c 编程，基本上没有以前的编程经验，现在我有一个奇怪的错误，想询问一些建议。

在程序流程中，一些输入数据突然发生变化，我看不出也无法解释为什么会发生这种情况。

1.程序启动 -> 用户获得菜单选择 -> 选择情况 1 -> 程序提供和额外的输入可能性......到目前为止，一切正常，正如我在 Debug模式下看到的那样。

2.用户输入一些数字，变量就会设置为该数字。(这发生在 input.c 中)

3.然后程序在main.c中继续到info_bottom()。(在circlemenu.c中)4.在那里调用函数 getRadius() ，该函数应该根据第 2 点中 input.c 中的用户输入集计算半径。

5.该计算函数位于circlefunctions.c

但是这里/那里有奇怪的事情。

如果我查看调试器，我会发现变量直径和半径都更改为一些奇怪的数字，而不是用户在第 2 点中指定的数字。据我判断，存储在指针中的数据不知何故被损坏了。不幸的是，我仍然缺乏调试经验，无法自己找出答案，所以希望有人能告诉我发生了什么事。在尝试删除程序中的所有全局变量时出现此问题。

我开始对专业程序员产生全新的尊重和理解，以及为什么有时需要这么长时间才能修复错误 o0。

(我也制作了相应的头文件，但我认为不需要将它们放在上面？)

main.c

#include <stdio.h>

#include "menu/menu.h"
#include "circle/circlemenu.h"
#include "input/input.h"

int main(void)
{
    while(1)
{
    menu();
    switch(menu_user_input())
{
    case 1:

        info_top();
        cir_user_input();
        info_bottom();
        break;

    case 2:
        system("cls");
        break;

    case 3:
        system("cls");
        break;

    case 8:
        system("cls");
        break;

    case 9:
        system("cls");
        break;

    case 0:
        return(0);

    default:
        system("cls");
        printf("\n **Wrong choice try again...**\n");
        break;
}

}
 return 0;
}

菜单.c

#include <stdio.h>
#include "menu.h"

void menu()
{
    printf(" \n Made by ************* 2015.\n");
    printf(" ----------------------------------------\n\n");
    printf(" 1. Calculate circle measurements. \n");
    printf(" 2. \n");
    printf(" 3. \n");
    printf(" 8. Info. \n");
    printf(" 9. Clear screen. \n");
    printf(" 0. Exit. \n \n");
    printf(" Make your choice and press enter: ");
}

输入.c

#include <stdio.h>
#include "input.h"

int menu_user_input()
{
    static int number;
    scanf(" %i", &number);
    return number;
}

float cir_user_input()
{
    static float diameter;
    scanf("%f", &diameter);
    return diameter;
}

圆菜单.c

#include <stdio.h>
#include "circlemenu.h"
#include "circlefunctions.h"

void info_top()
{
    system("cls");
    printf(" ----------------------------------------\n");
    printf(" Typ the diameter of the circle: ");
    return;
}

void info_bottom()
{
    printf(" ----------------------------------------\n");
    printf(" The radius = %f \n\n" , getRadius());
    printf(" The surface = %f \n\n" , getSurface());
    printf(" The outline = %f \n" , getOutline());
    printf(" ----------------------------------------\n");
    return;
}

圆函数.c

#include "circlefunctions.h"
#include "../input/input.h"
#define PI 3.14

double getRadius()
{
    static diameter = (*cir_user_input);
    double radius = diameter / 2;
    return radius;
}

double getSurface()
{
    double radius = getRadius();
    return PI * (radius * radius);
}

double getOutline(){
    double radius = getRadius();
    return 2 * PI * radius;
}

Answer 1

您在 getRadius 函数中像变量/指针一样使用 cir_user_input，尽管它是一个函数。令人惊讶的是，它的工作没有任何警告或错误(知道您正在使用什么编译器会很有趣)。

我认为您真正想要的是这样的，即调用cir_user_input，它将结果存储在浮点变量中。

double getRadius()
{
    float diameter = cir_user_input(); //float type; not static (see below)
    double radius = diameter / 2;
    return radius;
}

此外，main 函数中的 switch 语句仅调用用户输入函数 cir_user_input，但不调用任何计算例程(情况 1 )。由于函数的返回值不存储，所以以后无法使用。

此外，您似乎对 static 关键字的使用感到困惑:

This problem occured while trying to get rid of all global variables in my program.

当您使用static关键字在函数内声明变量时，它们在整个函数调用过程中保留其值，即它们实际上是全局的。

相反，如果您对全局变量/函数等使用static关键字，则它仅在您指定它的文件内可见。对于初学者来说，这通常会令人困惑，因为关键字单词的书写方式完全相同，请参阅 What does "static" mean? .

最诚挚的问候安德烈亚斯