hits_refactoring

[jichanan]

Related Issue

• 9

  
  

Motivation 🧐

• 조회수 관련 코드 가독성 향상
  

Key Changes 🗝

• [board.py] 69 \~ 84 : 함수 95 \~ 108 : 조회수 코드 본문

To Reviewers 🙏

-

Screenshot

-

Reference

-

[jichanan]

title, context, userid = cur.execute("SELECT title, context, userid FROM board where id=%s", [id]).fetchone()

'int' object has no attribute 'fetchone' 오류 발생

cur.execute(f'INSERT INTO hit_count (userid, board_no, hit_time) VALUES ({session_id}, {id}, {hit_time})')

이거 진짜 f-string으로 하면 보기 참 좋을 것 같아서 해보니깐 알 수 없는 오류들이..

1번째 오류: (1054, "Unknown column "dkswlcks77" in 'where clause'") ← 희안하게 {session_id}만 따옴표로 감싸주면 정상작동함 cur.execute(f'UPDATE hit_count SET hit_time = {current_time} WHERE userid = "{session_id}" AND board_no = {id}') ←이런식으로 <참고사항> print(sessionid) // 'dkswlcks77'_ print(type(session_id)) // str print(id) // '257' print(type(id)) // str hit_count 테이블의 userid와 board_no 둘 다 varchar %s 포맷팅은 정상작동

2번째 오류: cur.execute(f'UPDATE hit_count SET hit_time = {current_time} WHERE userid = {session_id} AND board_no = {id}')

+ f-string은 SQL Injection에 취약(?)

함수활용(코드에 이름 붙이기)

누나가 알려준대로 기능단위로 쪼개니깐 코드를 해석할 필요없이 쉽게 뭔 코드인지 빠르게 알 수 있어서 좋은 것 같아! 혹시 뭐 잘못 쪼갰거나 네이밍이 별로라거나 보완할 점 있으면 알려줘!

- 복습 (참고문헌:혼공파) 함수를 만들면 코드에 이름을 붙일 수 있습니다. 그리고 코드에 이름을 붙이면 쉽게 읽을 수 있습니다. 다음 코드를 보면 주석이 없어도 쉽게 코드를 이해 할 수 있습니다.(물론 area와 circumference라는 단어의 의미를 모르면 힘들겠지만요) 그런데 코드가 길어져서 오히려 복잡해졌다고 생각할수도 있는데, 함수 부분은 생략하고 코드 본문 부분만 살펴보면 주석이 전혀없어도 코드만 읽고 무엇을 하는 코드인지 쉽게 알 수 있습니다. 그러므로 한줄의 코드라도 의미를 가지고 있다면 함수로 만드는 것이 좋습니다.

[SooDevv]

지찬아~! 누나가 review해준거를 고칠 때, 이렇게 따로 PR 날릴 필요 없이 이전 PR에서 계속 작업해도돼!

PR 흐름을 정리하자면,

1. 1차로 작업한거 PR 날리기
2. review 받기
3. review 받은거에서 수정할 부분이 있으면 수정하고 commit & push (push하면 해당 PR에서 변경한 부분을 Commit단위로 볼 수 있음)
4. reviewer 가 approved 하면 PR merge 하기