0

ファイルパスをデータベースに保存したとします。これらのファイル パスをツリービューで表示したいと思います。正常に動作するサンプルを 1 つ見つけましたが、データベースに巨大なデータがいつ存在するかわかりません。ツリービューの作成がハングするか、時間がかかりすぎます。ここで私はコードを与えています。データベースに巨大なデータがある場合にパフォーマンスが向上するため、どれを修正できるかを確認して教えてください。

public static class MyDataBase
{
    private static List<string> fields = new List<string>();

    public static void AddField(string field)
    {
        fields.Add(field);
    }

    public static IList<string> FieldsInMyColumn()
    {
        return fields;
    }
}

public void CreateTreeView()
    {
        foreach (string field in MyDataBase.FieldsInMyColumn())
        {
            string[] elements = field.Split('\\');
            TreeNode parentNode = null;

            for (int i = 0; i < elements.Length - 1; ++i)
            {
                if (parentNode == null)
                {
                    bool exits = false;
                    foreach (TreeNode node in myTreeview.Nodes)
                    {
                        if (node.Text == elements[i])
                        {
                            exits = true;
                            parentNode = node;
                        }
                    }

                    if (!exits)
                    {
                        TreeNode childNode = new TreeNode(elements[i]);
                        myTreeview.Nodes.Add(childNode);
                        parentNode = childNode;
                    }
                }
                else
                {
                    bool exits = false;
                    foreach (TreeNode node in parentNode.Nodes)
                    {
                        if (node.Text == elements[i])
                        {
                            exits = true;
                            parentNode = node;
                        }
                    }

                    if (!exits)
                    {
                        TreeNode childNode = new TreeNode(elements[i]);
                        parentNode.Nodes.Add(childNode);
                        parentNode = childNode;
                    }
                }
            }

            if (parentNode != null)
            {
                parentNode.Nodes.Add(elements[elements.Length - 1]);
            }
        }
    }

 private void button1_Click(object sender, EventArgs e)
    {
        MyDataBase.AddField(@"c:\jsmith\project1\hello.cs");
        MyDataBase.AddField(@"c:\jsmith\project1\what.cs");
        MyDataBase.AddField(@"c:\jsmith\project2\hello.cs");
        MyDataBase.AddField(@"c:\jsmith\project1\tdp.cs");
        MyDataBase.AddField(@"c:\jsmith\project2\ship.cs");
        MyDataBase.AddField(@"d:\jsmith\project1\hello404.cs");
        MyDataBase.AddField(@"c:\jsmith1\project2\ship.cs");
        CreateTreeView();
    }

ありがとう

4

2 に答える 2

1

改善するために提案できる最初のことは、foreach の代わりに while を使用することです。

両方の場所でこれの代わりに：

bool exits = false;
foreach (TreeNode node in myTreeview.Nodes)
{
        if (node.Text == elements[i])
        {
                exits = true;
                parentNode = node;
        }
}

あなたが使用することができます

bool exits = false;
int j = 0;
while (!exits && j<myTreeview.Nodes.Count)
{
        if (myTreeview.Nodes[j].Text == elements[i])
        {
               exits = true;
               parentNode = node;
        }
        j++;
}

この方法では、Nodes コレクション全体を反復する必要がなく、親ノードが見つかった直後にループが終了します。もちろん、ノードというタイトルの要素[i]がない場合は、コレクション全体を調べます。

PS私はあなたがexists代わりに意味したと思いますexits

于 2012-11-06T14:44:58.087 に答える
1

フレームワークのバージョンによっては、次のようなことを試すことができます。

public void ProcessPath(IEnumerable<String> path,  TreeNodeCollection nodes)
{
    if (!path.Any())
        return;
    var node = nodes.Cast<TreeNode>().FirstOrDefault(n => n.Text == path.First());
    if (node == null)
    {
        node = new TreeNode(text: path.First());
        nodes.Add(node);
    }
    ProcessPath(path.Skip(1),node.ChildNodes);
}

public void CreateTreeView()
{
    foreach (string field in MyDataBase.FieldsInMyColumn())
        ProcessPath(field.Split('\\'),myTreeView.Nodes);
}

本当に大量の行がある場合は、ノードをクリックしたときにのみ子ノードをロードするソリューションを探す必要があります

于 2012-11-06T15:44:09.060 に答える