セグメンテーションフォールト（コアダンプ）私は事前にセット名を使用してアルファベット順に名前を置くコードを書くことになってる

Question

エラーソート文字列配列、およびこれは私がこれまで持っているものです。

#include <iostream> 
#include <iomanip> 
#include <string> 

using namespace std; 

int main() 

{ 

const int NUM_NAMES = 20; 
string temp;       
string names [NUM_NAMES] = {"Collins, Phil", "Simpson, Bart", "Allen, Jim", 
          "Griffen, Jim", "Stamey, Marty" "Rose, Geri", 
          "Taylor, Swift","Johnson, Jill", "Alison, Jeff", 
          "Looney, Joe", "Wolfe, Bill", "James, Jean", 
          "Weaver, Jim", "Pore, Bob", "Rutherford, Greg", 
          "Ravens, Renee", "Harrison, Rose", "Setzer, Cathy", 
          "Pike, Gordon", "Holland, Beth" }; 


for (int k=0; k < NUM_NAMES; k++) 
{  for (int i=0; i<NUM_NAMES; i++) 
     { 
      if (names[i] > names [i+1]) 
       { 
       temp = names[i]; 
       names [i] = names[i + 1]; 
       names [i + 1] = temp; 
       } 

     } 

} 
cout << names [0] << endl; 

int k = 1; 
int i = 0; 
while (k <= NUM_NAMES) 
{ 
     cout << names [i] << endl; 
k++; 
i++; 
} 


return 0; 
} 

それをエラーなしでコンパイルしますが、実行しようとすると "セグメンテーションフォルト（コアダンプ）"が表示されます...正しい方向のヘルプがあれば幸いです

Answer 1

これはあなたのものではなく、行：

names [i+1] = ... 

C++（および他の多くの言語）では、配列は0から索引付けされています。そのため、サイズNの配列は、0からN-1までのインデックスに対して有効です。 forループは0 ... NUM_NAMES - 1を繰り返しますが、最後にはというインデックスi+1を使用します。この問題を解決するための

、あなたが1 ... NUM_NAMES - 1を反復処理できます。names[i-1]とnames[i]上

for (int k=1; k < NUM_NAMES; k++) 

と仕事を：

if (names[i-1] > names [i]) 
{ 
    temp = names[i - 1]; 
    names [i - 1] = names[i]; 
    names [i] = temp; 
} 

（あるいは、それが今であるとして、それを残して、ループの実行条件をk < NUM_NAMES - 1に変更）

はまた、これに代えて：

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

あなたは、単に行うことができます：

std::swap(names[i], names[i + 1]);