c++ - 在循环中初始化变量

Question

我试图弄清楚初始化某些变量时的最佳做法是什么……我的代码现在看起来像这样：

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 循环中声明和初始化所有这些变量是更好的编程实践，还是应该像现在一样保留它？

Answer 1

您应该始终尽可能地缩小变量的范围。这将提高代码的可维护性，并减少出现错误的机会。

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

bar(k);

……对……

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

bar(k);