- Notifications
You must be signed in to change notification settings - Fork0
Feat/srp#12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
README.md Outdated
| -RetrievetheversionnumberofthePythonpackage | ||
| -RenderitselfasanHTMLelement | ||
| -파이썬패키지의버전번호를검색 | ||
| -그자체를HTML요소로써렌더링 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
itself의 지칭은VersionCommentElement 클래스가 될 것이나, 자칫하면 원문의 의도를 왜곡할 것 같아그 자체라는 직역안을 반영했습니다.클래스 로 변경하는 게 나아 보이는지 다른 분들의 의견이 궁금합니다. 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
요부분은 좀 더 논의가 필요할 것 같아요!!
뭔가 원문이 무슨 말을 하고 싶은지는 알 것 같은데, 한국어로 표현하자니 뭔가 애매한 그런 느낌...?
한번 다음에 같이 이야기해보아요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
저도 그 자체라는 직역은 처음보는 사람 입장에서 헷갈릴 여지가 있다고 생각해요.
문장의 의도는 bad case의 경우, 2가지 책임이 서로 dependent 하다는 걸 보여주고 싶은 것 같은데,
itself가 클래스의 멤버함수인def get_version(self) -> str: 라는 걸 명확히 하면 좋을 것 같아요.
README.md Outdated
| ### **단일 책임 원칙(Single Responsibility Principle (SRP))** | ||
| RobertC.Martinwrites: | ||
| RobertC.Martin는다음과같이저술한적이있습니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Robert C. Martin 는 다음과 같이 저술하였습니다.
로 축약해 보아도 괜찮을 것 같아욤! 😊
kyc3492 commentedJun 18, 2023
이 PR은 6.14. 야근세션 부로 해결된 PR입니다. 확인 이모지 남겨주시면 제가 병합 하겠습니다~~ |
SRP 부분 번역을 완료하여 PR 올립니다.
진행하면서 의문이 있었던 부분은 코멘트 남기겠습니다.
추가 개선사항이나 의문점이 있다면 자유롭게 코멘트 부탁드립니다! 😁