以输入指针

Question

可有人请我们的阵列我知道为什么人们不能使用输入流如下面的代码需要输入在指针数组为字符串：

char *names[5]; 

for(int i=0; i<5; i++) 
{ 
cout<<"enter name "<<i+1; 
cin>>names[i]; 
} 

Answer 1

变量“名”是一个未定义的char指针数组。这是你的第一个问题。这些指针是未定义的，没有为它们分配内存。 在您的原始代码示例中，有五个带有未定义值的char指针。这就是程序崩溃的原因，因为它试图访问指针中无效地址的内存。 另一个问题是没有分配内存来存放从stdin进入的字符数组。

你可以做这样的事情，让您的原来的例子，而不崩溃工作：

#include <iostream> 
#include <string.h> 

using namespace std; 

int main(int argc, char** argv) 
{ 
    char names[5][128]; 
    memset(names, 0, sizeof(names)); 

    for (int i = 0 ; i < 5 ; i++) 
    { 
     cout << "enter name " << i+1 << ": "; 
     cin >> names[i]; 
    } 

    for (int i = 0 ; i < 5 ; i++) 
    { 
     cout << names[i] << "\n"; 
    } 
} 

这种分配5个128个字符的字符串数组。它也使用memset（）清除数组。

因为这是C++，它似乎更有意义，做它的C++风格。

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

using namespace std; 

const int NUM_NAMES = 5; 

int main(int argc, char** argv) 
{ 
    vector<string> names; 
    names.resize(NUM_NAMES); 

    for (int i = 0 ; i < NUM_NAMES ; i++) 
    { 
     cout << "enter name " << i+1 << ": "; 
     cin >> names[i]; 
    } 

    for (int i = 0 ; i < NUM_NAMES ; i++) 
    { 
     cout << names[i] << "\n"; 
    } 
} 

Answer 2

免责声明：使用char*从std::cin阅读 通常是一个坏主意，我建议你学习如何在未来使用std::string。

此外这个问题本来会更适合codereview.SE，但现在我会忽略它。

下面是做你想做的事情的一个例子，但我强烈坚持你避免在任何真正的代码中做这件事。

char * names[5]; 

// Allocate names 
for (int i = 0; i < 5; i++) 
{ 
    // Use 256 characters as a buffer 
    // any names longer than that might cause errors 
    names[i] = new char[256]; 
} 

// Do IO 
for (int i = 0; i<5; i++) 
{ 
    // Append std::endl 
    std::cout << "enter name " << i + 1 << std::endl; 
    // This isn't very robust since the user can give anything as a name, 
    // and they can't use spaces because of how cin works in this example. 
    std::cin >> names[i]; 
} 

// Use names 
for (int i = 0; i < 5; i++) 
{ 
    // Printing names is just an example 
    // you could write the names to somewhere more important 
    // you could copy them to a smaller location to use less memory 
    std::cout << names[i] << std::endl; 
} 

// Dispose of names when done 
for (int i = 0; i < 5; i++) 
{ 
    // Check pointer isn't null 
    if (names[i] != nullptr) 
    { 
     // Never forget to delete 
     delete names[i]; 
    } 
}