搜索数组以查找匹配结果和 cout 结构成员 C++

Question

完成上学期要求作为期末项目的学业成绩跟踪计划。我没有在它到期之前完成它，我试图理解为什么某些功能没有按我希望的那样运行。

函数 (searchCourse) 旨在搜索结构中的数组。如果找到匹配项，则应将所有结构成员打印给用户。我正在按课程字母代号（即 CIT）搜索。我已经将一个包含三个 CIT 类的文件加载到程序中，并且在调用此函数时，它们无法识别。课程 alpha 指示符是 char 数据类型。如果它是字符串类型并在“for”循环中使用 strcmp，这会更容易完成吗？

//Prompts user to enter course alpha designator for comparison purposes. 
void searchCourse(courses course[])
{
    char desig[3];//User inputs course alpha designator to compare within loop
    //Assigned array value of 4 since strcmp requires 'desig' to have a size for comparison to course[i]
    int i;//Location / loop counter

    cout << endl << "Enter course Alpha-Designator (Ex: CIT, DFIA, etc.): " << endl;//User prompt
    cin >> desig;
    //Set course info headings
    cout << endl;
    cout << left << setw(10) << "Course" << left << setw(25) << "Title" << setw(7) << "Hours" << "Grade" << endl << endl;

    //begin for loop
    for (i = 0; i < courseArrayLength; i++)
    {
       if(course[i].courseName == desig) //If the value in course.courseName is equal to user 'desig'
                                               //Have to use strcmp function when comparing c-strings in array.
                                               //I tried to use the example in CH.8-2 and apply it here. 
        {
            cout << left << setw(4) << course[i].courseName << left << setw(6) << course[i].courseNum;//Print info formatting
            cout << left << setw(25) << course[i].courseTitle << left << setw(7) << course[i].courseHours << course[i].courseGrade << endl;
        }   
        else
        {
            cout << "No courses by that designator.";//Message to display if input 'desig' finds no matches.
            break;
        }
    }

}

如果有人愿意，完整的程序代码在 github 上。任何帮助表示赞赏。这是完成此项目的九个功能中的最后两个。如果我能对如何正确修复这个问题有所了解，那么下一个基本上是相同的。

提前致谢。

Answer 1

• desig 不够大：它也必须容纳空终止符。如果允许，请使用std::string。如果没有，请选择一个足够大的数组，例如char desig[ 1024 ]。
• 声明计数器变量i在for语句中：for ( int i = 0;…
• course[ i ].courseName == desig 比较两个指针，而不是两个字符串。使用std::strcmp：if ( 0 == strcmp( course[ i ].courseName, desig ) )...。如果您切换到std::string，您可以像以前一样使用比较。
• 您过早地打破循环：当您找到第一个不匹配的课程时（您可能稍后会找到它）。找到第一个匹配项后，您可能希望中断循环：删除 else 并将中断移动到 if 内。

欢迎来到 SO，祝你好运！

Answer 2

感谢大家的回复。我显然是编程新手，这个项目教会了我很多。所有的答案都给了我思考的余地，我最终完成了自己打算做的事情。无论如何，下面的代码可以工作。

void searchCourse(courses course[])
{
    string desig;//User inputs course alpha designator to compare within loop
    //Assigned array value of 4 since strcmp requires 'desig' to have a size for comparison to course[i]
    bool found = false;

    cout << endl << "Enter course Alpha-Designator (Ex: CIT, DFIA, etc.): " << endl;//User prompt
    cin >> desig;
    cout << endl;//Terminal formatting
    cout << left << setw(10) << "Course" << left << setw(25) << "Title" << setw(7) << "Hours" << "Grade" << endl << endl;
    //Format info header
    //begin for loop
    for (int i = 0; i < courseArrayLength; i++)//For loop to process array
    {
        while (i < courseArrayLength)//While array processes...
        {
            if(strcmp(course[i].courseName, desig.c_str()) == 0) //If the value in course.courseName is equal to user 'desig'
                {
                    found = true;//Boolean 'found' is set to true
                    cout << left << setw(4) << course[i].courseName << left << setw(6) << course[i].courseNum;//Print info formatting
                    cout << left << setw(25) << course[i].courseTitle << left << setw(7); 
                    cout << course[i].courseHours << course[i].courseGrade << endl;
                    break;//Required since nested in while loop
                }   

            else if(i < courseArrayLength)//Continue array process.
                {
                    i++;                   
                }
        }
        if(found == false)//If no courses found by user defined designator...
        {
            cout << "No courses by that designator." << endl;//Message to display if input 'desig' finds no matches.
            break;
        }  
    } 
}