使用命令行界面和 malloc 创建计算器，

Question

我正在研究 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;
}

Answer 1

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

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

或者，您可以修改SI的返回类型，使其返回一些结果——修改结构（返回类型Calc ）或仅返回单个结果（返回类型int ）。 然后，你还要加上return语句转换成函数返回值，并且，在main例程中，你可以使用x = SI(*pCalc); 将返回值赋给x 。

Answer 2

您可以将pCalc指针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;
}