編集履歴

質問編集履歴

書式の改善

2017/11/04 03:01

投稿

ganariya

スコア50

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -1,43 +1,495 @@
-現在、Siv3Dでゲーム制作をしています。
-独自のクラス[Block]を作り、それを vectorとして
-std::vector<Block> blocksと[Board]というクラス内で宣言して
-emplace_backで要素を追加し、
-[Board]というクラス内で使用しています。
-ブロックを真下に落とす関数を作っていたのですが、
-blocks[0]
-として使おうとすると
-Expression: vector subscript out of range
-Line:1234
-と表示され、アサーションが吐かれてしまいました。
-for(auto& x:  blocks)は使用でき、
-for(size_t i=0;i<blocks.size();i++)もエラーはありませんでした。
-blocks[0]やblocks.front()などだけエラーとなってしまいます。
-これは、vectorが、emplace_backとして追加したせいで、配列として機能せず、添字などの順番が決まっていないからでしょうか。
+###前提・実現したいこと
+現在C++とSiv3Dで落ち物パズルゲームを制作しています。
+###発生している問題・エラーメッセージ
+std::vectorを使用して'Block'クラスを格納しようとしていたのですが
+(std::vector<Block> bronzeと宣言)
+bronze[0]などとアクセスしようとするとエラーが起きてしまいました。
+**発生した箇所はboard.cpp内のdropBronze関数です。**
+```
+Expression: vector subscript out of range
+```
+###該当のソースコード
+```C+++
+#pragma once
+#include "Block.hpp"
+///<summary>
+///<para>Boardクラス</para>
+///<para>・Boardのサイズは大きい方</para>
+///<para>・位置＝画面上のもの</para>
+///<para>・座標＝二次元配列のこと</para>
+///</summary>
+class Board {
+public:
+    Board() = default;
+    Board(WhichPlayer _whichPlayer);
+    ~Board();
+public:
+    const Point		    BOARD_VISUAL_SIZE;			//見た目上の二次元配列
+    const Point		    BOARD_ACTUAL_SIZE;			//実際の二次元配列
+    const Point		    BOARD_POINT_1P;			//1Pの画面上の位置
+    const Point		    BOARD_POINT_2P;			//2Pの画面上の位置
+    const Point		    INIT_BRONZE_COORDINATE;		//ブロンズの基準の初期座標
+    const Point		    BOARD_RECTANGLE_SIZE;		//ボードのサイズ
+    const int		    BLOCK_SIZE;
+    const unsigned int	    BRONZE_MAX_SIZE;
+    const int		    RED_NUM;
+    const int		    BLUE_NUM;
+    const int		    GREEN_NUM;
+    const int		    YELLOW_NUM;
+    const int		    PURPLE_NUM;
+    const int		    SELULIAN_NUM;
+    const int		    WALL_NUM;
+    const int		    NONE_NUM;
+public:
+    void		    init();
+    void		    update();
+    void		    draw() const;
+    void		    boardInit();
+    void		    bronzeInit();
+    void		    boardDraw() const;
+    void		    bronzeDraw() const;
+    void		    addBlockToBronze(Point coordinate,BlockColor blockColor);
+    void		    moveBronzeIndicator(int direction);
+    void		    dropBronze();
+    void		    checkBoardConnection();
+    bool		    isCanMoveBronze(int direction);
+    bool		    isCanAddBlockToBronze();
+    bool		    isBronzeDontCollideWithBoard(Point _coordinate);
+    bool		    isReachFallingDefaultCount();
+private:
+    std::vector<std::vector<BlockColor>>      board;
+    std::vector<Block>	    bronze;
+    std::vector<Point>	    candidates;
+    Point		    bronzeIndicator;
+    WhichPlayer		    whichPlayer;
+    Point		    boardVisualPoint;
+    Rect		    rectangle;
+    int			    fallingCount;
+    int			    fallingDefaultCount;
+};
+#include "Board.hpp"
+Board::Board(WhichPlayer _whichPlayer) :
+    whichPlayer(_whichPlayer),
+    BOARD_VISUAL_SIZE(Point(7, 13)),
+    BOARD_ACTUAL_SIZE(Point(9, 15)),
+    BOARD_POINT_1P(150, 100),
+    BOARD_POINT_2P(850, 100),
+    INIT_BRONZE_COORDINATE(3,0),
+    BOARD_RECTANGLE_SIZE(280,520),
+    BLOCK_SIZE(40),
+    BRONZE_MAX_SIZE(5),
+    RED_NUM(0),
+    BLUE_NUM(1),
+    GREEN_NUM(2),
+    YELLOW_NUM(3),
+    PURPLE_NUM(4),
+    SELULIAN_NUM(5),
+    WALL_NUM(6),
+    NONE_NUM(7)
+{
+    init();
+}
+Board::~Board() {
+}
+void Board::init() {
+    boardInit();
+    bronzeInit();
+}
+void Board::update() {
+    if (Input::Key1.clicked) {
+	dropBronze();
+    }
+}
+void Board::draw() const {
+    rectangle.draw();
+    boardDraw();
+    bronzeDraw();
+}
+void Board::boardInit() {
+    board.resize(BOARD_ACTUAL_SIZE.x);
+    for (int i = 0; i < BOARD_ACTUAL_SIZE.x; i++) {
+	board[i].resize(BOARD_ACTUAL_SIZE.y);
+    }
+    for (int i = 0; i < BOARD_ACTUAL_SIZE.x; i++) {
+	for (int j = 0; j < BOARD_ACTUAL_SIZE.y; j++) {
+	    board[i][j] = static_cast<BlockColor>(NONE_NUM);
+	}
+    }
+    for (int i = 0; i < BOARD_ACTUAL_SIZE.x; i++) {
+	board[i][0] = static_cast<BlockColor>(WALL_NUM);
+	board[i][BOARD_ACTUAL_SIZE.y - 1] = static_cast<BlockColor>(WALL_NUM);
+    }
+    for (int i = 0; i < BOARD_ACTUAL_SIZE.y; i++) {
+	board[0][i] = static_cast<BlockColor>(WALL_NUM);
+	board[BOARD_ACTUAL_SIZE.x - 1][i] = static_cast<BlockColor>(WALL_NUM);
+    }
+    if (whichPlayer == WhichPlayer::PLAYER1) {
+	boardVisualPoint = BOARD_POINT_1P;
+    }
+    else {
+	boardVisualPoint = BOARD_POINT_2P;
+    }
+    rectangle = Rect(boardVisualPoint, BOARD_RECTANGLE_SIZE);
+}
+void Board::bronzeInit() {
+    bronzeIndicator = INIT_BRONZE_COORDINATE;
+    std::vector<Block>().swap(bronze);
+}
+void Board::bronzeDraw() const {
+    for (unsigned int i = 0; i<bronze.size(); i++) {
+	TextureAsset(L"Block")(BLOCK_SIZE*static_cast<int>(bronze[i].getBlockColor()), 0, BLOCK_SIZE, BLOCK_SIZE).draw(boardVisualPoint +BLOCK_SIZE*(bronzeIndicator+bronze[i].getPoint()));
+    }
+}
+void Board::boardDraw() const {
+    for (int i = 0; i < BOARD_VISUAL_SIZE.x; i++) {
+	for (int j = 0; j < BOARD_VISUAL_SIZE.y; j++) {
+	    if (static_cast<int>(board[i + 1][j + 1]) < 5) {
+		TextureAsset(L"Block")(BLOCK_SIZE*static_cast<int>(board[i + 1][j + 1]), 0, BLOCK_SIZE, BLOCK_SIZE).draw(boardVisualPoint + Point(i*BLOCK_SIZE, j*BLOCK_SIZE));
+	    }
+	}
+    }
+}
+void Board::addBlockToBronze(Point coordinate,BlockColor blockColor) {
+    for (auto& x : bronze) {
+	x.setPoint(x.getPoint() + Point(0, 1));
+    }
+    bronze.emplace_back(Block(blockColor,coordinate));
+}
+void Board::moveBronzeIndicator(int direction) {
+    bronzeIndicator += Point(direction, 0);
+}
+void Board::dropBronze() {
+    Point dropBaseCoordinate = Point(0,0);
+    for (auto& x : bronze) {
+    }
+ //   while (board[bronzeIndicator.x+1][(dropBaseCoordinate + bronze.front().getPoint()).y + 1] == BlockColor::NONE) {
+	//dropBaseCoordinate += Point(0, 1);
+ //   }
+    for (auto& x : bronze) {
+	board[bronzeIndicator.x + 1][x.getPoint().y + dropBaseCoordinate.y + 1] = x.getBlockColor();
+    }
+    bronze[0];
+    bronzeInit();
+}
+bool Board::isCanAddBlockToBronze() {
+    if (bronze.size() < BRONZE_MAX_SIZE) {
+	bool isCanBronzeAdd = true;
+	for (auto& x : bronze) {
+	    if (!isBronzeDontCollideWithBoard(x.getPoint() + bronzeIndicator + Point(0, 1))) {
+		isCanBronzeAdd = false;
+	    }
+	}
+	return isCanBronzeAdd;
+    }
+    return false;
+}
+bool Board::isCanMoveBronze(int direction) {
+    int valid = bronzeIndicator.x + direction;
+    if (valid >= 0 && valid < BOARD_VISUAL_SIZE.x) {
+	bool isCanBronzeMove = true;
+	for (auto& x : bronze) {
+	    if (!isBronzeDontCollideWithBoard(x.getPoint()+bronzeIndicator+Point(direction,0))) {
+		isCanBronzeMove = false;
+	    }
+	}
+	return isCanBronzeMove;
+    }
+    return false;
+}
+bool Board::isBronzeDontCollideWithBoard(Point _coordinate) {
+    if (board[_coordinate.x + 1][_coordinate.y + 1] == BlockColor::NONE) {
+	return true;
+    }
+    else {
+	return false;
+    }
+}
+```
+###試したこと
+brozneをfor文やfor auto文で回した場合エラーはありませんでした。
+添字アクセスまたは、front()などでアクセスしてしまうとエラーが発生しました。
 よろしくお願いいたします。