4

我正在尝试编写一个代码，使用 strstr 提取和标记之间的所有单词/字符串。但似乎它只是停留在提取的第一个字符串上，即“快速”。提取第一个字符串后如何让代码继续运行？

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

int main()
{

    char feed[] = "The <item> quick </item> brown <item> fox </item> jumps <item> over </item> the <item> lazy dog </item>";


    const char needle[] = "<item>";
    const char popo[] = "</item>";
    char *ret;
    char *ter;
    int n;
    n = 0;

    while (feed[n] != '\0')
    {
        ret = strstr(feed, needle)+6;
        ter = strstr(ret, popo);
        size_t len = ter - ret;
        char *res = (char*)malloc(sizeof(char)*(len+1));
        strncpy(res, ret, len);
        res[len] = '\0';

        printf("%s",res);
        n++;
    }
    return 0;
}
4

2 回答 2

2

您需要使ret指针指向字符串中的当前位置，并在每次迭代时按长度递增，并传递ret给第一个strstr()而不是feed，查看此实现

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

int main()
{

    char       feed[]   = "The <item> quick </item> brown <item> fox </item> "
                          "jumps <item> over </item> the <item> lazy dog </item>";
    const char needle[] = "<item>";
    const char popo[]   = "</item>";
    char      *head;
    int n;
    n = 0;

    head = feed;
    while (feed[n] != '\0')
    {
        char  *tail;
        char  *copy;
        size_t length;

        head = strstr(head, needle);
        /*            ^ always start at the current position. */
        if (head == NULL)
         {
            fprintf(stderr, "Invalid input...???\n");
            return -1;
         }
        tail   = strstr(head, popo);
        length = tail - head - 6;
        head  += 6;
        if (length < 0)
         {
            fprintf(stderr, "Invalid input...???\n");
            return -1;
         }
        copy = malloc(length + 1);
        if (copy != NULL)
         {
            memcpy(copy, head, length);
            copy[length] = '\0';

            printf("*%s*\n", copy);
            /* If you are not going to keep it, free it */
            free(copy);
         }
        head += length; /* <-- this is the imprtant thing */
        n++;
    }
    return 0;
}
于 2015-05-02T15:13:06.693 回答
1

在这条线上：

ret = strstr(feed, needle)+6;

您总是从feed字符串的开头开始搜索。您需要将不同的起点传递给strstr，您已经在ter. 所以你应该能够做这样的事情：

ter = feed;
while (ter != NULL) 
{
     ret = strstr(ter, needle) + 6;
...

有了这个，您的搜索开始将继续沿着feed字符串向下移动。

您的代码中还有其他一些问题：

  1. strstr()如果找不到匹配项，则可以返回 NULL - 你需要检查它，否则你的程序将崩溃。
  2. 你需要free()你的记忆malloc()
  3. 正如@iharob 指出的那样“不要施放malloc()”
于 2015-05-02T15:11:07.640 回答