在函数顶部或单独的范围内声明变量？

Question

哪个是优选的,方法1或方法2？

方法1:

LRESULT CALLBACK wpMainWindow(HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam)
{
    switch (msg)
    {
        case WM_PAINT:
        {
            HDC hdc;
            PAINTSTRUCT ps;

            RECT rc;
            GetClientRect(hwnd, &rc);           

            hdc = BeginPaint(hwnd, &ps);
            // drawing here
            EndPaint(hwnd, &ps);
            break;
        }
        default: 
            return DefWindowProc(hwnd, msg, wparam, lparam);
    }
    return 0;
}

方法2:

LRESULT CALLBACK wpMainWindow(HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam)
{
    HDC hdc;
    PAINTSTRUCT ps;
    RECT rc;

    switch (msg)
    {
        case WM_PAINT:
            GetClientRect(hwnd, &rc);

            hdc = BeginPaint(hwnd, &ps);
            // drawing here
            EndPaint(hwnd, &ps);
            break;

        default: 
            return DefWindowProc(hwnd, msg, wparam, lparam);
    }
    return 0;
}

在方法1中,如果在调用wpMainWindow函数时msg = WM_PAINT,它是否在开始时为堆栈上的所有变量分配内存？或者只有当它进入WM_PAINT范围时？

方法1只在消息是WM_PAINT时才使用内存,而方法2是否会使用内存而不管msg等于什么？

Answer 1

变量应尽可能在本地声明.

在函数顶部声明变量总是一种灾难性的不良做法.即使在C89/90语言中,变量只能在块的开头声明,最好将它们声明为尽可能本地,即在覆盖变量所需寿命的最小局部块的开头.有时甚至可能引入一个"冗余"的本地块,其唯一目的是"本地化"变量声明.

在C++和C99中,可以在代码中的任何地方声明变量,答案非常简单:再次,尽可能在本地声明每个变量,并尽可能接近第一次使用它的点.这个的主要原理是,在大多数情况下,这将允许您在声明点为变量提供有意义的初始化器(而不是在没有初始化器或使用虚拟初始化器的情况下声明它).

至于内存使用情况,通常一个典型的实现将立即(当你输入函数时)分配同时存在的所有变量所需的最大空间.但是,您的声明习惯可能会影响该空间的确切大小.例如,在此代码中

void foo() {
  int a, b, c;

  if (...) {
  }

  if (...) {
  }
}

所有三个变量同时存在,并且通常必须分配所有三个变量的空间.但是在这段代码中

void foo() {
  int a;

  if (...) {
    int b;
  }

  if (...) {
    int c;
  }
}

在任何给定时刻只存在两个变量,这意味着只有两个变量的空间将由典型的实现分配(b并且c将共享相同的空间).这是将变量声明为尽可能本地的另一个原因.

Answer 2

在案例1中是否在堆栈上分配的内容是实现定义的.甚至不需要实现堆栈.

这样做通常不会慢,因为操作往往是来自整个局部变量区域的堆栈指针的一个值的简单减法(对于向下增长的堆栈).

这里重要的是范围应该尽可能地本地化.换句话说,尽可能晚地声明变量,只要需要它们就可以保留它们.

请注意,此处的声明处于不同的抽象级别,以便为它们分配空间.实际空间可以在函数的开头(实现级别)分配,但是只能在它们作用域(C级)时使用这些变量.

信息的位置很重要,就像它的堂兄一样.

Answer 3

我喜欢方法3:

LRESULT wpMainWindowPaint(HWND hwnd)
{
    HDC hdc;
    PAINTSTRUCT ps;

    RECT rc;
    GetClientRect(hwnd, &rc);           

    hdc = BeginPaint(hwnd, &ps);
    // drawing here
    EndPaint(hwnd, &ps);
    return 0;
}

LRESULT CALLBACK wpMainWindow(HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam)
{
    switch (msg)
    {
        case WM_PAINT:      return wpMainWindowPaint(hwnd);
        default:            return DefWindowProc(hwnd, msg, wparam, lparam);
    }
}

如果它应该有自己的组织范围,它应该有自己的功能.如果您担心函数调用开销,请将其设置为内联.

Answer 4

由于优化代码是编译器的工作，而一个小时的编译时间比一个小时的时间便宜得多，而且如果我需要上下滚动代码以查看变量的声明位置，则会浪费我的时间，我认为我的公司希望我尽可能将所有内容都保留在本地。

我什至不是在谈论“最小的街区”，而是在“靠近使用它的地方”！

LRESULT CALLBACK wpMainWindow(HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam) 
{ 
    switch (msg) 
    { 
        case WM_PAINT: 
        { 
            RECT rc; 
            GetClientRect(hwnd, &rc);            

            { // sometimes I even create an arbitrary block 
              // to show correlated statements.
              // as a side-effect, the compiler may not need to allocate space for 
              // variables declared here...
              PAINTSTRUCT ps; 
              HDC hdc = BeginPaint(hwnd, &ps); 
              // drawing here 
              EndPaint(hwnd, &ps); 
            }
            break; 
        } 
        default:  
            return DefWindowProc(hwnd, msg, wparam, lparam); 
    } 
    return 0; 
}