ch 배열의 값을 확인해서 더 이상 반복문이 돌지 않도록 하는 것도 유효한가요?

private static int solution(int v, int cost) {
    int answer = 0;
    Queue<Edge> pq = new PriorityQueue<>();
    pq.offer(new Edge(v, cost));

    while (!pq.isEmpty()) {
       if (successTree()) { // 모든 노드가 트리에 추가되었을 때는 반복문을 더이상 돌지않는다.
          break;
       }

       Edge cur = pq.poll();

       if (ch[cur.v]) { 
          continue;
       }

       ch[cur.v] = true;
       answer += cur.cost;

       for (Edge edge : graph.get(cur.v)) {
          if (!ch[edge.v]) {
             pq.offer(edge);
          }
       }
    }

    return answer;
}

private static boolean successTree() {
    for (boolean check : ch) {
       if (!check) {
          return false; // 아직 연결 안 된 노드가 있다면 false를 return
       }
    }
    return true; // 이미 트리가 다 완성되었다면 true를 return
}

이렇게 pq를 도는 반복문에 successTree라는 메소드를 추가하여 모든 노드가(=도시) 트리에 포함되었다면 바로 true를 return 시켜 pq에 대한 반복문이 돌지 않도록 수정했습니다.

이게 코드 수행 시간에 더 효율적인 방법인지 강사님의 의견이 궁금하여 질문해봅니다!

저는 마지막 노드까지 트에 추가되었어도 pq에 남아있는 값들이 있어 의미없는 로직을 돌 것이라고 생각해 추가하게 되었습니다만 ch 배열을 탐색하는 데에 걸리는 시간이 더 걸려 해당 로직을 수행하는 것이 더 비효율적인지 궁금합니다.

또한, 이미 노드가 다 들어가있다면 그 뒤에 pq가 반복할 일이 없어 해당 메소드가 의미가 있는지 없는지 궁금합니다. 이전에 pq에 남아있는 노드들이 poll()될 수 있다고 생각했거든요..//

항상 강의 잘 보고있습니다. 감사합니다.