Question

为什么我会收到此分段错误（核心转储）？我最初被告知这是因为我的指针p_p_tictactoe = new char * [cols]，但我被告知不正确。每组代码的目的都在整个代码中被注释。代码正在运行，但我得到了这个结果。我知道for循环必须是这个问题的主要问题。

Please enter a number of rows: 4
Please enter number of columns: 3
Enter a single character for position( << i << ): a  
Enter a single character for position( << j << ): b  
Segmentation fault (core dumped)



 #include <iostream>

        using namespace std;

        int main()
        {
           // TODO:  Below each Bp_tictactoe` of type pointer-to-a-pointer-to-a-char
       char **p_p_tictactoe;


       // 2. Prompt your user to enter a number of rows, then a number of columns.
       //    store their answers in the variables `rows` and `cols`.
       char rows;
       char cols;
       cout << "Please enter a number of rows: ";
       cin >> rows;
       cout << "Please enter number of columns: ";
       cin >> cols;
       // 3. Allocate a 1-dimensional array of pointers-to-chars (length == `rows`)
       //    and store its address in `p_p_tictactoe`
       p_p_tictactoe = new char*[rows];
       // 4. Use a for-loop to allocate a dynamic array of chars (length == `cols`)
       //    for i from 0 to rows - 1 and store its address in `p_p_tictactoe[i]`.
       for (int i = 0; i < rows - 1; i++)
       {
         p_p_tictactoe = new char*[cols];
       }
       // 5. Use a for-loop to prompt the user to enter a char for each position in
       //    (" << i << ", " << j << "): "
       //    As you read each char, store it in the array.
       // 6. Use a nested for-loop to print the array, one row per line.  The chars
       //    for each row should be space-separated.  For example, if the array is
       //    2 x 3 and stores the values A, B, C, X, !, &, the output should look
       //    like:
       //       A B C
       //       X ! &\


       char new_input1;
       char new_input2;
       for (int i = 0; i < rows; i++)
       {
          for (int j = 0; j < cols; j++)
          {
             cout << "Enter a single character for position( << i << ): ";
             cin >> new_input1;
             cout << "Enter a single character for position( << j << ): ";
             cin >> new_input2;
             *p_p_tictactoe[i] = new_input1;
             *p_p_tictactoe[j] = new_input2;
             cout << *p_p_tictactoe[i] <<endl;
          }  
       }
       // *** Prevent memory leaks by deallocating dynamic memory when you are done
       // using it. ***

       // 7. Use a for-loop to delete each row of the dynamic array.                


       // 8. Delete the pointer-to-a-pointer to release the array of row pointers,
       //    and set its value to NULL to avoid accessing invalid memory.
for (int i = 0; i < 3; i++)
{
delete[] p_p_tictactoe[i];
delete[] p_p_tictactoe;
}
cout << "Bye!" << endl;
return 0;
    }

Answer 1

除此之外，您p_p_tictactoe的分配不正确。这是一个双指针，它只是意味着它是一个指针数组的指针。你的两步分配是正确的想法，但你在for循环中所拥有的是不正确的。在行p_p_tictactoe = new char*[rows]之后，您现在有一个指向char*类型数组的指针。因此，如果rows为4，那么您在内存中的内容现在看起来像是：

 p_p_tictactoe[0] == char* --> junk
              [1] == char* --> junk
              [2] == char* --> junk
              [3] == char* --> junk

您现在必须遍历这4个char*中的每一个并为它们分配空间。每个char*必须指向chars的数组。这是注释为您提供有关循环并使用p_p_tictactoe[i]索引的提示：

 for (int i = 0; i < rows - 1; i++)
 {
   p_p_tictactoe[i] = new char[cols];
 }

现在，对于cols == 3，在内存中你有：

 p_p_tictactoe[0] == char* --> 3 consecutive bytes
              [1] == char* --> 3 consecutive bytes
              [2] == char* --> 3 consecutive bytes
              [3] == char* --> 3 consecutive bytes

您发布的代码是内存泄漏。每次执行p_p_tictactoe = new char*[#]时，操作系统都会进入堆以获取足够的内存以进行分配。你没有跟踪前一个指针，也没有先释放它，所以分配的内存现在没有指向它。

同样的理论适用于释放记忆。你最后得到的不太正确。解除分配始终是分配的镜像。这显然是一个家庭作业，所以我不会发布那个代码，但它与分配相同，除了相反。

我强烈建议使用gdb，它是linux（或任何等效调试器）的文本调试器。如果您希望在C / C ++中成功编写代码，您必须了解内存如何在堆栈和堆上运行，并且您必须学习如何正确地管理它，否则您将陷入一个受伤的世界。 gdb一开始有点令人生畏，但它会让你打印出内存地址并检查内存，这对学习和重新执行你认为你知道的内容非常有帮助。

Answer 2

我认为这里的问题是你应该为双指针变量采用双数组..你将两个不同的东西重新分配给一个变量..

Answer 3

您的代码存在的一个问题是您的宽度和高度不会按照您认为的方式进行解释。当你这样写：

char width;
cin >> width;

...生成的程序读取一个字符并将其ASCII值分配给width。因此，如果您输入了＆＃39; 4，那么width == 52将评估为true， width == 4将为false。（但width == '4'将是true因为'4' == 52）。

此问题很容易解决：只需使用int代替char即可。你还没有保存任何内存，因为new可能正在创建单词对齐的指针。

除了Tarang Gupta指出的问题外，还有其他问题。

Answer 4

这是因为行：

*p_p_tictactoe[i] = new_input1;
*p_p_tictactoe[j] = new_input2;

您将其视为具有2维的数组。

错误地写了以下内容：

 p_p_tictactoe = new char*[rows];
 for (int i = 0; i < rows - 1; i++)
   {
     p_p_tictactoe = new char*[cols];
   }

您首先将p_p_tictactoe大小sizeof (char*[rows])和替换其内容的空间分配为p_p_tictactoe = new char*[cols]; (int)row次。代码中有很多未使用和未引用的内存。

即使您更改代码以防止此错误，以下内容仍会再次导致问题：

for (int i = 0; i < 3; i++)
{
    delete[] p_p_tictactoe[i];
    delete[] p_p_tictactoe;
}

您要删除p_p_tictactoe连续3次分配和引用的空间。它应该在循环之外。

分段故障。是因为糟糕的指针吗？

4 个答案: