PRB: Variables with Local Scope to Switch Won't Be Initialized

• Microsoft C for MS-DOS, versions 6.0, 6.0a, 6.0ax
• Microsoft C/C++ for MS-DOS, version 7.0
• Microsoft Visual C++ for Windows, 16-bit edition, versions 1.0, 1.5
• Microsoft Visual C++, 32-bit Editions, versions 1.0, 2.0, 2.1, 4.0, 5.0

SYMPTOMS

In Microsoft C, variables may be declared as local to a switch statement by defining them within the braces that make up the switch. They then have visibility and life for the duration of the switch statement. However, because of the design of a switch block, they will not be initialized when declared unless the variable is static or is declared after a case label.

RESOLUTION

If you must declare an initialized variable that is local to the switch statement and transient in duration, use brackets before and after the switch and place the declaration outside the switch statement.

MORE INFORMATION

The ANSI Standard (Section 3.6.4.2) states:

A "switch" statement causes control to jump to, into, or past the statement that is the switch body, depending on the value of a controlling expression, and on the presence of a "default" label and the values of any "case" labels on or in the switch body. A "case" or "default" label is accessible only within the closest enclosing "switch" statement.

The integral promotions are performed on the controlling expression. The constant expression in each "case" label is converted to the promoted type of the controlling expression. If a converted value matches that of the promoted controlling expression, control jumps to the statement following the matched "case" label. Otherwise, if there is a "default" label, control jumps to the labeled statement. If no converted case constant expression matches and there is no default label, no part of the switch body is executed.

   local_int NOT initialized 

• Declare it outside the switch statement, using braces to limit its scope if necessary. For example:

  
  {
     int local_int = 5;
     switch (param)
     {
        case 100:
           local_int+=5;
           rc = (local_int == 10 ? 1 : 0);
           break;
        default:
           break;
     }
  } 




• Declare after each case label in which it is to be used. For example:

  
  switch (param)
  {
     case 100:
     {
        int local_int = 10;
        rc = (local_int == 10 ? 1 : 0);
        break;
     }
     case 101:
     {
        int local_int = 11;
        rc = (local_int == 10 ? 1 : 0);
        break;
     }
     default:
        break;
  } 




• Declare the variable as static.

  
  /* Compile options needed: none
  */ 
  
  int _cdecl printf(const char *, ...);
  int switch_func(int);
  int main(void);
  
  int main(void)
  {
     int rc = switch_func(100);
  
     if ( rc )
        printf("local_int initialized\n");
     else
        printf("local_int NOT initialized\n");
  
     return( rc );
  }
  
  int switch_func(int param)
  {
     int rc = 0;
  
     switch (param)
     {
        int local_int = 5;
        case 100:
           local_int+=5;
           rc = (local_int == 10 ? 1 : 0);
           break;
        default:
           break;
     }
  
     return( rc );
  } 

Additional query words:

Keywords : kbLangC kbVC100 kbVC150 kbVC200 kbVC210 kbVC400 kbVC500
Version : 6.0 6.0a 6.0ax 7.0 1.0 1.5 2.0 2
Platform : MS-DOS NT WINDOWS
Issue type : kbprb