[SeongA] WEEK 15 solutions

[delight010]

답안 제출 문제

• Subtree of Another Tree #229
• Construct Binary Tree From Preorder And Inorder Traversal #253

작성자 체크 리스트

• Projects의 오른쪽 버튼(▼)을 눌러 확장한 뒤, Week를 현재 주차로 설정해주세요.
• 문제를 모두 푸시면 프로젝트에서 Status를 In Review로 설정해주세요.
• 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

검토자 체크 리스트

Important

본인 답안 제출 뿐만 아니라 다른 분 PR 하나 이상을 반드시 검토를 해주셔야 합니다!

• 바로 이전에 올라온 PR에 본인을 코드 리뷰어로 추가해주세요.
• 본인이 검토해야하는 PR의 답안 코드에 피드백을 주세요.
• 토요일 전까지 PR을 병합할 수 있도록 승인해주세요.

[yhkee0404]

마지막까지 깔끔하게 잘 푸셨네요! 그동안 수고 많으셨어요!

[yhkee0404]

알고달레 풀이 1의 index처럼 for 반복문이 시간 병목이 됩니다. 선형 시간으로 줄일 수는 있던데 좀 어렵더라고요!

Suggested change

	// Time O(N)
	// Time O(N^2)

[yhkee0404]

preorder.count와 inorder.count가 같음을 보장하려면 이렇게 줄일 수 있어요:

Suggested change

	rootNode.left = buildTree(Array(preorder[1..<rootIndex + 1]), Array(inorder[0..<rootIndex + 1]))
	rootNode.left = buildTree(Array(preorder[1..<rootIndex + 1]), Array(inorder[0..<rootIndex]))

[yhkee0404]

마찬가지로 알고달레 풀이 1에서처럼 Slice가 시간과 공간 병목이 될 수 있어요. 그래서 Slice 대신 Two Pointers를 사용하거나 또는 Swift에서는 다음과 같이 Array 형변환을 피하도록 고치면 선형 공간으로는 최적화할 수 있습니다:

Suggested change

	// Space O(N)
	func buildTree(_ preorder: [Int], _ inorder: [Int]) -> TreeNode? {
	if preorder.isEmpty {
	return nil
	}
	let totalCount = preorder.count
	let rootNode = TreeNode(preorder[0])
	var rootIndex = 0
	for (index, value) in inorder.enumerated() {
	if value == rootNode.val {
	rootIndex = index
	break
	}
	}
	rootNode.left = buildTree(Array(preorder[1..<rootIndex + 1]), Array(inorder[0..<rootIndex + 1]))
	rootNode.right = buildTree(Array(preorder[1 + rootIndex..<totalCount]), Array(inorder[1 + rootIndex..<totalCount]))
	// Space O(N)
	func buildTree(_ preorder: [Int], _ inorder: [Int]) -> TreeNode? {
	return buildTree(preorder[...], inorder[...])
	}
	func buildTree(_ preorder: ArraySlice<Int>, _ inorder: ArraySlice<Int>) -> TreeNode? {
	if preorder.isEmpty {
	return nil
	}
	let rootNode = TreeNode(preorder.first!)
	var rootIndex = 0
	for (index, value) in inorder.enumerated() {
	if value == rootNode.val {
	rootIndex = index
	break
	}
	}
	rootNode.left = buildTree(preorder[preorder.startIndex + 1..<preorder.startIndex + rootIndex + 1], inorder[inorder.startIndex..<inorder.startIndex + rootIndex])
	rootNode.right = buildTree(preorder[preorder.startIndex + 1 + rootIndex..<preorder.endIndex], inorder[inorder.startIndex + 1 + rootIndex..<inorder.endIndex])

[delight010]

오! 이건 생각지도 못한 부분이네요 조금 더 최적화할 수 있는 방법을 배워갑니다

[yhkee0404]

Suggested change

	// Space O(N)
	// Space O(M + N)

[yhkee0404]

사소한 간소화입니다 ㅎㅎ

Suggested change

	if let root = root, let subRoot = subRoot {
	if root.val == subRoot.val {
	return dfs(root.left, subRoot: subRoot.left) && dfs(root.right, subRoot: subRoot.right) || isSubtree(root.left, subRoot) || isSubtree(root.right, subRoot)
	}
	return isSubtree(root.left, subRoot) || isSubtree(root.right, subRoot)
	}
	return false
	}
	private func dfs(_ root: TreeNode?, subRoot: TreeNode?) -> Bool {
	if root == nil && subRoot == nil {
	return true
	}
	if let root = root, let subRoot = subRoot {
	if root.val == subRoot.val {
	return dfs(root.left, subRoot: subRoot.left) && dfs(root.right, subRoot: subRoot.right)
	}
	}
	return false
	if let root = root, let subRoot = subRoot {
	return dfs(root, subRoot) || isSubtree(root.left, subRoot) || isSubtree(root.right, subRoot)
	}
	return root == nil && subRoot == nil
	}
	private func dfs(_ root: TreeNode?, _ subRoot: TreeNode?) -> Bool {
	if let root = root, let subRoot = subRoot {
	return root.val == subRoot.val && dfs(root.left, subRoot.left) && dfs(root.right, subRoot.right)
	}
	return root == nil && subRoot == nil

[delight010]

짧으면서도 이해하기 쉬운 코드 덕분에 공부가 되었습니다.