해결된 질문
작성
·
24
·
수정됨
0
안녕하세요!
JWT 구현 중 verify_token 함수의 네이밍이 궁금합니다.
app/core/jwt_handler.py
def verify_token(token: str) -> int:
payload = decode_token(token)
return payload.get("uid")
verify라는 이름은 보통 boolean을 반환할 것 같은데, 실제로는 uid(int)를 반환하고 있어서 조금 헷갈립니다.
get_uid_from_token이나 extract_user_id 같은 이름이 더 명확하지 않을까요?
감사합니다!
답변 2
0
안녕하세요! 세심한 피드백 정말 감사합니다.
말씀해주신 부분이 정확합니다. 원래는 verify_token 함수가 토큰의 유효성 검증만 수행하고 Boolean 값을 반환하도록 구현되어 있었는데, 이후 리팩토링 과정에서 검증과 동시에 user id를 반환하는 형태로변경되면서 함수명도 함께 수정했어야 했습니다. 하지만 당시 네이밍을 그대로 두는 바람에 혼란을 드리게 되었습니다.
get_uid_from_token이나 extract_user_id처럼 실제 반환 값을 명확히 드러내는 네이밍이 훨씬 직관적이라는 점에 깊이 공감합니다. 현실적으로는 강의 전체를 수정하기는 어려운 상황이지만, 말씀해주신 의견은 매우 합당하고 실무적으로도 중요한 지적입니다.
좋은 피드백 주셔서 진심으로 감사드립니다. 이런 세심한 관점이 강의 품질을 개선하는 데 큰 도움이 됩니다.
0
안녕하세요, 인프런 AI 인턴입니다. 고리오영감님께서 JWT 구현 중 verify_token 함수의 네이밍에 대해 궁금증을 가지고 계신 것 같습니다.
코드에서 함수의 이름은 함수의 역할과 반환값을 명확하게 나타내야 다른 개발자들이 코드를 이해하기 쉽게 만듭니다. verify_token이라는 이름은 이름 그대로 토큰의 검증 여부를 나타내는 불리언 값을 반환할 것처럼 보입니다. 하지만 실제로는 토큰에서 사용자 ID를 추출하여 반환하고 있습니다.
고리오영감님께서 제안하신 get_uid_from_token이나 extract_user_id는 함수의 동작을 더 명확하게 설명할 수 있는 이름들입니다. 이러한 이름은 함수가 무엇을 하는지 명확하게 전달하며, 다른 개발자들이 코드를 이해하고 유지보수하는 데 크게 도움이 됩니다.
함수 이름을 변경하는 것은 함수의 사용 목적을 명확히 구별하고, 팀 협업 시 커뮤니케이션 오류를 줄이는 데 긍정적인 영향을 줍니다. 특히, 네이밍을 통해 코드의 의도를 명확히 하는 것은 좋은 프로그래밍 습관 중 하나입니다.
아래 관련된 질문과 토론 링크를 통해 비슷한 사례들을 참고해 보실 수 있습니다:
질문자님의 코딩 경험과 토론 기회를 바탕으로 적절한 네이밍을 통해 코드의 가독성을 높이시길 바랍니다.
저는 질문자님의 이해를 돕고자 유사한 답변 링크를 추천해드리고 있으며, 현재 베타 기능으로 답변이 만족스럽지 않을 수 있음을 양해 부탁드립니다. 🙏 추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.