Помилка: перейти до регістру


229

Я написав програму, яка передбачає використання операторів переключення ... Однак при компіляції це показує:

Помилка: перейти до регістру.

Чому це робиться?

#include <iostream>
#include <cstdlib>
#include <fstream>
#include <string>

using namespace std;

class contact
{
public:
    string name;
    int phonenumber;
    string address;
    contact() {
        name= "Noname";
        phonenumber= 0;
        address= "Noaddress";
    }
};

int main() {
    contact *d;
    d = new contact[200];
    string name,add;
    int choice,modchoice,t;//Variable for switch statement
    int phno,phno1;
    int i=0;
    int initsize=0, i1=0;//i is declared as a static int variable
    bool flag=false,flag_no_blank=false;

    //TAKE DATA FROM FILES.....
    //We create 3 files names, phone numbers, Address and then abstract the data from these files first!
    fstream f1;
    fstream f2;
    fstream f3;
    string file_input_name;
    string file_input_address;
    int file_input_number;

    f1.open("./names");
    while(f1>>file_input_name){
        d[i].name=file_input_name;
        i++;
    }
    initsize=i;

    f2.open("./numbers");
    while(f2>>file_input_number){
        d[i1].phonenumber=file_input_number;
        i1++;
    }
    i1=0;

    f3.open("./address");
    while(f3>>file_input_address){
        d[i1].address=file_input_address;
        i1++;
    }

    cout<<"\tWelcome to the phone Directory\n";//Welcome Message
    do{
        //do-While Loop Starts
        cout<<"Select :\n1.Add New Contact\n2.Update Existing Contact\n3.Display All Contacts\n4.Search for a Contact\n5.Delete a  Contact\n6.Exit PhoneBook\n\n\n";//Display all options
        cin>>choice;//Input Choice from user

        switch(choice){//Switch Loop Starts
        case 1:
            i++;//increment i so that values are now taken from the program and stored as different variables
            i1++;
            do{
                cout<<"\nEnter The Name\n";
                cin>>name;
                if(name==" "){cout<<"Blank Entries are not allowed";
                flag_no_blank=true;
                }
            }while(flag_no_blank==true);
            flag_no_blank=false;
            d[i].name=name;
            cout<<"\nEnter the Phone Number\n";
            cin>>phno;
            d[i1].phonenumber=phno;
            cout<<"\nEnter the address\n";
            cin>>add;
            d[i1].address=add;
            i1++;
            i++;
            break;//Exit Case 1 to the main menu
        case 2:
            cout<<"\nEnter the name\n";//Here it is assumed that no two contacts can have same contact number or address but may have the same name.
            cin>>name;
            int k=0,val;
            cout<<"\n\nSearching.........\n\n";
            for(int j=0;j<=i;j++){
                if(d[j].name==name){
                    k++;
                    cout<<k<<".\t"<<d[j].name<<"\t"<<d[j].phonenumber<<"\t"<<d[j].address<<"\n\n";
                    val=j;
                }
            }
            char ch;
            cout<<"\nTotal of "<<k<<" Entries were found....Do you wish to edit?\n";
            string staticname;
            staticname=d[val].name;
            cin>>ch;
            if(ch=='y'|| ch=='Y'){
                cout<<"Which entry do you wish to modify ?(enter the old telephone number)\n";
                cin>>phno;
                for(int j=0;j<=i;j++){
                    if(d[j].phonenumber==phno && staticname==d[j].name){
                        cout<<"Do you wish to change the name?\n";
                        cin>>ch;
                        if(ch=='y'||ch=='Y'){
                            cout<<"Enter new name\n";
                            cin>>name;
                            d[j].name=name;
                        }
                        cout<<"Do you wish to change the number?\n";
                        cin>>ch;
                        if(ch=='y'||ch=='Y'){
                            cout<<"Enter the new number\n";
                            cin>>phno1;
                            d[j].phonenumber=phno1;
                        }
                        cout<<"Do you wish to change the address?\n";
                        cin>>ch;
                        if(ch=='y'||ch=='Y'){
                            cout<<"Enter the new address\n";
                            cin>>add;
                            d[j].address=add;
                        }
                    }
                }
            }
            break;
        case 3 : {
            cout<<"\n\tContents of PhoneBook:\n\n\tNames\tPhone-Numbers\tAddresses";
            for(int t=0;t<=i;t++){
                cout<<t+1<<".\t"<<d[t].name<<"\t"<<d[t].phonenumber<<"\t"<<d[t].address;
            }
            break;
                 }
        }
    }
    while(flag==false);
    return 0;
}

1
Який код ви намагаєтеся скласти? Який компілятор ви використовуєте? Ви вклали кожен caseблок в дужки?
Коді Грей

2
ось одне вражаюче повідомлення про помилку
jozxyqk,

Відповіді:


437

Проблема полягає в тому, що змінні, оголошені в одній case, все ще помітні в наступних casesх, якщо не використовується явний { }блок, але вони не будуть ініціалізовані, оскільки код ініціалізації належить іншомуcase .

У наступному коді, якщо fooдорівнює 1, все нормально, але якщо він дорівнює 2, ми випадково використаємо iзмінну, яка існує, але, ймовірно, містить сміття.

switch(foo) {
  case 1:
    int i = 42; // i exists all the way to the end of the switch
    dostuff(i);
    break;
  case 2:
    dostuff(i*2); // i is *also* in scope here, but is not initialized!
}

Обгортання корпусу в явному блоці вирішує проблему:

switch(foo) {
  case 1:
    {
        int i = 42; // i only exists within the { }
        dostuff(i);
        break;
    }
  case 2:
    dostuff(123); // Now you cannot use i accidentally
}

Редагувати

Для подальшого уточнення switchвисловлювання є лише особливо вигадливим видом goto. Ось аналогічний фрагмент коду, що демонструє ту саму проблему, але використовує gotoзамість switch:

int main() {
    if(rand() % 2) // Toss a coin
        goto end;

    int i = 42;

  end:
    // We either skipped the declaration of i or not,
    // but either way the variable i exists here, because
    // variable scopes are resolved at compile time.
    // Whether the *initialization* code was run, though,
    // depends on whether rand returned 0 or 1.
    std::cout << i;
}

1
Дивіться цей виправлений звіт про помилки LLVM, щоб отримати інші пояснення: llvm.org/bugs/show_bug.cgi?id=7789
Francesco

70

Декларація нових змінних у випадах випадків - це те, що викликає проблеми. Закриття всіх caseоператорів у {}обмежує область нещодавно оголошених змінних поточним виконавчим випадком, який вирішує проблему.

switch(choice)
{
    case 1: {
       // .......
    }break;
    case 2: {
       // .......
    }break;
    case 3: {
       // .......
    }break;
}    

Інструкція з чистішого виправлення
yc_yuy

Чи буде якась проблема, якщо я покладу заяву про розрив всередині фігурних дужок?
Вішал Шарма

10

C ++ 11 стандарт для переходу через деякі ініціалізації

ЙоханнесД дав пояснення, тепер вже щодо стандартів.

C ++ 11 N3337 проекту стандарту 6.7 «Декларація заяви» говорить:

3 Можна перенести в блок, але не таким чином, що обходить декларації з ініціалізацією. Програма, яка стрибає (87) з точки, коли змінна з автоматичною тривалістю зберігання не входить в область, до якої вона знаходиться в області, неправильно формується, якщо змінна не має скалярного типу, типу класу з тривіальним конструктором за замовчуванням і тривіальною деструктор, версія, що відповідає кваліфікації одного з цих типів, або масив одного з попередніх типів і оголошується без ініціалізатора (8.5).

87) Перехід від умови заяви переключення до мітки справи вважається стрибком у цьому відношенні.

[Приклад:

void f() {
   // ...
  goto lx;    // ill-formed: jump into scope of a
  // ...
ly:
  X a = 1;
  // ...
lx:
  goto ly;    // OK, jump implies destructor
              // call for a followed by construction
              // again immediately following label ly
}

- кінцевий приклад]

Щодо GCC 5.2, повідомлення про помилку тепер говорить:

перетинає ініціалізацію

С

C дозволяє це: c99 перейти до ініціалізації минулого

Проект стандарту C99 N1256 Додаток I «Загальні попередження» , говорить:

2 Блок з ініціалізацією об'єкта, який має тривалість автоматичного зберігання, перескакується


6

Відповідь ЙоганнесД є правильною, але я вважаю, що це не зовсім ясно в аспекті проблеми.

Приклад, який він дає, декларує та ініціалізує змінну iу випадку 1, а потім намагається використати її у випадку 2. Його аргумент полягає в тому, що якби перемикач перейшов прямо у випадок 2, iвін використовувався б без ініціалізації, і саме тому є компіляція помилка. У цей момент можна було б подумати, що не було б жодної проблеми, якби змінні, оголошені у випадку, ніколи не використовувались в інших випадках. Наприклад:

switch(choice) {
    case 1:
        int i = 10; // i is never used outside of this case
        printf("i = %d\n", i);
        break;
    case 2:
        int j = 20; // j is never used outside of this case
        printf("j = %d\n", j);
        break;
}

Можна очікувати компіляції цієї програми, оскільки обидві iі jвикористовуються лише всередині випадків, що їх декларують. На жаль, у C ++ він не компілюється: як пояснив Ciro Santilli 包子 露 宪 六四 事件 法轮功 , ми просто не можемо перейти до цього case 2:, оскільки це може пропустити декларацію з ініціалізацією i, і хоча case 2вона не використовуєi , це все ще заборонено в C ++.

Цікаво, що з деякими змінами ( #ifdefдо #includeвідповідного заголовку і точці з коми після етикетки, тому що мітки можуть слідувати тільки заяви і декларація не враховується в якості операторів в C ), ця програма робить компіляції в C:

// Disable warning issued by MSVC about scanf being deprecated
#ifdef _MSC_VER
#define _CRT_SECURE_NO_WARNINGS
#endif

#ifdef __cplusplus
#include <cstdio>
#else
#include <stdio.h>
#endif

int main() {

    int choice;
    printf("Please enter 1 or 2: ");
    scanf("%d", &choice);

    switch(choice) {
        case 1:
            ;
            int i = 10; // i is never used outside of this case
            printf("i = %d\n", i);
            break;
        case 2:
            ;
            int j = 20; // j is never used outside of this case
            printf("j = %d\n", j);
            break;
    }
}

Завдяки онлайн-компілятору, наприклад http://rextester.com, ви можете швидко спробувати скласти його як C або C ++, використовуючи MSVC, GCC або Clang. Оскільки це завжди працює (просто пам’ятайте про встановлення STDIN!), Так як C ++ жоден компілятор не приймає його.

Використовуючи наш веб-сайт, ви визнаєте, що прочитали та зрозуміли наші Політику щодо файлів cookie та Політику конфіденційності.
Licensed under cc by-sa 3.0 with attribution required.