unsigned char*型を返す画像ロード関数を作りたい。

Question

提示コードですがどうすれば画像データを読み込んでそのデータをunsigned char*型で返せるのでしょうか？提示コードは正常に読み込まれておらすglTexImage2D();関数で例外エラーが出てしまいます。どうすればいいのでしょうか？この関数になりが足りていないのでしょうか？

//画像ファイルのバイナリを返す。　widht ,height は大きさで必要ない場合は指定しない。
byte* Game::LoadTexture(const char* str,int *width, int *height)
{
    cv::Mat image;
    image = cv::imread(str);

    if (image.empty() == true)
    {
        printf("image.empty()\n");
        return nullptr;
    }
    else {
        printf("load \n");
        cv::cvtColor(image, image, cv::COLOR_RGB2BGR); //色空間を変更


        int size = image.rows * image.cols;
        byte* bytes = new byte[size]{ 0 };
        std::memcpy(bytes, image.data, size * sizeof(byte));

        //　画像サイズを取得
        if (width != nullptr) 
        {
            *width = image.cols;    //  横
            printf("widht: %d\n",*width);
        }

        if (height != nullptr) 
        {
            *height = image.rows;   //  縦
            printf("height: %d\n", *height);

        }

        return bytes;   //  unsigned char*を返す
    }
}

Answer 1

何が足りないか？と言えば，

int size = image.rows * image.cols;

このサイズが全然足りないんじゃないですかね．
（１画素の情報に何byte必要なのか，という視点が抜けている）

あと，glTexImage2Dに食わせるのならばその際にフォーマットを指定すれば良いだろうからcvtColorとかしなくても良いのではないかと思う．

Comments

[退会済みユーザー]

int size = image.rows * image.cols * (int)sizeof(byte);
としましたが例外が発生します。
またglTexImage2D();関数を
glTexImage2D(GL_TEXTURE_2D, 0, GL_RGBA, width, height, 0, GL_RGBA, GL_UNSIGNED_BYTE, (void*)data);
としましが例外が発生します。GL_RGBに変更してみましたがだめでした。

[fana]

> int size = image.rows * image.cols * (int)sizeof(byte);

何を意図した変更なんですか？
sizeof(byte) の値が1であれば，何も変わらないですよねコレ．

cv::Mat における１画素の情報が {R,G,B}の3chであって且つそれぞれが8bitなのであれば，１画素のデータサイズは8*3=24bit なわけ．
だから，画素値をコピーするには最低でも (画素数*3)[byte] のバッファが必要でしょ，って話．

[退会済みユーザー]

なるほど。nt size = image.rows * image.cols * 4;　として治りました。