freetypeを使った文字描画で文字の四角い枠が表示される原因が知りたい。

Question

提示画像ですがなぜ提示コードの////コメント部でしっかりと終端文字を設定しているのも関わらず四角い文字の枠組みが表示されるのでしょうか？原因がわかりません。その先の for (int i = 0; txt[i] != L'\0'; i++)　分でしっかりと最後まで、まで見ているはずなのですがどもう最後以上まで？見てるみたいなのですがその場合どうすればいいのでしょうか？

void FrameWork::Text::Draw(glm::vec2 pos, const char* text, float scale, glm::vec3 color)
//void FrameWork::Text::Draw(glm::vec2 pos, std::string text,float scale, glm::vec3 color)
{
    setEnable();    //シェーダーを有効にする

    pos.y = windowContext->getSize().y - pos.y - charSize;

    //テクスチャをアクティブ
    glActiveTexture(GL_TEXTURE0);
    glBindVertexArray(vao);

    //Unform
    setUniform3f("textColor", color);
    setUniformMatrix4fv("uViewProjection", glm::ortho(0.0f, windowContext->getSize().x, 0.0f, windowContext->getSize().y));

    //char text[] = "テスト";
    //wchar_t txt[strlen(text)] = { L'\0' };
    wchar_t* txt = (wchar_t*)malloc(strlen(text));



    int i, j, f;
    for (i = 0, j = 0; text[j]; i++, j += f)
    {
        f = mbrtowc(txt + i, &text[j], MB_CUR_MAX, nullptr);
    }
///////////////////////////////////
    txt[j] = L'\0';
//////////////////////////////////
   
    for (int i = 0; txt[i] != L'\0'; i++)
    {
        //std::cout << txt[i] <<std::endl;
        unsigned int texture = 0;

        // load character glyph 
        FT_Load_Glyph(face, FT_Get_Char_Index(face, txt[i]), FT_LOAD_RENDER);


        // now store character for later use
        Character ch = {
            texture,
            glm::ivec2(face->glyph->bitmap.width, face->glyph->bitmap.rows),
            glm::ivec2(face->glyph->bitmap_left, face->glyph->bitmap_top),
            (unsigned int)face->glyph->advance.x

        };


        // generate texture
        glGenTextures(1, &ch.textureID);
        glBindTexture(GL_TEXTURE_2D, ch.textureID);

        glTexImage2D(
            GL_TEXTURE_2D,
            0,
            GL_RED,
            face->glyph->bitmap.width,
            face->glyph->bitmap.rows,
            0,
            GL_RED,
            GL_UNSIGNED_BYTE,
            face->glyph->bitmap.buffer
        );
        // set texture options
        glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, GL_CLAMP_TO_EDGE);
        glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, GL_CLAMP_TO_EDGE);
        glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_LINEAR);
        glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_LINEAR);
        // now store character for later use
       


        float xpos = pos.x + ch.Bearing.x * scale;
        float ypos = pos.y - (ch.Size.y - ch.Bearing.y) * scale;

        float w = ch.Size.x * scale;
        float h = ch.Size.y * scale;
        // update VBO for each character
        float vertices[6][4] = {
            { xpos,     ypos + h,   0.0f, 0.0f },
            { xpos,     ypos,       0.0f, 1.0f },
            { xpos + w, ypos,       1.0f, 1.0f },

            { xpos,     ypos + h,   0.0f, 0.0f },
            { xpos + w, ypos,       1.0f, 1.0f },
            { xpos + w, ypos + h,   1.0f, 0.0f }
        };
        // render glyph texture over quad
        // update content of VBO memory
        glBindBuffer(GL_ARRAY_BUFFER, vbo);
        glBufferSubData(GL_ARRAY_BUFFER, 0, sizeof(vertices), vertices);
        glBindBuffer(GL_ARRAY_BUFFER, 0);

        // render quad
        glDrawArrays(GL_TRIANGLES, 0, 6);



        // now advance cursors for next glyph (note that advance is number of 1/64 pixels)
        pos.x += ((ch.Advance >> 6) * scale); // bitshift by 6 to get value in pixels (2^6 = 64)
    }



    free(txt);
    txt = nullptr;
    glBindVertexArray(0);
    glBindTexture(GL_TEXTURE_2D, 0);
    setDisable();   //シェーダーを無効にする
}

Comments

[退会済みユーザー]

> wchar_t* txt = (wchar_t*)malloc(strlen(text));

正気ですか？

[退会済みユーザー]

なぜでしょうか？

[退会済みユーザー]

「malloc()」や「strlen()」の動きを理解してますか？
「wchar_t」の大きさは理解してますか？

仮に「テスト」というシフトJISのマルチバイト文字列があったとして。
1. 上記「テスト」を対象とした「strlen()」の結果
2. 「wchar_t」のサイズ
3. 「テスト」をワイド文字列として格納する際、必要なサイズ
4. 「malloc()」で実際確保されているサイズ
上記を説明できますか？

また、「テスト」ではなく「Test」の場合も合わせて説明していただければと思います。

Answer 1

該当処理は間違えています。
前回の質問のベストアンサーにある通り直せば動くでしょう。

    int i, j, f;
    for (i = 0, j = 0; text[j]; i++, j += f)
    {
        f = mbrtowc(txt + i, &text[j], MB_CUR_MAX, nullptr);
    }
///////////////////////////////////
    txt[j] = L'\0';
    // ↑ 間違えている
//////////////////////////////////

前回のベストアンサーであれば変換後文字数分の「i」を指定しており、
その場合は、変換後文字数の後にヌル文字により終端が行われることになります。

上記コードは、なぜか「j」となっており、変換前のバイト数をしていしております。
マルチバイト文字からの変換であれば、恐らく文字数とは一致してないでしょう。

デバッガで確認すればわかるものですが、変換後文字列の末尾とは異なる箇所に、
ヌル文字を代入しているため、範囲外を参照し、それが四角となって表示されているのでしょう。

Comments

[退会済みユーザー]

質問ですが txt の配列のその時入力したときの長さに自動で調整させたいのですがどうすればいいのでしょうか？

[退会済みユーザー]

> 質問ですが txt の配列のその時入力したときの長さに自動で調整させたいのですがどうすればいいのでしょうか？

思いつかないのであれば、まだ実装できる実力はないと思うので、固定長バッファで対応するほうがいいと思います。

[退会済みユーザー]

何故ここを前回のベストアンサーの内容から変えたのでしょうか。

というかデバッグしてますか？
「最後以上まで？」じゃなくて、デバッガで見れば一発でしょうに。

Answer 2

そもそもmbrtowc()を使うのならwchar_tの配列もforループ2回も不要です。

    for (int j = 0; text[j]; j += f)
    {
        wchar_t unicode_char;
        f = mbrtowc(&unicode_char, &text[j], MB_CUR_MAX, nullptr);

        unsigned int texture = 0;

        // load character glyph 
        FT_Load_Glyph(face, FT_Get_Char_Index(face, unicode_char), FT_LOAD_RENDER);