maker0927
170
2020-09-24 13:42:54
2
208

간단한 코드 리뷰 부탁드립니다.


형편없는 실력이지만 간단한 이차방정식 실근판별 코드를 짜보았습니다. 부족한점 마음껏 까주시면 감사하겠습니다. 개발언어는 python3 입니다. 

import math

print("<이차방정식 실근 판별기> \n----------------------")

print("이차항의 계수:")
quad_order = int(input())

print("일차항의 계수:")
fst_order = int(input())

print("상수항의 계수:")
constant = int(input())


def print_result(num):
    print("이 방정식은 총 " + str(num) + "개의 실근이 존재합니다.")


def judge(quad, fst, con):
    if quad == 0:
        print_result(1)

    statement = pow(fst, 2) - 4 * quad * con

    if statement > 0:
        print_result(2)
    elif statement == 0:
        print_result(1)
    else:
        print_result(0)


judge(quad_order, fst_order, constant)


0
  • 답변 2

  • 한번사는인생
    108
    2020-09-24 14:03:13

    간단한 코드이지만, 한번 리뷰를 남겨봅니다.


    일단 저는 C#, .Net을 하는 개발자이니, 파이썬에 대한 기초지식이 없는 점 양해 부탁드립니다.


    judge 라는 함수에서 print_result함수를 호출하는 하는 모습이 계속 보입니다.

    결과 값에 대해서 result라는 변수에 설정 후, return result; 하고 난 다음

    pirint_result 함수를 호출하는 것이 바람직해보입니다.

    judge 함수는 결과값을 가져오는 함수이지, 결과를 프린트하는 함수는 아니니깐요.


    그리고 result라는 변수에 초기 값을 0으로 설정하면, 마지막 else문이 필요없을거 같네요.

    항상 변수를 설정할때 초기값을 설정해주면 좋습니다.

  • maker0927
    170
    2020-09-24 14:26:28
    많은 도움이 되었습니다. 감사합니다 :)
  • 로그인을 하시면 답변 을 등록할 수 있습니다.