そもそもrepマクロとは

競技プログラミングで入力文字数を減らすためにしばしば使われる、以下のようなマクロである。

#define REP(i, n)    for (int i = 0; i < (n); i++)

repマクロのなにが問題か

現行のC++においてマクロは本当に必要でない限り利用すべきでない機能とされており、他の記法を使用できる限りそちらを使用するべきである。

C/C++のマクロは、単に引数をテキスト置換した上で実コードに変換される。

重要な点として、マクロの変換処理はスコープを一切考慮しないため、例えばnamespaceなどのステートメントで適用範囲を束縛することが出来ない。

本来C/C++のfor文は宣言からループ処理の記述を含めて一つのステートメントとされているが、構文規則を無視することで宣言部分のみを無理やり分割して変換しているため、注意しないと構文の整合性を損なう恐れがある。
引数nをカッコで括っているのはこれを防ぐためである。

意地悪な例としては以下のようなものがある。

void solve() {
 	int a = 8;
 
   	REP(i = a, 3)
   		std::cout << i << std::endl;
 
    std::cout << "Out: " << a << std::endl;
}

この関数の実行結果は以下のようになる。

1
1
1
Out: 3

この関数のループカウンタは正しく動作せず、変数aは代入されているはずの8ではなく、3を返す。
マクロが以下のように変換されるからである。

for (int i = a = 0; i = a < (3); i = a++)   // aの値が変更されている！

このfor文はループカウンタ変数iの代わりに変数aをインクリメントし、条件式を含む全ての式を実行するごとに、ループカウンタに値を代入してしまう。¹

Boost Libraryの中にirangeというループを処理するためのクラスがある。

for (int i : boost::irange(0, n)) {
    // 0からn-1までのループ
}

第一引数に0以外を指定すれば0以外で始まるループも指定可能。

template<typename F> constexpr void rep(int n, F function) {
    for (int i = 0; i < n; i++)
        function(i);
}

以下のように書けるようになる。

rep(n, [&](int i) {
    // do something
});

std::for_eachとほとんど同じように使えるので、タイプ数を減らすなら多分この方法が一番実用的。

良記事です。ループの途中抜けに対応していたりと色々すごい。

古い環境でミリ秒単位の高速化をするために、C時代から使われていたループ。
元の変数そのものをループカウンタとして使用するため、値が変わってしまうのがネック。
当然ながら、値がマイナスだった場合は（実質的に）無限ループしてしまう。

while (n--) {
    // do something
}

確かにタイプ量は少ないが…

多分そろそろRustに乗り換えたほうがいいです。

iにカッコをつければ抑制できるが、~~GCCの場合、カッコが必要ないケースでwarningが出る。~~int{i}と書けばwarningを抑制できます。 ↩