C++オーバーロード演算子++

Question

学校プロジェクトでは、センサーがアクティブかどうかを示すprivateプロパティactive（boolean）を持つC++でセンサークラスを実装する必要があります。 ++演算子をオーバーロードする必要があります。演算子++を使用すると、activeプロパティはtrueに設定されます。

Iは、下記（sensor.cpp）実装：私は、プログラムが実行される最後の方法を用いて（終了）を実行停止気づい

int main(int argc, char *argv[]) { 
    Sensor * sensor = new Sensor(1, "sample vendor"); 
    sensor->setStatus(false); 
    sensor++; 
    std::cout << "status: " << sensor->getStatus() << std::endl; 
} 

：

Sensor::Sensor(int id, std::string vendor) : _id(id), _vendor(vendor) { 
    std::cout << "Sensor created with id: " << Sensor::getId() << std::endl; 
    _status = false; 
} 

bool Sensor::getStatus() { 
    return _status; 
} 

void Sensor::setStatus(bool status) { 
    _status = status; 
} 

Sensor& Sensor::operator++() { 
    Sensor result = *this; 
    this->setStatus(true); 
    return result; 
} 

main.cppにしsensor-> setStatus（false）; main.cppにはありますが、私の端末にエラーは表示されず、コンパイラーは不平を言っていません。

誰かが私が何をやったのか、それをどのように修正して状態が真になるように修正できるのか考えていますか？ sensorがポインタであるので、予め

Answer 1

で

おかげで、sensor++はポインタないオブジェクトをインクリメントします。最も簡単な解決策は、最初にポインタを使用しないことです。別の解決策は、(*sensor)++を使用することです

int main() { 
    Sensor sensor{1, "sample vendor"}; 
    sensor.setStatus(false); 
    sensor++; 
    std::cout << "status: " << sensor.getStatus() << std::endl; 
} 

...

int main() { 
    std::unique_ptr<Sensor> sensor = 
     std::make_unique<Sensor>(1, "sample vendor"); 
    sensor->setStatus(false); 
    (*sensor)++; 
    std::cout << "status: " << sensor->getStatus() << std::endl; 
} 

あなたのコード内の別のエラーはここにある：

Sensor& Sensor::operator++() { 
    // You don't want to do this... it creates a copy! 
    Sensor result = *this; 
    this->setStatus(true); 
    // This is a dangling reference! 
    return result; 
} 

代わりにこれを使用します。

Sensor& Sensor::operator++() { 
    this->setStatus(true); 
    return *this; 
}