C ++:按编号错误

时间:2018-05-22 20:27:10

标签: c++ pointers linked-list nodes

我正在阅读的文件看起来像这样:

  

t 44

     

c 13

     

a 47

     

19

     

g 41

     

n 51

     

。 4

等等,总共有53行。我需要读取这些行并使用addInOrder函数将它们按顺序放入。我用cout进行了测试,我看到了

  

letter = t,num = 44,head = 44

     

letter = c,num = 13,head = 13

     

letter = a,num = 47,head = 13

     

信:[空格],num = 19,head = 13

     

信:g,num = 41,head = 13

     

信:n,num = 51,head = 13

     信:。 ,num = 4,head = 13

直到稍后当num = 12时头部才会改变,然后当num = 1时头部再次改变。然后头部等于1直到完成添加。

当我最后打印时,链接列表按第一个数字而不是整数排序,它遵循以下顺序:

  • 1
  • 10
  • 11
  • ...
  • 19
  • 2
  • 20
  • 21
  • ...
  • 29
  • 3
  • 30
  • 31

依此类推......我需要它:

  • 1
  • 2
  • 3
  • ...
  • 9
  • 10
  • 11

我的逻辑中是否存在addInOrder函数错误?

这是我的主要内容:

#include <fstream>
#include <iostream>
#include <string>
#include <sstream>
#include <stdio.h>
#include <ctype.h>

using namespace std;

struct ListNode
{
    string letter;
    string num;
    ListNode *next;
};

void addInOrder(ListNode *&h, string l, string n);
void printList(ListNode *h, int &lengthOfFile);
void deleteList(ListNode *&h);

int main()
{
    string letter;
    string num;
    string lines;
    int lengthOfFile = 0;
    const string FILENAME = "link to file";
    ifstream inFile(FILENAME);

    ListNode *head = nullptr;

    if (inFile)
    {
        string line;
        for (int lineNum = 1; getline(inFile, line); lineNum++)
        {
            stringstream ss(line);
            string word;

            for (int wordNum = 1; ss >> word; wordNum++)
            {
                if (wordNum == 1)
                {
                    char c = word[0];

                    if (isalpha(c))
                    {
                        letter = c;
                    }
                    else if (word == "!" or word == ".")
                    {
                        letter = word;
                    }
                    else if (word != "!" or word != ".")
                    {
                        letter = "  ";
                        num = word;
                        lengthOfFile++;
                        if (wordNum == 1)
                        {
                            addInOrder(head, letter, num);
                        }
                    }
                }
                if (wordNum == 2)
                {
                    num = word;
                    lengthOfFile++;
                }
                if (wordNum == 2)
                {
                    addInOrder(head, letter, num);
                }
            }
        }
    inFile.close();
    }

    printList(head, lengthOfFile);

    deleteList(head);
}

这是我按照数字顺序添加到链接列表的函数:

void addInOrder(ListNode *&h, string l, string n)
{
    ListNode *newNode;
    newNode = new ListNode;

    newNode->letter = l;

    newNode->num = n;

    if (h == nullptr)
    {
        h = newNode;
        newNode->next = nullptr;
    }
    else
    {
        ListNode *prev = nullptr;
        ListNode *curr = h;

        while ((curr != nullptr) && (curr->num < n))
        {
            prev = curr;
            curr = curr->next;
        }
        if (prev == nullptr)
        {
            h = newNode;
            newNode->next = curr;
        }
        else
        {
            prev->next = newNode;
            newNode->next = curr;
        }

    }
}

打印列表以确保索引:

void printList(ListNode *h, int &lengthOfFile)
{
    ListNode *ptr = h;  

    for(int i = 0; i < lengthOfFile; i++)
    {
        cout << ptr->letter << " ";
        cout << ptr->num << " ";
        cout << endl;
        ptr = ptr->next;
    }
    cout << endl;
}

删除列表:

void deleteList(ListNode *&h)
{
    ListNode *ptr;
    while (h != nullptr)
    {
        ptr = h;
        h = h->next;
        delete ptr;
    }
}

1 个答案:

答案 0 :(得分:2)

您正在比较string,而不是数字。列表按字母顺序正确排序。

如果您希望按数字顺序排序,则必须将字符串转换为整数(或任何其他数值)。