メモリリークを起こさずにローカル/一時オブジェクトへの参照を返すか？

Question

私は（私のコンパイラがそう言う！）、これは間違っている実現：

Rectangle& Rectangle::Overlap(const Rectangle& rectangle) { 

    Point topLeft(__max(this->GetVerticies()[0]->GetX(), rectangle.GetVerticies()[0]->GetX()) - __min(this->GetVerticies()[0]->GetX() + this->GetWidth(), rectangle.GetVerticies()[0]->GetX() + rectangle.GetWidth()), 
       (__max(this->GetVerticies()[0]->GetY(), rectangle.GetVerticies()[0]->GetY()) - __min(this->GetVerticies()[0]->GetY() + this->GetHeight(), rectangle.GetVerticies()[0]->GetY() + rectangle.GetHeight()))); 

    Point bottomRight(__min(this->GetVerticies()[0]->GetX() + this->GetWidth(), rectangle.GetVerticies()[0]->GetX() + rectangle.GetWidth()), topLeft.GetY() + __max(this->GetVerticies()[0]->GetY() + this->GetHeight(), rectangle.GetVerticies()[0]->GetY() + rectangle.GetHeight())); 

    return Rectangle(topLeft, bottomRight); 
} 

メモリリークを発生させずに計算された矩形を返すための正しい方法でしょうか？ Rectangle * result = new Rectangle（topLeft、bottomRight）として矩形を定義すると、逆参照されたポインタが返されますが、間違っているようです。助言がありますか？

Answer 1

いずれかの値でリターン：

あなたの関数本体を変更する必要、または結果を返すために追加のパラメータを追加しません

Rectangle Rectangle::Overlap(const Rectangle& rectangle); 

：

void Rectangle::Overlap(const Rectangle& rectangle, Rectangle& out); 

とアウトに結果を代入パラメータ。

Answer 2

戻り値の型を非参照（値）にします。次に、暗黙のコピーコンストラクタを使用して、返された値が正常になります。

Answer 3

戻り値の型をRectangleに変更してください。

Answer 4

ちょうど1つの参照ではなく、矩形を返します。