[김성현] Peer Review #1

[castlehyeon]

views.py line 25 :

details = get_object_or_404(Faq, id = object_id)

• 예외처리를 해주신 부분인 것 같은데, 전혀 생각 하지 못했던 부분입니다. 영채님 덕분에 get_object_or_404() 사용법을 알게 되었습니다. 공식문서에서 본인이 필요한 부분을 선택적으로 쓸 수 있다는게 대단하신 것 같습니다.

views.py line 9-19 :

class NList(ListView):
model = Faq
template_name = 'support/normal.html'

class AList(ListView): model = Faq template_name = 'support/account.html'

class EList(ListView): model = Faq template_name = 'support/etc.html'

-수업시간에 배웠던 클래스기반 뷰를 작성하셨네요! 덕분에 코드가 정말 간결해졌다고 느꼈습니다.

models.py line 38 :

   email_allow = models.BooleanField(verbose_name='이메일수신여부', default=False)

• 이메일 수신여부를 booleanfield타입으로 default값을 false로 준 것을 보고, 실제 모델을 설계하면 이런 식으로 가능하겠구나.라는 것을 알게 되었습니다.

기타

사실 templates에 대한 리뷰얘기를 하고 싶었는데, 4, 5주차 학습내용을 본인 스타일의 코드로 멋지게 잘 만드신 것 같아요. 같이 배우는 입장에서 본 받을 부분이 많은 분이세요! 세심한 코드리뷰도 항상 감사합니다!!!

[ElyaSetinal]

1. 여러개의 글 중에서 선택한 하나의 글을 보기위해 고유값인 id를 Primary Key로 받아 이를 탬플릿에 넘겨보려고 사용해봤습니다. 처음에는 detail = Faq.objects.get(id=id)로 해봤었는데, 뭔가 에러만 나서 바꾸게 되었네요. 좋게 봐주셔서 감사합니다.

2. 전체적으로 주석이 부족해서 코드가 보기 어려우셨을텐데, 세세하게 봐주셔서 감사합니다.