2012-10-17 82 views
2

我的插入排序的实现似乎与排序第一个元素的例外。我在这里有一个小测试用例。任何人都可以告诉我我的算法有什么问题吗?插入排序不排序的第一个元素?

#include <iostream> 
#include <string> 
#include <stdlib.h> 
using namespace std; 



void Insert(int *S, int k) 
{ 
     int key = S[k]; 
     int j = k-1; 
     while(j>0 && S[j] > key) 
     { 
       S[j+1] = S[j]; 
       j--; 
     } 

     S[j+1] = key; 
} 


void Insertionsort(int S[], int n) 
{ 
     if(n>1) 
       Insertionsort(S,n-1); 
     Insert(S,n); 

} 

int main() 
{ 
     srand (time(NULL)); 
     int S1_8[8]; 
     for(int i=0; i<8; i++) 
       S1_8[i] = rand()%100; 

     Insertionsort(S1_8,8); 

     for(int i=0; i<8; i++) 
     { 
       cout << S1_8[i] << endl; 
     } 

     return 0; 
} 
+0

它并不能说明问题,但肯定是有问题在最后一次迭代中,当调用“Insert(S,8)”时。根据将要访问S [8]的'Insert'函数的定义,这是一个不存在的元素。 – jogojapan

回答

5

首次Insert被调用时,它被传递int key = S[8];

S[8]是数组边界内不

作出这样的

void Insertionsort(int S[], int n) 
{ 
     if(n>1) 
       Insertionsort(S,n-1); 
     Insert(S,n-1); 

} 

此外,在您的病情时,它必须是

while(j>=0 && S[j] > key) 

Link to Code

+0

+1,但是你也将while循环中的条件改为了'j> = 0'而不是'j> 0'。这是正确的,但应在答案中提及并解释。 – jogojapan

+0

谢谢你们两位。如果我想计算由'插入'进行的组件智能比较的次数,我是否会在while循环内或while循环外部增加一个计数器? – Zack

+0

在while循环中。 –