各種オペレータの再設計 or 廃止

[shimodateakira]

各種オペレーターはnewしていない。

[shimodateakira]

operator +などは本来、newしたものに処理を行い、そのインスタンスへの参照を返す。
VirtualDirectoryに対してVirtualNodeを追加するような場合、VirtualDirectoryのインスタンスのディープコピーを行い、それに対して追加処理後、そのインスタンスへの参照を返す。
しかし、VirtualDirectoryのインスタンスのディープコピーは多大なオーバーヘッドを伴うケースもある為、これを考慮する必要がある。
単純に追加するだけならAddメソッドを使えば良いが、演算子オペレーターを使う場合はこのような考慮が必要という事が分かった。
今のところの方針としては、ディープコピーを行う方式に切り換え、オーバーヘッドに関する留意点についてはドキュメントに記載して、ユーザーの判断に任せる形にしたいと思う。
リンク辞書については、従来とパスの仕様が変わるわけじゃないから影響はないはず。

[shimodateakira]

C#14では+=演算子の演算子オーバーロードがサポートされる予定。それで実装すればオーバーヘッドはなくなる。多分、11月にリリースされるから、それまでこの件は保留かな。

[shimodateakira]

// + 単体 VirtualNode
public static VirtualDirectory operator +(VirtualDirectory directory, VirtualNode node)
{
    var cloned = (VirtualDirectory)directory.DeepClone(true);
    cloned.Add(node);
    return cloned;
}

// + 複数 VirtualNode (IEnumerable)
public static VirtualDirectory operator +(VirtualDirectory directory, IEnumerable<VirtualNode> nodes)
{
    var cloned = (VirtualDirectory)directory.DeepClone(true);
    cloned.AddRange(nodes);
    return cloned;
}

// + VirtualNodeName（新規ディレクトリ追加）
public static VirtualDirectory operator +(VirtualDirectory directory, VirtualNodeName nodeName)
{
    var cloned = (VirtualDirectory)directory.DeepClone(true);
    cloned.Add(new VirtualDirectory(nodeName));
    return cloned;
}

// += 単体 VirtualNode
public static void operator +=(VirtualDirectory directory, VirtualNode node)
{
    directory.Add(node);
}

// += 複数 VirtualNode (IEnumerable)
public static void operator +=(VirtualDirectory directory, IEnumerable<VirtualNode> nodes)
{
    directory.AddRange(nodes);
}

// += VirtualNodeName（新規ディレクトリ追加）
public static void operator +=(VirtualDirectory directory, VirtualNodeName nodeName)
{
    directory.Add(new VirtualDirectory(nodeName));
}

// - 単体 VirtualNode
public static VirtualDirectory operator -(VirtualDirectory directory, VirtualNode node)
{
    var cloned = (VirtualDirectory)directory.DeepClone(true);
    if (cloned.NodeExists(node.Name))
    {
        cloned.Remove(node);
    }
    return cloned;
}

// - 複数 VirtualNode (IEnumerable)
public static VirtualDirectory operator -(VirtualDirectory directory, IEnumerable<VirtualNode> nodes)
{
    var cloned = (VirtualDirectory)directory.DeepClone(true);
    foreach (var node in nodes)
    {
        if (cloned.NodeExists(node.Name))
        {
            cloned.Remove(node);
        }
    }
    return cloned;
}

// - VirtualNodeName（名前による削除）
public static VirtualDirectory operator -(VirtualDirectory directory, VirtualNodeName nodeName)
{
    var cloned = (VirtualDirectory)directory.DeepClone(true);
    var node = cloned.Get(nodeName, exceptionEnabled: false);
    if (node != null)
    {
        cloned.Remove(node);
    }
    return cloned;
}

// -= 単体 VirtualNode
public static void operator -=(VirtualDirectory directory, VirtualNode node)
{
    if (directory.NodeExists(node.Name))
    {
        directory.Remove(node);
    }
}

// -= 複数 VirtualNode (IEnumerable)
public static void operator -=(VirtualDirectory directory, IEnumerable<VirtualNode> nodes)
{
    foreach (var node in nodes)
    {
        if (directory.NodeExists(node.Name))
        {
            directory.Remove(node);
        }
    }
}

// -= VirtualNodeName（名前による削除）
public static void operator -=(VirtualDirectory directory, VirtualNodeName nodeName)
{
    var node = directory.Get(nodeName, exceptionEnabled: false);
    if (node != null)
    {
        directory.Remove(node);
    }
}