这是《 C语言》一书中的程序。
有一个错误:“strdup”的类型冲突!当遇到函数'strdup'时,但是如果将'strdup'更改为其他名称,例如'strdu',错误将消失。
我不知道为什么?顺便说一句,我使用code::blocks作为我的IDE。

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

#define MAXWORD 100

struct tnode {
    char *word;
    int count;
    struct tnode *left;
    struct tnode *right;
};

struct tnode *addtree(struct tnode *, char *);
struct tnode *talloc(void);

void treeprint(struct tnode *);
int getword(char *, int);
char *strdup(char *);

/*  word frequency count */
int main()
{
    struct tnode *root;
    char word[MAXWORD];

    root = NULL;
    while (getword(word, MAXWORD) != EOF)
        if (isalpha(word[0]))
            root = addtree(root, word);
    treeprint(root);
    return 0;
}
/* addtree: add a node with w, at or below p */
struct tnode *addtree(struct tnode *p, char *w)
{
    int cond;
    if (p == NULL) {       /* a new word has arrived */
        p = talloc();      /* make a new node */
        p->word = strdup(w);
        p->count = 1;
        p->left = p->right = NULL;
    } else if ((cond = strcmp(w, p->word)) == 0)
        p->count++;        /* repeated word */
    else if (cond < 0)     /* less than into left subtree */
        p->left = addtree(p->left, w);
    else                   /* greater than into right subtree */
        p->right = addtree(p->right, w);
    return p;
};

/* treeprint: in-order print of tree p */
void treeprint(struct tnode *p)
{
    if (p != NULL) {
        treeprint(p->left);
        printf("%4d %s\n", p->count, p->word);
        treeprint(p->right);
    }
}

/* talloc: make a tnode */
struct tnode *talloc(void)
{
    return (struct tnode *) malloc(sizeof(struct tnode));
};

char *strdup(char *s)  /* make a duplicate of s */
{
    char *p;

    p = (char *) malloc(sizeof(strlen(s)) + 1);
    if (p != NULL)
        strcmp(p, s);
    return p;
}
.... some other function ....

最佳答案

您不能拥有自己的函数,其名称以str开头。整个“命名空间”在C中保留。

在这种情况下,strdup()<string.h>的标准函数,您的函数声明会与该函数冲突。

请注意,仅停止使用<string.h>是不够的,该名称仍保留,因此您无法有效使用它。

还有两个注意事项:

  • 输入未写入,因此它应该是const指针。
  • Please don't cast the return value of malloc() in C
  • 您的strdup()工作方式严重损坏,当它表示strcmp()时会调用strcpy()
  • 您对sizeof(strlen(s))的使用是完全错误的,即使解决了strcmp() / strcpy()问题,也将引起严重的问题。

  • 合理的strdup()实现为:
    char * my_strdup(const char *s)
    {
      char *r = NULL;
      if(s != NULL)
      {
        const size_t size = strlen(s) + 1;
        if((r = malloc(size)) != NULL)
          memcpy(r, s, size);
      }
      return r;
    }
    

    我使用memcpy(),因为我知道长度,它可以更快。

    关于c - 错误: 'strdup'类型冲突!,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/21850356/

    10-13 02:59