c++ - 在其他重载中重用重载

Question

如果我有 setter 函数重载

void setDamageRange(const Range& r);
void setDamageRange(int min, int max);

而且我能够在另一个中使用其中一个重载

void Weapon::setDamageRange(const Range& r)
{
    setDamageRange(r.min, r.max);
}

void Weapon::setDamageRange(int min, int max)
{
    mDamageRange.min = min;
    mDamageRange.max = max;
}

我应该这样做吗？或者我应该像这样再次进行所有有效性检查和分配

void Weapon::setDamageRange(const Range& r)
{
    mDamageRange = r;
}

void Weapon::setDamageRange(int min, int max)
{
    mDamageRange.min = min;
    mDamageRange.max = max;
}

?

我的直觉告诉我，出于重用代码的目的，我应该选择第一个备选方案，因为这通常是很好的做法。但与此同时，感觉它在某种程度上使它变得困惑，特别是如果我要提供更多的函数重载，因为其中只有一个是“控制”函数。

如果我认为我并不真的需要“控制”函数并将其删除，我就必须重写所有其他重载的代码。

Answer 1

should I [call one overload from the other]?

当然!这是重用代码的绝佳方式。

Or should I do all validity checking and assignment again

通常，这会违背 Don't Repeat Yourself原则，因为相同的验证(如果有的话)必须进入两个地方。