2

出于某种原因，我无法正确获得这种名称。谁能告诉我它有什么问题？据我所知，问题是没有正确比较字符串。我之前尝试过字符串比较，我知道这种代码应该可以工作。这真的让我难过。

#include <iostream>
#include <fstream>
#include <string>
#include <vector>

using namespace std;

void sortNames(vector<string> &);

void main()
{
    vector<string> namesList;
    ifstream namesFile;
    namesFile.open("Names.txt");

    // Make sure the file exists.
    if (namesFile)
    {
        // Get the names from the file.
        string name;
        while (getline(namesFile, name))
            namesList.push_back(name);

        // Sort the imported names.
        sortNames(namesList);

        for (int i = 0; i < namesList.size(); i++)
            cout << namesList[i] << endl;
    }
    else
    {
        cout << "Data files are missing";
    }

    namesFile.close();
}

void sortNames(vector<string> &list)
{
    for (int i = 0; i < list.size(); i++)
    {
        // Find the lowest value after i.
        int lowIndex = i;
        for (int j = i + 1; j < list.size(); j++)
        {
            string name = list[i];
            string name2 = list[j];

            if (name > name2)
                lowIndex = j;
        }

        // Flip the elements if there was a value lower than i.
        if (i != lowIndex)
        {
            string temp = list[i];
            list[i] = list[lowIndex];
            list[lowIndex] = temp;
        }
    }
}
4

2 回答 2

5

这是问题所在：这条线

string name = list[i];

应该

string name = list[lowIndex];

您当前的实现j不是将 at 的元素与您迄今为止找到的最小字符串进行比较，而是将其与 index 的字符串进行比较i。这是不正确的，因为它没有找到最小的剩余字符串：相反，它会vector在 index 中找到小于当前元素的最后一个字符串i，这不是您想要的。

于 2013-03-08T22:53:03.547 回答
0

而不是string name = list[i];，你想要string name = list[lowIndex];

于 2013-03-08T22:54:02.453 回答