[feature/modeling] 알고별점

[HyunjinKIM-Chloe]

• 36 작업을 기반으로 함

• 한 카테고리당 100점으로 환산
• 별은 총 5개로 표시 (총 500점 만점)
• 별 표기 시, 반올림하여 정수화 / 우측 실 점수는 소수 1번째 자리까지 표기

[SeyoungKo]

1. 함수 내 함수 선언

   • 함수를 이중으로 선언하는 코드는 제가 처음봐서 그런지 생소하네요 이렇게 쓰신 이유가 무엇인지 궁금합니다! https://github.com/kohyunsil/Algo_camping/blob/9302bed3487dd5adeed498b7c2ce695c4aa9ece6/camping_modeling/algostar/cat_points.py#L28-L45

2. 외부 파일 호출 코드 config.py 등 외부 파일로 별도 분리 필요

   • 해당 클래스에서 수행하는 기능과 관련없는 코드들은 외부 파일로 분리시키는 게 좋을 것 같습니다. https://github.com/kohyunsil/Algo_camping/blob/9302bed3487dd5adeed498b7c2ce695c4aa9ece6/camping_modeling/algostar/cat_points.py#L69-L72

3. 클래스명 파스칼 케이스 사용 ex) Calc_logic -> CalcLogic https://github.com/kohyunsil/Algo_camping/blob/9302bed3487dd5adeed498b7c2ce695c4aa9ece6/camping_modeling/algostar/cat_points.py#L7

4. 중복되는 코드 별도 함수로 분리 https://github.com/kohyunsil/Algo_camping/blob/9302bed3487dd5adeed498b7c2ce695c4aa9ece6/camping_modeling/algostar/cat_points.py#L36-L57

말씀해주신 것보다 생각보다 중복되는 부분이 많이 없는 것 같네요 ! 제 생각이 옳은 답은 아니지만 참고하시고 코드 작성에 도움되셨으면 좋겠습니다.