C++ 而使用getline（）的循环没有中断

所以我有这部分代码，它工作得很好。然后我在下面添加了一些代码行，现在它在执行时崩溃了。从我做的测试来看，它似乎卡在了while循环中。。我是一个新手，我不知道这些变化是如何影响我的

编辑：程序编译时没有错误。 此消息在执行时显示： 此应用程序已请求运行时以异常方式终止它。 现在它不允许我使用调试器。。。我正在使用devc++

int main() {
    char fileName[] = "espion.txt";
    std:: string infoEspion;
    Espion *cur = 0, *first = 0, *last = 0, *add = 0;
    std :: ifstream readFile;

    readFile.open(fileName, std::ios::in);

    if(!readFile.is_open())
        exit(EXIT_FAILURE);

    while(std::getline(readFile, infoEspion)) {
        if(first == NULL){ //head
            add = new Espion;
            add -> name = infoEspion.substr(0,30);
            add -> next = NULL;
            first = add;
            cur = add;
            last = add;
        } else if(findName(first, infoEspion.substr(0,30)) == NULL ) {
            // adding only if not on the list already
            add = new Espion;
            add -> name = infoEspion.substr(0,30);
            add -> next = NULL;
            cur -> next = add;
            last = add;
            cur = add;
        }
    }
    printList(first);
    printList(sortAlpha(first));
    readFile.close();

    system("pause");
    return 0;
}

新代码 while循环没有考虑到如果文件以换行结束，最后一行将为空，因此substr30,20将引发异常

此外，循环代码有不必要的代码重复。您可以大大简化逻辑

试试这个：

#include <iostream>
#include <fstream>
#include <cstdlib>
#include <cstring>
#include <cstdio>
#include <cerrno>
#include <string>

struct Pays
{
    std::string name; /* nom du pays */
    Pays *next; /* pointeur sur le prochain pays */
};

struct Espion
{
    std::string name; /* nom de l’espion(ne) */
    Espion *next; /* pointeur sur le prochain espion */
    Pays *begListPays; /* tête de liste des pays visités */
    Pays *endListPays;
};

Espion* findName(Espion* ptr, std::string nameToCompare)// returns pointer or NULL
{
    while (ptr)
    {
        if(ptr->name.compare(nameToCompare) == 0)
            return ptr;
        ptr = ptr->next;
    }
    return NULL;
}

int main()
{
    char fileName[] = "espion.txt";
    std::string infoEspion, espionName, paysName;
    Espion *first = 0, *last = 0, *add;
    Pays *addP;
    std::ifstream readFile;

    readFile.open(fileName, std::ios::in);
    if (!readFile.is_open())
        exit(EXIT_FAILURE);

    while (std::getline(readFile, infoEspion))
    {
        if (infoEspion.empty())
            continue;

        espionName = infoEspion.substr(0,30);
        paysName = infoEspion.substr(30,20);

        add = findName(first, espionName);
        if (!add)
        {
            add = new Espion;
            add->name = espionName;
            add->next = NULL;
            add->begListPays = NULL;
            add->endListPays = NULL;
            if (!first) first = add;
            if (last) last->next = add;
            last = add;
        }

        addP = new Pays;
        addP->name = paysName;
        addP->next = NULL;
        if (!add->begListPays) add->begListPays = addP;
        if (add->endListPays) add->endListPays->next = addP;
        add->endListPays = addP;
    }

    printList(first);
    printList(sortAlpha(first));

    readFile.close();

    system("pause");
    return 0;
}

这样说，你应该利用C++的特性来管理更多的逻辑：

#include <iostream>
#include <fstream>
#include <cstdlib>
#include <cstring>
#include <cstdio>
#include <cerrno>
#include <string>
#include <list>
#include <algorithm>

struct Pays
{
    std::string name; /* nom du pays */
};

struct Espion
{
    std::string name; /* nom de l’espion(ne) */
    std::list<Pays> ListPays; /* tête de liste des pays visités */

    bool operator<(const Espion& rhs) const
    {
        return (name < rhs.name);
    }
};

struct isEspionName
{
    std::string _name;
    isEspionName(const std::string &nameToCompare) : _name(nameToCompare) {}

    bool operator()(const Espion& e) const
    {
        return (e.name == _name);
    }
};

int main()
{
    char fileName[] = "espion.txt";
    std::string infoEspion, espionName, paysName;
    std::list<Espion> ListEspion;
    Espion *e;
    std::ifstream readFile;

    readFile.open(fileName, std::ios::in);
    if (!readFile.is_open())
        exit(EXIT_FAILURE);

    while (std::getline(readFile, infoEspion))
    {
        if (infoEspion.empty())
            continue;

        espionName = infoEspion.substr(0,30);
        paysName = infoEspion.substr(30,20);

        std::list<Espion>::iterator iter = std::find_if(ListEspion.begin(), ListEspion.end(), isEspionName(espionName));
        if (iter != ListEspion.end())
        {
            e = &(*iter);
        }
        else
        {
            Espion addE;
            addE.name = espionName;
            ListEspion.push_back(addE);
            e = &(ListEspion.back());
        }

        Pays addP;
        addP.name = paysName;
        e->ListPays.push_back(addP);
    }

    readFile.close();

    printList(ListEspion);
    ListEspion.sort();
    printList(ListEspion);

    system("pause");
    return 0;
}

---

列表实现为一个双链接列表。如果你真的想要一个单一的链表，C++11在include中添加了一个std:：forward\u list类。

如果你能说出错误发生的地方，准确的错误信息会很有帮助。findName在哪里定义？我编辑了我的帖子。。我很确定这个错误发生在第二个if中，在崩溃之前什么也没有打印出来。e.name.compare\u name==0为什么不e.name==\u name因为原始代码使用的是compare，而我没有更改它。我认为当我尝试不同的东西时，它可能是有意义的。。但你是对的，我会简化它。谢谢你的帮助！！

#include <iostream>
#include <fstream>
#include <cstdlib>
#include <cstring>
#include <cstdio>
#include <cerrno>
#include <string>
#include <list>
#include <algorithm>

struct Pays
{
    std::string name; /* nom du pays */
};

struct Espion
{
    std::string name; /* nom de l’espion(ne) */
    std::list<Pays> ListPays; /* tête de liste des pays visités */

    bool operator<(const Espion& rhs) const
    {
        return (name < rhs.name);
    }
};

struct isEspionName
{
    std::string _name;
    isEspionName(const std::string &nameToCompare) : _name(nameToCompare) {}

    bool operator()(const Espion& e) const
    {
        return (e.name == _name);
    }
};

int main()
{
    char fileName[] = "espion.txt";
    std::string infoEspion, espionName, paysName;
    std::list<Espion> ListEspion;
    Espion *e;
    std::ifstream readFile;

    readFile.open(fileName, std::ios::in);
    if (!readFile.is_open())
        exit(EXIT_FAILURE);

    while (std::getline(readFile, infoEspion))
    {
        if (infoEspion.empty())
            continue;

        espionName = infoEspion.substr(0,30);
        paysName = infoEspion.substr(30,20);

        std::list<Espion>::iterator iter = std::find_if(ListEspion.begin(), ListEspion.end(), isEspionName(espionName));
        if (iter != ListEspion.end())
        {
            e = &(*iter);
        }
        else
        {
            Espion addE;
            addE.name = espionName;
            ListEspion.push_back(addE);
            e = &(ListEspion.back());
        }

        Pays addP;
        addP.name = paysName;
        e->ListPays.push_back(addP);
    }

    readFile.close();

    printList(ListEspion);
    ListEspion.sort();
    printList(ListEspion);

    system("pause");
    return 0;
}