인프런 커뮤니티 질문&답변

gytharj님의 프로필 이미지
gytharj

작성한 질문수

10주완성 C++ 코딩테스트 | 알고리즘 코딩테스트

1-L

1-L 질문입니다.

작성

·

168

0

안녕하세요.

저는 set을 사용해서 문제를 해결했습니다. 그런데 시간을 더 줄여보려고 코드를 수정했는데 틀리다고 나옵니다. 수정한 코드가 왜 잘못되었는지 모르겠습니다. 알려주시면 감사하겠습니다.

 

수정한 코드

http://boj.kr/e812a45ff3e34a8e9a22e97cce7e113f

 

원래 코드

http://boj.kr/039878b27e594e0c90895f1414ab061d

답변 1

0

큰돌님의 프로필 이미지
큰돌
지식공유자

안녕하세요 ㅎㅎ

        if (mat_num == num_found) break;

        auto it_found = mat_nums.find(M-mat_num);
        if (it_found != mat_nums.end())
        {
            cnt++;
            num_found = *it_found;

            //cout << "   found num : " << num_found;
        }

제 생각에 이부분은

정렬되어있는 set에서

1 2 3 4 5 6 7

이렇게 되어있을 때

3부분에서 5를 선택했다면

그 다음 5가 나왔을 때 break를 걸어서 전체 탐색을 막으려고 했던 것 같은데요.

 

 

괜찮은 로직인 거 같지만... 반례는 다음과 같습니다.

10 10 1 2 3 4 5 6 8 10 12 14

 

정답 : 2

gh님 코드 : 5

 

set을 이용하신다면 이렇게 고치시는 것은 어떨까요?

#include <iostream>
#include <string>
#include <set>

using namespace std;

int main() {
    int N, M;
    cin >> N >> M;

    set<int> st;

    for (int i = 0; i < N; i++) {
        int num;
        cin >> num;
        st.insert(num);
    }

    int cnt = 0;

    for (auto mat_num : st) {
        int complement = M - mat_num; 
        if (st.count(mat_num) && st.count(complement)) {
            cnt++;
        }
    } 
    cout << cnt / 2;
    return 0;
}

 

 

또 질문 있으시면 언제든지 질문 부탁드립니다.

좋은 수강평과 별점 5점은 제게 큰 힘이 됩니다. :)

감사합니다.

강사 큰돌 올림.

 

gytharj님의 프로필 이미지
gytharj

작성한 질문수

질문하기