0504-DailyAlgorithm

Author Avatar
John Doe May 04, 2018

3Sum

숫자 리스트를 받아서 3개의 합이 0인 조합들을 출력
오늘은 youtube를 보다가.. 이 문제를 풀어보기로 했다.
3sum

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
function threeNum(array){
//함수를 완성하세요
const minusList = []
const plusList = []
const zeroList = []

for(item of array){
if(item>0) plusList.push(item)
else if(item<0) minusList.push(item)
else zeroList.push(item)
}
// 2개짜리 set를 어떻게 만들지?

// 생각해보면 2개짜리는 딱히 ... for-for 두번 돌리면 됬는데 ...

const minusSet = minusList.reduce((ac,c,ci)=> {
return ac.concat(minusList.slice(ci+1).map(v=>[c,v]))
},[])
const plusSet = plusList.reduce((ac,c,ci)=> {
return ac.concat(plusList.slice(ci+1).map(v=>[c,v]))
},[])

// set이랑 이제 합을 돌리면서 0이면 push
const mResult = minusSet.reduce((ac,c)=> {
const sum = c.reduce((acc,cc)=>acc+cc)
return ac.concat(plusList.filter(v=>sum+v===0).map(v=>c.concat(v)))
},[])

const pResult = plusSet.reduce((ac,c)=> {
const sum = c.reduce((acc,cc)=>acc+cc)
return ac.concat(minusList.filter(v=>sum+v===0).map(v=>c.concat(v)))
},[])
let symmetryResult = [];
if(zeroList.length!==0){
const result =plusList.reduce((ac,c)=> {
return ac.concat(minusList.filter(v=>c+v===0).map(v=>[c].concat(v,0)))
},[])
symmetryResult = symmetryResult.concat(result)
}
console.log(symmetryResult)
const resultList = symmetryResult.concat(pResult,mResult)
return resultList;
}

console.log(threeNum([-3,1,2,4,-2,-2,0]))
// 중복 로직이 엄청 많다. 다 함수로 만들어서 재사용성을 할 수 있는 부분들인데 ...

잘한점 일지는 모르겠는데
이 문제의 특수한 경우 -2개랑 plus하나 더하거나 plus2개 더하고 -하나 더하는 두 가지 경우로만 나눠서 계산
-,-,- , +,+, +인 것을 피핟고자 이렇게 하긴 했는데 과연… 더 효율적이였나 의문이 들고 있다. 대규모라면 좀 더 낫긴 하겠지
for-for-for 돌면서 a,b,c 합을 구하는 과정 -> 어지간하면 직관적인 후자를 택하자 나누고 나누고 복잡해보이는 쪽 보다는
또 시간 분배… 알고리즘의 2시간 쏟음 …

메소드 체이닝이랑 concat등 한 꺼번에 처리하려고 하다 보니 디버깅이 오래걸렸다. 사소한 실수인데… 차근 차근 하는 습관을 나중에
동작 안 되는 것이 다 섞여 있는 것에서 찾으려 하니 디버깅이 상당히 오래걸렸다.
우선 주로 for문으로 먼저 짜고 정리할 떄 주로 쓰자… 처음부터 쓰면 뭔가 정신이 없을 떄가 많음 for of를 쓰면 array Method쓰는 것과 큰 차이도 별로 없음 :D