c - 投票系统实现中的错误

Question

我正在编写一个程序来创建选民 ID 列表，并允许他们投票给候选人。如果选民试图投票给不存在的候选人，则应将该名字添加到列表中。当我尝试使用“strcpy”复制已输入到candidate_name字段的名称时，出现段错误，我尝试直接复制它，但这会将所有先前节点的值覆盖为我最后输入的节点。正如之前链接列表被覆盖问题的解决方案之一中提到的，我也尝试了 strdup，但是当我这样做时，我没有得到我应该得到的结果。例如。如果我现在投票给“a”，并且在函数返回“EXISTS”后立即投票给“a”，但如果我尝试先投票给“a”，然后投票给“b”，然后再次投票给“a”，则函数返回“NEW”由于某种原因，检查列表中名称是否存在的循环仅执行一次，即使它必须执行两次。如果有人能告诉我应该做什么来纠正这个问题，我将不胜感激。另外，为什么“strcpy”会导致段错误？

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

typedef struct candidate{
    char *candidate_name;
    int vote_count;
    struct candidate *next;
}candidate;
struct candidate *candidate_head=NULL;
typedef struct voter{
    int voterid;
    int has_voted;
    struct voter *next;
}voter;
struct voter *voter_head=NULL;




    char *add_voter(int voterid){
        struct voter *v=voter_head;
        while(v!=NULL){
            if(v->voterid==voterid)
                return "Already exists";
            v=v->next;
        }

        voter *v1=malloc(sizeof(voter));
        v1->voterid=voterid;
        v1->has_voted=0;
        v1->next=voter_head;
        voter_head=v1;
        return "Added";
    }

    char *add_candidate(char *c_name, int voterid){
        int flag=0;
        struct voter *v=voter_head;
        while(v!=NULL){
            if(v->voterid==voterid){
                flag=1;
                if(v->has_voted==1){
                    return "ALREADY VOTED";
                }
                else{
                    v->has_voted=1;
                }
            }    
            v=v->next;
        }

        if(flag==0)
            return "Invalid Voter ID";

            struct candidate *c=candidate_head;
            if(candidate_head!=NULL)
            printf("see-%s\n",candidate_head->candidate_name);
                while(c!=NULL){
                    printf("name0 c%s\n",c->candidate_name);
                    printf("name1 c%s\n",c_name);
                    if(strcmp(c_name,c->candidate_name)==0){
                        c->vote_count++;
                        return "EXISTS";
                    }
                c=c->next;
                }

                struct candidate *c1=malloc(sizeof(candidate));
                // strcpy(c1->candidate_name,c_name);
                c1->candidate_name=strdup(c_name);
                printf("\nname1 %s\n",c1->candidate_name);
                printf("name2 %s\n",c_name);
                c1->vote_count=1;
                c1->next=candidate_head;
                candidate_head=c1;
                candidate_head->next = NULL;
                return "NEW";   


    }


    int main(){

        int voterid,count;
        char *cand=malloc(sizeof(char));
        struct candidate *c=malloc(sizeof(candidate));
        for(count=0;count<4;count++){
            printf("Enter voter ID\n");
            scanf("%d",&voterid);
            printf("%s\n",add_voter(voterid));  
        }

        for(count=0;count<4;count++){
            printf("Enter candidate name\n");
            scanf("%s",cand);
            printf("Enter voter ID\n");
            scanf("%d",&voterid);
            printf("%s\n", add_candidate(cand,voterid));

        }

        return 0;

    }

Answer 1

解决代码中与分配内存有关的错误后，它似乎运行时没有错误。

首先，该行:

char *cand=malloc(sizeof(char));//(主线第二行)

仅分配 1 个字节给预期的字符数组 cand，因此在下一行中，scanf() 返回
尝试编写超出字符串的内容。为您的候选人姓名分配更多内存。和 free() cand 当你完成了。
提供更多内存:

char *cand = malloc(100);//将保存 99 个字符的名称(加上\0)

第二，c1->candidate_name 从未被给予空间。您正在尝试复制到不属于您的内存中。
注意:行:

struct candidate *c1=malloc(sizeof(candidate));  

不为成员candidate_name分配内存，您必须明确地这样做

c1->candidate_name = malloc(SOME_NUMBER_OF_BYTES);
// then when you are done, free it:  
free(c1->candidate_name);