[C언어in] 아 진짜 제가 겨우 푼 예제좀 봐주세요 ㅠㅠ

Chapter 09 <함수란 무엇인가>

예제 6
두 정수를 입력받아 덧셈을 하는 함수 Add()와 연산을 몇 번 수행했는지 세는 Count() 함수를 작성하세요.
단, 총 연산은 100번으로 제한하며 두 정수 모두 0을 입력하면 종료합니다. 결과를 다음과 같이 출력하는 프로그램을 작성하세요. 정적 변수를 이용하세요.

제가 짠 코드....
#include <stdio.h>
void Add();
int Count();

int main()
{
        Add();

        return 0;
}

void Add()
{
        int x,y,temp,result;
        static int repeat=0;

        while(repeat<2)
        {
                printf("숫자 두 개를 입력하세요(0 0 -> exit) : ");
                scanf_s("%d %d",&x, &y);

                if((x==0)&&(y==0))
                        break;

                else
                {
                        result=x+y;
                        repeat=Count();
                        printf("덧셈 결과 : %d n",result);
                }
        }

        if((x==0)&&(y==0))
                printf("총 연산 수 : %d n",repeat);

        else if(repeat<2)
                Add();

        else
                printf("제한 횟수 초과로 인해 프로그램을 종료합니다. n");
}

int Count()
{
        static int repeat=0;

        repeat++;
        return repeat;
}

일단, 잘 돌아가는데요....

아......문제는 없는것 같은데...

너무.. 조잡하게 짠 것 같아서 ....

잘 짠건지 잘 모르겠어요...

보시기에 어떠세요? 괜찮은 것 같으신지...

ps 제가 맨날 오류도 안 났는데 '제가 짠 코드 괜찮나요?' 라고 여쭤보는게
    독학중인데.. 짜긴 짰는데 더 간결하고 잘 짤 수 있을 것 같은데(배운 범위 내에서도)
    그냥 우걱우걱 어거지로 짠 것 같은 코드라고 제 개인적으로 느낄때가 잦아서...
    알고리즘? 이라고 하나요 그런게 전혀 훈련이 안 되있다고 하나 해서..
    이렇게 예제 하나 풀고 올리고 하네여 ㅠㅠ;

잡담 | 1644명이 읽었어요. 18.222.225.11

비공개 손님 … 2015-02-22 19:24:24

이 책에 모범답안이 전혀 없어서 더 갈피를 못 잡는것도 있는것 같아요..

비공개 손님 … 2015-02-22 19:32:45

문제의 의도는 함수 입력변수로 두 숫자를 넣고 함수 내부에서는 덧셈이랑 카운트만 하는 형태를 원하는것 같은데

비공개 손님 … 2015-02-22 19:33:27

아 저건 명세서에 있는거구나;; 근데 지적할게 너무 많아서 어디부터 지적해야될지 모르겠어요;; add함수에서 다시 add함수를 호출할 필요는 전혀 없는거같음;

비공개 손님 … 2015-02-22 19:34:04

4// 그니까 Add에 인자 2개 넣고 Add안에 덧셈이랑 카운트 하는 형태 말씀하시는건가여?

제가 난독이 있나 문제 볼때마다 너무 애매해서 ㅠㅠㅠㅠ

비공개 손님 … 2015-02-22 19:35:00

5// 아...그런가여........

휴.... 창의력이 딸리는건가 어떤식으로 해야 할지를 모르겠어여

비공개 손님 … 2015-02-22 19:35:31

반복문 하라는 말씀이신가요?

비공개 손님 … 2015-02-22 19:38:20

지문을 잘 이해를 못해서 저렇게 짠 건가 어ㅏㅣㅁㄴㅇ러ㅣㄴㅁㅇ러ㅣㅁㄴㅇ러ㅣㅏㄴ

문제 자체가 명확히 이해가 잘 안되여

비공개 손님 … 2015-02-22 19:40:30

9// sudo코드 네이버에 검색해봤는데 명확한 정보가 잘 안나와서...
혹시 어떤건지 여쭤봐도 되나요?

비공개 손님 … 2015-02-22 19:50:26

12///아아 무슨 말씀이신지 잘 알겠어요

감사합니다 도움 많이 됐어요

비공개 손님 … 2015-02-22 20:00:44

pseudo code

비공개 손님 … 2015-02-22 20:03:56

아니다 pseudocode

비공개 손님 … 2015-02-22 20:14:53

sudo code....

비공개 손님 … 2015-02-22 21:35:28

역시 여긴 쪼렙은 오면 안되..

비공개 손님 … 2015-02-22 22:00:49

sudo marry awesomesauce ydk928 --forced

비공개 손님 … 2015-02-23 17:29:32

#include <stdio.h>

void Add(int, int);
void Count(void);

const int COUNT_LIMIT = 100; //최대 연산수
static int count = 0; //연산이 행해진 횟수

int main(void){
int i=0, k=0; //사용자가 입력한 두개의 정수를 보관하는 그릇

while(true){
printf("input two decimal value : ");
scanf("%d %d", &i, &k);
if(i==0 && k==0){
printf("\n\n==Program End==\n\n");
break;
}else{
if(count > COUNT_LIMIT){
printf("\n\nIt calculate more than limit\n\n");
break;
}else{
Add(i, k);
}
}
}
}

void Add(int x, int y){
Count();
printf("%d + %d = %d // now in %03dth caculate \n", x, y, x+y, count);
return;
}

void Count(){
count += 1;
return;
}

비공개 손님 … 2015-02-23 17:32:55

들여쓰기가 다 없어져버려서.. 보기가 그렇네요 일단 이코드두 모법적인 코드가 아니에요. 저두 알고리즘 등신이라.. ㅎㅎㅎㅎ 같이 짱구 굴리면서 답을 찾아보아요.

비공개 손님 … 2015-02-23 19:14:35

저 다시 짜봤는데 이거 어떤가요?

#include <stdio.h>
int Count();
int Add(int a,int b);
int main()
{
const int LIMIT=100;
int a,b;

while(1)
{
printf("숫자 두 개를 입력하세요(0 0->exit) : ");
scanf_s("%d %d",&a,&b);

if((a==0)&&(b==0))
break;

printf("덧셈 결과 : %d \n",Add(a,b));

if(Count()==LIMIT)
{
printf("%d번 제한으로 인해 프로그램이 종료됩니다. \n",LIMIT);
break;
}
}

if(Count()-1<LIMIT)
printf("총 연산 수 : %d \n",Count()-2);

return 0;
}

int Count()
{
static int repeat=0;

return ++repeat;
}

int Add(int a,int b)
{
return a+b;
}

비공개 손님 … 2015-02-23 19:20:11

21// 님은 상수,정전변수를 전역으로 설정하셧네여..

특별한 이유가 있으셔서 저렇게 하신건가요?

비공개 손님 … 2015-02-23 19:31:31

21// 글구 님

if(count > COUNT_LIMIT){
printf("\n\nIt calculate more than limit\n\n");
break;

이부분에 count 에 +2 해주시거나 +1>=이렇게 하셔야 될것 같아여..

if(count+2 > COUNT_LIMIT){
printf("\n\nIt calculate more than limit\n\n");
break;

혹은

if(count+1 >= COUNT_LIMIT){
printf("\n\nIt calculate more than limit\n\n");
break;

이런 식으루 ㅠㅠ;;

그렇게 안하면 LIMIT보다 2번 더 함 (예를들어 리밋이 1인데 3번까지 연산함)

비공개 손님 … 2015-02-24 10:40:13

#include <stdio.h>
void Add(int, int);
void Count(void);

const int LIMIT = 3; //최대 연산수
static int count = 0; //연산이 행해진 횟수

int main(void){
int i=0, k=0; //사용자가 입력한 두개의 정수를 보관하는 그릇

while(count < LIMIT){
printf("input two decimal value : ");
scanf("%d %d", &i, &k);
if( (i==0 && k==0) ) break;
Add(i, k);
}
return 0;
}

void Add(int x, int y){
Count();
printf("%d + %d = %d // now in %03dth caculate \n", x, y, x+y, count);
}

void Count(){
count += 1;
}

말씀해주신 버그 수정 및 좀더 메인함수를 간결하게 손봤어욤

비공개 손님 … 2015-02-24 10:45:04

static변수랑 상수를 메인바깥(전역변수) 로 쓴것은.. 제 개인적인 기준일뿐이긴 한데요. 보통 static변수나 상수는 프로그램 전체에서 참조해야하는 일이 많은 경우 또는 작동에 치명적인 영향을 주는 변수들을 많이 선언하기때문에, 바깥으로 빼서 따로 관리해요. 이렇게 해두면 나중에 순서도나 프로그램 명세서 만들때 프로그램 윗단만 보고 작성하면 되어서욤.

비공개 손님 … 2015-02-24 16:08:55

새로 올려주신 소스도 지금 쭉 읽어보았는데욤. 명세표 대로 잘 작동이 되기땜에 잘 만드셨어욤. 제 개인적으론 1.정해진 기간내에 명세표에 기재된대로 작동하는가? 가 제일 중요하고, 2.유지관리하기에 적합한 구조인가? 를 보는데... 1번은 잘 만족하는거 같은데 2번에선 약간 문제가 되지 않을까 싶어욤.
그리고 함수 이름은 대문자로 시작하는것 보단 소문자로 변경하시는게 일반적인 관습이에욤. 그리고 함수명을 좀더 설명적으로 바꾸는게 좋을것 같아요. Add 는 sum 정도로 바꾸면 의미가 더 정확해지는것 같고, Count는 set_count_increase() get_current_count() 등으로 분할해서 카운트를 증가시키는 경우와 현재의 카운트수만 가져오는 경우로 분할해주면 좀더 명확할것 같아욤.

비공개 손님 … 2015-02-24 16:12:44

27번에 덧붙임 // 여기서 LIMIT 와 count 는 전체 함수(프로그램 전반에 걸쳐) 사용할 변수이므로 전역으로 뺀거에요. 전역변수를 너무 과도하게 설정하는건 유지보수 측면에서 나빠진다고 보통 말하고 있기때문에, 꼭 필요한게 아니면 전역변수는 줄이는게 좋겠지요.

비공개 손님 … 2015-02-27 23:36:51

27,28,29// 아.. 감사합니다 ㅠㅠ 제가 실전으로 써 본적이 없어서 어떤식으로 유지보수하는지를 전혀 모르거든여 ㅠㅠ;;

감사합니당

댓글을 작성하실 수 없습니다.
(권한이 없는 회원레벨)

목록으로

△ 이전글: [스압] OAuth 2.0으로 가입/로그인절차 작성하는데 질문이 있습니다. [5]

▽ 다음글: 코드공유는 이걸로

TOP arrow_upward