使用字符串指针的分段错误

Question

我是一个C++新手，我试图把实践中的指针与字符串。我所做的程序只是将用户类型的字符串存储在命令行中。但我得到段错误，不知道为什么。

这是代码：

#include <cstdio> 
#include <cstdlib> 
#include <iostream> 

using namespace std; 

//This code is meant to learn how to use pointers and strings 
//Ask the user for who are they in the family and save it in string array! 


void print_string (string* Value, int const nSize); 
int get_names(string* Family_input); 

int main (int nNumberofArgs, char* pszArgs[]) 
{ 

    cout << "Tis program stores your family members\n"; 
    cout<< "Type the names and write 0 to exit\n"; 

    string familia_string; 
    string* familia = &familia_string; 

    int family_number; 

    family_number=get_names(familia); 

    cout << "The family members are: "; 

    print_string(familia, family_number); 

    cout << endl; 

    return 0; 
} 

int get_names(string* Family_input) 
{ 
    int i=0; 
    string input=""; 
    string old_input=""; 
    while (input!="0") 
    { 
    cout << "type " << i <<" member\n"; 
    //cin >> *(Family_input+i); 
    //input=*(Family_input+i); 
    cin >> input; 
    *(Family_input + old_input.length()) = input; 
    old_input=input; 
    i++; 
    } 
    return i; 

} 

void print_string (string* Value, int const nSize) 
{// I don't want to &psValue to be changed! 
    for (int i=0; i<nSize; i++) 
    { 
    cout << *(Value+i) << " "; 
    //&psValue++; 
    } 
} 

我不知道这是否是因为我不接受字符串的正确尺寸，或者我没有使用正确的指针或者是我有在使用偏移量之前分配内存。

Answer 1

由于您没有为字符串数组分配内存，因此会发生seg故障。

*(Family_input + old_input.length()) = input; 

这是完全废话。如果你有一个数组，你只能增加一个索引而不是字符串的长度。

Answer 2

正如@kleszcz已经指出，该行

*(Family_input + old_input.length()) = input; 

是错误的。您正在访问您不应该访问的内存。

最简单的解决方法是改变get_names略：

int get_names(string* Family_input) 
{ 
    int i=0; 
    string input=""; 
    while (input!="0") 
    { 
     cout << "type " << i <<" member\n"; 
     cin >> input; 
     *Family_input += input; // Just keep on appending to the input argument. 
     *Family_input += "\n"; // Add a newline to separate the inputs. 
     i++; 
    } 
    return i; 
} 

也改变print_string到：

void print_string (string* Value) 
{ 
    cout << *Value; 
} 

当然，print_string变得如此简单，你不需要把它在所有。

您可以更改get_names以使用参考参数而不是指针参数。这是一个更好的做法。

int get_names(string& Family_input) 
{ 
    int i=0; 
    string input=""; 
    while (input!="0") 
    { 
     cout << "type " << i <<" member\n"; 
     cin >> input; 
     Family_input += input; // Just keep on appending to the input argument. 
     Family_input += "\n"; // Add a newline to separate the inputs. 
     i++; 
    } 
    return i; 
} 

然后，将呼叫更改为get_names。除了使用

family_number=get_names(familia); 

使用

family_number=get_names(familia_string); 

Answer 3

如果你想保存在不同的字符串对象不同的名字，我建议：

#include <iostream> 
#include <string> 
#include <cstdlib> 

using namespace std; 

void print_string(string** Value, int const nSize); 
void get_names(string** Family_input, int count); 


int main(int nNumberofArgs, char* pszArgs[]) { 

cout << "This program stores your family members\n"; 
int family_number; 
cout << "Types the number of family members"; 
cin >> family_number; 

/*allocate the number of string pointers that you need*/ 
string** familia = new string*[family_number]; 

cout << "Type the names\n"; 

get_names(familia, family_number); 

cout << "The family members are: "; 

print_string(familia, family_number); 

cout << endl; 

return 0; 
} 

void get_names(string** Family_input, int count) { 

for(int i = 0 ; i < count; i++){ 
    cout << "type " << i << " member\n"; 
    /*create a string obj in the heap memory*/ 
    string *input = new string (""); 
    // using that way you get only a single word 
    cin >> *input; 
    /*create a string object on the stack and put its pointer in the family_array*/ 
    Family_input[i] = input; 
} 
} 

void print_string(string** Value, int const nSize) { 
for (int i = 0; i < nSize; i++) { 
    cout << *(Value[i]) << " "; 
} 
}