C++でrep関数に関してのトラブル

Question

C++でエラーメッセージが出てしまい解決できません

エラーメッセージは次のとおりです
./Main.cpp: In function ‘int main()’:
./Main.cpp:3:18: error: expected ‘,’ or ‘;’ before ‘for’
 #define rep(i,n) for (int i = 0;i < (n);i++)
                  ^
./Main.cpp:23:3: note: in expansion of macro ‘rep’
   rep(i,N)
   ^
./Main.cpp:23:7: error: ‘i’ was not declared in this scope
   rep(i,N)
       ^
./Main.cpp:3:33: note: in definition of macro ‘rep’
 #define rep(i,n) for (int i = 0;i < (n);i++)
                                 ^

ソースコードは次のとおりです

#include <bits/stdc++.h>
using namespace std;
#define rep(i,n) for (int i = 0;i < (n);i++)
using ll = long long;

int main()
{
ll N;
cin >> N;
vector<vector<ll>> set(N + 1,vector<ll>(3));
rep(i,3)
{
set.at(0).at(i) = 0;
}
rep(i,N)
{
rep(j,3)
{
cin >> set.at(i + 1).at(j);
}
}
string ans = "Yes"
rep(i,N)
{
ll t,x,y;
t = set.at(i + 1).at(0) - set.at(i).at(0);
x = set.at(i + 1).at(1) - set.at(i).at(1);
y = set.at(i + 1).at(2) - set.at(i).at(2);
if(t < x + y || (t - x - y) % 2 == 1)
{
ans = "No";
break;
}
}
cout << ans << endl;
return 0;
}

どなたかどこを直せば良いか分かる方はいませんか

Answer 1

string ans = "Yes" // セミコロンがない！
rep(i,N)

---

本題ではありませんが、特に理由もないのにrepのようなマクロを使うのはあまりおすすめしません。素直にforを書いたほうが読みやすいです。

Comments

[mogurin]

うわー、すごい初歩的なミス…
ありがとうございました

Answer 2

string ans = "Yes"
rep(i,N)

Answer 3

rep(j,3)

マクロ展開すると、forの後に文がないですね

わざわざわかりにくくするようなマクロはダメですよ