2017-05-14 5 views
2

私が書いた以下のC#アルゴリズムは、O(n)時間に無向グラフのサイクルの存在を検出します。再帰を防ぎ、辞書やハッシュセットを使ってハッシングを利用します。しかし、私がもっと良くすることができる方法はありますか?これは、無向グラフのサイクルを検出するための最良のアルゴリズムですか?

void Main() 
{ 
    var graph = new Dictionary<int, HashSet<int>> 
    { 
     { 0, new HashSet<int> { 4 } }, 
     { 1, new HashSet<int> { 2, 3, 4 } }, 
     { 2, new HashSet<int> { 1 } }, 
     { 3, new HashSet<int> { 1, 4 } }, 
     { 4, new HashSet<int> { 0, 1, 3 } } 
    }; 

    Console.WriteLine(HasCycle(graph, 0)); 
} 

bool HasCycle(Dictionary<int, HashSet<int>> graph, int start) 
{ 
    var stack = new Stack<int>(); 
    var visited = new HashSet<int>(); 
    stack.Push(start); 
    var curr = start; 
    var prev = -1; 
    while (stack.Count > 0) 
    { 
     prev = curr; 
     curr = stack.Pop(); 
     visited.Add(curr); 
     HashSet<int> neighbors; 
     if (graph.TryGetValue(curr, out neighbors) && neighbors != null) 
     { 
      foreach (var neighbor in neighbors) 
      { 
       if (!visited.Contains(neighbor)) 
       { 
        stack.Push(neighbor); 
       } 
       else if (neighbor != prev && neighbors.Contains(prev)) 
       { 
        return true; 
       } 
      } 
     } 
    } 
    return false; 
} 
+0

すでに訪問したノードで反復処理を行うと、サイクルが発生します。 – arboreal84

+0

しかし、*無向グラフの場合は、既に訪問したノードが、あなたが来たノードでないことを確認する必要があります。それが私の「else」ステートメントで捕捉しようとするロジックです。 –

答えて

関連する問題