1) 문제
코딩테스트를 준비하는 머쓱이는 프로그래머스에서 문제를 풀고 나중에 다시 코드를 보면서 공부하려고 작성한 코드를 컴퓨터 바탕화면에 아무 위치에나 저장해 둡니다. 저장한 코드가 많아지면서 머쓱이는 본인의 컴퓨터 바탕화면이 너무 지저분하다고 생각했습니다. 프로그래머스에서 작성했던 코드는 그 문제에 가서 다시 볼 수 있기 때문에 저장해 둔 파일들을 전부 삭제하기로 했습니다.
컴퓨터 바탕화면은 각 칸이 정사각형인 격자판입니다. 이때 컴퓨터 바탕화면의 상태를 나타낸 문자열 배열 wallpaper가 주어집니다. 파일들은 바탕화면의 격자칸에 위치하고 바탕화면의 격자점들은 바탕화면의 가장 왼쪽 위를 (0, 0)으로 시작해 (세로 좌표, 가로 좌표)로 표현합니다. 빈칸은 ".", 파일이 있는 칸은 "#"의 값을 가집니다. 드래그를 하면 파일들을 선택할 수 있고, 선택된 파일들을 삭제할 수 있습니다. 머쓱이는 최소한의 이동거리를 갖는 한 번의 드래그로 모든 파일을 선택해서 한 번에 지우려고 하며 드래그로 파일들을 선택하는 방법은 다음과 같습니다.
-
드래그는 바탕화면의 격자점 S(
lux,luy)를 마우스 왼쪽 버튼으로 클릭한 상태로 격자점 E(rdx,rdy)로 이동한 뒤 마우스 왼쪽 버튼을 떼는 행동입니다. 이때, "점 S에서 점 E로 드래그한다"고 표현하고 점 S와 점 E를 각각 드래그의 시작점, 끝점이라고 표현합니다. -
점 S(
lux,luy)에서 점 E(rdx,rdy)로 드래그를 할 때, "드래그 한 거리"는 |rdx-lux| + |rdy-luy|로 정의합니다. -
점 S에서 점 E로 드래그를 하면 바탕화면에서 두 격자점을 각각 왼쪽 위, 오른쪽 아래로 하는 직사각형 내부에 있는 모든 파일이 선택됩니다.
예를 들어 wallpaper = [".#...", "..#..", "...#."]인 바탕화면을 그림으로 나타내면 다음과 같습니다.
이러한 바탕화면에서 다음 그림과 같이 S(0, 1)에서 E(3, 4)로 드래그하면 세 개의 파일이 모두 선택되므로 드래그 한 거리 (3 - 0) + (4 - 1) = 6을 최솟값으로 모든 파일을 선택 가능합니다.
(0, 0)에서 (3, 5)로 드래그해도 모든 파일을 선택할 수 있지만 이때 드래그 한 거리는 (3 - 0) + (5 - 0) = 8이고 이전의 방법보다 거리가 늘어납니다.
머쓱이의 컴퓨터 바탕화면의 상태를 나타내는 문자열 배열 wallpaper가 매개변수로 주어질 때 바탕화면의 파일들을 한 번에 삭제하기 위해 최소한의 이동거리를 갖는 드래그의 시작점과 끝점을 담은 정수 배열을 return하는 solution 함수를 작성해 주세요. 드래그의 시작점이 (lux, luy), 끝점이 (rdx, rdy)라면 정수 배열 [lux, luy, rdx, rdy]를 return하면 됩니다.
2) 문제 풀이
func solution(_ wallpaper:[String]) -> [Int] {
var maxY = 0
var minY = wallpaper.count + 1
var minX = wallpaper.first?.count ?? 0 + 1
var maxX = 0
for (index, target) in wallpaper.enumerated() where target.contains("#") {
if let targetIndex = target.firstIndex(of: "#") {
let lux = target.distance(from: target.startIndex, to: targetIndex)
minX = min(minX, lux)
}
if let targetIndex = target.lastIndex(of: "#") {
let rdx = target.distance(from: target.startIndex, to: targetIndex) + 1
maxX = max(maxX, rdx)
}
minY = min(minY, index)
maxY = max(maxY, index + 1)
}
return [minY, minX, maxY, maxX]
}결과
3) 코드 개선
🚨문제점 분석
1. 초기값 설정의 논리적 오류
minY = wallpaper.count + 1,minX = wallpaper.first?.count ?? 0 + 1minY는 충분히 큰 값이어야 함 →Int.max사용minX는wallpaper.first가 nil일 경우 1이 되어 예상한 값이 나오지 않을 수도 있음 →minX = (wallpaper.first?.count ?? 0) + 1
2. 첫 번째 #만 검사해서 minX를 구함
#이 여러 개 있을 수 있는데, 첫 번째#의 위치만 사용함- 루프 중간에 더 왼쪽에 위치한
#이 있을 수 있기 때문에, 정확한 최소 X값을 구하기 위해 모든#의 인덱스를 구해야 함
3. 전체 루프를 반복하면서도 줄마다 여러 번 문자열 탐색
firstIndex(of:),lastIndex(of:)를 매 루프마다 수행함- 줄 전체를 한 번 스캔하면서 동시에
minX,maxX를 구하는 방식으로 변경
func solution(_ wallpaper: [String]) -> [Int] {
var minY = Int.max
var maxY = Int.min
var minX = Int.max
var maxX = Int.min
for (y, row) in wallpaper.enumerated() {
for (x, char) in row.enumerated() {
if char == "#" {
minY = min(minY, y)
maxY = max(maxY, y + 1)
minX = min(minX, x)
maxX = max(maxX, x + 1)
}
}
}
return [minY, minX, maxY, maxX]
}결과
