動的配列のサイズとgetline（）でのクラッシュ。

Question

私は入力として名前をとり、最終的にバイナリ検索&を並べ替えるプログラムに取り組んできました。しかし、配列を動的なサイズにしようとすると（ループの繰り返しごとに1つずつ増加する）、さまざまな問題が発生しました。

私は文字列配列を20個の要素で構成することができますし、プログラムは動作しますが、私の割り当てに余分なクレジットはそれを動的なサイズにすることです。現在、プログラムは "getline（cin、Names [x]）;"に達するとエラーコードなしでクラッシュします。 私は周りを探索してきましたが、この場合は配列の代わりにベクトルを行うほうが簡単かもしれないことは分かっていますが、この割り当てでベクターを使用することは許されていません。それが作成された後

おかげ

オリジナルコードは

using namespace std; 
#include <iostream> 
#include <string> 

void main() 
{ 
    int x = 0; 
    string * Names = new string[x]; 
    bool NameInputEnd(0); 

    cout << " Enter your names to be sorted\n"; 
    cout << "To exit just press [Enter] at any time\n"; 

    do 
    { 
     cout << x << endl; 
     cout << "\n< Name " << (x + 1) << " > = "; 

     !!**CRASHES HERE**!! 

     getline(cin, Names[x]); 

     if (Names[x].empty() || x == 19) 
     { 
      cout << "\nFinal Name Amount = " << (x + 1) << endl << endl; 
      NameInputEnd = 1; 
      continue; 
     } 

     x++; 

    } while (NameInputEnd == 0); 

    delete [] Names; 
} 

変更

int tempsize(1), x(0); 
string * Names = new string[tempsize]; 
... 

do 
{ 
... 
    x++; 
    tempsize++; 
}while (NameInputEnd == 0); 

Answer 1

配列のサイズを変更することはできません。あなたはそれを破壊し、既存のデータのコピーを持つ新しい配列を作成する必要があります。たとえば：

#include <iostream> 
#include <string> 
#include <algorithm> 

using namespace std; 

void main() 
{ 
    int x = 0; 
    int capacity = 20; 
    string * Names = new string[capacity]; 
    string Name; 

    cout << " Enter your names to be sorted\n"; 
    cout << "To exit just press [Enter] at any time\n"; 

    do 
    { 
     cout << x << endl; 
     cout << "\n< Name " << (x + 1) << " > = "; 

     if ((!getline(cin, Name)) || Name.empty()) 
      break; 

     if (x == capacity) 
     { 
      int newCapacity = capacity + 20; 
      string *newNames = new string[newCapacity]; 
      copy(Names, Names + x, newNames); 
      delete [] Names; 
      Names = newNames; 
      capacity = newCapacity; 
     } 

     Names[x] = Name; 
     ++x;  
    } 
    while (true); 

    cout << "\nFinal Name Amount = " << x << endl << endl; 

    delete [] Names; 
} 

は、あなたは本当にかかわらず、std::vectorを使用する必要があります。

#include <iostream> 
#include <string> 
#include <vector> 

using namespace std; 

void main() 
{ 
    vector<string> Names; 
    string Name; 

    Names.reserve(20); // optional 

    cout << " Enter your names to be sorted\n"; 
    cout << "To exit just press [Enter] at any time\n"; 

    do 
    { 
     cout << Names.size() << endl; 
     cout << "\n< Name " << (Names.size() + 1) << " > = "; 

     if ((!getline(cin, Name)) || Name.empty()) 
      break; 

     Names.push_back(Name); 
    } 
    while (true); 

    cout << "\nFinal Name Amount = " << Names.size() << endl << endl; 
}