Skip to content

Commit df7e7e2

Browse files
committed
MORE input validation
1 parent 65180aa commit df7e7e2

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

monopoly.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -795,7 +795,7 @@ bool financial_menue(player &p){
795795
std::cout<< "99 | Finished in this menue" <<std::endl;
796796
std::cout<<"Wich do you choose?"<<std::endl;
797797
std::cin>>sel;
798-
if(sel !=99){
798+
if(sel !=99 && sel < filteredTileListPlayer.size() && std::cin.good()){
799799
p.money += (0.5*filteredTileListPlayer[sel].buyPrice);
800800
gameBoard[filteredTileListPlayer[sel].tileIndex].isMortgaged = true;
801801
filteredTileListPlayer.erase(filteredTileListPlayer.begin() + sel);
@@ -835,7 +835,7 @@ bool financial_menue(player &p){
835835
std::cout<< "99 | Finished in this menue" <<std::endl;
836836
std::cout<<"Wich do you choose?"<<std::endl;
837837
std::cin>>sel;
838-
if(sel !=99){
838+
if(sel !=99 && sel < filteredTileListPlayer.size() && std::cin.good()){
839839
if (p.money < (0.55*filteredTileListPlayer[sel].buyPrice)){
840840
displayGameBoard();
841841
std::cout<<"Not enough Money. 😢"<<std::endl;
@@ -910,7 +910,7 @@ bool building_menue(player &p){
910910
std::cout<< "99 | Finished in this menue" <<std::endl;
911911
std::cout<<"Wich do you choose?"<<std::endl;
912912
std::cin>>sel;
913-
if(sel !=99 && sel < filteredTileListPlayer.size()){
913+
if(sel !=99 && sel < filteredTileListPlayer.size() && std::cin.good()){
914914
if (p.money < gameBoard[filteredTileListPlayer[sel]].housePrice){
915915
displayGameBoard();
916916
std::cout<<"Not enough Money. 😢"<<std::endl;
@@ -964,7 +964,7 @@ bool building_menue(player &p){
964964
std::cout<< "99 | Finished in this menue" <<std::endl;
965965
std::cout<<"Wich do you choose?"<<std::endl;
966966
std::cin>>sel;
967-
if(sel !=99 && sel < filteredTileListPlayer.size()){
967+
if(sel !=99 && sel < filteredTileListPlayer.size() && std::cin.good()){
968968
p.money += (0.5*gameBoard[filteredTileListPlayer[sel]].housePrice);
969969
gameBoard[filteredTileListPlayer[sel]].upgradeStage--;
970970
if(gameBoard[filteredTileListPlayer[sel]].upgradeStage == 0){

0 commit comments

Comments
 (0)