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

안녕하세요 승호님 ㅎㅎ

잘 짜셨는데요. ㅎㅎ

go라는 함수요 이렇게 해야 되는게 아닐까요? l, r부분을 좀 다듬어 봤습니다.

#include <bits/stdc++.h>
#pragma warning(disable:4996)
using namespace std;
int n, m ,j, rets=0;
int l, r;

void go(int n) {
    if (n <= r && n > l) {
        return;
    }
    if (r < n) {
        rets += n - r;
        r = n;
        l = r - m + 1;
    } else if (l > n) {
        rets += l - n;
        l = n;
        r = l + m - 1;
    }
}

int main()
{
	ios_base::sync_with_stdio(0);
	cin.tie(NULL);cout.tie(NULL);
	cin >> n >> m;
	cin >> j;
	r = m;
	for (int i = 0;i < j;i++) {
		int temp = 0;
		cin >> temp;
		go(temp);
	}
	cout << rets << "\n";
	
}

그리고 n을 전역변수로 받았는데 이를 지역함수로 중복해서 쓰는 것은 피하셔야 합니다. 이 코드 같은 경우는 어차피 매개변수로 넘겨서 하기 때문에 상관없지만 실수를 유발할 수 있습니다. (매개변수로 안넘기거나 그럴때)

이문제 같은 경우는 n이라는 것 말고도 충분히 다른 변수명을 쓸 수 있구요.

이런 작은 차이가 코테에서 맞왜틀에 빠질 가능성을 없게 만듭니다.

void go(int point) {
    if (point <= r && point > l) {
        return;
    }
    if (r < point) {
        rets += point - r;
        r = point;
        l = r - m + 1;
    } else if (l > point) {
        rets += l - point;
        l = point;
        r = l + m - 1;
    }
}

이렇게요 ㅎㅎ

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

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

감사합니다.

강사 큰돌 올림.