[c++] クラスの宣言で初期化をする方法

Question

提示コードですがクラス部の///コメント部にある乱数のクラスの宣言をすると、エラーになるのですが
提示コードコンストラクタのような書き方をしたいのですがこれはどうやってコンパイルエラーを回避するのでしょうか？

コンストラクタ

/*#################################################
# 初期化
###################################################*/    
Game::Game(SDL_Renderer *ren)
{
    render = ren;

    sprite = LoadImage("ast/piece.png");
    position = glm::ivec2(3,0);

    std::random_device seed_gen;
    std::default_random_engine engine(seed_gen);
    std::uniform_int_distribution<> dist(0, 9);

}

Error

shigurechan@shigurechan-System-Product-Name:~/Program/SDLGame/Tetris$ sh run.sh
g++ -c -MMD -MP src/Game.cpp -o obj/Game.o
In file included from src/Game.cpp:9:
src/Game.hpp:57:39: error: ‘seed_gen’ is not a type
   57 |     std::default_random_engine engine(seed_gen());
      |                                       ^~~~~~~~
src/Game.hpp:58:42: error: expected identifier before numeric constant
   58 |     std::uniform_int_distribution<> dist(0, 7);
      |                                          ^
src/Game.hpp:58:42: error: expected ‘,’ or ‘...’ before numeric constant
make: *** [Makefile:13: obj/Game.o] エラー 1


          Compile Error

クラス

#ifndef ___GAME_HPP__
#define ___GAME_HPP__
#include <SDL2/SDL.h>
#include <glm/glm.hpp>
#include <iostream>
#include <locale.h>
#include <random>

#define byte char
#define PIECE_SIZE ( (int) sprite.surface->h )  //ピースの大きさ
#define PIECE_NUM      ((int)4)                 //ピースの数
#define PIECE_WIDTH   ((int)4)                  //ピースの数
#define PIECE_HEIGHT   ((int)4)                 //ピースの数

#define STAGE_HEIGHT ((int)20)  //ステージの大きさ
#define STAGE_WIDTH ((int)10)   //ステージの大きさ

#define STAGE_POSITION_X ((int) 300)
#define STAGE_POSITION_Y ((int) 200)


class Game
{
public:
    Game(SDL_Renderer *ren);
    ~Game();

    void Update();
    void Render();

    bool getIsExit();
private:


    enum class PieceType
    {
        Red,
        Blue,
        Orange,
        Yellow,
        Purple,
        Cyan,
        Green,
        Black,
        white
    };
//////////////////////////////////////////////////////////////////////////////////////
  std::random_device seed_gen;
    std::default_random_engine engine(seed_gen);
    std::uniform_int_distribution<> dist(0, 9);
//////////////////////////////////////////////////////////////////////////////////////



    typedef struct Sprite_Image
    {
        SDL_Surface *surface = NULL;
        SDL_Texture *texture = NULL;
    }Sprite;

    Sprite sprite;

    glm::ivec2 position;
    PieceType nowPiece;
    void RenderPiece(glm::ivec2 pos,PieceType type);


    void Event();
    void RenderSprite(glm::ivec2 pos,PieceType piece);
    Sprite LoadImage(const char* fileName);

    

    bool exit = false;
    SDL_Event event;

    SDL_Renderer *render;
};



#endif

Comments

[maisumakun]

「エラーになる」とだけ書いて具体的なエラー内容を提示しないのは、何らかのポリシーがあって意図的にやっていることなのでしょうか。

[episteme]

> 提示コードですがクラス部のコメント部にある乱数のコードですがそのように実装するとエラーになるのですがこの場合はコンストラクタのような書き方をしたいのですがこれはどうやってコンパイルエラーを回避するのでしょうか？

日本語でおねがいします。

Answer 1

エラーメッセージ

src/Game.hpp:57:39: error: ‘seed_gen’ is not a type
   57 |     std::default_random_engine engine(seed_gen());
      |                                       ^~~~~~~~

にある通り

    std::default_random_engine engine(seed_gen);

の部分を

    std::default_random_engine engine(seed_gen());

に変えると動くのではないでしょうか?

https://cpprefjp.github.io/reference/random/default_random_engine.html