1

我正在研究 malloc 和命令行界面，我必须在其中创建和简单的计算器。我已经成功运行了程序，但我想从主程序打印我的结果，而不是从函数中，我无法从主程序运行它。每次我运行它都会显示垃圾值。什么地方出了错？

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

typedef struct {
    int num1;
    int num2;
    uint8_t *ope;
    int result;
} Calc;

void SI( Calc c) {
    if(strcmp(c.ope,"add")==0)
    {
        printf(" result is : %d\n :",c.result = (c.num1 + c.num2 ));
    }
    else if(strcmp(c.ope,"sub")==0)
    {
        printf(" result is :%d\n :",c.result = (c.num1 - c.num2 ));
    }
}

int main(int argc,char *argv[]) {
    Calc *pCalc = (pCalc *) malloc(sizeof(Calc));

    pCalc ->ope = (argv[1]);
    pCalc ->num1   = atoi (argv[2]);
    pCalc ->num2   = atoi (argv[3]);
    SI(*pCalc );

    printf("result is: %d\n", pCalc ->result);  // I want to print result here

    free(pCalc );

    return 0;
}
4

2 回答 2

2

SI(*pCalc );将结构的值（本质上是一个副本）传递给SI，并void SI( Calc c)声明SI有一个c初始化为传递值的参数。更改只影响参数c；它们不影响.SIcmain

您可以更改SI(*pCalc );为，SI(pCalc);以便它传递地址pCalc而不是其值，并且您可以更改函数声明，void SI(Calc *c)以便其参数c是指针并初始化为传递的地址。里面，SI改成c.。c->运算符访问结构的.成员，而->运算符访问指向的结构的成员。

或者，您可以修改 的返回类型SI，使其返回一些结果——修改结构（返回类型Calc）或只是单个结果（返回类型int）。然后，您将return在函数中添加一条语句以返回值，并且在main例程中，您可以使用x = SI(*pCalc);将返回的值分配给x.

于 2021-11-14T12:10:14.750 回答
0

您可以将指针传递pCalc给函数SI（如@EricPostpischil 所建议的那样）并将其设置在result那里。

此外，还添加了一些修复。请阅读评论// CHANGE HERE。

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

typedef struct
{   int num1;
    int num2;
    uint8_t *ope;
    int result;
} Calc;

// CHANGE HERE - accept pointer argument
void SI(Calc* c) {
    if (c == NULL)
    {
        return;
    }
    // CHANGE HERE - use strncmp instead of strcmp
    if (strncmp(c->ope, "add", 3) == 0)
    {
        c->result = (c->num1 + c->num2);
    }
    else if (strncmp(c->ope, "sub", 3) == 0)
    {
        c->result = (c->num1 - c->num2);
    }
}

int main(int argc,char *argv[]) {
    // CHANGE HERE - command line arguments validation
    if (argc != 4)
    {
        printf("Unexpected number of arguments\n");
        exit(1);
    }

    // CHANGE HERE - pCalc -> Calc
    Calc *pCalc = (Calc *) malloc(sizeof(Calc));

    pCalc ->ope = (argv[1]);
    pCalc ->num1   = atoi (argv[2]);
    pCalc ->num2   = atoi (argv[3]);
    SI(pCalc);  // CHANGE HERE - pass the pointer

    printf("result is: %d\n", pCalc->result);  // I want to print result here

    free(pCalc);

    return 0;
}
于 2021-11-14T12:20:19.607 回答