1-js/03-code-quality/02-coding-style/ 코드 리뷰 부탁 드립니다 ~ :)#75
1-js/03-code-quality/02-coding-style/ 코드 리뷰 부탁 드립니다 ~ :)#75devhaeyeon wants to merge 2 commits intojavascript-tutorial:masterfrom
Conversation
|
|
Violet-Bora-Lee
left a comment
Violet-Bora-Lee
left a comment
There was a problem hiding this comment.
원문과 번역문의 line이 매칭되어야 합니다.
이부분이 반영되면 다시 리뷰 시작하도록 하겠습니다.
감사합니다.
| # 코딩 스타일 | ||
|
|
||
| Our code must be as clean and easy to read as possible. | ||
| 코드는 깨끗하 고 읽기 쉽게 작성하는 것이 실질적으로 프로그래밍의 기술입니다. |
There was a problem hiding this comment.
코드는~~~ 프로그래밍의 기술입니다. 라는 문장이 어색해보이네요.
구글 번역기에 따르면, 아래와 같이 번역이 되는데,
코드는 깨끗하고 읽기 쉬워야합니다. 이 문장이 낫지 않을까요?
| 코드는 깨끗하 고 읽기 쉽게 작성하는 것이 실질적으로 프로그래밍의 기술입니다. | ||
|
|
||
| That is actually the art of programming -- to take a complex task and code it in a way that is both correct and human-readable. | ||
| 복잡한 작업을 수행하고 사람이 읽을 수 있는 방식으로 코드를 작성합니다. |
There was a problem hiding this comment.
번역에 누락이 있습니다.
That is actually the art of programming
재번역을 요청합니다.
| --> | ||
|
|
||
| Now let's discuss the rules and reasons for them in detail. | ||
| 위의 치트 시트에 기재된 규칙과 세세한 내용을 설명하겠습니다. |
There was a problem hiding this comment.
치트 시트와 치트시트를 통일해 주시기 바랍니다.(띄어쓰기)
| Now let's discuss the rules and reasons for them in detail. | ||
| 위의 치트 시트에 기재된 규칙과 세세한 내용을 설명하겠습니다. | ||
|
|
||
| ```warn header="Irony Detected" |
|
|
||
| ```warn header="Irony Detected" | ||
| Nothing is set in stone here. These are style preferences, not religious dogmas. | ||
| 꼭 해야만 하는 정해진 규칙이 아닙니다. |
There was a problem hiding this comment.
Nothing is set in stone here. 은 수사의 용도로 사용된 문장이긴 합니다만, 번역해주시기 바랍니다.
|
|
||
| In most JavaScript projects curly braces are written in "Egyptian" style with the opening brace on the same line as the corresponding keyword -- not on a new line. There should also be a space before the opening bracket, like this: | ||
| 대부분의 자바스크립트 프로젝트에서 중괄호는 "Egyptian" 스타일입니다. "Egyptian" 스타일은 새로운 키워드가 아닌 해당 키워드와 같은 행에 여는 중괄호로 작성되는 것입니다. 아래의 예시 코드에서 처럼 여는 중괄호 앞에는 공백 한 칸이 있어야 합니다. | ||
|
|
There was a problem hiding this comment.
원문과 번역문의 line수를 맞춰야합니다.
| Here are the annotated variants so you can judge their readability for yourself: | ||
| 단일 행 구조는 에지 케이스에서 중요합니다. 그렇다면 대괄호를 사용해야할까요? 그렇다면 어디에 대괄호를 사용할까요? | ||
|
|
||
| 다음은 주석이 달린 코드 입니다. 코드를 보면 가독성을 판단할 수 있습니다. |
| - For very short code, one line is acceptable. For example: `if (cond) return null`. | ||
| - But a separate line for each statement in brackets is usually easier to read. | ||
| 요약 : | ||
|
|
There was a problem hiding this comment.
line break를 삭제해 원문과 줄수를 맞춰주시기 바랍니다.
| - **세로 들여쓰기 : 로직이 있는 블록 사이를 나눌 때 빈 행** | ||
|
|
||
| Even a single function can often be divided into logical blocks. In the example below, the initialization of variables, the main loop and returning the result are split vertically: | ||
| 단일 함수에서 자주 로직 블록으로 나눌 수 있습니다. |
There was a problem hiding this comment.
111번째줄과 112번째줄은 나눠지면 안됩니다.
|
Please make the requested changes. After it, add a comment "/done". |
|
안녕하세요. 작업 요청드렸으나 시간이 많이 지체되어 본 PR은 종료하겠습니다. |
4 participants
No description provided.