看教程过程中感觉有些不顺的地方 by yyxygit · Pull Request #584 · javascript-tutorial/zh.javascript.info

yyxygit · 2020-01-07T05:55:47Z

目标章节：

1-js/04-object-basics/06-constructor-new/
1-js/05-data-types/01-primitives-methods/
1-js/05-data-types/02-number/

当前上游最新 commit：da7c9de7

所做更改如下：

文件名	参考上游 commit	更改（理由）
task.md	`d3dbf3b`	一行翻译
solution.md	`d3dbf3b`	与英文版同步
article.md	`d3dbf3b`	一行翻译
article.md	`d3dbf3b`	同步英文版示例，更新翻译

…n issue

… version

bemself

感谢校正，校对完成~

javascript-translate-bot · 2020-01-07T12:18:11Z

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

yyxygit · 2020-01-08T06:36:01Z

客气啦，其实也不知道改的有没问题 😓

bemself · 2020-01-12T07:37:47Z

请问，是把这些更新的内容在webstorm编辑器内我自己的分支提交吗？还是在github上操作？ @YAYAXUEYU, 在你提交这个PR的那个分支里提交即可, 看了下貌似这个分支 yyxygit:fix-yyxygit<https://github.com/yyxygit/zh.javascript.info/tree/fix-yyxygit>, 您再查看下?

yyxygit · 2020-01-12T07:55:55Z

@bemself push上来了，292de64

bemself · 2020-01-12T08:23:37Z

@yyxygit 好像有冲突文件呢, 您查看解决一下? 另外看到多出来2个文件(之前是5个好像), 回网我看下多出来的文件修改~

yyxygit · 2020-01-12T08:42:52Z

@bemself 在github上解决冲突了，然后点击了merge，您看是这样处理吗？
是的，之前有提交了其他文件修改，Map&Set那节参考《ES标准入门》第三版（阮一峰）对应Map章节翻译
ps：太客气了，不需要用“您”啊 😓 明显您才是前辈

bemself · 2020-01-12T08:56:08Z

@yyxygit 👍 等回网看下新增加的文件. 另外提醒一下那个done的标记不是写在"commit message"里, 而是像我们现在这样对话的 "issue comment"里面.

P.S. 那好,俺以后不用您了, 不过俺也不是前辈, 大家一起学习拉

yyxygit · 2020-01-12T09:02:29Z

/done
@bemself 好的，一起学习 :)

bemself

@yyxygit 感谢，校对完成~
其中有一些是以前的翻译里的，看到就顺便麻烦你也顺便改一下了~
另外想提醒一下，尽量每个 PR 在独立的分支里，提了之后如有其它文章的改动可放新分支提新 PR~

bemself · 2020-01-12T13:18:18Z

-在我们学习了那些之后，我们将回到对象，在<info:prototypes>和<info:classes>章节中深入介绍它们。
-```
+在我们学习了那些之后，我们将回到对象，在 <info:prototypes> 和 <info:classes> 章节中深入介绍它们。
+


这一行貌似被删掉了？

218 行添加了<>前后空格
219 行有三个反引号
没有删掉啊？

@yyxygit 219 行的三个反引号我这边显示删掉了呢，截图如下~

@bemself webstorm 分支 fix-yyxygit里是有的，pull了分支代码就没了，
再提交以下吧

bemself · 2020-01-12T13:30:01Z

 ```

-正如我们看到的那样，`Map` 的键可以被转化为字符串而不是像普通对象那样不能转化， 任何类型的键都是可能的.
+如我们所见，与对象不同，键不会转换为字符串。键（key）可以是任何类型。


这里出现两次键，而这个带英文注释的“键（key）”放在后面。建议：

（key）放在第一个键的后面，第二个键就不需要了

另外查一下，如果前文已经有过（key）注释了，这里也就不需要了~

收到，搜过中文网页了，全页没有 ”键（key）“形式注释过，42行改为：

如我们所见，与对象不同，键（key）不会转换为字符串。键可以是任何类型。

这样？

@yyxygit 是的~

bemself · 2020-01-12T13:36:33Z


-使用对象作为键是 `Map` 最出名也是最重要的特点. 对于字符键, `Object` （普通对象）能正常使用, 但是使用键就会产生意料之外的效果.
-
+使用对象作为键是 `Map` 最值得注意和重要的功能之一。对于字符串键， `Object` （普通对象）能正常使用，但对于对象键则不能。


Suggested change

使用对象作为键是 `Map` 最值得注意和重要的功能之一。对于字符串键， `Object` （普通对象）能正常使用，但对于对象键则不能。

使用对象作为键是 `Map` 最值得注意和重要的功能之一。对于字符串键，`Object`（普通对象）能正常使用，但对于对象键则不能。

https://github.com/sparanoid/chinese-copywriting-guidelines#%E5%85%A8%E5%BD%A2%E6%A8%99%E9%BB%9E%E8%88%87%E5%85%B6%E4%BB%96%E5%AD%97%E7%AC%A6%E4%B9%8B%E9%96%93%E4%B8%8D%E5%8A%A0%E7%A9%BA%E6%A0%BC
加强的英文单词，前后有全角中文标点符号的，不需要加空格，是吗？

@yyxygit 对的，如果前后有全角中文标点符号，可以不加空格

bemself · 2020-01-12T13:51:18Z

- `map.entries()` -- returns an iterable for entries `[key, value]`, it's used by default in `for..of`.
+- `map.keys()` -- 返回键名的遍历器，
+- `map.values()` -- 返回键值的遍历器，
+- `map.entries()` -- 返回实体 `[key, value]` 的遍历器，默认在for..of中使用。


Suggested change

- `map.entries()` -- 返回实体 `[key, value]` 的遍历器，默认在for..of中使用。

- `map.entries()` －返回实体 `[key, value]` 的遍历器，默认在 `for..of` 中使用。

请问，英文中两个半角中划线 --
替换成一个全角中文中划线？

@yyxygit 是的，英文中两个半角中划线 -- 替换成一个全角中文中划线

bemself · 2020-01-12T13:51:31Z

- `map.values()` -- returns an iterable for values,
- `map.entries()` -- returns an iterable for entries `[key, value]`, it's used by default in `for..of`.
+- `map.keys()` -- 返回键名的遍历器，
+- `map.values()` -- 返回键值的遍历器，


Suggested change

- `map.values()` -- 返回键值的遍历器，

- `map.values()` －返回键值的遍历器，

bemself · 2020-01-12T13:51:42Z

- `map.keys()` -- returns an iterable for keys,
- `map.values()` -- returns an iterable for values,
- `map.entries()` -- returns an iterable for entries `[key, value]`, it's used by default in `for..of`.
+- `map.keys()` -- 返回键名的遍历器，


Suggested change

- `map.keys()` -- 返回键名的遍历器，

- `map.keys()` －返回键名的遍历器，

bemself · 2020-01-12T13:54:15Z

@@ -325,4 +325,4 @@ set.forEach((value, valueAgain, set) => {
 - `set.clear()` -- 清空 set。


Suggested change

- `set.clear()` -- 清空 set。

- `set.clear()` －清空 set。

bemself · 2020-01-12T13:54:42Z

@@ -325,4 +325,4 @@ set.forEach((value, valueAgain, set) => {
 - `set.clear()` -- 清空 set。
 - `set.size` -- 返回当前全部元素的数量。


Suggested change

- `set.size` -- 返回当前全部元素的数量。

- `set.size` －返回当前全部元素的数量。

javascript-translate-bot · 2020-01-12T14:02:46Z

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

yyxygit · 2020-01-14T05:58:52Z

/done

bemself

感谢翻译，校对完成~ Map 文件中有些历史遗留问题，不在本 PR 里面改了，回头我单开一个 PR 更正

yyxygit added 3 commits December 30, 2019 09:42

object#copying-by-reference - Comparison by reference：tiny translatio…

8fc0502

…n issue

https://zh.javascript.info/constructor-new#zong-jie：sync with english…

d3dbf3b

… version

tiny update

da7c9de

javascript-translate-bot added the review needed label Jan 7, 2020

javascript-translate-bot requested a review from a team January 7, 2020 05:55

yyxygit changed the title ~~tiny update~~ 看教程过程中感觉有些不顺的地方 Jan 7, 2020

Merge branch 'master' into fix-yyxygit

e7f1316

bemself added the 正在校对 label Jan 7, 2020

bemself suggested changes Jan 7, 2020

View reviewed changes

javascript-translate-bot added changes requested and removed review needed labels Jan 7, 2020

bemself removed the 正在校对 label Jan 7, 2020

1-js/05-data-types/07-map-set/article.md: update

ea658eb

/done （pull/584）

292de64

Merge branch 'master' into fix-yyxygit

5d465f2

javascript-translate-bot added review needed and removed changes requested labels Jan 12, 2020

1-js/05-data-types/08-weakmap-weakset/article.md：tiny update

e0d8baf

bemself suggested changes Jan 12, 2020

View reviewed changes

javascript-translate-bot removed the review needed label Jan 12, 2020

javascript-translate-bot added the changes requested label Jan 12, 2020

yyxygit added 2 commits January 14, 2020 13:55

1-js/05-data-types/07-map-set/article.md changes

9465c36

Merge remote-tracking branch 'origin/fix-yyxygit' into fix-yyxygit

e9c8bd8

javascript-translate-bot added review needed and removed changes requested labels Jan 14, 2020

yyxygit and others added 2 commits January 14, 2020 14:02

1-js/04-object-basics/06-constructor-new/article.md add ```

c16d18a

correct some format issues

26694c4

bemself approved these changes Jan 14, 2020

View reviewed changes

javascript-translate-bot added needs +1 and removed review needed labels Jan 14, 2020

bemself merged commit 487b2bc into javascript-tutorial:master Jan 14, 2020


		使用对象作为键是 `Map` 最出名也是最重要的特点. 对于字符键, `Object` （普通对象）能正常使用, 但是使用键就会产生意料之外的效果.

		使用对象作为键是 `Map` 最值得注意和重要的功能之一。对于字符串键， `Object` （普通对象）能正常使用，但对于对象键则不能。

	使用对象作为键是 `Map` 最值得注意和重要的功能之一。对于字符串键， `Object` （普通对象）能正常使用，但对于对象键则不能。
	使用对象作为键是 `Map` 最值得注意和重要的功能之一。对于字符串键，`Object`（普通对象）能正常使用，但对于对象键则不能。

	- `map.entries()` -- 返回实体 `[key, value]` 的遍历器，默认在for..of中使用。
	- `map.entries()` －返回实体 `[key, value]` 的遍历器，默认在 `for..of` 中使用。

	- `map.values()` -- 返回键值的遍历器，
	- `map.values()` －返回键值的遍历器，

	- `map.keys()` -- 返回键名的遍历器，
	- `map.keys()` －返回键名的遍历器，

		@@ -325,4 +325,4 @@ set.forEach((value, valueAgain, set) => {
		- `set.clear()` -- 清空 set。

	- `set.clear()` -- 清空 set。
	- `set.clear()` －清空 set。

		@@ -325,4 +325,4 @@ set.forEach((value, valueAgain, set) => {
		- `set.clear()` -- 清空 set。
		- `set.size` -- 返回当前全部元素的数量。

	- `set.size` -- 返回当前全部元素的数量。
	- `set.size` －返回当前全部元素的数量。

Conversation

yyxygit commented Jan 7, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

目标章节：

当前上游最新 commit：da7c9de7

所做更改如下：

Uh oh!

bemself left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

javascript-translate-bot commented Jan 7, 2020

Uh oh!

yyxygit commented Jan 8, 2020

Uh oh!

bemself commented Jan 12, 2020 via email

Uh oh!

yyxygit commented Jan 12, 2020

Uh oh!

bemself commented Jan 12, 2020

Uh oh!

yyxygit commented Jan 12, 2020

Uh oh!

bemself commented Jan 12, 2020

Uh oh!

yyxygit commented Jan 12, 2020

Uh oh!

bemself left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

yyxygit Jan 14, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

javascript-translate-bot commented Jan 12, 2020

Uh oh!

yyxygit commented Jan 14, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

bemself left a comment

Choose a reason for hiding this comment

yyxygit commented Jan 7, 2020 •

edited

Loading

yyxygit Jan 14, 2020 •

edited

Loading

yyxygit commented Jan 14, 2020 •

edited

Loading