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.

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

zh-cn: update the concat() methods #24970

Merged
merged 3 commits into from
Dec 7, 2024
Merged

zh-cn: update the concat() methods #24970

merged 3 commits into from
Dec 7, 2024

Conversation

Yanko1013
Copy link
Contributor

请指教 :D

@Yanko1013 Yanko1013 requested a review from a team as a code owner December 6, 2024 17:03
@Yanko1013 Yanko1013 requested review from t7yang and removed request for a team December 6, 2024 17:03
@github-actions github-actions bot added the l10n-zh Issues related to Chinese content. label Dec 6, 2024
Copy link
Contributor

github-actions bot commented Dec 6, 2024

@Yanko1013
Copy link
Contributor Author

@yin1999 非常感谢指正!另外我有个问题:...It should only... 中的 It,我应该翻译为“这种情况”,还是应该作其他说明?

Though technically permitted, calling String.prototype.concat() with no arguments is a useless operation, because it does not result in observable copying (like {{jsxref("Array.prototype.concat()")}}), since strings are immutable. It should only happen if you are spreading an array of strings as arguments, and that array happens to be empty.

尽管技术上允许,但不带参数地调用 String.prototype.concat() 毫无意义,因为它不会(像 {{jsxref("Array.prototype.concat()")}})返回浅拷贝——字符串是不可变的。仅当你将一个字符串数组作为参数展开,并且该数组恰好为空时,才会发生这种情况。

与此同时,我去修改我以前错误提交的 l10n.sourceCommit。

@Yanko1013 Yanko1013 requested a review from yin1999 December 7, 2024 07:57
Copy link
Member

@yin1999 yin1999 left a comment

Choose a reason for hiding this comment

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

我认为翻译成“这种情况”是合理的 👍

@yin1999 yin1999 changed the title zh-cn: update "concat" zh-cn: update the concat() methods Dec 7, 2024
@yin1999 yin1999 enabled auto-merge (squash) December 7, 2024 12:14
@yin1999 yin1999 merged commit 2d93cab into mdn:main Dec 7, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-zh Issues related to Chinese content.
Projects
Development

Successfully merging this pull request may close these issues.

2 participants