插入排序算法,我的代码有什么问题?

时间:2013-04-07 15:18:36

标签: c++ sorting insertion-sort

我正在尝试使用InsertSort算法来输入存储在向量中的字符串。

我所做的是在矢量中输入一些字符串, 然后我使用insertionsort对矢量进行排序。

但我不确定为什么它不起作用!有人能指出我正确的方向吗?

#include <iostream>
#include <cstring>
#include <vector>
#include <cstdlib>
using namespace std;

int main (){
    vector <string> names; //vector to store
    string input; //input is the variable of strings

    cout<<"Input a list of names \n";
    cout<<"To end the list type 'END'" <<endl;

    while (true){
        getline(cin, input);

        if (input =="END")
        break;

        names.push_back(input); //push into vector names
    }


  //my insertsort starts here
  string temp;
  int i;
  for (int j = 1; j < names.size(); j++){
        i = j - 1;        

        while ((i>0) && (names[i]>names[j]) ) {

            names[i+1] = names[i];

            i=i-1;         
                                                 }

        names[i+1] = names[j];

    }



    for (unsigned int i = 0; i<names.size (); i++)
    cout<<names[i]<<endl;
    cout<<endl;
    system("pause");
}

非常感谢

编辑: 我将输入字符串,例如我将输入:

彼得 苹果 兔

我希望的输出是按字母顺序排列的:

苹果 彼得 兔

目前输入示例,我得到: 彼得 苹果 兔

编辑3:

我的插入排序现在看起来像这样:

 string temp;
  int i;
  for (int j = 1; j < names.size(); j++){
        i = j - 1;        
        temp = names[j];

 while ((i>=0) && (names[i]>names[j]) ) {
    names[i+1] = names[i];
    i=i-1;                                                  
                                        }

 names[i+1] = temp;
    }

1 个答案:

答案 0 :(得分:2)

你错过了一点:

 //You have to remember names[j] before while loop
 //the variable temp is never used
 temp = names[j];
 while ((i>=0) && (names[i]>temp) ) {
    names[i+1] = names[i];
    i=i-1;                                                  
 }
 names[i+1] = temp;
 // since names[j] was already been filled by other words during swapping

如果不需要使用插入排序,则最好使用stl排序算法。