Форум программистов, компьютерный форум CyberForum.ru

Не понятные ошибки - C++

Восстановить пароль Регистрация
 
Рейтинг: Рейтинг темы: голосов - 29, средняя оценка - 5.00
Chelioss
179 / 179 / 4
Регистрация: 08.01.2011
Сообщений: 1,131
08.01.2011, 16:46     Не понятные ошибки #1
Ошибки в след. коде:
.cpp(32) : error C2065: S: необъявленный идентификатор
.cpp(39) : error C2065: S: необъявленный идентификатор
.cpp(41) : error C2046: недопустимый вариант выбора
Первый ошибки, думаю, понятно где всплывают. Третья ошибка всплывает на case 2:

Не могу понять в чем же проблема.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
#include <iostream>
using namespace std;
 
char* zamenaStr1Str2(char* str, char* str1, char* str2);
void testirovanie();
 
void main()
{
    setlocale(LC_ALL,"rus");
    char otv='y';
    do
    {
        cout << "Выберите 1, чтобы включить основную функцию. Выберите 2, чтобы включить функцию тестирования\n";
        int vibor;
        cin >> vibor;
        switch(vibor)
            case 1 :
                    int S=0;
                    char str1[80], str2[80], otv1='y';
                    cout << "Введите str1\n";
                    cin >> str1;
                    cout << "Введите str2\n";
                    cin >> str2;
                    do
                    {
                        char str[80];
                        cout << "Введите str\n";
                        cin >> str;
                        if(char *resultat=zamenaStr1Str2(str,str1,str2))
                        {
                            cout << resultat << endl;
                            S++;
                        }
                        else cout << "Изменений не было\n";
                
                        cout << "Ввести новую строку str?\n";
                        cin >> otv1;
                    }while(otv1=='y');
                    cout << "Количество строк, подвергшихся изменений равно " << S << endl;
                    break;
            case 2 : testirovanie();    break;
 
            cout << "Повторить?\n";
            cin >> otv;
    }while(otv=='y');
}
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Алексaндр
 Аватар для Алексaндр
131 / 108 / 5
Регистрация: 04.12.2010
Сообщений: 313
08.01.2011, 16:54     Не понятные ошибки #2
.cpp(32) : error C2065: S: необъявленный идентификатор -
.cpp(39) : error C2065: S: необъявленный идентификатор - у вас эти 2 ошибки там, где S. Вы не там объявили эту переменную похоже. Объявите сдесь :
C++
1
2
cout << "Выберите 1, чтобы включить основную функцию. Выберите 2, чтобы включить функцию тестирования\n";
                int vibor, S=0;char str1[80], str2[80], otv1='y';
.cpp(41) : error C2046: недопустимый вариант выбора - попробуйте после switch(vibor) поставить фигурные скобки и после тела case 2 закрыть
silent_1991
Эксперт C++
4938 / 3014 / 149
Регистрация: 11.11.2009
Сообщений: 7,024
Завершенные тесты: 1
08.01.2011, 16:55     Не понятные ошибки #3
Если я правильно помню, чтобы объявлять переменные внутри меток оператора switch, нужно тело соответствующей метки заключать в фигурные скобки.
Chelioss
179 / 179 / 4
Регистрация: 08.01.2011
Сообщений: 1,131
08.01.2011, 16:57  [ТС]     Не понятные ошибки #4
Можно S и там объявить, но почему тут то не работает?
Какое сравнивание vibor и функции вы увидели? Там же сравнивается vibor с 2, а testirovanie() это то, что будет выполнятся, если vibor==2.

Вот код, в котором работает выбор

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
#include <iostream>
using namespace std;
char* poiskPstr_sposob1(char *str, char *pstr);
int ravnoPstr_sposob1(char *str,char *pstr);
char* poisk2Pstr(char *,char *,char *);
void testirovanie(void);
void main()
{
    setlocale(LC_ALL,"rus"); 
    char otv;
    do
    {
        int vibor;
        cout << "Нажмите 1, чтобы решить вашу задачу.\nНажмите 2, чтобы включить функцию тестирования.\n  ";
        cin >> vibor;
        switch(vibor)
        {
            case 1 :
                char str[80],pstr1[80],pstr2[80], *result;
                cout << "Введите строку\n"; cin >> str;
                cout << "Введите подстроку pstr1, которая будет заменяться подстрокой pstr2\n"; cin >> pstr1;
                cout << "Введите подстроку pstr2\n"; cin >> pstr2;
                result=poisk2Pstr(str,pstr1,pstr2);
                cout << "Итоговая строка: " << result << endl; 
                break;
            case 2: testirovanie(); break;
            default: cout << "Вы ввели не правильно\n";
        }
        cout << "Повторить? (y/n)\n  ";
        cin >> otv;
    }while(otv=='y');
}
Алексaндр
 Аватар для Алексaндр
131 / 108 / 5
Регистрация: 04.12.2010
Сообщений: 313
08.01.2011, 16:59     Не понятные ошибки #5
Цитата Сообщение от Алексaндр Посмотреть сообщение
попробуйте после switch(vibor) поставить фигурные скобки и после тела case 2 закрыть
!!!!!
Chelioss
179 / 179 / 4
Регистрация: 08.01.2011
Сообщений: 1,131
08.01.2011, 17:04  [ТС]     Не понятные ошибки #6
Спасибо. Эти ошибки вроде исчезли, но вот появились новые ошибки:
.cpp(42) : error C2360: пропуск инициализации 'otv1' из-за метки 'case'
.cpp(20): см. объявление 'otv1'
.cpp(42) : error C2360: пропуск инициализации 'S' из-за метки 'case'
.cpp(19): см. объявление 'S'
.cpp(43) : error C2361: пропуск инициализации 'otv1' из-за метки 'default'
.cpp(20): см. объявление 'otv1'
.cpp(43) : error C2361: пропуск инициализации 'S' из-за метки 'default'
.cpp(19): см. объявление 'S'


C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
#include <iostream>
using namespace std;
 
char* zamenaStr1Str2(char* str, char* str1, char* str2);
void testirovanie();
 
void main()
{
    setlocale(LC_ALL,"rus");
    char otv='y';
    do
    {
        cout << "Выберите 1, чтобы включить основную функцию. Выберите 2, чтобы включить функцию тестирования\n";
        int vibor;
        cin >> vibor;
        switch(vibor)
        {
            case 1 :
                    int S=0;
                    char str1[80], str2[80], otv1='y';
                    cout << "Введите str1\n";
                    cin >> str1;
                    cout << "Введите str2\n";
                    cin >> str2;
                    do
                    {
                        char str[80];
                        cout << "Введите str\n";
                        cin >> str;
                        if(char *resultat=zamenaStr1Str2(str,str1,str2))
                        {
                            cout << resultat << endl;
                            S++;
                        }
                        else cout << "Изменений не было\n";
                
                        cout << "Ввести новую строку str?\n";
                        cin >> otv1;
                    }while(otv1=='y');
                    cout << "Количество строк, подвергшихся изменений равно " << S << endl;
                    break;
            case 2 : testirovanie();break;
            default : cout <<"Введено не правильное число.\n";
        }
 
            cout << "Повторить?\n";
            cin >> otv;
    }while(otv=='y');
}
Алексaндр
 Аватар для Алексaндр
131 / 108 / 5
Регистрация: 04.12.2010
Сообщений: 313
08.01.2011, 17:13     Не понятные ошибки #7
а если попробывать переменные перед switch() объявить?
Chelioss
179 / 179 / 4
Регистрация: 08.01.2011
Сообщений: 1,131
08.01.2011, 17:23  [ТС]     Не понятные ошибки #8
Так получается, но все таки хотелось бы разобраться в чем проблема(
Vulkan_Sh
3 / 3 / 0
Регистрация: 08.01.2011
Сообщений: 22
08.01.2011, 18:22     Не понятные ошибки #9
case 2 : testirovanie();break;
default : cout <<"Введено не правильное число.\n";
}
помоему после @default : cout <<"Введено не правильное число.\n";@ надо поставить break;
ForEveR
Модератор
Эксперт C++
 Аватар для ForEveR
7927 / 4709 / 318
Регистрация: 24.06.2010
Сообщений: 10,524
Завершенные тесты: 3
08.01.2011, 18:29     Не понятные ошибки #10
Vulkan_Sh, После default-а break не нужен
asics
Freelance
Эксперт C++
 Аватар для asics
2838 / 1775 / 144
Регистрация: 09.09.2010
Сообщений: 3,842
08.01.2011, 18:29     Не понятные ошибки #11
Цитата Сообщение от Vulkan_Sh Посмотреть сообщение
помоему после @default : cout <<"Введено не правильное число.\n";@ надо поставить break;
Да нет.
d_eva
5 / 5 / 1
Регистрация: 04.02.2011
Сообщений: 6
04.02.2011, 13:48     Не понятные ошибки #12
char otv='y';

Инициализация недопустипа при объявлении.
Т.е.:

Правильно:
char otv;
otv='y';

Неправильно:
char otv='y';
silent_1991
Эксперт C++
4938 / 3014 / 149
Регистрация: 11.11.2009
Сообщений: 7,024
Завершенные тесты: 1
04.02.2011, 13:51     Не понятные ошибки #13
d_eva, это вы сами придумали или прочитали где-то?
d_eva
5 / 5 / 1
Регистрация: 04.02.2011
Сообщений: 6
04.02.2011, 14:05     Не понятные ошибки #14
Цитата Сообщение от silent_1991 Посмотреть сообщение
d_eva, это вы сами придумали или прочитали где-то?
Как раз сегодня на уроке обсуждали. "По книжному" - нельзя одновременно объявлять и инициализировать в составе конструкции switch (считается распространенной ошибкой).
А по-моему - (апробировано) можно, но взять case в скобки:
case 'n':{
действие...
}break;

Работает и при правильной инициализации, и при фигурных скобках.

Считаете иначе?
silent_1991
Эксперт C++
4938 / 3014 / 149
Регистрация: 11.11.2009
Сообщений: 7,024
Завершенные тесты: 1
04.02.2011, 14:07     Не понятные ошибки #15
d_eva, во!
d_eva
5 / 5 / 1
Регистрация: 04.02.2011
Сообщений: 6
04.02.2011, 14:11     Не понятные ошибки #16
Цитата Сообщение от silent_1991 Посмотреть сообщение
d_eva, во!
считаю не совсем корректным... Объявлять можно и без скобок, т.е. если правильно объявить (речь идет только о составе в конструкции switch):
int a;
a= 5
(а не int a=5; )
то скобки не нужны.
silent_1991
Эксперт C++
4938 / 3014 / 149
Регистрация: 11.11.2009
Сообщений: 7,024
Завершенные тесты: 1
04.02.2011, 14:19     Не понятные ошибки #17
d_eva,

Не по теме:

я, честно говоря, с самого начала не понял, что вы говорите об объявлении и инициализации внутри метки case. Но раз вы уточнили, решил показать, что подобную мысль я уже высказывал. В частности, если очень хочется объявить переменные с одинаковыми именами внутри всех меток case (хотя разумнее было бы объявить одну переменную до switch) - то без фигурных скобок не обойтись.

MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
04.02.2011, 14:23     Не понятные ошибки
Еще ссылки по теме:

C++ Почему выводит не понятные символы, а не то что вводилось пользователем?
Возникли совсем не понятные мне ошибки в программе C++
не понятные определения макросов MFC C++

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
d_eva
5 / 5 / 1
Регистрация: 04.02.2011
Сообщений: 6
04.02.2011, 14:23     Не понятные ошибки #18
Цитата Сообщение от silent_1991 Посмотреть сообщение
d_eva,

Не по теме:

я, честно говоря, с самого начала не понял, что вы говорите об объявлении и инициализации внутри метки case. Но раз вы уточнили, решил показать, что подобную мысль я уже высказывал. В частности, если очень хочется объявить переменные с одинаковыми именами внутри всех меток case (хотя разумнее было бы объявить одну переменную до switch) - то без фигурных скобок не обойтись.

ну тогда Вы просто умничка! А я невнимательно прочитала Ваши рекомендации. ))
Yandex
Объявления
04.02.2011, 14:23     Не понятные ошибки
Ответ Создать тему
Опции темы

Текущее время: 02:46. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru