ввод строки в c++

[ZwS]

возник небольшой вопросик по си. есть вот такой код

Код: [Выделить]

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

int rate;

...

void menu()
{
std::string skill;
std::cout << "Выберите уровень сложности (easy, medium, hard): ";
std::cin >> skill;
if(skill.compare("easy") == 0) rate = 15;
else if(skill.compare("medium") == 0) rate = 25;
else if(skill.compare("hard") == 0) rate = 35;
else menu();
}

...

int main()
{

...

menu();

...

return 0;
}
проблема заключается вот в чем: на строке std::cin >> skill; программа зависает если ввести easy, medium или hard. решил так потому, что если после этой сторчки добавить std::cout << "работает \n"; вывод надписи не осуществляется.
если составить программу такого плана:

Код: [Выделить]

#include <iostream>
#include <string>

int rate;

int main()
{
std::string skill;
std::cout << "Выберите уровень сложности (easy, medium, hard): ";
std::cin >> skill;
std::cout << "работает \n";
if(skill.compare("easy") == 0) rate = 15;
else if(skill.compare("medium") == 0) rate = 25;
else if(skill.compare("hard") == 0) rate = 35;
else main();
}все отрабатывается на ура. в чем может быть дело?

[Saha]

Первый вариант программы отрабатывается без проблем
 
Может, дело во всем остальном коде?
(компилил g++ 4.3.3)

[ZwS]

видимо так оно и есть. закоментировал остальные строки и программа выполнилась. только вычислить ошибку не получается.
компилю тоже при помощи g++

(Нажмите, чтобы показать/скрыть)

Код: [Выделить]

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

class Map
{
public:
char ships[10][10];
void GeneratePos(unsigned int cells);
};

const std::string ver("0.1a");
int rate;
Map player1, enemymap1, player2, enemymap2, player, enemymap, enemyrealmap;


int int_rand(int n)
{
return int(rand()/(RAND_MAX + 1.0) * n );
}

void menu()
{
std::string skill;
std::cout << "Выберите уровень сложности (easy, medium, hard): ";
std::cin >> skill;                                             \\та самая проблемная строка
if(skill.compare("easy") == 0) rate = 15;
else if(skill.compare("medium") == 0) rate = 25;
else if(skill.compare("hard") == 0) rate = 35;
else menu();
}

//расстановка кораблей
void Map::GeneratePos(unsigned int cells)
{
int m, n, dir;
bool flag = false;
while(flag == false)
{
n = int_rand(9);
m = int_rand(9);
dir = int_rand(2);
switch(cells)
{
case 1 : if (((ships[n-1][m-1] == ' ') || ((n == 1) && (m == 1))) &&
((ships[n-1][m] == ' ') || (n == 1)) &&
((ships[n-1][m+1] == ' ') || ((n == 1) && (m == 9))) &&
((ships[n][m-1] == ' ') || (m == 1)) &&
((ships[n][m+1] == ' ') || (m == 9)) &&
((ships[n+1][m-1] == ' ') || ((n == 9) && (m == 1))) &&
((ships[n+1][m] = ' ') || (n == 9)) &&
((ships[n+1][m+1] == ' ') || ((n == 1) && (m == 1))))
{
ships[n][m] = '█';
flag = true;
}
case 2 : if ((dir = 0) and (n+2 <= 9))
{
if (((ships[n-1][m-1] == ' ') || ((n == 1) && (m == 1))) &&
((ships[n-1][m] == ' ') || (n == 1)) &&
((ships[n-1][m+1] == ' ') || ((n == 1) && (m == 9))) &&
((ships[n][m-1] == ' ') || (m == 1)) &&
((ships[n][m+1] == ' ') || (m == 9)) &&
((ships[n+1][m-1] == ' ') || (m == 1)) &&
((ships[n+1][m+1] == ' ') || (m == 9)) &&
((ships[n+2][m-1] == ' ') || ((n == 9) && (m == 1))) &&
((ships[n+2][m] == ' ') || (n == 9)) &&
((ships[n+2][m+1] == ' ') || ((n == 1) && (m == 1))))
{
ships[n][m] = '█';
ships[n+1][m] = '█';
flag = true;
}
}
if ((dir == 1) && (m+2 <= 9))
{
if  (((ships[n-1][m] == ' ') || (n == 1)) &&
((ships[n+1][m] == ' ') || (n == 9)) &&
((ships[n-1][m+1] == ' ') || (n == 1)) &&
((ships[n+1][m+1] == ' ') || (n == 9)) &&
((ships[n][m+2] == ' ') || (m == 9)) &&
((ships[n-1][m+2] == ' ') || ((n == 1) && (m == 9))) &&
((ships[n+1][m+2] == ' ') || ((n == 1) && (m == 1))) &&
((ships[n][m-1] == ' ') || (m == 1)) &&
((ships[n-1][m-1] == ' ') || ((n == 1) && (m == 1))) &&
((ships[n+1][m-1] == ' ') || ((n == 9) && (m == 1))))
{
ships[n][m] = '█';
ships[n][m+1] = '█';
flag = true;
}
}
case 3 : if ((dir = 0) and (n+3 <= 9))
{
if (((ships[n-1][m-1] == ' ') || ((n == 1) && (m == 1))) &&
((ships[n-1][m] == ' ') || (n == 1)) &&
((ships[n-1][m+1] == ' ') || ((n == 1) && (m == 9))) &&
((ships[n][m-1] == ' ') || (m == 1)) &&
((ships[n][m+1] == ' ') || (m == 9)) &&
((ships[n+1][m-1] == ' ') || (m == 1)) &&
((ships[n+1][m+1] == ' ') || (m == 9)) &&
((ships[n+2][m-1] == ' ') || (m == 1)) &&
((ships[n+2][m+1] == ' ') || (m == 9)) &&
((ships[n+3][m-1] == ' ') || ((n == 9) && (m == 1))) &&
((ships[n+3][m] == ' ') || (n == 9)) &&
((ships[n+3][m+1] == ' ') || ((n == 1) && (m == 1))))
{
ships[n][m] = '█';
ships[n+1][m] = '█';
ships[n+2][m] = '█';
flag = true;
}
}
if ((dir == 1) &&  (m+3 <= 9))
{
if  (((ships[n-1][m] == ' ') || (n == 1)) &&
((ships[n+1][m] == ' ') || (n == 9)) &&
((ships[n-1][m+1] == ' ') || (n == 1)) &&
((ships[n+1][m+1] == ' ') || (n == 9)) &&
((ships[n-1][m+2] == ' ') || (n == 1)) &&
((ships[n+1][m+2] == ' ') || (n == 9)) &&
((ships[n][m+3] == ' ') || (m == 9)) &&
((ships[n-1][m+3] == ' ') || ((n == 1) && (m == 9))) &&
((ships[n+1][m+3] == ' ') || ((n == 1) && (m == 1))) &&
((ships[n][m-1] == ' ') || (m == 1)) &&
((ships[n-1][m-1] == ' ') || ((n == 1) && (m == 1))) &&
((ships[n+1][m-1] == ' ') || ((n == 9) && (m == 1))))
{
ships[n][m] = '█';
ships[n][m+1] = '█';
ships[n][m+2] = '█';
flag = true;
}
}
case 4 : if ((dir == 0) && (n+4 <= 9))
{
if (((ships[n-1][m-1] == ' ') || ((n == 1) && (m == 1))) &&
((ships[n-1][m] == ' ') || (n == 1)) &&
((ships[n-1][m+1] == ' ') || ((n == 1) && (m == 9))) &&
((ships[n][m-1] == ' ') || (m == 1)) &&
((ships[n][m+1] == ' ') || (m == 9)) &&
((ships[n+1][m-1] == ' ') || (m == 1)) &&
((ships[n+1][m+1] == ' ') || (m == 9)) &&
((ships[n+2][m-1] == ' ') || (m == 1)) &&
((ships[n+2][m+1] == ' ') || (m == 9)) &&
((ships[n+3][m-1] == ' ') || (m == 1)) &&
((ships[n+3][m+1] == ' ') || (m == 9)) &&
((ships[n+4][m-1] == ' ') || ((n == 9) && (m == 1))) &&
((ships[n+4][m] == ' ') || (n == 9)) &&
((ships[n+4][m+1] == ' ') || ((n == 1) && (m == 1))))
{
ships[n][m] = '█';
ships[n+1][m] = '█';
ships[n+2][m] = '█';
ships[n+3][m] = '█';
flag = true;
}
}
if ((dir == 1) &&  (m+3 <= 9))
{
if  (((ships[n-1][m] == ' ') || (n == 1)) &&
((ships[n+1][m] == ' ') || (n == 9)) &&
((ships[n-1][m+1] == ' ') || (n == 1)) &&
((ships[n+1][m+1] == ' ') || (n == 9)) &&
((ships[n-1][m+2] == ' ') || (n == 1)) &&
((ships[n+1][m+2] == ' ') || (n == 9)) &&
((ships[n-1][m+3] == ' ') || (n == 1)) &&
((ships[n+1][m+3] == ' ') || (n == 9)) &&
((ships[n][m+4] == ' ') || (m == 9)) &&
((ships[n-1][m+4] == ' ') || ((n == 1) && (m == 9))) &&
((ships[n+1][m+4] == ' ') || ((n == 1) && (m == 1))) &&
((ships[n][m-1] == ' ') || (m == 1)) &&
((ships[n-1][m-1] == ' ') || ((n == 1) && (m == 1))) &&
((ships[n+1][m-1] == ' ') || ((n == 9) && (m == 1))))
{
ships[n][m] = '█';
ships[n][m+1] = '█';
ships[n][m+2] = '█';
ships[n][m+3] = '█';
flag = true;
}
}
}
}
}

void gameprepare()
{
int i, j;
for (i=0; i < 10; i++)
{
for (j=0; j < 10; j++);
{
player1.ships[i][j] = ' ';
player2.ships[i][j] = ' ';
enemymap1.ships[i][j] = ' ';
enemymap2.ships[i][j] = ' ';
}
}
player1.GeneratePos(1);
player2.GeneratePos(1);
player1.GeneratePos(2);
player2.GeneratePos(2);
player1.GeneratePos(3);
player2.GeneratePos(3);
player1.GeneratePos(4);
player2.GeneratePos(4);
}

void MapDraw()
{
unsigned int i, j;
for(i = 0; i < 10; i++)
{
std::cout << i+1;
for(j = 0; j < 10; j++)
{
std::cout << player1.ships[i][j] << ' ';
}
std::cout << "/n";
}
}

int main()
{
std::cout << "SeaBattle " << ver << "\n";
menu();
gameprepare();
MapDraw();
return 0;
}


это порт моего морского боя с паскаля (на нем я время от времени упражняюсь).

[Kwah]

Бесконечной рекурсией балуемся? Выхода из menu() при правильном вводе как бы и нет
Пустую строку тоже наверное надо как-то обрабатывать отдельно, ибо нажимать энтер можно долго.

[Nubaseg]

В меню, все верно. Хотя я бы переделал его так:
int menu()
{
   std::string skill;
   std::cout << "Выберите уровень сложности (easy, medium, hard): ";
   std::cin >> skill;                                             //та самая проблемная строка
   if(skill.compare("easy") == 0) return 15;
   else if(skill.compare("medium") == 0) return 25;
   else if(skill.compare("hard") == 0) return 35;
   else menu();
}
А в мейне rate = menu();

Но проблема не в нем, у тебя цикл похоже бесконечный - GeneratePos
flag - false
так вот я не вникал в логику, но в каждом case идет дополнительный if, все они без else, т.е. вполне реально что flag может никогда не стать true, а цикл идет while(flag == false). 90% что проблема здесь, разбирайся.
Пользователь решил продолжить мысль 10 Апреля 2010, 00:03:35:З.Ы. писать проги без трейса и дебагера = садомазо

[Saha]

Конечно, рекурсия в данном месте как бы совсем не к месту =)
Но с логикой работы функции menu() все в порядке, никакого зацикливание не будет (только переполнение стека будет когда-нибудь после до-о-олгого неправильного ввода =)

P.S. С функцией menu() все в порядке, дальше где-то ошибка, ищи =)
(Да, дебагер наверняка поможет=)

[Nubaseg]

Продебажил твою прогу в CodeBlocks -да как я и предполагал зацикливается  GeneratePos.
поставь себе CodeBlocks из репов, там есть нормальный дебагер.

[ZwS]

заменил функцию GeneratePos на более простую и все заработало, спасибо.
а может ктото объяснить почему ошибка происходит не там где вызывается глючная функция?

[maskimko]

А еще, чтобы не писать каждый раз std::
делаем using namespace std;
Пользователь решил продолжить мысль 13 Апреля 2010, 15:41:17:А еще можно идентифицировать функцию как void menu(void);

[Yurror]

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