3 - Bishoe01 #16

bishoe01 · 2025-04-29T23:29:42Z

🔗 문제 링크

1267. Count Servers that Communicate

✔️ 소요된 시간

50분

✨ 수도 코드

BFS풀이

통신되는 컴퓨터가 몇개인지 찾아라! 문제라서 BFS를 적용했습니다. 문제에서 티를 많이 내고 있기도 했구요.
connectCnt 하나만 있는 컴퓨터는 작용을 못하기 때문에 하나는 카운팅해주긴하는데 BFS돌면서 이게 1보다 커졌을때(즉, 통신되는 다른 컴퓨터 친구를 만났을때 이제 그 값이 실제 정답카운트 answer에 추가가 되는 구조입니다.
처음에는 그냥 붙어있는거만 되는지알았는데, 다 풀고나서 보니까 같은 행/열 이면 연결이 되는거라더구요. 그래서 그부분을 들어내고 행/열 각각에 대한 반복문을 돌려줬습니다.

인접한 컴퓨터가 아니다 ?!

같은 행과 열을 구하라 하더라구요.. 문제를 제대로 읽어야겠습니다.

Two servers are said to communicate if they are on the same row or on the same column.

 while pointer < queue.count {
                    let target = queue[pointer]
                    let currentX = target.0, currentY = target.1
                    print(target.0, target.1)
                    for direction in 0 ..< 4 {
                        let nextX = currentX + dx[direction], nextY = currentY + dy[direction]
                        if isRange(x: nextX, y: nextY) {
                            queue.append((nextX, nextY))
                            visitedGraph[nextX][nextY] = true
                            connectCnt += 1
                        }
                    }

                    pointer += 1
                }

이렇게 그냥 일반적인 DFS 인접 찾기에서 ,, 아래처럼 변경해줬습니다.

while pointer < queue.count {
                    let tmp = queue[pointer]
                    let currentX = tmp.0, currentY = tmp.1

//                     굴러갈 수 있는지 확인   / 2차 풀이 : 사실상 이부분만 고쳐주면 될 것같은데, 행 - 열이라

//                    행 계산
                    for row in 0 ..< grid.count {
                        if isRange(x: row, y: currentY) {
                            queue.append((row, currentY))
                            visitedGraph[row][currentY] = true
                            connectCnt += 1
                        }
                    }
//                    열 계산
                    for col in 0 ..< grid[0].count {
                        if isRange(x: currentX, y: col) {
                            queue.append((currentX, col))
                            visitedGraph[currentX][col] = true
                            connectCnt += 1
                        }
                    }

                    pointer += 1
                }

Index out of range

return x >= 0 && y >= 0 && x < grid.count && y < grid[0].count && grid[x][y] == 1 && visitedGraph[x][y] == false

를 비롯해서
for row in 0 ..< grid[0].count , for col in 0 ..< grid.count 였다가 계속 틀려서 보니까 반대였네요..
for row in 0 ..< grid.count , for col in 0 ..< grid[0].count
급하게 풀어서 맨날 틀리고 바꾸고 하다보니 저 나름대로의 기준이 덜잡혔나봅니다.. 이후엔 틀리기전에 바꾸도록 노력해보겠습니닷..

개선

아마 행 과 열에 있는 컴퓨터 찾는 로직이 저렇게 대충 처리해주는게 아니라 더 좋은 방식이 있을 것 같은데, 한 번 리뷰해주시면 좋을 것 가타요..! 세션가기전에 푼거라 저도 찾아보고 공유드리도록 하겠습니다..!

MuchanKim

핀이 문제를 제대로 읽으라는 힌트를 줬기때문에 어렵지않게 풀 수 있었습니다.
(문제 그림만 보면 당연히 인접한 컴퓨터만 서버 연결 된 것처럼 보임;;)

입력된 grid의 행, 렬에 위치한 서버 개수만 세주면 됩니다.
왜냐하면 '인접'한 서버가 아니기에 같은 위치에 2개 이상 있으면 걔네는 연결된거니까요.

(필기하면서 했는데 너무 지저분해서 시간 들여 여기에 써봅니다.)

일단 아래 grid를 예시로 볼게요.

행을 기준으로 봐볼게여

[1]--[1]  [ ]  [ ]   (행에 서버 2개: 통신 가능)
[ ]  [ ]  [1]  [ ]   (행에 서버 1개: X)
[ ]  [ ]  [1]  [ ]   (행에 서버 1개: X)
[ ]  [ ]  [ ]  [1]   (행에 서버 1개: X)

각 서버가 위치한 행에 다른 서버가 1개만 있으면 그 서버는 연결된 서버겠죠??

열을 기준으로도 한번 봐보입시더.

[1]  [1]  [ ]  [ ]
[ ]  [ ]  [1]  [ ]
           |
[ ]  [ ]  [1]  [ ]
[ ]  [ ]  [ ]  [1]

3(컴터기준2)열만 서버 2개고 얘네만 연결 가능합니다.

이로써 알 수 있는 사실

각 서버가 위치한 행에 다른 서버가 자신포함 2개만 있으면 그 서버는 연결된 서버다.
각 서버가 위치한 열에 다른 서버가 자신포함 2개만 있으면 그 서버는 연결된 서버다.
= 각 서버에 행or열에 서버가 자신포함 2개 이상이면 그 서버는 연결된 서버다.

그리고 우린 연결된 서버를 세어주면 됩니다.

무의코드

전 행 서버 개수, 열 서버 개수를 따로 선언해서 했는데 더 나은 방법이 있을지 궁금하네요

class Solution {
    func countServers(_ grid: [[Int]]) -> Int {
        let rows = grid.count
        let cols = grid[0].count
        
        var rowServer = Array(repeating: 0, count: rows)
        var colServer = Array(repeating: 0, count: cols)
        

        // 각 행, 열 서버 수 채워넣기
        for i in 0..<rows {
            for j in 0..<cols {
                if grid[i][j] == 1 {
                    rowServer[i] += 1
                    colServer[j] += 1
                }
            }
        }
        
        // 순회하면서 서버가 있는 경우, 행 or 열의 카운트가 2 이상인지 체크하고 맞으면 +1
        var res = 0
        for i in 0..<rows {
            for j in 0..<cols {
                if grid[i][j] == 1 && (rowServer[i] > 1 || colServer[j] > 1) {
                    res += 1
                }
            }
        }
        return res
    }
}

sep037 · 2025-05-07T01:28:28Z

저도 같은 행과 열에 1대 이상 있어야 통신이 가능하다는 걸 인지했는데
모르겠어서 이거 참고했어요

저기 링크 안에서는 C++ 코드로 작성되어 있는데 스위프트로 바꿔보았습니다
근데 무랑 완전 똑같아요

class Solution {
    func countServers(_ grid: [[Int]]) -> Int {
    let n = grid.count
    let m = grid[0].count
    
    var answer = 0
    var rows = [Int](repeating: 0, count: n)
    var cols = [Int](repeating: 0, count: m)
    
    // 각 행과 열의 서버 개수를 센다
    for i in 0..<n {
        for j in 0..<m {
            if grid[i][j] == 1 {
                rows[i] += 1
                cols[j] += 1
            }
        }
    }
    
    // 통신 가능한 서버만 카운트한다
    for i in 0..<n {
        for j in 0..<m {
            if grid[i][j] == 1 && (rows[i] > 1 || cols[j] > 1) {
                answer += 1
            }
        }
    }
    
    return answer
}
}

YooGyeongMo

캬 ~ finn 쌤의 BFS 코드를 보고 큐한번 구현해보고싶다는 욕구가 막 솟아나네요 !!! 무릎을 탁 치고갑니다 !

YooGyeongMo · 2025-05-07T12:45:25Z

bishoe01/BFS_DFS/Count Servers that Communicate.swift

+//                    열 계싼
+                    for col in 0 ..< grid[0].count {


ㅋㅋ 열 계싼 ~ finn 코드 컨트리뷰터 주시나용 ? ㅎㅎ

@YooGyeongMo zㅋㅋㅋㅋㅋㅋ아니 다른분들 코드보다가 현웃터졌네요 ㅋㅋㅋ아

굉장히 부끄럽네요

YooGyeongMo · 2025-05-07T12:46:38Z

bishoe01/BFS_DFS/Count Servers that Communicate.swift

+                while pointer < queue.count {
+                    let tmp = queue[pointer]
+                    let currentX = tmp.0, currentY = tmp.1
+
+//                     굴러갈 수 있는지 확인   / 2차 풀이 : 사실상 이부분만 고쳐주면 될 것같은데, 행 - 열이라
+
+//                    행 계산
+                    for row in 0 ..< grid.count {
+//                        print("next:", nextX, nextY)
+                        if isRange(x: row, y: currentY) {
+//                            print("Hi")
+                            queue.append((row, currentY))
+                            visitedGraph[row][currentY] = true
+                            connectCnt += 1
+//                            print("ROW", row, currentY)
+                        }
+                    }


로그 찍어보시는게 정말 대단한거같습니다! finn !!

게다가 직접 큐를 구현 하셨군요. 엄청 힘드셨을텐데..tmp.0 tmp.1로 접근하셨네요 !!!!

예전 BFS/DFS할때 킨더선생님이 pointer로 큐가 되는 것처럼하시길래 계속 잘 써먹고 있습니다 ㅎㅎ..

헉 왠지 뭔가 익숙하다 싶었는데 그때 영감을 받아서 잘쓰고 계시는군요!

YooGyeongMo · 2025-05-07T12:48:45Z

bishoe01/BFS_DFS/Count Servers that Communicate.swift

+                        if isRange(x: currentX, y: col) {
+//                            print("Hi")
+                            queue.append((currentX, col))
+                            visitedGraph[currentX][col] = true
+                            connectCnt += 1
+//                            print("COL", currentX, col)
+                        }
+                    }
+
+                    pointer += 1
+                }


같은 하이로 로그 찍으면 헷갈리시진 않으셨나요? ㅋㅋ

저의 코드는 그냥 일반 배열 여러개 생성하고 카운트하는것이었어요 사실 처음에 보자마자 BFS로 해야겠다라고 고민없이 고 하려다가

저는 오히려 영어 문제가 더 조건이 확실한거같아서 좋더라구요 !!!

여기 보시는것처럼 딱 or로 주니 저는 그냥 배열에 카운트해도 괜춘하지않을까 ? 250250이어서 조건부로 카운팅하면 완탐이 아니라 O(MN) 으로할 수 있지않을까생각했던거같아요 !

class Solution { func countServers(_ grid: [[Int]]) -> Int { let m = grid.count let n = grid[0].count var rowCount = Array(repeating: 0, count: m) var colCount = Array(repeating: 0, count: n) for i in 0..<m { for j in 0..<n{ if grid[i][j] == 1 { rowCount[i] += 1 colCount[j] += 1 } } } var result = 0 for i in 0..<m { for j in 0..<n { if grid[i][j] == 1 && (rowCount[i] > 1 || colCount[j] > 1) { result += 1 } } } return result } }

로 풀었습니다 !!!

성능도 실제로 더 좋았습니닷 !!

영어 문제가 더 조건이 확실하게 느껴지시는군요..

저는 바~로 페이지 번역기 돌렸읍니다.........
.....

@giljihun 길ㅋㅋ 사실 저도 영어 겁나 못합니다ㅋㅋㅋ 뭐랄까 or? and ? 논리연산자가 좀 더 직관적인거같더라구요 !!

이야 데미안쌤은 항상 runTime상위에 랭크되시네용.. 리스펙

같은 하이로 로그 찍으면 헷갈리시진 않으셨나요? ㅋㅋ

A : 다 주석되어있어서 그래보이실 수 있지만,, 항상 한쪽은 주석일거라 괜찮습니다..ㅋㅋ 사실 더 명확하게 로그찍어주는게 더 이상적이긴합니다.. 다음번엔 그렇게 해볼게용.

영어 문제가 더 조건이 확실하게 느껴지시는군요..

A : 저도 예시케이스보고 잘 모르겠다싶으면 바로 번역기로..

giljihun

~~리트 코드는 항상 두렵네요.~~

giljihun · 2025-05-07T13:32:11Z

bishoe01/BFS_DFS/Count Servers that Communicate.swift

+        func isRange(x: Int, y: Int) -> Bool {
+            return x >= 0 && y >= 0 && x < grid.count && y < grid[0].count && grid[x][y] == 1 && visitedGraph[x][y] == false
+        }


isRange()를 보고서 가독성 측면에서 조금 리팩토링하면 좋지않을까? 생각이 들었어요!

현재 isRange()는 범위 체크, 서버 유효성?( == 1 + 방문 여부)를 함께 처리하고 있는걸로 보여요.
의미는 명확한데, 이게 뭉쳐있어서 가독성이 조금 떨어질 수 있다? 생각이 들어요!

func isValidServer(x: Int, y: Int) -> Bool { guard (0..<grid.count).contains(x), (0..<grid[0].count).contains(y) else { return false } return grid[x][y] == 1 && !visitedGraph[x][y] }

guard를 활용해보죠!! 범위 체크를 먼저해서 main 조건에 더 집중해보는 겁니다.

함수명도 조금 더 메인 기능에 집중해서 바꿔봤습니다.

그 본 코드에 적용해도 조금만 바꾸면 문제 없을 것 같아요 아마도..?!!?!

코드는 다른 분들과 유사하여 생략합니다!

추가적으로 리트코드가 마냥 불친절하진 않네요
뭔가 프로그래머스 상위호환 같아요 ㅋㅋ

우와 길선생님 리뷰는 항상 퀄리티가 참 좋다고 느낍니다..
예전에 처음 DFS/BFS배울때는 isRange가 많은 것을 수행하지않았었는데,
문제를 풀수록 더 깔끔하게 압축하고 싶은 욕심에 어느샌가 이름만 isRange지 다 해주고 있었네요..
guard써주신 야무진 함수 한 수 배워갑니다..!

프로그래머스 상위호환맞는 것 같아요..! 일단 입력받을때의 불편함이 없어서 참 좋습니당 ( 결제하면 디버깅을 비롯한 추가적인 기능도 있는 듯해요)

alstjr7437

이번주차 바빠서 문제를 아직 못풀어봣네요 ㅠㅠ
리뷰는 다른 분들이 너무 잘달아주셔서 몇개 코멘트 추가밖에 없었어요 ㅠ
다음부터 잘하도록 하겠습니다!

alstjr7437 · 2025-05-09T04:50:35Z

bishoe01/BFS_DFS/Count Servers that Communicate.swift

+                while pointer < queue.count {
+                    let tmp = queue[pointer]
+                    let currentX = tmp.0, currentY = tmp.1
+
+//                     굴러갈 수 있는지 확인   / 2차 풀이 : 사실상 이부분만 고쳐주면 될 것같은데, 행 - 열이라
+
+//                    행 계산
+                    for row in 0 ..< grid.count {
+//                        print("next:", nextX, nextY)
+                        if isRange(x: row, y: currentY) {
+//                            print("Hi")
+                            queue.append((row, currentY))
+                            visitedGraph[row][currentY] = true
+                            connectCnt += 1
+//                            print("ROW", row, currentY)
+                        }
+                    }


헉 왠지 뭔가 익숙하다 싶었는데 그때 영감을 받아서 잘쓰고 계시는군요!

solve : leetcode 1267

7db3c66

bishoe01 added the 리뷰 기다리는 중 🔥 label Apr 29, 2025

bishoe01 self-assigned this Apr 29, 2025

github-actions bot requested review from MuchanKim, YooGyeongMo, alstjr7437, giljihun and sep037 April 29, 2025 23:49

MuchanKim approved these changes May 4, 2025

View reviewed changes

sep037 added the bishoe01 label May 4, 2025

YooGyeongMo approved these changes May 7, 2025

View reviewed changes

giljihun approved these changes May 7, 2025

View reviewed changes

alstjr7437 approved these changes May 9, 2025

View reviewed changes

sep037 approved these changes May 9, 2025

View reviewed changes

sep037 added 리뷰 완료 ✔️ and removed 리뷰 기다리는 중 🔥 labels May 9, 2025

bishoe01 merged commit de77f9c into main May 9, 2025
6 checks passed

alstjr7437 deleted the 3-Bishoe01 branch May 23, 2025 06:13

3 - Bishoe01 #16

3 - Bishoe01 #16

Uh oh!

Conversation

bishoe01 commented Apr 29, 2025

🔗 문제 링크

✔️ 소요된 시간

✨ 수도 코드

인접한 컴퓨터가 아니다 ?!

Index out of range

개선

Uh oh!

MuchanKim left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

무의코드

Uh oh!

sep037 commented May 7, 2025

Uh oh!

YooGyeongMo left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

YooGyeongMo May 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

같은 하이로 로그 찍으면 헷갈리시진 않으셨나요? ㅋㅋ

영어 문제가 더 조건이 확실하게 느껴지시는군요..

Uh oh!

giljihun left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

alstjr7437 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

MuchanKim left a comment •

edited

Loading

YooGyeongMo left a comment •

edited

Loading

YooGyeongMo May 7, 2025 •

edited

Loading