0

我正在从事一项介绍内存和指针动态分配原则的作业。我过去做过一个简单的程序,它接受 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 分段错误(核心转储)

无论我放什么 int 都会发生这种情况。我在 cin << num; 之后放了一个 cout 语句 看看程序是否能做到这一点,但它从来没有。

任何帮助是极大的赞赏。抱歉,如果这是有史以来最基本的错误。

4

3 回答 3

2
int **intPoint;

此时在您的代码中,intPoint没有指向任何内容,因为您尚未为其分配值。

*intPoint = new int[num];

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

尝试:

int *intPoint;

intPoint = new int[num];

现在您正在设置intPoint',使其指向您分配的整数。

于 2013-09-27T07:33:23.720 回答
0

使用 std::vector 代替 int 或字符串数​​组。

说,

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

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

于 2013-09-27T08:16:38.797 回答
0

你得到一个分段错误的原因是你取消引用一个未初始化的指针。

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.
于 2013-09-27T07:42:05.037 回答