持续接受三个数字并打印三个数字中的最大值的程序

Question

持续接受三个数字并打印三个数字中的最大值的程序。 我收到运行时错误（SIGABRT） 这是我的代码

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

int main()
{
    char ch[30];
    char *c[3];
    long int i[3], mx_2;
    int o;
    while((fgets(ch,sizeof(ch),stdin))&&ch!='\0')
    {
        c[0] = (char *)malloc(10);
        c[1] = (char *)malloc(10);
        c[2] = (char *)malloc(10);
        c[0] = strtok(ch," ");
        c[1] = strtok(NULL," ");
        c[2] = strtok(NULL," ");
        i[0] = atoi(c[0]);
        i[1] = atoi(c[1]);
        i[2] = atoi(c[2]);
        mx_2 = i[0] > i[1] ? (i[0] >i[2] ? i[0] : i[2]) : (i[1] > i[2] ? i[1] : i[2]);
        printf("%ld\n",mx_2);
        fflush(stdin);
        for (o = 0; o < 3; o++) {
            free(c[o]);
        }
    }

    return 0;
}

任何帮助家伙 谢谢

Answer 1

当您执行c[0] = malloc(10); 和一些句子之后您执行c[0] = strtok(...); 时，您将用strtok(3) 的结果覆盖c[0] 的指针值（它本身就是一个指针），不复制字符串内容。当您到达free(3) 的for 循环时，您将传递给free() 函数由strtok 给出的值，而不是从malloc() 给出的值（当您重新分配数组@987654331 时，这些值将永远丢失@values) 所以这是你的SIGABRT 最可能的原因。

顺便说一句，您甚至不需要在程序中执行任何malloc 和free。只需获得一个足够长的缓冲区来存储一整行输入，然后使用 strtok 获取所有块。另一方面，您必须测试strtok 结果，如果没有更多数据（您只输入两个值）它将返回NULL。

这段代码不仅可以处理三个值，还可以处理不超过数组大小的任何数字：

#include <stdio.h> /* for input output routines like fgets */
#include <stdlib.h> /* for the constant EXIT_SUCCESS */
#include <limits.h> /* for INT_MAX and INT_MIN */
#include <string.h> /* for strtok */

int main()
{
    char buffer[1024];
    while (fgets(buffer, sizeof buffer, stdin)) {
        /* we have one full line of input up to sizeof buffer chars.  */
        int max = INT_MIN;
        int min = INT_MAX;
        char *s = strtok(buffer, " \t\n");
        if (!s) {
            fprintf(stderr, "Invalid line\n");
            continue;
        }
        while (s) {
            int x = atoi(s);
            if (x > max) max = x;
            if (x < min) min = x;
            s = strtok(NULL, " \t\n");
        }
        if (max != INT_MIN)
            printf("MAX: %d\n", max);
        if (min != INT_MAX)
            printf("MIN: %d\n", min);
    } /* while */
    return EXIT_SUCCESS;
} /* main */

Answer 2

更好地使用 scanf 来完成您的任务：

 scanf(" %d %d %d", i, i+1, i+2);//note the leading space in the format