add TypeId.md #27
Open
add TypeId.md #27
+242
−1
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
anyモジュールのフォルダーにTypeId.mdを追加
TypeId構造体のリファレンスを追加
Member
|
レビューをもう少しお待ちください． |
hinshiba
requested changes
Jan 21, 2026
Member
hinshiba
left a comment
hinshiba
left a comment
There was a problem hiding this comment.
貢献ありがとうございます．
Danger of Improper VarianceとExamplesについては確認お願いします．
他はこだわりがあるなら，そちらでもいいかなと考えています．
| ## 解説 | ||
|
|
||
| 型に対するグローバルに一意な識別子を提供する構造体。 | ||
| 不透明なオブジェクトであるため内部を直接見ることはできないが、cloneやdebugといった基本的な操作は可能である。 |
Member
There was a problem hiding this comment.
不透明というのは日本語として不自然な気がします．
内部が隠蔽されているオブジェクト
とかがより実態に即していると思いますがどうでしょうか?
| ## 解説 | ||
|
|
||
| 型に対するグローバルに一意な識別子を提供する構造体。 | ||
| 不透明なオブジェクトであるため内部を直接見ることはできないが、cloneやdebugといった基本的な操作は可能である。 |
Member
There was a problem hiding this comment.
debugといった...操作というのは不自然な気がします．Debug traitによる表示なのだから操作としては表示です．
トレイトにするならDebug...という基本的なトレイトを実装している．
操作にするなら表示...という基本的な操作は可能
がいいのではないでしょうか?
|
|
||
| 現在、`TypeId`は`'static`制約を満たすもののみ利用できるが、これは将来解除される可能性がある。 | ||
|
|
||
| また、`Hash`や`Ord`、`PartialOrd`を実装しているが、ハッシュ値や順序はRustのリリースごとに違うため、これらに依存した設計は避けるべきである。 |
| 現在、`TypeId`は`'static`制約を満たすもののみ利用できるが、これは将来解除される可能性がある。 | ||
|
|
||
| また、`Hash`や`Ord`、`PartialOrd`を実装しているが、ハッシュ値や順序はRustのリリースごとに違うため、これらに依存した設計は避けるべきである。 | ||
|
|
Member
There was a problem hiding this comment.
Danger of Improper VarianceとExamplesが省略されています．
これは閲覧者が公式には追加の記述があることを一時的にでも書いておくことが親切かと思われます．
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
概要
anyモジュールのフォルダーにTypeId.mdを追加
変更内容
チェックリスト
mdbook buildを実行して確認した