embedにおけるnamesって何？

[ikngtty]

https://github.com/Huyu2239/ExpandBot/blob/f28edd6de2954a6bbb2a4a37cff8b61103f80353/lib/embed.py#L4-L11
https://github.com/Huyu2239/ExpandBot/blob/f28edd6de2954a6bbb2a4a37cff8b61103f80353/lib/embed.py#L23
https://github.com/Huyu2239/ExpandBot/blob/f28edd6de2954a6bbb2a4a37cff8b61103f80353/lib/embed.py#L71-L87

この names が何を示してるのか"説明"してみて、それを元に適切な変数名考えてみて。
https://github.com/Huyu2239/ExpandBot/discussions/53#discussioncomment-829467
https://github.com/Huyu2239/ExpandBot/discussions/41#discussion-3397834
https://xn--97-273ae6a4irb6e2hsoiozc2g4b8082p.com/%E3%82%A8%E3%83%83%E3%82%BB%E3%82%A4/%E5%90%8D%E5%89%8D%E9%87%8D%E8%A6%81/

[Huyu2239]

embedの作成関数を一つにするため、各種設定によってばらばらになる要素をまとめてdictにし、それをembed作成関数に渡します

なので、embed_values, embed_variableですかね

[Huyu2239]

@ikngtty
ダメです？

[ikngtty]

よさそう。
俺なら embed_params とかにするかも。

[aiotter]

names は namedtuple か Dataclass か何かとして関数外で定義すべきだと思う。
どんなフィールドを持つオブジェクトなのかが compose_embed の中だけで決まってて、他の関数がそれを参照するというのは、 compose_embed が特別視されていて不自然。
compose_embed も他の関数も一律に、別所で定義された names のクラスを参照するというのが自然。

[Huyu2239]

クラス使います
命名はEmbedParamsで