なぜforループは正しくありませんか？

Question

Visual Studioでは、forループが正しくないと伝えています。 エラーメッセージは以下のとおりです。OK

予期しない

• bool型が欠落している未宣言の識別子
• です。前}

---

に関する情報：

が長い-MoveNextブール

for (size_t i = 0, bool ok = recordset.Select(Adress::getSQLStatement() + "Where A05.recid = %ld", i); ok; ok = recordset.MoveNext(), i++) { 
    at(i).Save(recordset); 
} 

Answer 1

それは、ようだStenSoftは語りました。しかし、ループの最初のステートメントで無名構造を定義し、それを初期化することができます。

#include <iostream> 
using namespace std; 

int main() { 
    for (struct {size_t i; bool ok;} s = {0, true}; s.ok; ++s.i) { 
     s.ok = s.i < 10; 
     cout << s.i; 
    } 
    return 0; 
} 

しかし、IMHO、while it worksでも、それは価値があります。コードを再構成する方が良い。

Answer 2

まずオフを返す-recordset.Select、あなたはもちろん、あなたを書き換えることができますループはそうです：

{ 
    bool ok = recordset.Select(...); 
    for (std::size_t i = 0; ok; ok = recordset.MoveNext(), ++i) 
    { 
     /* ... */ 
    } 
} 

しかし、ここでのメタレッスンは、ほぼすべてのループがfor -loopsであり、構造が異なると思うときは、もう一度考えてください。あなたのロジックをより明確にするために、おそらくfor -loopsの書き換えがあります。現在のコードでは、最初のエラーと後で「レコードがありません」というエラーは区別されません。新しいコードでは、それは今、明示的に可能です：

if (bool ok = recordset.select(...)) 
{ 
    for (std::size_t i = 0; ok; ok = recordset.MoveNext(), ++i) { /* ... */ } 
} 
else 
{ 
    // handle initial error 
} 

私は多分、冗長ok変数を取り除くになります。

if (recordset.select(...)) 
{ 
    for (std::size_t i = 0; ; ++i) 
    { 
     /* ... */ 

     if (!recordset.MoveNext()) break; 
    } 
} 
else 
{ 
    // handle initial error 
}