Не могу понять в чем дело.

[Terabaytus]

Читаю Стивина Прата вот листинг приведенный в его книге

Код: (C++)

#include "stdafx.h"
#include <iostream>
using namespace std;
void showmenu () ; // прототипы функций
void report () ;
void comfort () ;
int main ()
{
showmenu () ;
int choice;
cin >> choice ;

while (choice != 5)

switch (choice )
{
case 1: cout << "\a\n" ;
break;
case 2: report () ;
break;
case 3: cout << "The boss was in all day. \n" ;
break;
case 4: comfort () ;
break;
default: cout << "That ' s not а choice .\n" ;
showmenu () ;
cin >> choice ;
cout << "Bye!\n" ;
return 0;
}
}
void showmenu ()
{
cout << "Please enter 1, 2, 3, 4, or 5:\n"
"1) alarm 2) report \n"
"3) alibi 4) comfort \n"
"5) quit\n ";
};
void report ()
{
cout << "I t's been an excellent week for business .\n"
"Sales are up 120% . Expenses are down 35% .\n" ;
};
void comfort ()
{
cout << "Your employee s think you are the finest CEO\n"
"in the industry . The board of directors think\n"
"you are the finest СЕО in the indus try . \n" ;
}

При его его выполнении работает не корректно, выбираешь любое действие и оно зацикливается такое впечатление, что оператор break игнорируется компилятором, но самое интересное загружал его в visual C++ 2008 почти все нормально, а в visual C++ 2010 происходит выше описанная хрень, в чем может быть дело

[Finch]

А где в swicth определение, как реагировать на 5 пункт меню?

[Terabaytus]

Вводя 5, прога выводит Bye и закрывает консоль. Именно так написано в книге и именно так было в visual C++ 2008.

[Dimka]

Если в книге именно такой код, то книгу надо на помойку сразу.

[Finch]

Код:

default: cout << "That ' s not а choice .\n" ;
showmenu () ;
cin >> choice ;
cout << "Bye!\n" ;
return 0;

При коде 5, у тебя будет выполняться данная ветка. Где правильность логики? Кстати при вводе 6 таже самая ветка будет работать.

Добавлено через 3 минуты и 20 секунд:
Dimka, Я встречал много толковых книг, где были ошибки в коде. Наверно специально туда вставляются. Чтоб читатель думал

[Terabaytus]

Да издание Прата Лекции и упражнения грешит опечатками я как начинающий и то это замечаю, это скорей проблемы перевода, но остается вопрос почему в разных студиях по разному работает?

[Finch]

Тут склорее всего дело не в студии, а в оптимизации. Компилируй и исполняй в Debug режиме.

[Dimka]

Finch, да причём тут ошибки. Ты на main посмотри, на цикл, на сам подход к структуризации кода - это же ужас, почти "спагетти" с точки зрения control flow. Либо в книжке всё же не совсем так, и мы видим код преимущественно автора темы - начинающего, либо книжку на помойку.

Оно мало того, что по всем кроме 1-4 выйдет, так ещё и зациклится насмерть при 1-4.

P.S. А вообще хорошая книжка содержит правильный пример без ошибок и несколько упражнений, в том числе и вида "найди ошибку".

[Finch]

Насчет зацикливания, я даже не обратил внимание. Тогда понятно, почему оптимизатор так играется
Наши процессоры настолько быстры, что даже бесконечный цикл выполняют за 2 секунды (c) Sun Inc.

[Dimka]

Правильный control flow будет:

Код: (C++)

int choice;
do
{
   showmenu();
   cin >> choice;
   switch(choice)
   {
   case 1:
      cout << "bla-bla-bla" << endl;
      break;
   case 2:
      report();
      break;
   case 3:
      cout << "bla-bla-bla" << endl;
      break;
   case 4:
      comfort();
      break;
   case 5:
      cout << "Bye!" << endl;
      break;
   default:
      cout << "Invalid choice" << endl;
   }
}
while(choice != 5);

Минимум кода, и очевидно, что куда следует.

[Terabaytus]

Спасибо за код и ответы. Все разобрался косяк за мной все дело оказалось в расстановке фигурных скобок недоглядел. Потому и в студиях разность получилась, в одной код был правильно написан в другой нет. У меня другой вопрос кто ни будь может мне объяснить как можно в консоли заставить в вращаться геометрическую фигуру желательно с простеньким примером?

Код: (C++)

#include "stdafx.h"
#include <iostream>
using namespace std;
void showmenu () ; // прототипы функций
void report () ;
void comfort () ;
int main ()
{
showmenu () ;
int choice;
cin >> choice ;
while (choice != 5)
{
switch (choice )
{
          case 1: cout << "\a\n" ;
break;
          case 2: report () ;
break;
          case 3: cout << "The boss was in all day. \n" ;
break;
         case 4: comfort () ;
break;
          default: cout << "That ' s not а choice .\n" ;
}
showmenu () ;
cin >> choice ;
}
cout << "Bye!\n" ;
return 0;
}

void showmenu ()
{
cout << "Please enter 1, 2, 3, 4, or 5:\n"
"1) alarm 2) report \n"
"3) alibi 4) comfort \n"
"5) quit\n ";
};
void report ()
{
cout << "I t's been an excellent week for business .\n"
"Sales are up 120% . Expenses are down 35% .\n" ;
};
void comfort ()
{
cout << "Your employee s think you are the finest CEO\n"
"in the industry . The board of directors think\n"
"you are the finest СЕО in the indus try . \n" ;
}

[Вад]

Terabaytus, что значит "в консоли" и "геометрическую фигуру"? Фигура сама по себе, консоль сама по себе. Рассчитываешь положение фигуры, рисуешь. Положение поменялось - пересчитал, перерисовал. Первая часть задачи сугубо на геометрию, а вторая - исключительно вопрос представления ("рисования") фигуры теми или иными средствами.

[Terabaytus]

Видимо у нас разное представление поправьте если я ошибаюсь, для меня консоль это черное окошко в которое компилятор выводит результаты написанного мой кода. Так вот я рисую с помощью символов(0,1) фигуру и хочу чтобы она вращалась по часовой стрелке как это можно осуществить по координатам до этого я дойти не могу для этого и прошу пример, но то что ее можно перерисовать каждый  сдвиг а потом выводить эти рисунки на экран друг за другом (как на страницах бумажного блокнота )  это мне ясно.   

[Dimka]

Terabaytus, как сказал Вад, такие задачи решаются декомпозицией по частям.

1) Математическая модель фигуры. Какая именно фигура, какими точками, отрезками, кривыми она должна представляться? Где центр вращения в так называемых мировых координатах? Какой код написать, чтобы все точки фигуры поворачивались в нужную тебе сторону относительно центра вращения?

2) Как фигура должна рисоваться на экране. Это какие-то линии. (Текстур у тебя, похоже, не будет.) Каково разрешение экрана? (У консоли это обычно 80х25 или 80х50 позиций.) Как выводятся пикселы? В простейшем случае это будет символ заливки. В более сложном, исходя из анализа соседних символов подбирается символ псевдографики или алфавитно-цифровой, чтобы картина смотрелась чётче. С алгоритмом такого анализа и подбора нужно определиться, если пойти более сложным путём. Как отобразить мировую систему координат в экранную (какие масштабные коэффициенты, как повёрнуто и смещено начало координат - обычно в левом верхнем углу, а ось Y направлена вниз)?

3) Как организовать вывод и анимацию? Если консоль работает в телетайпном режиме, то нужно составить алгоритм заполнения экрана символами, и организовать поток сменяющих друг друга экранов. Если консоль работает в программируемом режиме (с перемещением курсора и сменой цветов), то организовать циклическую очистку экрана и отрисовку очередного кадра.

[Terabaytus]

Вот вот с третьим пунктом мне все ясно в этом направлении я и буду двигаться, а еще вопрос, что нужно сделать с кодом или дописать в нем, что бы скрыть от посторонних глаз, если возможно я это представлю себе в виде exe файла на который кликаешь и код начинает работать?

[Dimka]

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

[Джон]

Ключевая фраза:

если возможно я это представлю себе в виде exe файла на который кликаешь и код начинает работать?

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

[Terabaytus]

Все ясно спасибо.