Question

我试图理解为什么我的代码崩溃了。我有一系列结构，如下所示：

typedef struct contact {

    char cFirstName[10];
    char cLastName[10];
    char cTelphone[12];

} address ; // end type

在代码中我像这样初始化数组：

address myContacts[5];

for ( i = 0; i < 5 ; i++ ){
        strcpy(myContacts[i].cFirstName, "0");
        strcpy(myContacts[i].cLastName,"0");
        strcpy(myContacts[i].cTelphone,"0"); 
    }

这有效：

for ( i = 0; strcmp(myContacts[i].cFirstName,"0") != 0 ; i++ ){                                             
        printf("\nmyContacts[%d].cFirstName: %s", i, \
        myContacts[i].cFirstName );
    }// end for

所以，我只打印出有内容的联系人。

但是，我无法理解为什么我的搜索联系功能不起作用：

void searchContact( address * myContacts,    char * name ){
    int found = 1;
    int i = 0;

    for ( i = 1; found != 0 ;i++ ){
    found=strcmp(myContacts[i-1].cFirstName, name);

    printf(" Name Found %s",   myContacts[i-1].cFirstName);
    }
} // end of searchContacts

我这样称呼这个函数：

printf("\nEnter a name or part of a name to search:\n");
            fscanf(stdin, "%s", buffer);
            getchar(); // clear the last enter
            printf("\nThe line you entered was:\n");
            printf("%s\n", buffer);
            searchContact( myContacts, buffer );

如果我搜索现有名称，则找到它，一切正常。但是，寻找不存在的名称会导致分段错误。这里有一件显而易见的事吗？

Answer 1

问题在于：

        for ( i = 1; found != 0 ;i++ ){
        found=strcmp(myContacts[i-1].cFirstName, name);

        printf(" Name Found %s",   myContacts[i-1].cFirstName);
        }

您需要添加for ( i = 1; found != 0 && i < num_of_contacts ;i++ )之类的内容否则你会离开你的阵列！

Answer 2

是的：你循环越过数组的末尾。你的循环根本没有限制。

您应该将myContacts上的循环限制为它实际拥有的值的数量。

Answer 3

如果搜索未找到任何结果，您永远不会结束循环

Answer 4

问题在于：

for ( i = 1; found != 0 ;i++ ) {
    found=strcmp(myContacts[i-1].cFirstName, name);
}

如果找不到name，则循环继续超出数组的末尾。您需要在for循环中添加一个额外的测试，以使其在没有找到匹配项的情况下到达数组末尾时终止。

碰巧，我不明白为什么你的for循环从1开始。这样做会更自然：

for (i=0; found!=0 && i<5; i++) {
    found = strcmp(myContacts[i].cFirstName, name);
}

此外，您的found变量的名称很差。它可能应该被称为notfound，因为当找不到名称时它是1，但是当它有0时它是！{/ p>

strcmp的分段错误？

4 个答案: