Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Unicode, String internals#426

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
nazarepiedady merged 3 commits intojavascript-tutorial:masterfromnotFaceroll:master
Mar 24, 2024

Conversation

notFaceroll
Copy link
Contributor

No description provided.

@CLAassistant
Copy link

CLAassistant commentedJan 24, 2024
edited
Loading

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@nazarepiedadynazarepiedady left a comment

Choose a reason for hiding this comment

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

I review some words that should be revised.

@@ -1,172 +1,167 @@

#Unicode,String internals
#Unicode,detalhes internos de Strings

Choose a reason for hiding this comment

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

It should be "das Strings" because "String" is singular and "Strings" plural.

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

Strings, both singular and plural, can be referenced with the preposition "de".

There are some examples of this in both singular and plural in the pt-br version ofMDN docs - Strings

Copy link
Member

@nazarepiedadynazarepiedadyJan 25, 2024
edited
Loading

Choose a reason for hiding this comment

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

Well, we are not translating to Brazilian Portuguese, only Portuguese.

String translates directly toSequência de Caracteres andStrings translates directly toSequências de Caracteres, so we are not talking about the same thing.

To avoid confusion, I recommend reverting the translation to just useString instead ofStrings.

```

As we already know, JavaScriptstringsare based on[Unicode](https://en.wikipedia.org/wiki/Unicode):each character is represented by a byte sequence of 1-4 bytes.
Como já sabemos,stringsem JavaScript são baseadas em[Unicode](https://en.wikipedia.org/wiki/Unicode):cada caractere é representado por uma sequência de bytes de 1 a4 bytes.

Choose a reason for hiding this comment

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

It should be "já sabemos que, as Strings da JavaScript são baseadas no".

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

I believe this change is also not necessary.

Since the word "linguagem" is omitted before "JavaScript", it can also be referred to with the preposition "em".

This can also be based on the pt-br version ofMDN docs - Strings

Copy link
Member

@nazarepiedadynazarepiedadyJan 25, 2024
edited
Loading

Choose a reason for hiding this comment

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

The wordlinguagem is implied because JavaScript is a programming language.

And just because others do different does not make it right.

`XX`deve ser composto por dois digitos hexadecimais com um valor entre`00`e`FF`,assim`\xXX`representa o caractere cujo o código Unicode é`XX`.

Because the`\xXX`notation supports only two hexadecimal digits, it can be used only for the first 256Unicode characters.
Porque a notação`\xXX`suporta apenas dois dígitos hexadecimais, ela pode ser usada apenas para os primeiros 256caracteres Unicode.

Choose a reason for hiding this comment

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

It should start by "uma vez que a notação".

@javascript-translate-bot

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

@notFaceroll
Copy link
ContributorAuthor

/done

Copy link
Member

@nazarepiedadynazarepiedady left a comment

Choose a reason for hiding this comment

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

It seems good enough to be applied.

@nazarepiedady
Copy link
Member

@peruibeloko and@jonnathan-ls, could you review this to check if it is okay to pass?

Copy link
Contributor

@peruibelokoperuibeloko left a comment

Choose a reason for hiding this comment

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

Algumas mudanças rápidas, nada demais

```warn header="Advanced knowledge"
The section goes deeper into string internals. This knowledge will be useful for you if you plan to deal with emoji, rare mathematical or hieroglyphic characters, or other rare symbols.
```warn header="Conhecimento avançado"
Esta seção aprofunda nos detalhes internos das strings. Este conhecimento será útil se você planeja lidar com emojis, caracteres matemáticos raros, hieróglifos ou outros símbolos raros.
Copy link
Contributor

Choose a reason for hiding this comment

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

Eu odeio a palavra "hieróglifos" usada em diversos trechos que tratam de Unicode, sinto que é um pouco xenofóbico. Eu sempre traduzo por "logogramas" ou outros nomes mais apropriados.

Copy link
Member

Choose a reason for hiding this comment

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

@peruibeloko, não é xenofóbico, é um termo muito usado na arqueologia.
Recomendo as pessoas a entenderem as coisas e dos porquês (se existirem) antes de partilharem opiniões sobre assuntos fora do seu domínio, mas geralmente se está fora do seu domínio não dê opiniões.

Copy link
Contributor

Choose a reason for hiding this comment

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

@nazarepiedady sim, é um termo muito usado na arqueologia, mas nos trechos que me refiro, o autor não usa a palavra nesse sentido.

Repare que é possível ver pelo contexto, não só nesse artigo mas em outros também, que o terno hieróglifos está sendo usado no lugar de logogramas. Hieróglifos são especificamente os usados pelos egípcios antigos, logogramas são qualquer sistema de escrita que usa um único caractere para representar um morfema, palavra ou conceito.

Hieróglifos são um tipo de logograma, mais especificamente ideograma, já que usam representações visuais dos conceitos.

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

@peruibeloko entendo, obrigado pelo esclarecimento!
Mas agora,@nazarepiedady e@peruibeloko, o que não ficou muito claro pra mim é: mantenho como está ou faço a alteração para "logograma"?

Copy link
Member

@nazarepiedadynazarepiedadyMar 9, 2024
edited
Loading

Choose a reason for hiding this comment

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

@notFaceroll, keep it as it is.

Tecnicamente, pares substitutos também são detectáveis pelos seus códigos: se um caractere tem o código no intervalo de`0xd800..0xdbff`,então ele é a primeira partedo parsubstituto.O próximocaractere (segunda parte) deve ter o código no intervalo`0xdc00..0xdfff`.Esses intervalos são reservados exclusivamente para pares substitutos por padrão.

So the methods [String.fromCodePoint](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/fromCodePoint)and [str.codePointAt](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/codePointAt)were added in JavaScriptto deal with surrogate pairs.
Então, os métodos [String.fromCodePoint](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/fromCodePoint)e [str.codePointAt](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/codePointAt)foram adicionados no JavaScriptpara lidar com pares substitutos.
Copy link
Contributor

Choose a reason for hiding this comment

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

Sempre que possível, é bom usar o link para a MDN em pt-BR

Copy link
Member

Choose a reason for hiding this comment

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

@notFaceroll, could you check?

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

Posso sim

Então, os métodos [String.fromCodePoint](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/fromCodePoint)e [str.codePointAt](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/codePointAt)foram adicionados no JavaScriptpara lidar com pares substitutos.

They are essentially the same as [String.fromCharCode](mdn:js/String/fromCharCode)and [str.charCodeAt](mdn:js/String/charCodeAt),but they treat surrogate pairs correctly.
Eles são essencialmente os mesmos que [String.fromCharCode](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/fromCharCode)e [str.charCodeAt](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/charCodeAt),mas eles tratam pares substitutos corretamente.
Copy link
Contributor

Choose a reason for hiding this comment

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

O mesmo aqui

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

Pra já

@javascript-translate-bot

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

@notFaceroll
Copy link
ContributorAuthor

/done

@nazarepiedady
Copy link
Member

@peruibeloko,@jonnathan-ls, could you review this pull request?

We need to review and apply all current pull request to leave the pull request section clean to receive new pull requests.

@nazarepiedadynazarepiedady merged commitbc56aba intojavascript-tutorial:masterMar 24, 2024
@javascript-translate-bot

Thank you 💖 I updated the Progress Issue#1 🎉 🎉 🎉

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@nazarepiedadynazarepiedadynazarepiedady approved these changes

@odsantosodsantosAwaiting requested review from odsantos

@jonnathan-lsjonnathan-lsAwaiting requested review from jonnathan-ls

+1 more reviewer

@peruibelokoperuibelokoperuibeloko approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@notFaceroll@CLAassistant@javascript-translate-bot@nazarepiedady@peruibeloko

[8]ページ先頭

©2009-2025 Movatter.jp