структура выбора switch

[AcTiV]

я сначала выбираю 3(матем формулы), потом выбираю ромб, в результате оно автоматически пишет что я нажал 4 в этом выборе

но я только выбрал что буду выполнять действия с ромбом
почему мой выбор 4 switch (SelectToForOfGeom )
присваивается под этому выбору switch (SelectOption ) который мне ошибку выводит, а этого он не должен делать

это при том что я еще не вводил номер выбора
да, если он был что мне известно у ромбе и я нажал 4 , тогда я понимаю что макс выбор 3, но оно даже выбора не предоставляет, а сразу переходит к ошибке.

т.к. я еще не выбрал что мне известно в ромбе
вот скрин, думаю на скрине более яснее

[_Bers]

AcTiV, телепаты в отпуске.

1. Предоставьте исходный код, который можно скопипастить и запустить на выполнение на своих локальных машинах.

2. Предоставьте полное подробное описание проблемы.
Например: "при запуске программы, в консольке появляется меню, в котором можно выбрать... , я выбрал..., а она потом вместо того что бы ...., сделала вот так...."

Вот эта фраза, лично мне ни о чем не говорит:

"я сначала выбираю 3(матем формулы), потом выбираю ромб, в результате оно автоматически пишет что я нажал 4 в этом выборе"

[AcTiV]

ок,

http://rghost.ru/private/44213347/7b...4df4ec93bc664d - сам код

проблема:
программа мне предоставляет выбор функций
я выбираю 3 (матем формулы). дальше...
я выбираю ромб (4 выбор) программа продолжает правильно выполнятся до

PHP код:

//формула ромба
        case 4:
            cout << "Что вам дано в задании про ромб?" << endl;
            cout << "1 - высота и сторона\n" 
                << "2 - сторона и кут\n" 
                << "3 - 2 диагонали" << endl;
//==========дальше уже неправильно===========================================
            switch (SelectOption)
            {
                //известно сторону и высоту
                case 1:
                    cout << "Вам известно высоту и сторону" << endl;
                    cout << "Введите сторону: " ;
                    cin >> a;
                    cout << "Введите высоту: " ;
                    cin >> h;
                    cout << "Площадь ромба = " << a * h << endl;
                break;
                //известно сторону и кут
                case 2:
                    cout << "Вам известно сторону и кут" << endl;
                    cout << "Введите сторону: " ;
                    cin >> a;
                    cout << "Введите кут (sin): " ;
                    cin >> l;
                    cout << "Площадь ромба = " << pow((double)a, 2) * sin((double)l) << endl;
                break;
                //известно 2 диагонали
                case 3:
                    cout << "Вам известно 2 диагонали" << endl;
                    cout << "Введите первую диагональ: " ;
                    cin >> b;
                    cout << "Введите вторую диагональ: " ;
                    cin >> c ;
                    cout << "Площадь ромба = " << (b * c) / 2 << endl;
                break;
                default:
                    cout << "\t\t\tНЕПРАВИЛЬНЫЙ ВВОД. Максимальный выбор 3" << endl;
                    FormulsOfGeometrix();
                break;
            }
        break; 


в этом коде ясно, что если я нажму 4(выбор ромба) то напишет текст

PHP код:

cout << "Что вам дано в задании про ромб?" << endl;
            cout << "1 - высота и сторона\n" 
                << "2 - сторона и кут\n" 
                << "3 - 2 диагонали" << endl; 


но дальше уже ошибка, но я должен был выбрать что мне известно у ромбе, а здесь сразу ошибка

PHP код:

default:
                    cout << "\t\t\tНЕПРАВИЛЬНЫЙ ВВОД. Максимальный выбор 3" << endl;
                    FormulsOfGeometrix();
                break; 


[_Bers]

я не увидел сам код. Ссылка у меня не открылась.


Выложите код вот сюда: http://liveworkspace.org/

Или просто прямо на форум

[ACE Valery]

Уважаемый, а вы не можете код ТЕКСТОМ присылать? Тут программисты, а не компиляторы сидят. Или вы думаете, что кто-то будет ваше творение с картинки перенабирать? Уважайте время помогающих вам людей. А чтоб дальше вы не повторяли своих ошибок, вам штраф.

Поо теме: так бывает. Почему - хз, не копала так глубоко. Но иногда при операторе выбора, вложенном в оператор выбора берется введенное значение внешнего оператора выбора. Очистите поток ввода. Или сделайте пару getchar'ов

[AcTiV]

ссылка

вложенный файл

[_Bers]

Ага. Вижу. В общем смотри:

cout << "Вы выбрали развед 3. Геометрические формулы" << endl;
    cout << "Что бы выбрать, просто введите номер формулы и нажмите Enter" << endl;
    cout << "Список формул: \n"
        << "1)Площадь квадрата\n"
        << "2)Площадь прямоугольника\n"
        << "3)Площадь паралелограмма\n"
        << "4)Площадь ромба\n"
        << "5)Площадь трапеции\n"
        << "6)Площадь круга\n"
        << "7)Площадь эллипса\n"
        << "8)Площадь Равнобедренный треугольника\n"
        << "9)Площадь Равносторонний треугольника\n" << endl;

    cin >> SelectToForOfGeom;

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

case 4:
            cout << "Что вам дано в задании про ромб?" << endl;
            cout << "1 - высота и сторона\n"
                << "2 - сторона и кут\n"
                << "3 - 2 диагонали" << endl;
            switch (SelectToOption)
            { 
                 ...

Когда ты попадаешь в блок обработки ромба, ты нигде заного не спрашиваешь у пользователя его новый выбор. SelectToOption нигде не изменяется.

[_Bers]

То есть смотри, я тупо добавил возможность выбрать пункт меню:

//формула ромба
        case 4:
            cout << "Что вам дано в задании про ромб?" << endl;
            cout << "1 - высота и сторона\n"
                << "2 - сторона и кут\n"
                << "3 - 2 диагонали" << endl;
            cin >> SelectToOption; //                 <---- добавил
            switch (SelectToOption)
            {
                //известно сторону и высоту
                case 1:
                    cout << "Вам известно высоту и сторону" << endl;
                    cout << "Введите сторону: " ;
                    cin >> a;
                    cout << "Введите высоту: " ;
                    cin >> h;
                    cout << "Площадь ромба = " << a * h << endl;
                break;
                //известно сторону и кут
                case 2:
                    cout << "Вам известно сторону и кут" << endl;
                    cout << "Введите сторону: " ;
                    cin >> a;
                    cout << "Введите кут (sin): " ;
                    cin >> l;
                    cout << "Площадь ромба = " << pow((double)a, 2) * sin((double)l) << endl;
                break;

[_Bers]

У тебя код читается очень легко, но он крайне не гибкий. Можно было сделать проще, но в тоже время гибче и покороче:

инкапсулировать меню с возможностью выбора

[AcTiV]

о спасибо большое, забыл, наверно вчера пересидел и на мелочь не обращал

p.s. я учту ваши пожелания насчет улучшения кода