Question

我写了一个程序，下面显示的问题是当我使用gets（）函数然后在代码块16.01中执行代码时调用gets函数然后突然代码块停止工作。任何人都可以帮我解决这个问题。

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

#define PI 3.1415926535897932384626433832795

struct elements
{
    float A, L, x1, y1, x2, y2;
    double deg, c, s, E, esm[4][4];
    int n1, n2;
} * ele;

int nonod, noele;
void elestiffmat(int);

void main()
{
    int i;
    char ch;
    printf("\nEnter the number of nodes you want to create:");
    scanf("%d", &nonod);
    printf("\nEnter the number of elements you want to create:");
    scanf("%d", &noele);
    if (noele < (nonod - 1))
    {
        printf("\nThe problem is wrongly modeled.\nPress any key to abort.");
        getch();
        exit(0);
    }
    ele = (struct elements*)malloc(noele * sizeof(struct elements));
    for (i = 0; i < noele; i++)
    {
        printf("\nCreating element %d", i + 1);
        printf("\nCross sectional area?");
        scanf("%f", &ele[i].A);
        printf("\nYoung's Modulus?");
        scanf("%Lf", &ele[i].E);
        printf("\nStarting node number?");
        scanf("%d", &ele[i].n1);
        printf("\nIts coordinates:");
        scanf("%f%f", &ele[i].x1, &ele[i].y1);
        printf("\nEnding node number?");
        scanf("%d", &ele[i].n2);
        printf("\nIts coordinates:");
        scanf("%f%f", &ele[i].x2, &ele[i].y2);
        ele[i].L = sqrt(pow((ele[i].x2 - ele[i].x1), 2) + pow((ele[i].y2 - ele[i].y1), 2));
        if ((ele[i].x2) - (ele[i].x1) == 0)
        {
            if (ele[i].y2 > ele[i].y1)
            {
                ele[i].deg = PI / 2.0;
            }
            if (ele[i].y2 < ele[i].y1)
            {
                ele[i].deg = (3.0 * PI) / 2.0;
            }
        }
        if ((ele[i].y2) - (ele[i].y1) == 0)
        {
            if (ele[i].x2 > ele[i].x1)
            {
                ele[i].deg = 0.0;
            }
            if (ele[i].x2 < ele[i].x1)
            {
                ele[i].deg = PI;
            }
        }
        if (((ele[i].y2) - (ele[i].y1)) / ((ele[i].x2) - (ele[i].x1)) < 0)
        {
            ele[i].deg = PI + atanf(((ele[i].y2) - (ele[i].y1)) / ((ele[i].x2) - (ele[i].x1)));
        }
        if (((ele[i].y2) - (ele[i].y1)) / ((ele[i].x2) - (ele[i].x1)) > 0)
        {
            ele[i].deg = atanf(((ele[i].y2) - (ele[i].y1)) / ((ele[i].x2) - (ele[i].x1)));
        }
        // printf("%g",ele[i].deg);
        ele[i].c = cos(ele[i].deg);
        ele[i].s = sin(ele[i].deg);
        // printf("c=%g\ts=%g",ele[i].c,ele[i].s);
        elestiffmat(i);
    }
    getch();
}
void elestiffmat(int i)
{
    char choice;
    ele[i].esm[0][0] = ele[i].c * ele[i].c * ele[i].A * ele[i].E / ele[i].L;
    ele[i].esm[0][1] = ele[i].s * ele[i].c * ele[i].A * ele[i].E / ele[i].L;
    ele[i].esm[0][2] = -ele[i].c * ele[i].c * ele[i].A * ele[i].E / ele[i].L;
    ele[i].esm[0][3] = -ele[i].s * ele[i].c * ele[i].A * ele[i].E / ele[i].L;
    ele[i].esm[1][0] = ele[i].s * ele[i].c * ele[i].A * ele[i].E / ele[i].L;
    ele[i].esm[1][1] = ele[i].s * ele[i].s * ele[i].A * ele[i].E / ele[i].L;
    ele[i].esm[1][2] = -ele[i].s * ele[i].c * ele[i].A * ele[i].E / ele[i].L;
    ele[i].esm[1][3] = -ele[i].s * ele[i].s * ele[i].A * ele[i].E / ele[i].L;
    ele[i].esm[2][0] = -ele[i].c * ele[i].c * ele[i].A * ele[i].E / ele[i].L;
    ele[i].esm[2][1] = -ele[i].s * ele[i].c * ele[i].A * ele[i].E / ele[i].L;
    ele[i].esm[2][2] = ele[i].c * ele[i].c * ele[i].A * ele[i].E / ele[i].L;
    ele[i].esm[2][3] = ele[i].s * ele[i].c * ele[i].A * ele[i].E / ele[i].L;
    ele[i].esm[3][0] = -ele[i].s * ele[i].c * ele[i].A * ele[i].E / ele[i].L;
    ele[i].esm[3][1] = -ele[i].s * ele[i].s * ele[i].A * ele[i].E / ele[i].L;
    ele[i].esm[3][2] = ele[i].s * ele[i].c * ele[i].A * ele[i].E / ele[i].L;
    ele[i].esm[3][3] = ele[i].s * ele[i].s * ele[i].A * ele[i].E / ele[i].L;
    printf("\nDo you want to print the element stiffness matrix (y/n)?");
    gets(choice);
    if (choice == 'y' || choice == 'Y')
    {
        printf("\nThe element stiffness matrix of the element %d is:-", i + 1);
        int j, k;
        for (j = 0; j < 4; j++)
        {
            printf("\n");
            for (k = 0; k < 4; k++)
            {
                printf("%15g", ele[i].esm[j][k]);
            }
        }
    }
}

Answer 1

如果我重建（你的代码），我会收到这样的警告

然后，尝试使用scanf(" %c", &choice);

％c转换说明符不会自动跳过任何前导空格，因此如果输入流中有一个迷路换行符。 scanf调用将立即使用它。

解决此问题的一种方法是在转换说明符之前的格式字符串中放置一个空格：

scanf(" %c", &choice);

格式字符串中的空白告诉scanf跳过前导空格，并且将使用％c转换说明符读取第一个非空白字符。检查重建，我用

没有警告

每当gets（）语句遇到时，用户输入的字符（带空格的字符串）将被复制到变量中。
如果用户开始接受字符，并且新行字符然后出现换行符不会被复制到字符串中变量（即名称）。
终止空字符会自动附加到复制到字符串vriable（即名称）
gets（）使用stdin（Standered Input Output）作为源，但它没有在结果字符串中包含结束换行符并且确实如此不允许为字符串变量指定最大大小（可以导致缓冲区溢出）。

gets（）函数无法阻止您输入字符，因此应该避免使用。

我希望这完全使用。对不起，如果有任何错误:)我学习

Answer 2

我不能说为什么它会停止工作＆＃34;在您的系统上，虽然已经提到过，但您传递的参数不正确，因此预计会出现未定义的行为。请参见手册页中的以下内容：

char * gets（char * s）;

说明

永远不要使用此功能。

gets（）从stdin读取一行到s指向的缓冲区，直到终止换行符或EOF，它替换为   空字节（＆＃39; \ 0＆＃39;）。不执行缓冲区溢出检查          （见下面的BUGS）。

所以有两个错误，传递一个char而不是一个缓冲区指针，并首先使用 gets 。该手册进一步解释了为什么使用获取

这么危险

BUGS   永远不要使用gets（）。因为没有它是不可能的   事先知道数据get（）将读取多少个字符，以及   因为gets（）将继续存储超过结束的字符   缓冲，使用起来非常危险。它已被用来打破   计算机安全。改为使用fgets（）。

有关更多信息，请参阅CWE-242（又名＆＃34;使用固有危险   功能＆＃34;）http://cwe.mitre.org/data/definitions/242.html

简单地说，假设您已声明选择如下：

char choice[2]; // e.g. 'y' plus terminating null byte

没有什么可以阻止用户进入＆＃34;是＆＃34;或其他任何超出缓冲区大小的东西。

请记住，您将传递指针，并且每次将指针传递给缓冲区时，接收函数也应该接收缓冲区大小并确保它不会越过其边界。由于此函数不会将大小作为参数，因此现在不推荐使用它（在这种情况下，读取＆＃34;禁止＆＃34;

无论如何，有很多选择，这是一个潜在的解决方案。

替换＆＃34;字符选择＆＃34;用：

int choice, c;

而不是＆＃34;得到（选择）＆＃34;你可以用这个：

while( (c = getchar()) != '\n' && c != EOF )
    /* discard */ ;
choice = getchar();

注意＆＃34; getchar（）等同于getc（stdin）＆＃34;。它实际上并没有直接从键盘上读取。这就是我们在提示之前添加while循环以从输入流中丢弃未处理数据的原因。有关详细信息，请参阅this answer on c-faq。

使用获取功能时，代码块已停止工作

2 个答案: