Question

我正在制作国际象棋引擎并使用此代码：

#pragma once

    #include "SFML\Graphics.hpp"
    #include "Grid.h"

    enum PieceType { //place gamePieces into Grid not field, cus grid is already in field
        W_BISHOP,
        W_PAWN,
        W_KING,
        W_QUEEN,
        W_ROOK,
        W_KNIGHT,
        B_BISHOP,
        B_PAWN,
        B_KING,
        B_QUEEN,
        B_ROOK,
        B_KNIGHT
    };

class GamePieces //TODO PUT TEAM IN GRID, ACCESS GAME PIECES IN GRID TO MOVE THEM
{
public:
    GamePieces(){}
    GamePieces(PieceType& type, Coords& position) : m_type(type),m_position(position) {
        switch (type) {
            case W_BISHOP:
                m_gamePiece.loadFromFile("w_bishop");
                m_gamePieceSprite.setTexture(m_gamePiece);
                break;

           //REST OF CASES OF PIECETYPE ARE PRETTY MUCH THE SAME

    }
    ~GamePieces();
private:
    sf::Texture m_gamePiece;
    sf::Sprite m_gamePieceSprite;
    Coords m_position;
    PieceType m_type;
};



enum TeamColor {
    BLACK,
    WHITE
};

struct Team {
    //16 pieces in regular chess
    //map that assigns specific game pieces coordinates

    GamePieces m_chessPieces[16];
    TeamColor color;

    Team() {}

    Team(TeamColor& c) {
        Coords coord;

        switch (c) {
            case WHITE: 

                for (int i = 0; i < 8; i++) {
                    coord.m_x += 52.5;
                    coord.m_y += 52.5;
                    m_chessPieces[i] = GamePieces(PieceType::B_PAWN, coord);
                }
                break;

剪掉我不需要的部分，但基本上错误发生在这一行：

GamePieces(PieceType::B_PAWN, coord);

并且它说GamePieces的构造函数没有指定的参数列表，但确实如此！

Answer 1

这是因为您尝试将rvalue（PieceType::B_PAWN）分配给非const引用。该语言不允许您这样做。

解决方案是使构造函数按值或const引用：

GamePieces(PieceType type, const Coords& position) //...

Answer 2

临时不能绑定到非const引用，将构造函数更改为

GamePieces(const PieceType& type, const Coords& position)

或

GamePieces(PieceType type, const Coords& position)

Answer 3

您的构造函数无理由地通过非const引用获取其参数。

非const引用不能绑定到PieceType::B_PAWN等常量。

除非要修改被调用函数中的参数，否则应该按值传递枚举。您应该通过const引用传递Coord之类的结构，或者如果它们足够小且足够简单（或者通过const引用是一个安全的回退选项），有时会通过值传递。

Answer 4

你的功能似乎错了。删除引用：

GamePieces(PieceType type, Coords position)

更新：请注意，上面的行是解决问题的最简单方法。正如之前的评论所说，你可以通过值或const引用来获取参数。

在这种情况下，第一个是非常小的对象，因此您可以按值获取它。第二个可以通过cont引用，因为它可能更大（8个字节？）。从这个意义上说，@ TartanLlama的答案是完美的。

尽管如此，通过值获取两者是一个很好的解决方案，因为您的方法是内联的，编译器有机会优化代码（例如，使用寄存器而不是堆栈来传递参数）。

Answer 5

语法：

 GamePieces(PieceType::B_PAWN, coord);

不正确。如果是enum变量，则不必声明枚举类型。正确的语法如下：

 GamePieces(B_PAWN, coord);

宣言：

GamePieces(PieceType& type, Coords& position)

不正确，因为您使用非const引用。您必须将其更改为以下内容：

GamePieces(const PieceType& type, const Coords& position)

类的构造函数与参数列表不匹配....但它确实......？

5 个答案: