Skip to content
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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[오타수정] 2.8 기본 연산자와 수학 #1452 #1497

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[오타수정] 2.8 기본 연산자와 수학 #1452 #1497

wants to merge 2 commits into from

Conversation

cottonpup
Copy link
Contributor

@cottonpup cottonpup commented Aug 2, 2022 •
edited
Loading

Pull Request 체크리스트

TODO

  • 번역 규칙을 확인하셨나요?
    • 줄 바꿈과 단락을 '원문과 동일하게' 유지하셨나요?
    • 맞춤법 검사기로 맞춤법을 확인하셨나요?
    • 마크다운 문법에 사용되는 공백(스페이스), 큰따옴표("), 작은따옴표('), 대시(-), 백틱(`) 등의 특수문자는 그대로 두셨나요?
  • 로컬 서버 세팅 후 최종 결과물을 확인해 보셨나요?
  • PR 하나엔 번역문 하나만 넣으셨나요?
  • 의미 있는 커밋 메시지를 작성하셨나요?
    • 예시
      • [프락시] 번역
      • [프락시] 과제 번역
      • [if문과 조건부 연산자 '?'] 리뷰
      • [주석] 2차 리뷰
      • [Date 객체와 날짜] 번역

Summary

[오타수정] 2.8 기본 연산자와 수학 #1452
띄어쓰기, 피동형 문장 등을 수정하였습니다.

  • 연산자라고 => 연산자 라고
  • 앞쪽 네 연산자는 => 앞쪽, 네 연산자는
  • x = value을 => x = value를
  • 쓰여지고 => 쓰이고
  • 되도록이면 => 되도록

  • 하는것 보다 => 하는 것 보다
  • 권유드립니다 => 권유해 드립니다

@javascript-translate-bot javascript-translate-bot requested a review from a team August 2, 2022 14:18
Violet-Bora-Lee
Violet-Bora-Lee reviewed Aug 2, 2022
View reviewed changes
1-js/02-first-steps/08-operators/3-primitive-conversions-questions/solution.md Outdated
@@ -18,7 +18,7 @@ undefined + 1 = NaN // (6)
```

1. 피 연산자 중 하나가 문자열인 `"" + 1`에서 `1`은 문자형으로 변환됩니다. 따라서 공백과 문자열 1을 더한, `"" + 1 = "1"`과 같은 효과를 발휘하죠. 그다음 연산 `"1" + 0`에도 같은 규칙이 적용됩니다.
2. 뺄셈 연산자 `-`는 기타 수학 연산자처럼 숫자형만을 인수로 받습니다. 빈 문자열 `""`는 숫자 `0`으로 변환되기 때문에 결과는 `-1`이 됩니다.
2. 뺄셈 연산자 `-`는 기타 수학 연산자처럼 숫자형만을 인수로 받습니다. 빈 문자열 `""`은 숫자 `0`으로 변환되기 때문에 결과는 `-1`이 됩니다.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

""<--이 기호는 쌍따옴표라고 부르기 때문에 쌍따옴표는으로 하는게 맞을것 같은데요.
혹시 은으로 바꾸신 이유가 있을까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

쌍따옴표는이 더 정확한 것 같습니다. 커밋 수정하였습니다.

@Violet-Bora-Lee Violet-Bora-Lee added discussion needed This issue is under discussion and removed review needed labels Aug 2, 2022
@Violet-Bora-Lee
Copy link
Member

리뷰 후 수정 내용도 커밋 하나에 합치기 부탁드릴게요.. :)

@Violet-Bora-Lee Violet-Bora-Lee added changes requested and removed discussion needed This issue is under discussion labels Aug 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Violet-Bora-Lee Violet-Bora-Lee Violet-Bora-Lee left review comments

Assignees
No one assigned
Labels
changes requested
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cottonpup @Violet-Bora-Lee @javascript-translate-bot