链表/指针的问题

时间:2011-09-18 14:02:17

标签: c

我目前遇到链接列表和C指针的问题。我遇到的问题是将数据添加到链表。目前我有:

struct str_pair{ 
char ip  [50] ;
char uri [50] ;
struct str_pair *next ;
};

struct str_pair *it ;

struct str_pair *header = NULL; // Start of linked list
struct str_pair *ptr; // Moves along the list
struct str_pair *ptr2; // Another pointer
struct str_pair *ptr3;

void addData(char *addURI, char *addIP){

    struct str_pair *tmp, *tmp2;

    tmp = (str_pair*)malloc(sizeof(str_pair)); // Create new space in str_pair
    strncpy(tmp->uri, addURI, 49);
    strncpy(tmp->ip, addIP, 49);
    tmp->next = NULL;

    if (header == NULL) { header = tmp; }
    else
    {
        tmp2 = header;
        while (tmp2->next != NULL) { tmp2 = tmp2->next; }
        tmp2->next = tmp;
    }
}

我要做的是通过参数传递一个URL和一个IP地址,它应该将这些值添加到链接列表中。

以下是调用此函数的代码:

int main(int argc, char *argv[])
{
    int incrItems=0;
    int j;

    header = NULL;

    for(j = 1; j < argc; j++)
    {
        char ch=argv[j][0];

        switch(ch)
        {
        case 'A' :
            {
                char *newURI = argv[j+1];
                char *newIP = argv[j+2];
                incrItems++;
                addData(newURI,newIP);
                j=j+2;
                break;
            }

*Snipped the rest as its unnecessary*

我遇到的问题是传递的参数没有被添加到链表中。编译时不会显示错误。

3 个答案:

答案 0 :(得分:0)

for(j = 1; j < argc; j++)
    {
    switch(argv[j][0]) { /* no need to copy */
    case 'A' :
            incrItems++;
            /* assert (j+2 < argc); */
            addData(argv[j+1], argv[j+2]); /* no need to copy */
            j=j+2;
            break;
    case 'B' :
    default:
      ...
            break;
        }
     }

编辑:注意:以上不是解决方案,只是一个提示。 另一个暗示:

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

void addData(char *addURI, char *addIP){

    struct str_pair *tmp, **hnd;

    tmp = malloc(sizeof *tmp);
    /* assert (tmp != NULL); */
    strncpy(tmp->uri, addURI, 49); tmp->uri[49] = 0;
    strncpy(tmp->ip, addIP, 49); tmp->ip[49] = 0;
    tmp->next = NULL;

    for (hnd = &header; *hnd; hnd = &(*hnd)->next) {;}
    *hnd = tmp;
}

答案 1 :(得分:0)

如果你在for,你为什么要增加j变量?另外,尝试在add函数中放入一些printf,以了解参数是否正确。

答案 2 :(得分:0)

尽管wildplasser提供了很好的提示,但代码工作得非常好。

int main(int argc, char *argv[])
{
    int incrItems=0;
    int j;

    header = NULL;

    for(j = 1; j < argc; j++)
    {
        char ch=argv[j][0];

        switch(ch)
        {
        case 'A' :
            {
                char *newURI = argv[j+1];
                char *newIP = argv[j+2];
                incrItems++;
                printf(" Adding %s %s\n", newURI, newIP);
                addData(newURI,newIP);
                j=j+2;
                break;
            }
        }
     }
printf(" J at end is %d\n",j);

it = header;
if(it != NULL)
do {
        printf(" %s %s\n",it->ip, it->uri);
        it = it->next;
}while(it != NULL);

    }