Почему этот блок корпуса не выполняется?

Question

Я только что начал пробовать C++ пару недель назад. У меня довольно приличная хватка на Java, прежде чем пытаться использовать C++. Многие говорили мне, что они действительно похожи в смысле синтаксиса.

Внизу есть инструкция переключателя, инициирующая сцену боя. Всякий раз, когда я выбираю вариант для борьбы, он просто закрывает программу.

Вот мой код:

#include "stdafx.h" 
#include <iostream> 
#include <cstdlib>  // For rand() 
#include <string> 
#include <sstream> 
#include <algorithm> // transform() 
#include <cctype>  // toupper(), tolower() 
#include <functional> // ptr_fun() 
#include <time.h> 
// PUT S*** BELOW THIS POINT 
//____________________________________________________________________________ 

using namespace std; 

int main() 
{ 
    /* Expirimental text based adventure game. 
    * Mainly being used for practice methods. 
    * Developed by Zack Cook. 

    Generic title. Bad story. Bad interactions. 

    Lots and lots of bad, bad code. Be warned. 
    */ 

    string charName; 

    string charChoice; 

    int charDecision; 

    int playerHealth = 100; 
    int randomNumber; 
    int orcHealth = 100; 

    srand (time(NULL)); 

    cout << "_____________________________________________" << endl; 
    cout << "| ##### #########  ###  ### ### |" << endl; 
    cout << "| ### ### ###  ### ### ### ### |" << endl; 
    cout << "| ###  ###  ### ### ##### |" << endl; 
    cout << "|  ###  ###  #########  ### |" << endl; 
    cout << "| ### ### ### ###  ### ### |" << endl; 
    cout << "| #####  ### ###  ### ### |" << endl; 
    cout << "_____________________________________________" << endl; 
    cout << "" << endl; 
    cout << "" << endl; 
    cout << "Welcome player. What is your name?" << endl; 
    getline(cin, charName); 
    yesOrNo: 
    cout << "Hello " << charName << ". Are you ready to begin?" << endl; 

    getline(cin, charChoice); 

    transform(charChoice.begin(), charChoice.end(), charChoice.begin(), toupper); 
    cout << charChoice << endl; 
    if(charChoice == "YES"){ 
     cout << "Good. Let's begin." << endl; 
    } 
    else if(charChoice == "NO"){ 
     system ("exit"); 
    } 
    else 
    { 
     cout << "That is not a good answer." << endl; 
     goto yesOrNo; 
    } 

    cout << "Our story begins with a wanderer named " << charName << " passing through the small town of Hark's Pass." << endl; 
    cout << "A little cozy village with no more than 30 or so home stayers.\nThe village men work hard on the farms during the day." << endl; 
    cout << "The women cater to the children, and other house hold chores.\nIn the evening, most of the village turns to The Rusty Trough for a bit of drink." << endl; 
    cout << "As the sun starts to set, our wanderer, " << charName << ", starts foot towards The Rusty Trough." << endl; 
    cout << "As " << charName << " enters the tavern, a heavily drunken Orc man stumbles towards the entrance." << endl; 
    cout << "\"I've never seen you 'round here!\" The orc says to our wanderer. \"I think it's time to teach these adventure types what we really think about 'em\"" << endl; 

    cout << "" << endl; 
    cout << "What will you do?" << endl; 
    cout << "1| Get ready to fight!" << endl; 
    cout << "2| Call for help!" << endl; 
    cout << "3| Try to run!" << endl; 
    cout << "4| Do nothing at all!" << endl; 
    cout << "5| Try to reason!" << endl; 

    cin >> charDecision; 

    switch(charDecision) 
    { 
    case '1': 
     do{ 
      cout << "FIGHT" << endl; 
      randomNumber = rand() % 100 + 1; 
      if(randomNumber >= 50){ 
       orcHealth = orcHealth - (randomNumber - (randomNumber/5)); 
      cout << "You hit the orc! He now has " << orcHealth << " life left!" << endl; 
      } 
      else 
      { 
       playerHealth = playerHealth - (randomNumber - (randomNumber/5)); 
       cout << "The orc hit you! You now have " << playerHealth << " life left!" << endl; 
      } 
     }while(playerHealth || orcHealth != 0); 
     break; 

    default: 
     break; 
    } 

    return 0; 
} 

Answer 1

В вашей инструкции по эксплуатации сравнивается int charDecision с '1', что является char.

Вы со стандартного ввода в int, что означает charDecision будет содержать 1. Затем сравните 1 против '1', который переводит до 49, когда отливают в int. Таким образом, ваш вход никогда не будет соответствовать (если вы не вводите 49).

Исправление: Сравнить с 1 или сделать charDecision a char.

Answer 2

Переменная charDecision объявлена ​​как int.

Стандартные классы потоков ввода-вывода C++ (в том числе cin и cout, среди прочего) относительно интеллектуальны и «делают правильную вещь» в зависимости от типа переменной, которую вы ему даете.

Когда выполняется выражение cin >> charDecision, cin считывает все, что выглядит как число , и преобразует его в собственное представление числа. Итак, когда вы вводите 1 в этом меню, то, что хранится, является буквальным номером 1. Ваш коммутатор проверяет буква символ'1', который имеет целочисленное значение 49, поэтому он не будет соответствовать, так как 49! = 1 .

Вы хотите, чтобы либо изменить тип charDecision к charили тест число 1 вместо символа '1'.

Answer 3

Ваша основная проблема - использовать goto, избавиться от него!

yesOrNo: 
cout << "Hello " << charName << ". Are you ready to begin?" << endl; 

getline(cin, charChoice); 

transform(charChoice.begin(), charChoice.end(), charChoice.begin(), toupper); 
cout << charChoice << endl; 
if(charChoice == "YES"){ 
    cout << "Good. Let's begin." << endl; 
} 
else if(charChoice == "NO"){ 
    system ("exit"); 
} 
else 
{ 
    cout << "That is not a good answer." << endl; 
    goto yesOrNo; 
} 

можно заменить

do 
{ 
    cout << "Hello " << charName << ". Are you ready to begin?" << endl; 

    getline(cin, charChoice); 

    transform(charChoice.begin(), charChoice.end(), charChoice.begin(), toupper); 
    cout << charChoice << endl; 
    if(charChoice == "YES"){ 
     cout << "Good. Let's begin." << endl; 
     startGame();//it is the function where you should place your game logic, the "switch" block in your case 
     //if you want to get back to main menu after the end of the game you can delete next line: 
     break;//leave this loop 
    } 
    else if(charChoice == "NO"){ 
     break;//leave this loop 
    } 
    else 
    { 
     cout << "That is not a good answer." << endl; 
    } 
} while (true); 

делают также таким образом, вы можете вернуться в главное меню после окончания игры.
Я также рекомендую вам разделить логику игры на разные функции, код будет намного проще писать и поддерживать. Также я бы рекомендовал переписать свою игру объектно-ориентированным способом после того, как вы закончите, это может быть хорошей практикой для вас. И забудьте о goto, он может превратить любой код в кусок дерьма.

Answer 4

while (playerHealth != 0 || orcHealth != 0) Я думаю, что это проблема.