文字列の初期化に失敗する

Question

これは私のプログラムです。出力は一連の 'a'文字でなければなりませんが、何らかの理由でそれが出力されません。どうして？

#include <iostream> 

using namespace std; 

const int NAME_LENGTH = 16; 

struct Record { 
    char hotel_name[NAME_LENGTH]; 
}; 

int main() { 
    int amount = 5; 
    for (int i = 0; i < amount; i++) { 
     Record * elementToBeAdded = new Record; 
     for (int j = 0; j < NAME_LENGTH; j++)  
      elementToBeAdded->hotel_name[i] = 'a'; 
     elementToBeAdded->hotel_name[NAME_LENGTH-1] = '\0'; 
     cout << "string-" << elementToBeAdded->hotel_name << "-\n\n"; 
    } 
} 

Answer 1

あなたは混同iとjを得た：

for (int j = 0; j < NAME_LENGTH; j++)  
    elementToBeAdded->hotel_name[i] = 'a'; //<-- should be j here 

また、あなたは外側のループの最後で、delete elementToBeAdded;を記憶しなければならない漏れています。あなたは

 elementToBeAdded->hotel_name[i] = 'a'; 

にタイプミスがあるため

Answer 2

あなたは私は、jはない意味します。

Answer 3

あなたの最も内側のループは、それが確実にjを使用しようとするときiを使用している：

elementToBeAdded->hotel_name[i] = 'a'; 

結果として、あなたはRecord＃1のchar[]の0番目の要素、またの1番目の要素を設定することは決してないだろう＃2などです。つまり、最初のものを除く最も外側のループを通過するたびに、ホテル名の最初のcharが初期化されずに残ることになります。おそらく\0です。

また、あなたが作成しているRecordオブジェクトが削除されませんされ、これは各繰り返しでメモリをリークしています。

Answer 4

elementToBeAdded->hotel_name[i] = 'a';

、今、あなたが間違いを知っていることを

elementToBeAdded->hotel_name[j] = 'a';

Answer 5

なるアルゴリズムのヘッダにある代わりにstd::fill_nを使用する必要があります。

#include <algorithm> 

// ..... 

for (int i = 0; i < amount; i++) {   

    Record * elementToBeAdded = new Record; 
    std::fill_n(elementToBeAdded->hotel_name, NAME_LENGTH-2, 'a'); 
    elementToBeAdded->hotel_name[NAME_LENGTH-1] = '\0';   

    cout << "string-" << elementToBeAdded->hotel_name << "-\n\n"; 
    delete elementToBeAdded; 
} 

Answer 6

上記のソースコードに誤スペルがありますので：

for (int j = 0; j < NAME_LENGTH; j++)  
    elementToBeAdded->hotel_name[i] = 'a'; 

インデックスはこちら[J]でなければなりません。