c - C 中 strstr() 中 for 循环的错误使用

Question

我正在尝试用 C 语言学习字符串处理。我写了一个程序，它存储了一些音乐轨道，并帮助用户检查他/她想到的歌曲是否存在于存储的轨道中。这是通过要求用户输入一串字符来完成的。然后程序使用 strstr() 函数检查输入的单词/文本是否与任何轨道上的任何单词/文本匹配，如果匹配，则显示轨道编号和名称。

我编写的代码在一开始就可以正常运行。但是有一个很大的缺陷，就是for循环的错误使用。我会先展示程序和两段输出-

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

//Define tracks array
char tracks[][80]={     //[][80] is a 2D array- for storing tracks and the size of each track, which is 80 at max
"I left my heart in Harvard med school",
"Newark, Newark- a wonderful town",
"Dancing with a dork",
"From here to maternity",
"The girl from Iwo Jima",
};

//Define function to search for text in tracks array

void findTrack(char search_for[])
{
    int i;
    for(i = 0; i < 5; i++)
    {
       if(strstr(tracks[i], search_for))    
       {
          printf("\n Match found\n");       
          printf("\n Track[%i]: %s\n", i, tracks[i]);
          break;
       }
       else
           printf("\n No matching tracks found!");
       }
}

int main()
{
   char search_for[80];
   printf("\n Enter the text to search:\n");
   scanf("%79s", search_for);
   findTrack(search_for);
   return 0;
}

第一个输出-

 Enter the text to search:
Harvard

 Match found

 Track[0]: I left my heart in Harvard med school

如您所见，此输出是正确的。字Harvard出现在我存储在 tracks 中的第一首轨道中大批。这是在 for loop 的第一次迭代中发现的在findTrack里面函数，因此输出正确。

但是，如果我再次运行该程序，这次我将要搜索的文本指定为 town ，结果输出是 -

第二个输出-

Enter the text to search:
town

 No matching tracks found!
 Match found

 Track[1]: Newark, Newark- a wonderful town

发生这种情况是因为在 for 的第一次迭代中循环即:tracks[0] , strstr没有找到任何匹配项，因此它打印了 else 中的语句循环的一部分。当循环完成迭代 1 时，for tracks[1] , 在 town 中找到匹配项, 因此它打印了 if循环的一部分。

同样，如果我再次运行程序并给出 Jima作为要搜索的文本，输出是-

 Enter the text to search:
Jima

 No matching tracks found!
 No matching tracks found!
 No matching tracks found!
 No matching tracks found!
 Match found

 Track[4]: The girl from Iwo Jima 

我认为这个缺陷是由于不正确的 for 循环造成的。但我不确定如何使用 for 循环正确执行此操作。任何有关纠正此问题的建议将不胜感激。谢谢!

编辑

在我的代码中，我使用了 for用 i<5 循环因为我知道有 5 tracks存储在我的程序中。然而，更好的编码方式是运行一个循环，而不管/在轨道数量未知的情况下。这可能发生在我的轨道数量发生变化的情况下，因此每次都不断更改 for 循环效率很低。有关如何实现此更改的任何建议？

Answer 1

您只想在检查整个数组 tracks 是否匹配后打印 Not found。所以你可以使用一个标志并在循环后检查它:

void findTrack(char search_for[])
{
  int i;
  int found = 0;
  for(i=0;i<5;i++)
  {
    if(strstr(tracks[i], search_for))   
    {
    found = 1;
    printf("\n Match found\n");     
    printf("\n Track[%i]: %s\n", i, tracks[i]);
    break;
    }
  }
  if (!found)     printf("\n No matching tracks found!");
}