根据用户输入的玩家数量创建玩家对象？

Question

背景：
我创建了一个玩家类，我想问用户有多少玩家要玩这个游戏？根据用户输入，我正在尝试创建播放器类的许多实例。但是，我使用以下链接来帮助我：
Create object using user input
http://www.cplusplus.com/forum/beginner/197342/
所以，我尝试了他们的解决方案：

#include "Player.h"

int main() {

    int totalPlayers = -1;
    cout << "Enter total number of players: ";
    while ((totalPlayers < 1) && (totalPlayers > 5)) {
        cout << "How many players will be playing? (1-5): ";
        cin >> totalPlayers;
    }
    vector<Player> players(totalPlayers);

    system("pause");
}    

我收到错误：16LinearChess.exe 中 0x763F40B2 处的未处理异常：Microsoft C++ 异常：内存位置 0x003BF690 处的 std::length_error。
所以，我用谷歌搜索了这个确切的错误并找到了这个链接：Error : std::length_error at memory location
所以，首先他的代码与我的代码无关，但错误是一样的。我不明白答案，但我认为我必须使用堆内存创建实例。所以我试了一下：

#include "Player.h"

int main() {

    int totalPlayers = -1;
    cout << "Enter total number of players: ";
    while ((totalPlayers < 1) && (totalPlayers > 5)) {
        cout << "How many players will be playing? (1-5): ";
        cin >> totalPlayers;
    }
    vector<Player> *players = new Player(totalPlayers);
    delete[] players;

    system("pause");
}    

我收到两个错误：
严重性代码描述项目文件行抑制状态 错误（活动）E0144 类型“玩家 *”的值不能用于初始化类型“std::vector *”的实体 16LinearChess D:\Keshav\Programming Languages\C++\Beginner\ 01 Michael Dawson\16LinearChess\LinearChess.cpp 64


严重性代码描述项目文件行抑制状态 错误（活动）E0289 构造函数“Player::Player”的实例与参数列表不匹配 16LinearChess D:\Keshav\Programming Languages\C++\Beginner\01 Michael Dawson\16LinearChess\LinearChess.cpp

这是我的 Player 类：

#include <iostream>

class Player : public Board {
protected:
    int m_position;
    Log logger;
    int m_playerNumber;
public: 
    static int m_numberOfPlayers;
    Player() :m_position(0) {
        ++m_numberOfPlayers; 
        m_playerNumber = m_numberOfPlayers;
    }

    void m_SetPlayerPosition(int &position) {
        if ((position < 0) || (position > 100)) {
            m_position = 0;
            logger.Error("Position cannot be less than or greater than 100. Your position has been reset to 0 because you fell out of the map.");
        }
        else {
            m_position = position;
        }
        m_SetBoardPosition(m_position, m_numberOfPlayers); // update the position on the board.
    }

    friend ostream &operator << (ostream &os, Player &player) {
        os << "Player position on board: " << player.m_position << "\nPlayer Number: " << player.m_playerNumber << '\n';
        return os;
    }
};

int Player::m_numberOfPlayers = 0; // initializing total number of players.

谢谢！

Answer 1

问题在于您的 while 循环：

    int totalPlayers = -1;
    cout << "Enter total number of players: ";
    while ((totalPlayers < 1) && (totalPlayers > 5)) {
        cout << "How many players will be playing? (1-5): ";
        cin >> totalPlayers;
    }

while 循环只会在条件为真时运行，但条件永远不会为真，因为没有数字可以小于 1，也不能大于 5。由于条件不为真，while 循环永远不会运行，totalPlayers 将等于 -1，如果您尝试访问数组索引，这是您永远不会想要的。

将其更改为：totalPlayers < 1 || totalPlayers > 5 改为 ||，您应该没问题。

对于错误：

Unhandled exception at 0x763F40B2 in 16LinearChess.exe: Microsoft C++ exception: std::length_error at memory location 0x003BF690.

您的代码引发了异常，因为 totalPlayers 等于 -1。所以你基本上是这样做的：

vector<Player> players(-1);

这是没有意义的，因为您正在创建一个包含 -1 元素的数组？所以代码抛出了一个异常，告诉你出了点问题。 std::length_error 应该给出错误提示。

正如许多 cmets 所说，不要这样做：

vector<Player> *players = new Player(totalPlayers);

vector 的全部目的是让您不要那样做。您的第一个示例运行良好：

vector<Player> players(totalPlayers);

Answer 2

如果std::vector 尝试将大小调整为大于max_size() 的大小，则会出现std::length_error。大多数情况下，这是由错误的输入引起的——例如-1，它将被转换为std::vector<T>::size_type 的最大unsigned 值。

---

从你分享的代码来看，问题实际上是由于你的while循环条件：

    while ((totalPlayers < 1) && (totalPlayers > 5))

totalPlayers 不可能同时小于1 和 大于5——所以永远不会进入这个循环。由于您分配给totalPlayers 的默认值是-1，因此您调整大小的大小会溢出并成为最大的unsigned 值——这会触发std::length_error。

解决此问题应该可以解决您的错误。