이 코드, 더 효율적으로 코딩이 가능할까요?

Question

안녕하세요 오늘 주사위 문제 질문을 올렸었는데요방금 막 실행가능한 프로그램 만들기를 끝냈습니다. 혹시 더 효율적으로 수정할 만한 부분이 보인다면, 알려주시는 대로 최대한 수정하겠습니다[문제]1에서부터 6까지의 눈을 가진 3개의 주사위를 던져서 다음과 같은 규칙에 따라 상금을 받는 게임이 있다.같은 눈이 3개가 나오면 10,000원+(같은 눈)×1,000원의 상금을 받게 된다.같은 눈이 2개만 나오는 경우에는 1,000원+(같은 눈)×100원의 상금을 받게 된다.모두 다른 눈이 나오는 경우에는 (그 중 가장 큰 눈)×100원의 상금을 받게 된다.#include #include #include int main(){ int i; int data[3]; char a; srand((int)time(NULL)); for(i=0; i<3; i++){ data[i]=rand()%6+1; } for(i=0; i<3; i++){ printf("%d ", data[i]); } int temp=0;//임시변수 설정 for (int i=0; i<2; i++)//버블 정렬 { for(int j=0;j<2-i;j++) { if (data[j]>data[j+1]) { temp = data[j]; data[j] = data[j+1]; data[j+1] =temp; } } } switch ((data[0] == data[1]) + (data[1] == data[2])) { case 0 : printf(" %d", data[2] * 100); break; case 1 : printf(" %d", 1000 + data[1] * 100); break; case 2 : printf(" %d", 10000 + data[1] * 1000); break; } }https://okky.kr/questions/1488619?topic=questions&page=1 : 이전 코드

kenu,전독시 · Accepted Answer

통과 못하셨나요?과도한 최적화는 over egineering 이라고 알고 있어서, 그리고, acmicpc도 커뮤니티 공간이 있는데, 여기에 자꾸 올리시면 ... #말잊못https://www.acmicpc.net/board/search/all/problem/2480

옥다방고양이 · Answer

✅️개념, 기능이 이해되었다면, 거기에 너무 집착하지 마시고, 빨리 다음 레벨로 진도 빼시기 바랍니다.✅️윗분의 말씀대로, 실무에서는 오버엔지니어링은 오히려 독이 되는 경우가 있습니다.✅️실무에서는 가독성도 굉장히 중요한 요소입니다.왜냐하면, 실무에서는 혼자서 전부다 만드는게 아니거든요. 협업해야 하거든요.

Throwaway · Answer

이 문제에만 국한되서 푸는 거라면 그냥 쓰여져있는데로 하면 되겠는데요?maxSameDieCount = 0rollToCount = 0memo = map[int]intFor i = 0; i < 3; i++:memo[data[i]] += 1maxRoll = max(maxRoll, data[i])maxSameDieCount = max(maxSameDie, memo[data[i]])if maxSameDieCount is 3:return 10000+maxRoll*1000if maxSameDieCount is 2:return 1000+maxRoll*100if maxSameDieCount is 1:return maxRoll * 100

mirheeoj · Answer

랜덤으로 생성한 숫자를 그즉시 찍으면 되지 따로 또 루프를 돌 이유가 없습니다. 가장 큰 눈 역시 숫자 생성 단계에서 바로 검사해 저장하면 되고 따로 정렬을 할 이유가 없습니다.같은 게 몇 번 나왔는지도 숫자 생성단계에서 바로 체크하시면 됩니다. 상금도 그 때 업뎃하시면 되고요.

이 코드, 더 효율적으로 코딩이 가능할까요?

4개의 답변

기술 목록