프로그래밍

3206,

4/161

yj6393

[c언어]십진수를 이진수로 변환해주는 프로그램

http://www.hackerschool.org/HS_Boards/zboard.php?id=QNA_programming&no=3506 [복사]

int two(int num)
{
        int tw[8];
        int i;

        for(i=0;i<=8;i++)
        {
                if(num%2==1)
                {
                        tw[i]=1;
                        num/=2;
                }
                else
                {
                        tw[i]=0;
                        num/=2;
                }
        }
        for(;i>=0;i--)
                printf("%d",tw[i]);
        return 0;
}

int main()
{
        int num;

        printf("숫자 입력: ");
        scanf("%d",&num);
        two(num);
        return 0;
}
여기서 문제점을 알켜주세요.
글고 만약 코딩을 더 짧게하는 방법이 있으면 갈켜주세요

Hit : 7804 Date : 2012/10/11 05:24


tlsdmq12	개인적으로 num /= 2 를 if 문의 밖으로 빼는 것이 좋다고 생각합니다. 공통된 부분을 저렇게 각각 쓰게 되면,,,, 나중에 프로그램의 덩치가 커져서 수정할 일이 생겼을때, 수정을 하였는데도 결과가 원하는 결과가 안 나올때, 저런 것이 이유가 될 수 있습니다. 그리고 int형의 매개변수를 받는데,,,, int형은 십진수 9자리, 2진수 32자리의 숫자를 입력 받을 수 있는데 8칸의 배열로는 전부다 표현하기에는 굉장히 작을 수 밖에 없습니다. 혹여 8칸만 필요한 것이라면 매개변수의 크기를 판단해주는 에러 처리를 해주던가, 조금 space를 많이 차지하더라도 32칸으로 만들어 주는 것이 좋습니다. 뭐, 꼭 int형 배열에 넣어서 표시하는 것은 좋은 것인지는 잘 모르겠지만, 프로그램의 자체 기능을 제외하고 부분적으로 추가해주면 좋을 것 같은 부분입니다. 뿐만아니라 출력해주는 부분도,,, 만약 8칸을 다쓰지 않게 되는경우에는 수를 표시한 뒷부분에 0이 계속 들어가게 되고, 제대로 된 결과는 나오지 않을 것입니다. 다른 int 형 변수 하나를 추가하던가,, 해서 입력한 index번호를 저장하고 출력하는 for문은 i를 이용하는 것이 아닌, 그 index번호를 저장하고 있는 int형 변수를 이용하는 것이 좋을 것 같습니다,	2012/10/11
tlsdmq12	아 그리고,,, 꼭 return을 사용하지 않으실 것이라면, 사용자 정의 함수의 리턴은 void로 설정하시는 것이 좋다고 생각하고, 제일 좋다고 생각하는 것은 결과를 int * two(int num) 으로 하여서 배열을 리턴해주고 main에서 출력해주는 것이 좋다고 생각합니다. two() 함수는 그저 2진수를 나타내주는 배열을 구하는 기능만을 하는 것이 좋다는 것이 제 생각입니다.	2012/10/11
jwkzzangs	#include <stdio.h> #include <windows.h> int main(void) { DWORD UserInput = 0; char Buf[0x100] = {0, }; scanf("%d", &UserInput); itoa((int)UserInput, Buf, 2); printf("%s\n", Buf); return 0; } 가장 짧게 하는 방법입니다.	2012/10/12
kumi123	일단, 식이 좀 안맞구요.. 2진수로 변환후에.. i=9 가 되어있는 상태에서 출력을 하시면 배열주소값을 넘어 버려서 주소값 + tw[7]~~~~~tw[0] 으로 뜨거든요.. 그래서 조건문에 if(num==1) { tw[i]=1; break; } 라는 문장을 추가하시면 바로 작동할겁니다	2012/11/24