6

特定の変数を初期化するときのベストプラクティスを理解しようとしています...私のコードは現時点では次のようになっています。

int nHexCount = 0;
int prevState = sc.state;

bool bOnlySpaces = true;
bool bIsValidLabel = true;
bool bIsHotstring = false;
bool bIsValidName = true;
bool bIsValidExpStart = false;                         

bool fInExpression = false;
bool fInStringBlock = (sc.state == SCE_AHKL_STRINGOPTS || sc.state == SCE_AHKL_STRINGBLOCK);

for (; sc.More(); sc.Forward()) {

    if (sc.atLineStart) {
        if (!fInStringBlock && sc.state != SCE_AHKL_COMMENTBLOCK)
            sc.SetState(SCE_AHKL_DEFAULT);

        // Reset Status
        prevState = sc.state;

        bOnlySpaces = true;
        bIsValidLabel = true;
        bIsHotstring = false;
        bIsValidName = true;
        bIsValidExpStart = false;

        fInExpression = false;
    }

...

ご覧のとおり、これらの変数のほとんどは、プログラムが作業中の編集コンポーネントで新しい行を見つけるたびにリセットされます...

質問は次のようになります。

forループ内でこれらすべての変数を宣言して初期化する方がプログラミングの練習としては優れていますか、それとも現時点のままにしておく必要がありますか？

4

1 に答える 1

12

変数の範囲は常に可能な限り減らす必要があります。これにより、コードの保守性が向上し、バグの可能性が減少します。

// bad
int i, j, k;
k = 0;
for (i = 0; i < X, ++i)
{
  j = foo(i);
  k += j;
} 

bar(k);

...vs..。

// better
int k=0; // needs scope outside loop
for (int i = 0; i < X, ++i)
{
  int j = foo(i);
  k += j;
} 

bar(k);
于 2012-11-01T22:30:05.760 に答える