2

我的构造函数需要一个std::array. 我试图给它一个默认值,但是在没有参数的情况下调用构造函数会给出这个错误:

$ g++ -std=c++11 -Wall -Werror -Wextra -pedantic-errors test.cpp Position.cpp -o test
test.cpp: In function ‘int main()’:
test.cpp:14:29: error: array must be initialized with a brace-enclosed initializer
  Position *y = new Position();
                             ^
test.cpp:14:29: error: too many initializers for ‘std::array<unsigned char, 8ul>’

行号不同。这是我的代码:

// test.cpp
int main() {
    Position *x = new Position({1,1,1,1,1,1,1,1}); // works
    Position *y = new Position(); // does not work
}

// Position.cpp
#include <cstdint>
#include <array>
#include "Position.h"
Position::Position( std::array<uint8_t,8> columns_ ) {
    columns = columns_;
}

// Position.h
#ifndef POSITION_H
#define POSITION_H
#include <array>
class Position {
    public:
    Position( std::array<uint8_t,8> = {0,0,0,0,0,0,0,0} );
    private:
    std::array<uint8_t,8> columns;
};
#endif

我做错了什么?

4

2 回答 2

5

您需要重复类型:

Position( std::array<uint8_t,8> = std::array<uint8_t,8>{0,0,0,0,0,0,0,0} );

虽然这(或使用双括号)有效,但当采用默认值时,它会在每个调用者的一方创建一个副本。重载的构造函数可以避免这种代码膨胀,因此我建议优先使用重载而不是默认参数。

于 2013-10-03T07:10:14.617 回答
4

第二种情况运行所需的双括号:

Position( std::array<uint8_t,8> = {{0,0,0,0,0,0,0,0}} );
于 2013-10-03T07:16:02.810 回答