연속 부 수열 합의 개수(js, Lv.2)

https://school.programmers.co.kr/learn/courses/30/lessons/131701

프로그래머스

 

문제 설명

철호는 수열을 가지고 놀기 좋아합니다. 어느 날 철호는 어떤 자연수로 이루어진 원형 수열의 연속하는 부분 수열의 합으로 만들 수 있는 수가 모두 몇 가지인지 알아보고 싶어졌습니다. 원형 수열이란 일반적인 수열에서 처음과 끝이 연결된 형태의 수열을 말합니다. 예를 들어 수열 [7, 9, 1, 1, 4] 로 원형 수열을 만들면 다음과 같습니다.

원형 수열은 처음과 끝이 연결되어 끊기는 부분이 없기 때문에 연속하는 부분 수열도 일반적인 수열보다 많아집니다.
원형 수열의 모든 원소 elements가 순서대로 주어질 때, 원형 수열의 연속 부분 수열 합으로 만들 수 있는 수의 개수를 return 하도록 solution 함수를 완성해주세요.

---

제한사항

• 3 ≤ elements의 길이 ≤ 1,000
• 1 ≤ elements의 원소 ≤ 1,000

---

입출력 예elementsresult

[7,9,1,1,4]

18

---

입출력 예 설명

입출력 예 #1
길이가 1인 연속 부분 수열로부터 [1, 4, 7, 9] 네 가지의 합이 나올 수 있습니다.
길이가 2인 연속 부분 수열로부터 [2, 5, 10, 11, 16] 다섯 가지의 합이 나올 수 있습니다.
길이가 3인 연속 부분 수열로부터 [6, 11, 12, 17, 20] 다섯 가지의 합이 나올 수 있습니다.
길이가 4인 연속 부분 수열로부터 [13, 15, 18, 21] 네 가지의 합이 나올 수 있습니다.
길이가 5인 연속 부분 수열로부터 [22] 한 가지의 합이 나올 수 있습니다.
이들 중 중복되는 값을 제외하면 다음과 같은 18가지의 수들을 얻습니다.
[1, 2, 4, 5, 6, 7, 9, 10, 11, 12, 13, 15, 16, 17, 18, 20, 21, 22]

---

<시도한 코드>

function solution(elements) {
    //연속된 수의 합의 개수
    let arr=0;
    let result = 0;
    let count = 0;
    for(let i=1; i<=elements.length; i++){ // 갯수
        for(let j=0;j<elements.length; j++){// 인덱스 값 지칭
            if(arr == elements[j]){
                continue;
            }
            arr += elements[j]
            let sum = arr; // 일시적 합 저장소
            count++; // 갯수 세기            
            if(count == i){
                if(sum != arr){
                result++;
                }
                arr = elements[j]; 
                count = 0;
            }
        }
    }
    return result 
}

 

문제점 분석:

1. 변수 초기화 문제:
   • let arr=0; 변수는 부분 수열의 합을 저장하는 데 사용되지만, 이를 초기화하고 갱신하는 방식이 올바르지 않습니다.
2. 조건문 논리 오류:
   • if(arr == elements[j]) 조건문은 의미가 없습니다. 이 조건이 참인 경우 현재 요소를 건너뛰게 되는데, 이는 불필요합니다.
3. 부분 수열 합 계산 문제:
   • 부분 수열의 합을 계산하는 방식이 올바르지 않으며, 원형 수열을 고려하지 않습니다.

---

<챗gpt가 수정한 정답 코드>

 

Set 객체의 add와 size 함수를 이용하여 풀었다.

function solution(elements) {
    const circular = elements.concat(elements);
    const set = new Set();
    for (let i = 0; i < elements.length; i++) {
        let sum = 0;
        for (let j = 0; j < elements.length; j++) {
            sum += circular[i + j];
            set.add(sum);
        }
    }
    return set.size;
}