c++ - 谁能解释一下为什么这段代码不起作用？

Question

我正在学习 C++。现在，我正在尝试制作一个与对象的重载运算符相关的示例。我的对象(称为 Contador)具有不同的方法和变量，可帮助用户计算迭代次数。

对象的头文件:

class Contador
{
private:
    int* Valor;
    int* Salto;

public:
    Contador(int Valor_Inicio = 0, int Salto = 1);
    ~Contador();

inline int Get_Valor() const { return *Valor; }
inline int Get_Salto() const { return *Salto; }

inline void Incremento() { Set_Valor(Get_Valor() + Get_Salto()); }

inline void operator++ () { Set_Valor(Get_Valor() + Get_Salto()); }

void Set_Valor(int Valor);
void Set_Salto(int Salto);

};

对象的cpp文件:

// Librerias Propias
#include "Contador.h"

Contador::Contador(int Valor_Inicio, int Salto)
{
    Set_Valor(Valor_Inicio);
    Set_Salto(Salto);
}

Contador::~Contador()
{
    delete Contador::Valor;
    delete Contador::Salto;
}

void Contador::Set_Valor(int Valor)
{
    delete Contador::Valor;
    Contador::Valor = new int(Valor);
}

void Contador::Set_Salto(int Salto)
{
    delete Contador::Salto;
    Contador::Salto = new int(Salto);
}

示例的 main() 函数有 2 个不同的 for 循环。在第一个中，我调用了 Incremento() 方法，在第二个中，我调用了重载运算符。

主要功能:

void main()
{
    // Genero el elemento de analisis.
    Contador* contador = new Contador();

    // Realizo el bucle con la función de incremento.
    std::cout << "Incremento()" << std::endl;
    for (contador->Set_Valor(0); contador->Get_Valor() < 3; contador->Incremento())
    {
        // Escribo algo.
        std::cout << "Iteracion actual: " << contador->Get_Valor() << std::endl;
    }

    // Realizo el bucle on el operador sobrecargado
    std::cout << "operador sobrecargado" << std::endl;
    for (contador->Set_Valor(0); contador->Get_Valor() < 3; contador++)
    {
        // Escribo algo.
        std::cout << "Iteracion actual: " << contador->Get_Valor() << std::endl;
    }
}

问题出现在主函数通过第二个循环的第一个迭代时。它在 Get_Valor() 方法中抛出一个异常。

在我看来，它在某个地方更改了指针 Valor 的内存地址，但我找不到位置。

有人能帮帮我吗？ 谢谢。

Answer 1

contador++ 并没有按照您的想法去做 - contador 是一个指针，而不是 Contador，所以它会使 contador 指向不存在的东西。
您需要取消引用指针。

但是，*contador++ 也会递增 contador - 它是 *(contador++) - 和 (*contador)++ 无法编译，因为您只重载了前缀运算符(后缀运算符的原型(prototype)为 operator++(int)。

所以，++*contador 会做你想做的事。

通过在不必要的情况下不使用指针，您可以避免许多类似的问题和笨拙的语法。