Question

我正在进行一项任务，即引入内存和指针动态分配的原理。我过去制作了一个简单的程序，它接受了5个名字和5个分数，然后使用选择排序将它们按降序排列。我现在的任务是回到同一个程序并询问用户他们想要输入多少分数，然后使用指针动态分配必要的内存量。这是我第一次使用指针和这些概念，所以我仍然试图弄明白。

我得到了要编译的代码，但是只要我输入任意整数，我想输入多少分数就会得到分段错误错误（这是程序要求的第一件事）

我确定在我调用和声明函数的过程中会有一些错误，所以如果有什么我只是拼命改变请告诉我，但是现在我不明白为什么我的程序在崩溃的地方崩溃。这是我的代码

#include <iostream>
using namespace std;

void initializeData(string *names[], int *scores[], int num);
void displayData(string *names[], int *scores[], int num);
void sortData(string *names[], int *scores[], int num);

int main()
{
int num;
int **intPoint;
string **strPoint;

 cout << "How many scores would you like to enter?: ";
 cin >> num;

 cout << " core dumped? ";

*intPoint = new int[num];
*strPoint = new string[num];

initializeData(strPoint,intPoint,num);
sortData(strPoint,intPoint,num);
displayData(strPoint,intPoint,num); 

return 0;
}

void initializeData(string *names[], int *scores[], int num)
{
        for(int i=0;i<num;i++)
        {
                cout << "Please input the name for score: " << i+1 << ": " << endl;
                cin >> *(names[i]);   
                cout << "Please input the score for player: " << i+1 << ": " << endl;
                cin >> *(scores[i]);
        }
}

void sortData(string *names[], int *scores[], int num)
{
 int minIndex,minValue,x;
 string stringTemp;

             for(int i = 0;i<(num-1);i++)
             {
                minIndex = i;
                minValue = *(scores[i]);

                for(x= i+1;x<num;x++)
                {
                        if(*(scores[x]) > minValue)   
                         {
                          minValue = *(scores[x]);
                          minIndex = x;
                         }
                }

                *(scores[minIndex])=*(scores[i]);
                *(scores[i]) = minValue;

                stringTemp = *(names[minIndex]);
                *(names[minIndex]) = *(names[i]);
                *(names[i]) = stringTemp;
        }
}

void displayData(string *names[], int *scores[], int num)
{
cout << "Top scorers: " << endl;
         for(int i=0;i<num;i++)
        {
                cout << names[i] <<": ";
                cout << scores[i] << endl;
        }
}

和我目前的输出：你想进入多少分数？：10 分段错误（核心转储）

无论我放在哪里，都会发生这种情况。我在之后发了一个cout声明 cin＆lt;＆lt; NUM;看程序是否达到了那么远但它永远不会。

非常感谢任何帮助。对不起，如果这是有史以来最基本的错误。

Answer 1

int **intPoint;

在您的代码中，intPoint未指向任何内容，因为您尚未为其指定值。

*intPoint = new int[num];

然后你取消引用它，但它没有指向任何东西。

尝试：

int *intPoint;

intPoint = new int[num];

现在您要设置intPoint的值，使其指向您分配的整数。

Answer 2

您遇到分段错误的原因是您取消引用未初始化的指针。

int **intPoint; // intPoint is declared a pointer to a 'pointer to an int';
                // but currently it points to nothing
*intPoint = new int[num]; // *intPoint "dereferences" intPoint, i.e., assigns w/e it 
                          // pointed to (which is nothing) to a pointer.

与其他人建议的一样，你在这里不需要双指针。

int *intPoint; // intPoint is a pointer to an int
intPoint = new int[num]; // notice how we didn't dereference intPoint.
                         // all we did was assign to our newly minted memory.

Answer 3

在int或string数组的位置使用std :: vector。

说，

  std::vector<int> scores;
  std::vector<string> names;

这样你可以避免所有的麻烦。这很简单而且优雅。

动态分配和数组/指针的分段错误

3 个答案: