声明与指针

Question

我有使用指针2层结构没有影响，形成一个链表，我创建了这些工种操作：

typedef struct hashtag { 
    char *text; 
    int count; 
} *Item; 

typedef struct node { 
    Item item; 
    struct node *next; 
} *link; 

我在与所有的指针一对夫妇的问题相同的功能。

/* adds hashtag (Item Type) to linked list (Link Type) */ 

void add_hashtag(char *word){ 
    int search_result, i; 
    for (i = 0; i < strlen(word); i++) 

    /* converts word to all lowercase before adding */ 

    token[i]=tolower(*(token + i)); 
    Item buffer = (Item) malloc(sizeof(struct hashtag)); 
    strcpy(buffer->text,word); 

    /* Runs through the list to see if the hashtag was already added */ 

    search_result = (search_hashtag(head, buffer)); 
    if (search_result!=NULL){ 

     /* Increase count (occurrence) of hashtag if it already exists (returns link type or NULL if not found) */ 

     increase_hashtag_count(search_result); 
    } 

    /* Create new struct type hashtag */ 
    new_hashtag(buffer); 

} 

警告：赋值时将指针整数，未作施放[-Wint转换] search_result =（search_hashtag（头，缓冲液））;

警告：（！search_result = NULL）指针和整数 如果进行比较{

的tolower()功能和search_result()不与指针正常工作，而且我无法调试这一点。

编辑：tolower()固定的，我误读的文档

Answer 1

作为每tolower()的man page，

返回的值是转换后的信，或c如果转换是不可能的。

这意味着，它不变化传递的参数，它return S中的结果作为函数调用的返回值和你的情况，你忽略了返回值，使得整个函数调用毫无意义。

所以，你需要有另一个数组存储的tolower()返回值来获取转换后的小写串。

这就是说，对于第二种情况，您已经定义search_result为int但后来，你想它比对NULL，这是一个空指针常量。您需要相应地更正数据类型。

Answer 2

函数中有几个错误。

首先，目前还不清楚什么是可变token哪里以及如何声明，使用和设置

for (i = 0; i < strlen(word); i++) 

/* converts word to all lowercase before adding */ 

token[i]=tolower(*(token + i)); 
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ 

希望检查调用malloc是全成。例如

Item buffer = (Item) malloc(sizeof(struct hashtag)); 
if (buffer) 
{ 
    //...other stuff 
} 

buffer-> text的内存未分配。所以这个声明

strcpy(buffer->text,word); 

有未定义的行为。

变量search_result被声明为具有类型int。你应该把它与一个整数，而不是一个指针比较

search_result = (search_hashtag(head, buffer)); 
if (search_result!=NULL){ 
^^^^^^^^^^^^^^^^^^^^^^^^^ 

看来，如果主题标签已经存在，你不应该在此声明一个新的主题标签添加到列表中

new_hashtag(buffer); 

这足以增加现有主题标签的计数（发生）。