작성한 코드들이 네이밍 규칙에 맞지 않게 쓰여 있어서 C++ Core Guidelines를 기준으로 수정하였다.
전반적으로 고친 부분들을 정리하면 아래와 같다.
- 함수, 변수, 상수 : snake_case
- 함수는 소문자로 시작, 네이밍에 타입 정보 넣지 않음
- 타입 별칭 : 소문자, 언더스코어 없는 이름
- std 네임스페이스 명시적 사용
(using namespace std를 쓰는 경우 편리하지만, 규모가 커지고 라이브러리가 많아질수록 버그와 유지보수 어려움을 유발하기 때문.
이름 충돌 가능성, 코드의 명확성 저하, 헤더 파일 문제 등 존재)
- 상수는 constexpr 사용.
(constexpr은 컴파일 타입에 값이 반드시 결정되는 상수 만들 때 사용.
컴파일 타임 상수를 명확히 표현해 컴파일러 최적화에 좋음. 또한 컴파일 타임에 계산 가능 값은 미리 연산하여 성능 향상을 기대할 수 있다.)
(++ 매개변수에서 const를 쓰는 이유는 함수 내 해당 인수를 변경하지 않겠다는 의도를 명확히 하여 코드 안정성과 가독성에 도움이 되기 때문이다.)
알고리즘 풀 때처럼 임의로 작성했더니 이런 사태가 되어버렸다..그래도 초반에 고쳐서 다행이고, 앞으로도 언어에 맞는 네이밍 규칙을 잘 지켜서 가독성 좋은 코드 작성에 대해서도 좀 더 고민해봐야겠다.