Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RemoveChildNode(this) をいい感じにする #662

Open
wistnki opened this issue Nov 30, 2020 · 3 comments
Open

RemoveChildNode(this) をいい感じにする #662

wistnki opened this issue Nov 30, 2020 · 3 comments
Assignees
Labels
project:C# Engine altseed/Altseed2-csharp

Comments

@wistnki
Copy link
Member

wistnki commented Nov 30, 2020

No description provided.

@wistnki wistnki self-assigned this Nov 30, 2020
@wraikny wraikny added the project:C# Engine altseed/Altseed2-csharp label Feb 2, 2021
@wistnki
Copy link
Member Author

wistnki commented Mar 8, 2021

どうするんだっけ
(this) は往々にして自明なので、RemoveChildNode を自動詞的に使いたいってことだった気がする。
削除的な意味合いの自動詞 remove はなさそうなので、 attach(To) / detach に変更か

@wraikny
Copy link
Member

wraikny commented Mar 9, 2021

@wistnki this.Parent. RemoveChildNode(this) になってしまい、 Parentは自明だよねーという。
this.RemoveFromParent() 的な?
RemoveFromNodeTreeとか
わからんけど

@wistnki
Copy link
Member Author

wistnki commented Mar 23, 2021

よく考えたら

Parentは自明

とはという感じがしてきた。 とはいえ、 Parent.RemoveChildNode(this) ってすることは多い気がするので、Append と AppendTo みたいな糖衣メソッドの一種としてあってもいい気はする。
一方で似た働きをするメソッドがいっぱいあるの初心者に優しくなくない?という気もする。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
project:C# Engine altseed/Altseed2-csharp
Projects
None yet
Development

No branches or pull requests

2 participants