Question

最初，我的实验室传递了三个参数：addFractionJesseR（* lFrac，* rFrac，** resFrac）;但我发现我不能通过三个论点。我不得不把它改成** resFrac = addFractionJesseR（* lFrac，* rFrac）;现在我在编译方面遇到了问题。我知道我的指针和双指针在某处超出范围，但我无法发现在哪里。调试器指向第二行作为问题：

#include "Monster.h"

Monster::Monster()
{
    //ctor
}

Monster::~Monster()
{
    //dtor
}

由以下人员调用：

FractionJesseR& FractionJesseR::operator=(const FractionJesseR& arg) {
  num = arg.num;
  denom = arg.denom;
  return *this;
}

** resFrac = addFractionJesseR（* lFrac，* rFrac）;最初是addFractionJesseR（* lFrac，* rFrac，** resFrac）;

由以下人员调用：

FractionJesseR& addMenu(FractionJesseR* lFrac, FractionJesseR* rFrac) {
  int option;
  FractionJesseR** resFrac = new FractionJesseR*();
......
    case 2:
      cout << "Calling add() --\n\n";
      **resFrac = addFractionJesseR(*lFrac, *rFrac);
      break;
    ......

* resFrac = addMenu（lFrac，rFrac）最初是addMenu（lFrac，rFrac，＆amp; resFrac）

（是的，我确实在我的所有指针上调用了delete，我仍然是Stack Overflow的新手，并且学会只提供相关的代码片段）我需要帮助指向正确的方向。我认为我的指针超出了addMenu或displayMenu中的某个范围......也许我正在取消引用双指针错误？

非常感谢任何帮助！

编辑：

void displayMenu() {
  int option;
  FractionJesseR *lFrac = nullptr;
  FractionJesseR *rFrac = nullptr;
  FractionJesseR *resFrac = nullptr;
......
    case 2:
      cout << "  Adding Option --\n\n";
      if (lFrac == nullptr && rFrac == nullptr) {
        cout << "    Not a proper call as no Fractions are available!\n\n";
      }
      else {
        *resFrac = addMenu(lFrac, rFrac);
      }
      break;

Answer 1

您遇到内存管理问题。

我假设您使用的是某些版本的Visual Studio。调试器通常会标记要执行的下一行，因此您将在

上崩溃

num = arg.num;

这是因为this为空，因为

FractionJesseR** resFrac = new FractionJesseR*();

在免费商店上分配一个指针（这是不寻常的）并用0初始化它（由于括号）。这样：

**resFrac

首先取消引用resFrac，给出一个空指针，然后再将其取消引用。取消引用空指针是未定义的行为。在您的情况下，它会在第一个语句中导致赋值运算符崩溃。

显而易见的解决方案是停止使用指针和手动内存管理。充其量，使用对象：

FractionJesseR resFrac
// ...
resFrac = addFractionJesseR(*lFrac, *rFrac);

最糟糕的是，使用智能指针：

auto resFrac = std::make_unique<FractionJesseR>();
// ...
*resFrac = addFractionJesseR(*lFrac, *rFrac);

你不想做什么（除非你被迫，因为这是一项任务，在这种情况下我会质疑动机）：

auto resFrac = new FractionJesseR;
// ...
*resFrac = addFractionJesseR(*lFrac, *rFrac);
// ...
delete resFrac;

addFractionJesseR()返回对在免费商店中分配的值的引用。你要在哪里删除它？每个new必须与delete匹配。

如果你真的想要一个手动内存管理的例子，你不应该混合指针和引用：

FractionJesseR* addFractionJesseR(FractionJesseR* a, FractionJesseR* b)
{
    auto n = a->getNum() * b->getDenom() + a->getDenom() * b->getNum();
    auto d = a->getDenom() * b->getDenom();

    if (d < 0)
    {
        d = -d;
        n = -n;
    }

    return new FractionJesseR(n / gcd(n, d), d / gcd(n, d));
}

FractionJesseR* addMenu(FractionJesseR* a, FractionJesseR* b)
{
    // ...
    FractionJesseR* resFrac = addFractionJesseR(a, b);
    // ...
    return resFrac;
}

void displayMenu()
{
    // ...
    FractionJesseR *resFrac = addMenu(lFrac, rFrac);
    // ...
    delete resFrac;
}

请注意，addFractionJesseR()中分配的指针会返回addMenu()，然后将其返回displayMenu()，删除它。

operator =使用双指针进行重载以进行分数数学运算

1 个答案: