减少代码重复和使用模板替代 #define

Question

我有这样的代码：

#define defaultOnTimer(N, X) \
 if (Timers.N) Timers.N --; \
 if (!Timers.N) S.X = D.X; \

defaultOnTimer(t1, sig1); 
defaultOnTimer(t2, sig2); 
defaultOnTimer(t3, sig3); 
defaultOnTimer(t4, sig4); 

尽管这样做很有效，但我在想是否可能使用C++模板来完成同样的工作。将来可以扩展“defaultOnTimer”以执行其他操作，因此不希望只是根据需要复制它，并且必须找到并修改每个副本。

或者这只是一个过于复杂的工作，我应该坚持使用#define？

请注意，S.X可以是任何类型。

- squidge

---

没有模板不能完全做到这一点。如果你愿意稍微改变界面（并且S与D是相同类型），它们可以做类似的事情。 - n. m.

Answer 1

看起来很糟糕，但它在某种程度上能够工作：

template<typename A, size_t N, typename B, size_t X>
void onTimer()
{
    A& n = *(A*)(&Timers + N);
    if (n)
    {
        --n;
    }
    if (!n)
    {
        *(B*)(&S + X) = *(B*)(&D + X);
    }
}

这里有一个例子：

struct Timer
{
    int t1, t2, t3, t4;
};
struct Sig
{
    char sig1, sig2, sig3, sig4;
};

Sig S, D;
Timer Timers;

void a()
{    
    onTimer<int, offsetof(Timer, t1), char, offsetof(Sig, sig1)>();
}

但是，D、S和Timers应该是全局变量。您可以通过将它们作为函数参数传递来避免这种情况：

template<typename A, size_t N, typename B, size_t X>
void onTimer(Timer* timer, Sig* sigS, Sig* sigD)
{
    A& n = *(A*)(timer + N);
    if (n)
    {
        --n;
    }
    if (!n)
    {
        *(B*)(sigS + X) = *(B*)(sigD + X);
    }
}

void a()
{    
    Sig S, D;
    Timer Timers;
    onTimer<int, offsetof(Timer, t1), char, offsetof(Sig, sig1)>(&Timers, &S, &D);
}

我想这个回答解决了你的问题，但它仍然是一个不太理想的解决方案。

- Teivaz