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)++无法编译，因为您仅重载了前缀运算符（后缀运算符具有原型） operator++(int)。

因此，++*contador将做您想要的事情。

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

Answer 2

表达式contador++递增contador（指针）指向的地址！因此，在第一次迭代之后，指针将完全无效。

要调用增量运算符，您需要：++(*contador)，它首先取消指向所指向对象的指针，然后影响该对象的增量运算符。

Answer 3

3个编码问题：

main应该返回int。
Valor和Salto未在构造函数中初始化。 Contador::Set_Valor和Contador::Set_Salto需要初始化的指针（在删除它们时）。

简单的解决方法是：
```
class Contador
{
private:
    int* Valor = nullptr;
    int* Salto = nullptr;
//...
};
```
最后一个问题在您的上一个循环中：
```
for (contador->Set_Valor(0); contador->Get_Valor() < 3; contador++)
```
由于condator是一个指针（不是指向数组），因此访问condator[1]将是UB。您想要++(*condator)（operator++ ()是前递增，而operator++ (int)是后递增）。

最后，避免使用所有这些指针将简化代码（并且不打扰您打破3的规则）：

class Contador
{
private:
    int Valor;
    int Salto;

public:
    Contador(int Valor = 0, int Salto = 1) : Valor(Valor), Salto(Salto) {}
    ~Contador() = default;

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

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

    void Set_Valor(int Valor) { this->Valor = Valor;}
    void Set_Salto(int Salto) { this->Salto = Salto;}
};

int main()
{
    Contador contador;

    std::cout << "Incremento()" << std::endl;
    for (contador.Set_Valor(0); contador.Get_Valor() < 3; contador.Incremento())
    {
        std::cout << "Iteracion actual: " << contador.Get_Valor() << std::endl;
    }

    std::cout << "operador sobrecargado" << std::endl;
    for (contador.Set_Valor(0); contador.Get_Valor() < 3; ++contador)
    {
        std::cout << "Iteracion actual: " << contador.Get_Valor() << std::endl;
    }
}

Answer 4

除了以前的答案。如我所见，Contador* contador = new Contador();代码还包含UB（未定义的行为）此调用等于带有参数Contador（0，1）的构造函数它们将先执行Set_Valor和Set_Salto并先调用delete的操作，但是目前不保证此变量的内容为nullptr，因此您可能会破坏数据。同样，如果编译器看到UB，则它可能会优化所有其他代码，因为它已经是UB，并且它可以更改行为，无论如何它都希望将其完全丢弃。 https://devblogs.microsoft.com/oldnewthing/20140627-00/?p=633

有人可以向我解释为什么此代码无法正常工作吗？

4 个答案: