Corrected some typos & added some explanations... #2
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
띄어쓰기나 자바스크립트 실행결과 오류 정정
Update ch1-00.md
Update ch1-01.md
added `{:.1}` for the example of Rust code...
smoothed the explanation...
replaced `{}` to `dict()` in the Python function `fibonacci` & renamed the Rust function `fib` as `fibonacci`, for easy comparison...
added some further explanations about the expression `a..b`
Liberatedwinner
changed the title
steadydoer
reviewed
Feb 22, 2023
Indosaram
reviewed
Feb 22, 2023
src/ch3-03.md
Outdated
| ```python | ||
| def fibonacci(n): | ||
| cache = {} | ||
| cache = dict() |
Owner
There was a problem hiding this comment.
{}가 더 많이 사용되는 표현이라고 생각하는데, 혹시 dict()로 쎠주신 이유가 있을까요?
There was a problem hiding this comment.
use-dict-literal / R1735 에 따르면 {} 가 올바른 코드이며 특별한 이유가 없는한 dict() 를 사용할 일이 없습니다.
Author
There was a problem hiding this comment.
피드백 감사합니다! 예제라서 실행 속도가 상대적으로 관건이 아니라고 생각했고, Rust 쪽에서는 vec![0, 1]으로 쓰여있기에 코드의 문자열 비교가 상대적으로 용이하게끔 문자열인 dict()로 입력했습니다. 지금 생각해보면 주 독자층이 Python에 익숙하신 분들일테니 괜한 사족을 붙인 게 되었네요. {}로 복구하겠습니다.
Indosaram
reviewed
Apr 3, 2023
|
|
||
| 파이썬은 최근 가장 인기가 높은 언어 중 하나입니다. 간결한 문법과 범용성 덕분에 서버 개발부터 딥러닝 모델 개발까지 다양한 분야에서 사용되고 있습니다. 하지만 파이썬은 개발 시의 높은 생산성을 위해 코드 실행 속도를 일정 부분 포기한 언어입니다. 특히 파이썬의 태생적 한계인 GIL(Global interpreter lock)때문에 빠른 연산이 필요한 작업이나 멀티스레딩 프로그램에서 좋은 성능을 내기 어려운 단점이 있습니다. | ||
| 파이썬(Python)은 최근 가장 인기가 높은 언어 중 하나입니다. 간결한 문법과 범용성 덕분에, 파이썬은 서버 개발부터 딥러닝 모델 개발까지 다양한 분야에서 사용되고 있습니다. 하지만 파이썬은 개발 시의 높은 생산성을 위해 코드 실행 속도를 일정 부분 포기한 언어입니다. 특히 파이썬의 태생적 한계인 GIL (Global interpreter lock) 때문에 빠른 연산이 필요한 작업이나 멀티스레딩 프로그램에서 좋은 성능을 내기 어려운 단점이 있습니다. | ||
| [파이썬이 인기가 높은 언어라는 것은 언제를 기준한 것인지 추가가 필요합니다. 최근이라는 말로 미루어보아, 책을 집필하는 2023년 2월을 기준한 것으로 보면 될까요?]: # |
Indosaram
reviewed
Apr 3, 2023
|
|
||
| 러스트는 높은 추상성으로 높은 생산성을 가지고 있는 동시에 C/C++의 99%에 가까운 성능을 가지고 있어서 빠른 연산속도가 필요한 분야에서 각광받고 있습니다. 2022년 스택오버플로우 개발자 설문조사의 "Loved vs Dreaded(사랑하는 언어 대 두려운 언어)"에서 러스트는 86.73%라는 높은 비율로 가장 사랑받는 언어로 선정되었습니다. 참고로, 러스트는 현재 7년 연속 가장 사랑받는 언어 1위로 선정되었습니다. | ||
| 러스트(Rust)는 높은 추상성으로 높은 생산성을 가지고 있는 동시에 C/C++의 99%에 가까운 성능을 가지고 있어서 빠른 연산속도가 필요한 분야에서 각광받고 있습니다. 2022년 스택 오버플로우(Stack Overflow) 개발자 설문조사 "Loved vs Dreaded (사랑하는 언어 대 두려운 언어)"에서, 러스트는 86.73%라는 높은 비율로 가장 사랑받는 언어로 선정되었습니다. 참고로, 개발자들의 러스트 사랑은 2022년에 한정한 것이 아닙니다. 러스트는 2022년까지 7년 연속으로 가장 사랑받는 언어 1위로 선정되었습니다. | ||
| [러스트가 1위로 선정된 것은 동일한 설문주체에 의한 설문조사인가요? 어디에서 실시한 설문인지 명시하는 것이 좋아보입니다.]: # |
Owner
|
안녕하세요, 열심히 책 내용을 검토해주셔서 감사합니다. 다만 오탈자나 사실관계상 오류가 아닌 책 내의 어투나 문체 등은 따로 수정하고 싶지 않습니다. 괜찮으시다면 오류가 있는 부분만 다시 커밋을 올려주실 수 있으실까요? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR 실수 실례합니다.
약간의 내용 추가를 진행했습니다.
확인 부탁드립니다.