删除遗留代码中的宏

Question

我有很多使用以下形式的宏的遗留代码：

#define FXX(x) pField->GetValue(x)

宏强制变量 pField 位于范围内：

.....
FIELD *pField = ....
.....
int i = FXX(3);
int j = FXX(5);

有没有办法在不触及用户代码的情况下替换宏？

由于 FXX(x) 具有函数调用风格，因此我考虑了内联函数或类似的东西。

UPD： 人们只是习惯了宏观，而我想保持原样。

Answer 1

使用 find & 怎么样？在您最喜欢的编辑器中替换 函数...我认为它在您在问题中给出的示例中可以正常工作。将 FXX 替换为 pField->GetValue，然后删除 #define 行

Answer 2

pField 是什么（除了 Systems Hungarian 的一个令人厌恶的好例子之外）？如果碰巧它是一个全局变量或单例或我们只需要其中之一的东西，我们可以做一个像这样的漂亮技巧：

int FFX(int x)
{
    static FIELD *pField = ...; // remove this line if pField is global
    return pField->GetValue(x);
}

将 int 类型更改为您需要它进行操作的任何类型，甚至是模板（如果您需要它支持多种类型）。

@epatel 建议的另一种选择是使用您最喜欢的文本编辑器的查找和替换，只需将所有 FFX(x) 行更改为 pField->GetValue(x)，从而消除代码中的宏调用。如果你想保留函数调用，你可以将 FFX(x) 更改为 FFX(pField, x) 并将宏更改为采用两个参数（或将其更改为一个带有两个参数的函数）。但此时您也可以删除宏。

第三种选择是不修复未损坏的部分。该宏不是特别好，但是尝试删除它可能会带来更大的问题。宏并不是撒旦的产物（尽管它至少在地狱里有一些亲戚）。

Answer 3

您需要的是一个依赖于所定义变量的函数。唯一的方法是在与函数相同的作用域中声明该变量。但随后您的函数将使用该函数，而不是从调用函数的位置声明的函数。

所以我相当有信心这是不可能完成的。

Answer 4

好吧，如果您可以将此函数定义放在 pField 已在范围内的位置：

int FXX(int x) { return pField->GetValue(x); }

否则，无法在不影响现有代码的情况下将 pField 放入函数中。

在这种情况下，使用宏可能是最好的选择。宏可能是邪恶的，但有时它们是必要的。请参阅 http://www.parashift.com/c++ -faq-lite/big-picture.html#faq-6.15

Answer 5

我会保留原样，但只是为了讨论，并且根据代码的哪些部分是“不可触及的”，您可以定义一个接受 pField 的函子，并在同一范围内创建变量后进行初始化：

class FFX_t {
   FFX_t( FIELD * pField ) : field_(pField) {}
   int operator()( int index ) {
      return field_->GetValue( index );
   }
private:
   FIELD *field_;
};
// usage:
void f() {
   FIELD * pField = //...
   FFX_t FFX(pField); // added after pField construction
   // ...
   int a = FFX(5);
}

但是我坚持认为，为了工作代码而改变它不会真正增加任何价值是没有用的。