我的宏是不是有点太调皮了？ [关闭]

Question

我做了一个宏。我担心它可能有点“太顽皮”，因为宏是邪恶的。

这是没有宏的代码：

case SDLK_a:
    _buffer_.InsertAtCursor('a');
    _buffer_.MutableCursor().Right();
    break;

这里是宏定义及其用法：

#define SDLK_KEY_CASE_MACRO(X, Y) \
{\
case X: \
    _buffer_.InsertAtCursor(Y); \
    _buffer_.MutableCursor().Right(); \
    break;\
}

SDLK_KEY_CASE_MACRO(SDLK_b, 'b');
SDLK_KEY_CASE_MACRO(SDLK_c, 'c');
SDLK_KEY_CASE_MACRO(SDLK_d, 'd');
SDLK_KEY_CASE_MACRO(SDLK_e, 'e');
SDLK_KEY_CASE_MACRO(SDLK_f, 'f');
SDLK_KEY_CASE_MACRO(SDLK_g, 'g');
SDLK_KEY_CASE_MACRO(SDLK_h, 'h');
SDLK_KEY_CASE_MACRO(SDLK_i, 'i');
SDLK_KEY_CASE_MACRO(SDLK_j, 'j');
SDLK_KEY_CASE_MACRO(SDLK_k, 'k');
SDLK_KEY_CASE_MACRO(SDLK_l, 'l');
SDLK_KEY_CASE_MACRO(SDLK_m, 'm');
SDLK_KEY_CASE_MACRO(SDLK_n, 'n');
SDLK_KEY_CASE_MACRO(SDLK_o, 'o');
SDLK_KEY_CASE_MACRO(SDLK_p, 'p');
SDLK_KEY_CASE_MACRO(SDLK_q, 'q');
SDLK_KEY_CASE_MACRO(SDLK_r, 'r');
SDLK_KEY_CASE_MACRO(SDLK_s, 's');
SDLK_KEY_CASE_MACRO(SDLK_t, 't');
SDLK_KEY_CASE_MACRO(SDLK_u, 'u');
SDLK_KEY_CASE_MACRO(SDLK_v, 'v');
SDLK_KEY_CASE_MACRO(SDLK_w, 'w');
SDLK_KEY_CASE_MACRO(SDLK_x, 'x');
SDLK_KEY_CASE_MACRO(SDLK_y, 'y');
SDLK_KEY_CASE_MACRO(SDLK_z, 'z');

显然，这为我节省了很多打字时间和很多代码行。代码也编译运行正常。

• 但是这个宏是不是太调皮了？ （它是否可能会产生意想不到的行为，或者我有什么理由不应该在我的代码中使用这种“作弊”？）

Answer 1

像任何健全的系统一样，SDL_SCANCODE_A（又名SDLK_a）在一个连续的整数范围内，并且一直到SDL_SCANCODE_Z。 'a' 到 'z' 也是如此。

if( val >= SDLK_a && val <= SDLK_z ) {
  _buffer_.InsertAtCursor( 'a' + (val-SDLK_a) );
  _buffer_.MutableCursor().Right();
} else {
  // handle other cases
}

这比一堵宏墙更高效、更短、更清晰。

您可能需要重复数字。

对于特殊字符（制表符、引号等），我不会依赖精心安排的内容。在那里你会想要一张短桌

struct SDL_table {
  unsigned code; 
  char value;
};

SDL_table table[] = {
  {SDL_SCANCODE_TAB, '\t'},
  // ... etc
};

然后，在之前的检查失败后，快速搜索该表

auto it = std::find_if( std::begin(table), std::end(table), [&](SDL_table t) { return t.code == val; } );
if (it != std::end(table)) {
  _buffer_.InsertAtCursor( it->value );
  _buffer_.MutableCursor();
} else {
  // deal with unrecognized scancode
}

此时我可能会重构 _buffer_ 代码。

char SDLK_to_char( unsigned sdlk ) {
  if( sdlk >= SDLK_a && sdlk <= SDLK_z )
     return 'a' + (val-SDLK_a);
  if( sdlk >= SDLK_0 && sdlk <= SDLK_9 )
     return '0' + (sdlk-SDLK_0);
  struct SDLK_entry {
    unsigned code; 
    char value;
  };

  static const SDLK_entry table[] = {
    {SDL_SCANCODE_TAB, '\t'},
    // ... etc
  };

  auto it = std::find_if( std::begin(table), std::end(table),
   [&](auto&& t) { return t.code == sdlk; }
  );
  if (it != std::end(table))
    return it->value;
  return 0; // null for failure
}

然后

char c = SDLK_to_char( sdlk );

if (c) {
  _buffer_.InsertAtCursor( c );
  _buffer_.MutableCursor().Right();
}

Answer 2

好吧，我会避免使用宏 - 即使我必须用案例编写完整的 switch。

但首先我会考虑另一种方法。

例如将SDLK_... 转换为... 的查找表。在 C++ 中，您可以基于 std::map

喜欢：

  std::map<SDLK_type, char> look_up_table = {{SDLK_a, 'a'}, {SDLK_b, 'b'}};

  auto i = look_up_table.find(SDLK_value);
  if (i != look_up_table.end())
  {
    char c = i->second;
    _buffer_.InsertAtCursor(c);
    _buffer_.MutableCursor().Right();
  }
  else
  {
    std::cout << "Not found" << std::endl;
  }