需要帮助将 C 宏重写为函数

Question

我需要一些帮助以更类型安全的方式重写以下行并将其重写为函数，但是该代码是在函数内部定义的事实使我很难想到一种聪明的方法，因为显然它将涉及声明几个参数。

#define CHECK(id) if(table->cells[id]) isgood[table->cells[id]-1] = 0;

其中table是一个struct，isgood是一个int。

Answer 1

直接翻译（如果 table->cells[id] 是 int）：

void check(int id, int*isgood) { if (id) isgood[id-1] = 0; }

调用：

check(table->cells[id], isgood);

但是，我会对此进行重命名/修改。 我特别想改一下名字。 也没有错误检查 - 即，如果 table->cells[id]==0，您将尝试设置 isgood[-1]，这会很糟糕。

Answer 2

显然这需要声明
几个论据

有什么问题吗？

Answer 3

为什么不只是一个接收 table 和 id 的函数来执行此操作呢？

void foo(TableType & t, int id)
{
    if (t.cells[id]) 
        isgood[t.cells[id]-1] = 0;
}

ps

这确实是一个糟糕的宏。 这个名字很有误导性。

pps

整个事情相当奇怪，我无法理解这个函数的逻辑。 这究竟要达到什么目的？

Answer 4

如果您使用 C++，我会考虑将 check 作为表的成员函数，这似乎是一个很好的候选类：

class Table {
    //...
    public bool check(int id) {
        if (this->cells[id]) {
            this->isGood[id] = 0;
            // the line you have, isgood[table->cells[id]-1] = 0 looks buggy:
            // you treat table->cells[id] as a true/false value one line ago;
            // probably not a valid array index? I'm taking a stab at what to do.
        }
    }
}

Answer 5

通常最好不要在宏中引用变量。

首先，确保名称有意义。 你在检查什么？ 还有副作用吗？

void update_valid_cells(Table& table, int id, BoolArray& validArray)
{
     if(table.cells[id]==NULL) return;
     validArray[id]-1=false;
}

Answer 6

我认为 C99 可以将函数限定为内联函数，这样您就可以在不使用宏的情况下获得无函数调用的加速。 此外，大多数 C 编译器都支持 __inline 等扩展来实现此目的。