인프런 영문 브랜드 로고
인프런 영문 브랜드 로고

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

호두님의 프로필 이미지
호두

작성한 질문수

홍정모의 따라하며 배우는 C언어

7.6 소수 판단 예제

교수님 안녕하세요.

해결된 질문

작성

·

158

0

[질문1]

9:05초

    isPrime = true;
    for (div = 2; div < number; ++div)
    {
        if (number % div == 0)
        {
            isPrime = false;
        }
        else
        {
            isPrime = true;
        }
   }

이렇게 else문을 쓰면 안되는 이유가 
1. number에 4를 입력

2. 4 % 2, 4 % 3
4 % 2에서 isPrime이 false가 됐는데 for문은 어쨌든 div가 4보다 작을 때까지 도니까 div가 3인 경우에 isPrime이 true가 돼서 그런건가요?

요렇게 break;를 넣고 else문을 넣어도 되는거죠?

    for (div = 2; div < number; ++div)
    {
        if (number % div == 0)
        {
            isPrime = false;
            break;
        }
        else
        {
            isPrime = true;
        }
   }

[질문2]

소수 판별을 함수에서 해서 리턴값으로 true, false를 전달해주도록 작성했습니다.


#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>
#include <stdbool.h>

bool isPrime(unsigned number)
{
    unsigned i;
    for (i = 2; i < number; ++i)
    {
        if (number % i == 0)
        {
            return false;
        }
        else
        {
            return true;
        }
    }
}

int main()
{
    unsigned int a = 0;
    scanf("%u", &a);

    bool flag = isPrime(a);
    if (flag)
    {
        printf("%u is a prime number", a);
    }
    else
    {
        printf("%u is not a prime number", a);
    }

    return 0;
}

그런데 visual studio community 2019에서는

1>------ Build started: Project: prime.c, Configuration: Debug x64 ------

1>main.c

1>C:\Users\neuez\source\repos\test\prime.c\main.c(20): warning C4715: 'isPrime': not all control paths return a value

1>Done building project "prime.c.vcxproj".

========== Build: 1 succeeded, 0 failed, 0 up-to-date, 0 skipped ==========

not all control paths return a value라고 워닝이 뜨는데 
모든 패스에 대해 값을 리턴해주도록 작성했다고 생각하는데 

왜 워닝이 뜨는 지 모르겠네요 .

같은 코드를 visual studio code에서 gcc를 이용해서 컴파일 하면 warning이 안뜹니다.

PS C:\coding\lj_test> gcc a.c
PS C:\coding\lj_test> .\a.exe
3
3 is a prime number
PS C:\coding\lj_test> .\a.exe
4
4 is not a prime number
PS C:\coding\lj_test> .\a.exe
22

22 is not a prime number 

감사합니다.

답변 1

1

홍정모님의 프로필 이미지
홍정모
지식공유자

1. 논리구조가 동일하면 break를 넣어도 상관 없습니다. 다만, 다른 사람이 봐도 읽기 좋은 코드를 만들려면 어떻게 해야 하는 지에 대해서는 경험이 더 필요합니다. 코드가 그 자체로 이해가 잘 되는 형태라면 가장 좋지만 차선책으로는 주석을 많이 사용합니다. 강의에서는 말로 설명 드리기 바쁘기 때문에 주석을 많이 사용하지 않았습니다.

2. 컴파일러의 생각(?)을 이해하려고 노력해보세요. for문 끝나고 return을 하나 더 넣어주면 warning은 사라질 것 같네요. 

*인프런 질문에서 답 잘 받으시는 요령에서 1번 참고하세요.

https://blog.naver.com/atelierjpro/221547580916

호두님의 프로필 이미지
호두

작성한 질문수

질문하기