C 段错误源于读取标准输入然后操作该数据

Question

我有一个问题，我试图从标准输入读取单词，然后对其进行操作，然后稍后打印到标准输出。但是，当我不重定向标准输出时，如果我尝试打印从 STDIN 扫描的任何单词(我可以使用 put() 来显示信息到控制台，这很好)，它会在 2 个位置出现段错误，或者当我尝试访问每个单词的固有数据(就像我稍后使用的 strcasecmpr() )。这里有人可以帮忙吗？

int main(int argc, char** argv)
{
    int aflag = 0;
    int bflag = 0;
    int cflag = 0;
    int c;

    //int opteval
    Instruction head= NULL;
    Instruction first= NULL;
    init(head);
    puts("Init for head done");
    //While there are more instructions
    char line [128];
    printf("SdlJ");
    int numlines = 0;
    first = head; //Setting pointer to first instruction in list CHECK THIS LATER
    int index = 0;
    int rc = 0;
    char instr[20], r1[300], r2[300], r3[300], r4[300];
    instr[0] = '\0';
    r1[0] = '\0';
    r2[0] = '\0';
    r3[0] = '\0';
    r4[0] = '\0';

    while( fgets ( line, sizeof line, stdin) != NULL){
        puts(line);
        rc = sscanf(line, "%s %s %s %s %s", instr,r1,r2,r3,r4);
        if(rc == 4)
            puts("here");
        puts(instr);
        //printf("sfds%ssfsd", instr); SEGFAULT CASE 1: THIS WOULD SEG FAULT IF I UNCOMMENTED IT
        //return 1;
        chooseOp(head, instr);
        createReg(head, r1, 0);

        if(r2[0] != '=' && r2[0] != '\0')
            createReg(head, r2, 1);

        if(r3[0] != '='&& r3[0] != '\0')
            createReg(head, r3, 2);

        if(rc < 5)
            createReg(head, r4, 3);
        puts("here");
        Instruction next = NULL;
        init(next);
        numlines++;
        head->next = next;
        next->prev = head;
        head = next;
        //Point prevs's next to new head, and that heads prev to the previous head.
    }
}

void chooseOp(Instruction head, char* token){
    //Simple function to find the opcode responsible for this instruction
    //printf("sfdjls");

    puts("segfaulting");
    // SEGFAULT PLACE 2, IT SEGFAULTS RIGHT AT THIS STRCASECMP BELOW
    if(strcasecmp("nop", token) == 0){
        head->opcode= NOP;
        head->ismem=1;
        head->latency = 1;
    }
    else if(strcasecmp("addI", token) == 0){
        head->opcode= ADDI;
        head->ismem=1;
        head->latency = 1;
    }
    else if(strcasecmp("add", token) == 0){
        head->opcode= ADD;
        head->ismem=1;
        head->latency = 1;
    }
    else if(strcasecmp("subI", token) == 0){
        head->opcode= SUBI;
        head->ismem=1;
        head->latency = 1;
    }
    else if(strcasecmp("sub", token) == 0){
        head->opcode= SUB;
        head->ismem=1;
        head->latency = 1;
    }
    else if(strcasecmp("mult", token) == 0){
        head->opcode= MULT;
        head->ismem=1;
        head->latency = 3;
    }
    else if(strcasecmp("div", token) == 0){
        head->opcode= DIV;
        head->ismem=1;
        head->latency = 3;
    }
    else if(strcasecmp("load", token) == 0){
        head->opcode= LOAD;
        head->ismem=0;
        head->latency = 5;
    }
    else if(strcasecmp("loadI", token) == 0){
        head->opcode= LOADI;
        head->ismem=0;
        head->latency = 1;
    }
    else if(strcasecmp("loadAO", token) == 0){
        head->opcode= LOADAO;
        head->ismem=0;
        head->latency = 5;
    }
    else if(strcasecmp("loadAI", token) == 0){
        puts("here23");
        head->opcode= LOADAI;
        head->ismem=0;
        head->latency = 5;
    }
    else if(strcasecmp("store", token) == 0){
        head->opcode= STORE;
        head->ismem=0;
        head->latency = 5;
    }
    else if(strcasecmp("storeAO", token) == 0){
        head->opcode= STOREAO;
        head->ismem=0;
        head->latency = 5;
    }
    else if(strcasecmp("storeAI", token) == 0){
        head->opcode= STOREAI;
        head->ismem=0;
        head->latency = 5;
    }
    else if(strcasecmp("output", token) == 0){
        head->opcode= OUTPUT;
        head->ismem=0; //Need to ask prof about this
        head->latency = 1;
    }

    puts("here");
}

这是我应该阅读的示例:

 loadI 1024     => r0 
 loadI 0    => r1 
 storeAI r1     => r0, 0 

等等。

请帮忙。

编辑:这是我的 init(head) 代码

void init(Instruction head)
{
    head = (Instruction) malloc(sizeof(struct Command));
    head->opcode = 0;
    head->next = NULL;
    head->prev = NULL;
    head->ismem = -1;
    head->firstReg = NULL;
    head->secondReg = NULL;
    head->outputReg1 = NULL;
    head->outputReg2 = NULL;
    head->successors = NULL;
    //head->depends = NULL;
    head->numInstructions = 0;
    //head->lineNum = -1;
    head->priority = 0;
    head->numdepends = 0;
    head->cycle = 0;
    head->delay = -1;
}

Answer 1

这不会变成一个巨大的调试 session ，但这是代码中的一个基本问题，必须首先解决。

void init(Instruction head) // note: passed by-value
{
    head = (Instruction) malloc(sizeof(struct Command));
    // etc...
}

考虑一下:采用一个您想要用来将调用方 int 变量设置为某个值的函数。你会这样做:

void func(int *p) // note by-address
{
    *p = 42;
}

// caller code
int a = 0;
func(&a); // pass address

指针没有什么不同。指针保存地址，但也有地址(这将它们与数组区分开来，顺便说一句，数组只是地址)。您的 init() 应该如下所示:

void init(Instruction* head) // note: by address
{
    Instruction p = malloc(sizeof(struct Command)); // note no cast
    p->->opcode = 0;
    p->next = NULL;
    // etc...

    // save result to address given by out-parameter
    *head = p;
}

调用方:

Instruction head = NULL;
init(&head); // note address passed.

这应该能让你继续前进。

<小时/>

旁注

C 程序员喜欢星号。真的。不开玩笑。它们是代码中的巨大旗帜，从山顶尖叫着“这是一个指针”。将其隐藏在 typedef 中很少有帮助。事实上，我只见过三种可以说是合法的案例，其中之一是转移注意力。

1. 库 API 的抽象不透明“句柄”类型。这是常见且完全合法的。
2. 函数指针类型。通常用于确保不同的“回调”机制使用正确的类型(例如 typedef int (*callback_type)(void);)。
3. 避免在同一行上出现错误的变量声明。 (这是红鲱鱼)。

其中的第三个得到了最好的证明:

int* a,b; // declares one pointer-to-int, one int.

typedef int *int_ptr;
int_ptr a,b; // declared two pointers-to-int.

虽然这看起来很有帮助，但实际上没有。当您稍后在代码中尝试此操作时，编译器将崩溃:

*b = something;

所以无论如何(通常)你都会发现自己。

底线，不要害怕星号。如果您愿意，他们会帮助您更清楚地看到事情。