我正在创建一个双向链表，它将按字母顺序排列名称列表，但我不确定在 int main() 函数中放入啥

Posted 2023-02-21

【中文标题】我正在创建一个双向链表，它将按字母顺序排列名称列表，但我不确定在 int main() 函数中放入啥

【英文标题】：I am creating a doubly linked list that will alphabetize a list of names, but I am unsure of what to put in the int main( ) function我正在创建一个双向链表，它将按字母顺序排列名称列表，但我不确定在 int main() 函数中放入什么

【发布时间】：2011-11-10 08:17:26

【问题描述】：

最终，程序将按字母顺序打印出名称列表，以及与该名称关联的其他属性。换句话说，输出屏幕将如下所示：

Ares: Greek, fire, sword.
Freia: Norse, water, bow and arrow.
Poseidon: Greek, horses, ocean.
Thor: Norse, chariot, hammer.
Zeus: Greek, cloud, lightning.

同样，名字在这个列表中是按字母顺序排列的，但属性会打印在它们旁边。关于我的 int main()，我不确定我应该如何开始对这些名称进行排序并将它们按顺序排列。我有一个必须排序的未排序列表（使用将这些名称添加/插入到正确顺序的函数）。

    //
    // This is a standard library support code to the chapters of the book
    // "Programming -- Principles and Practice Using C++" by Bjarne Stroustrup
    //

    #ifndef STD_LIB_FACILITIES_GUARD
    #define STD_LIB_FACILITIES_GUARD 1

    #include <cmath>
    #include <iostream>
    #include <vector>
    #include <stdexcept>
    #include <string>

    using namespace std;

    //------------------------------------------------------------------------------

    // Helper function to show an error message
    inline void error(const string& errormessage)
    
    throw runtime_error(errormessage);
    

    //------------------------------------------------------------------------------

    #endif // STD_LIB_FACILITIES_GUARD

    //------------------------------------------------------------------------------

    struct Link 
    string name;
    string mythology;
     string vehicle;
    string weapon;

    Link* prev;
    Link* succ;
    Link(const string& n, const string& a, const string& b, const string&c,Link* p = 0,    
    Link* s = 0)
: name(n), mythology(a), vehicle(b), weapon(c), prev(p), succ(s)  
    ;



    Link* insert(Link* p, Link* n)    // insert n before p; return n
    
    if (n==0) return p;
    if (p==0) return n;
     n->succ = p;        // p comes after n
    if (p->prev) p->prev->succ = n;
     n->prev = p->prev;    // p's predecessor becomes n's predecessor
     p->prev = n;        // n becomes p's predecessor
     return n;
     

    void print_all(Link *p)
    
Link *current;
current = p;
while(current)

    cout<<"For this link we have: \n";
    cout<<"Name: "<<current->name<<".\n";
    cout<<"Info1: "<<current->mythology<<".\n";
    cout<<"Info2: "<<current->vehicle<<".\n";
    cout<<"Info3: "<<current->weapon<<".\n";
    current = current->succ;


    
    Link * add_after_find(Link *p, Link *n,const string& s )
        Link *current = 0;
current = p;
/* empty list */
if(p == 0)
   cout<<"List is empty so string not found so not added after it. \n";

    return  0;

/*  DO WE NEED ONE LINK ONLY */
else if(p->succ == 0)   /* one link only */

    if(p->name == s)
    

        /* add after link with s */
        /* p in front */
        p->succ = n;
        n->prev = p;
        p->prev = 0;
        n->succ = 0;

        return p;      
       /* end of if names =  */
    else 
        cout<<"String not found in link listed so not added. \n";
        return p;

    

  /* end of one link */

else /* two or more links */

   
    current = p;
    while(current->succ)
    
        if (s == current->name)


        

            /* then n goes AFTER this link */
            n->prev = current;
            n->succ = current->succ;
            current->succ = n;


            return p;
          /* end of name matches */

        else 
        
            current = current->succ;
        
    // end of while
    /* if outside of while then we are at last link with a current -> name 
     so s not found  */
    cout<<"String is not found so not add after it. \n";
    return p;
  // end of else 2 or more
  // end of function

---

int main()
      
    Link*newlist = new Link("Thor","Norse","chariot","hammer");
    newlist = add_after_find(newlist,new Link("Hera","Greek", "horse", "arrow"),"Thor");
    newlist = add_after_find(newlist,new Link("Poseidon","Greek", "ocean", "trident"),"Freia");
     newlist = add_after_find(newlist,new Link("Ares","Greek", "fire", "sword"),"Poseidon");
     newlist = add_after_find(newlist,new Link("Zeus","Greek", "cloud", "lightning"),"Ares");

    print_all(newlist);
    cout<<"Now let's alphabetize these five gods.\n";
    system("Pause");
    return 0;

【问题讨论】：

如果这是作业，请标记它。

tl;博士您有实际问题，还是正在寻找某人为您解决这个问题？

哦。忘记指定了。我主要是在找人用一些伪代码来帮助指导我。

@user994559：您应该处理代码的格式，特别是 if 结构周围的缩进和格式。代码的格式对阅读代码和避免细微的错误有很大帮助，例如，你有一行 ifs 后跟一个缩进的行，它不是 if 分支的一部分，这使得它很难立即阅读。

另外，您的代码中还有其他问题：您的程序泄漏内存（Poseidon 会发生什么情况？），这是一个设计问题，因为您的界面不允许您处理这种特殊情况.您可能想要实现find 和add，然后在它们之上提供add_after_find，以便更优雅地处理错误条件。您的列表不变量（prev/succ 指针指的是列表中的前一个/下一个元素）并非在所有情况下都得到维护（如果列表包含多个元素，如果有的话，当您找到插入位置时会发生什么更多元素？）

【参考方案1】：

我假设这是作业，如果不是，简单的答案是您应该在该容器中使用 std::list 和 sort 方法。

在main 中输入什么？最有可能是：sort_list( newlist )，其中指针通过引用传递（因为列表的头部可能会改变）。至于如何实现，这取决于您要实现的排序算法，最简单的可能是冒泡排序，列表的下一个最佳选择是归并排序。为他们谷歌，如果您需要算法方面的帮助，请回来询问。

与此同时，您可能想要解决我作为 cmets 提出的问题：代码格式化、内存泄漏（在插入时如果找不到位置，以及在程序结束时）、正确性一直以来的数据结构......我没有做过深入的分析，但我觉得你的add_after_find在需要在列表尾部添加一个元素时可能会失败......甚至在你开始之前考虑排序，您应该确保输入正确。与开始添加更多代码相比，调试当前问题更容易。

【讨论】：

一边调试一边调试比最后调试要容易得多。从长远来看，在排序之前确定您拥有正确的数据将为您省去很多麻烦。