std::shared_ptr<std::vector<std::shared_ptr<XXX>>>のイテレーターを使って構造体の要素を指したい。

Question

二つ目の提示コードですがstd::shared_ptr<std::vector<std::shared_ptr<CharacterString>>>のイテレーターで提示コード二つ目のコメント名部のstd::shared_ptr<CharacterString> it = *itr;の一行がもどかしいのですがこれは一行で書く方法はないのでしょうか？
コメント部のコードのようにしたいです。

        //文字構造体
        struct Character
        {
            unsigned int textureID; //グリフのテクスチャID
            glm::ivec2   size;      //グリフサイズ
            glm::ivec2   bearing;   //グリフのベースライン
            unsigned int advance;   //次のグリフまでのオフセット
            wchar_t character;      //文字
        };
        
        //文字列構造体
        struct CharacterString
        {
            std::vector<Character> text;    //文字列
            glm::vec4 color;                //描画色
            glm::lowp_i16vec2 textSize;     //文字列のサイズ
            unsigned short pixelSize;       //ピクセルサイズ
            std::string fontName;           //フォント名
            FT_Face face;                   //フェイス 
        };

        std::shared_ptr<std::vector<std::shared_ptr<CharacterString>>> characterText;    //描画文字

// ##################################### デストラクタ ##################################### 
FrameWork::Text::~Text()
{
    glBindVertexArray(0);
    glBindBuffer(GL_ARRAY_BUFFER, 0);
    glDeleteVertexArrays(1, &vao);
    glDeleteBuffers(1, &vbo);

    //グリフ解放 
    for (std::vector<std::shared_ptr<CharacterString>>::iterator itr = characterText->begin(); itr != characterText->end(); itr++)
    {
///////////////////////////////////////////////////////////////
       std::shared_ptr<CharacterString> it = *itr;
       FT_Done_Face(*itr->face);
       //FT_Done_Face(it->face);
/////////////////////////////////////////////////////////////
    }


    FT_Done_FreeType(ft);
}

Comments

[actorbug]

> std::shared_ptr<CharacterString> it = *itr;
> FT_Done_Face(*itr->face);
これが現状のコードで
> //FT_Done_Face(it->face);
これが希望のコードということで、本当に間違いないですか?
現状のコードで未使用のitをわざわざ定義している意味が分かりません。

[退会済みユーザー]

ではどうすればいいのでしょうか？どうしないとコンパイルエラーになるので仕方なく書いたのですが？

[actorbug]

現状のコードがこちらで、
> std::shared_ptr<CharacterString> it = *itr;
> FT_Done_Face(it->face);
希望のコードがこちらではないかという指摘です。
> //FT_Done_Face(*itr->face);
これなら、現状のコードが二行で希望のコードが一行なので、意味が分かります。

[退会済みユーザー]

> これなら、現状のコードが二行で希望のコードが一行なので、意味が分かります。
とはどいう意味でしょうか？もう少し詳細によろしいでしょうか？

[actorbug]

分かりづらいようなので、聞き方を変えます。
質問からすると、現状のコードは以下で、これであればコンパイルが通ることになります。
> std::shared_ptr<CharacterString> it = *itr;
> FT_Done_Face(*itr->face);
しかし、このコードをコンパイルできるなら、一行目を消してもコンパイルが通るはずです。
そうなると、「これは一行で書く方法はないのでしょうか？」という質問の答えは、「一行目を消せばよい」になってしまいます。
本当にこのコードでコンパイルが通るのですか。

[int32_t]

https://teratail.com/questions/341915 こちらの質問が参考になるかと思います。

Answer 1

範囲for文でvectorの要素を型推論させればよいかと思います。

for ( auto&& it : characterText ) {
       FT_Done_Face(it->face);
}

Comments

[退会済みユーザー]

確かにautoは便利なのですが初心者なので型についてわかってる方がいいのでちょっとやりたいこととあｈ違いますｗ

[Serbonis]

長い名前の同じ型をなんども記述したくないのであればエイリアスを利用するという方法もあります。
using sp_CS = std::shared_ptr<CharacterString>;

[退会済みユーザー]

それも利用したない場合は自分の提示コードのようなやり方しか方法はないのでしょうか？

[Serbonis]

ちょっと質問の意図が見えないですが、長い記述や冗長な記述を回避したいというわけではないのですね。型の情報も意識したいのであれば、範囲for文でautoの型推論を使わずに型を明示的に記述されたらどうでしょうか？

for ( std::shared_ptr<CharacterString> it : characterText ) {
FT_Done_Face(it->face);
}

[退会済みユーザー]

>
for (std::shared_ptr<CharacterString> it : characterText) {
FT_Done_Face(it->face);
}
ですがエラーになるのですがそもそも出来ないのでしょうか？

[Serbonis]

characterText → *characterText ですね

Answer 2

修正依頼を理解してもらうまで時間がかかりそうなので、先にこちらの想定する回答を出しておきます。

これで解決しませんか

FT_Done_Face((*itr)->face);