AkiraNetwork / AkiraNetwork/VirtualStorageLibrary

ノードをストレージに追加するAdd系メソッドにおいて内部でクローンする処理について

Open
#122 0 comments 0 reactions 1 assignee Claimed by @shimodateakira View on GitHub
enhancement
Dominant language
C#
Stars
20
Forks
0
PR merge metrics
No merged PRs in 30d

Description

一般的なコレクションクラスのAddメソッドは内部的で勝手にクローンはしない。
現在のこのライブラリのAddメソッドは追加対象のノードがストレージに追加済みのノードだったら自動的にクローンをして、それを追加し、追加したものを復帰値として返すというメソッド仕様となっている。
このメソッド仕様は一般的なAddメソッドとはかけ離れている為、対応が必要ではないか?

対応案

1. 呼出元でクローンをし、メソッド仕様としては一般的なコレクションクラスのAddと同じようにシンプルに追加だけにする。
2. メソッド仕様は今までと同じにして、内部的にクローンしている事が分かるようにメソッド名を変更する。

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.