Почему я не могу использовать «константу» внутри оператора switch в области видимости? - PullRequest
10 голосов
/ 23 марта 2012

С этим кодом:

public partial class Form1 : Form
{
    private static readonly int TABCONTROL_BASICINFO = 0;
    private static readonly int TABCONTROL_CONFIDENTIALINFO = 1;
    private static readonly int TABCONTROL_ROLESANDSECURITY = 2;
    private static readonly int TABCONTROL_INACTIVEINFO = 3;
. . .
int ActiveTabPage = tabControlWorker.SelectedIndex;
switch (ActiveTabPage) {
    case TABCONTROL_BASICINFO:
        if (currentNode == "NodeBuckingham") {
        } else if (currentNode == "NodeNamath") {
        } else if (currentNode == "NodeParsons") {
        } else {
        }
    break;

... Я должен заменить «TABCONTROL_BASICINFO» на «0», или я получаю «Постоянное значение ожидается "

Небеса Мургатройду! Разве он не может посмотреть и увидеть, что TABCONTROL_BASICINFO равен 0?

Ответы [ 4 ]

20 голосов
/ 23 марта 2012

Если вы хотите сделать его константным выражением для компилятора, объявите его как const:

// Note that static readonly is implied here
private const int TABCONTROL_BASICINFO = 0;

или следуйте соглашениям об именах .NET ...

private const int TabControlBasicInfo = 0;

Или используйте перечисление, учитывая, что вы в основном получили фиксированный набор значений:

private enum TabControlType
{
    // Could ditch the explicit values here if you want
    BasicInfo = 0,
    ConfidentialInfo = 1,
    ...
}

Кстати, вы также можете включить строки в C #, так что:

 if (currentNode == "NodeBuckingham") {
 } else if (currentNode == "NodeNamath") {
 } else if (currentNode == "NodeParsons") {
 } else {
 }

Может стать:

 switch (currentNode) {
     case "NodeBuckingham":
         ...
         break;
     case "NodeNamath":
         ...
         break;
     case "NodeParsons":
         ...
         break;
     default:
         ...
         break;
 }
13 голосов
/ 23 марта 2012

A readonly переменная не является константой.Значение не известно во время компиляции, но может быть инициализировано либо в объявлении (как вы это сделали), либо в конструкторе класса (в данном случае, статическом конструкторе для вашего класса).

Подробнее см.

http://msdn.microsoft.com/en-us/library/acdd6hb7(v=vs.71).aspx

Вы можете изменить его на:

private const int TABCONTROL_BASICINFO = 0; 

Если вам не нужно что-то вычислять для инициализации переменной,объявите это как const.Это будет немного эффективнее.

7 голосов
/ 23 марта 2012

Это не константа; переменная только для чтения . Если вы хотите, чтобы она была постоянной, пометьте ее как постоянную.

private const int TabControlBasicInfo = 0;

Или даже лучше:

private enum TabControl
{
    BasicInfo,
    ...
}
...
switch(whatever)
{
    case TabControl.BasicInfo:
5 голосов
/ 23 марта 2012

Статическое чтение только не постоянная переменная. Это может быть инициализировано. Тогда как " const " является константой.

...