Skip to content

Add translation of BigInt chapter#813

Merged
iliakan merged 8 commits intojavascript-tutorial:masterfrom
pahuta:add-chapter-of-bigInt
Feb 5, 2020
Merged

Add translation of BigInt chapter#813
iliakan merged 8 commits intojavascript-tutorial:masterfrom
pahuta:add-chapter-of-bigInt

Conversation

@pahuta
Copy link
Contributor

@pahuta pahuta commented Jan 14, 2020

Perform translation of commit from this comment #770 (comment)

@javascript-translate-bot javascript-translate-bot added the review needed Review needed, please approve or request changes label Jan 14, 2020
@javascript-translate-bot javascript-translate-bot requested a review from a team January 14, 2020 14:36
@claassistantio
Copy link

claassistantio commented Jan 14, 2020

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

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.

/done

Copy link
Contributor

@stalanov stalanov Jan 16, 2020

Choose a reason for hiding this comment

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

typo in работать

Copy link
Contributor Author

Choose a reason for hiding this comment

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

/done

Copy link
Contributor

Choose a reason for hiding this comment

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

оператор не поддерживается при работе с BigInt числамИ

Copy link
Contributor Author

Choose a reason for hiding this comment

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

/done

Copy link
Contributor

Choose a reason for hiding this comment

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

English line translated below

Copy link
Contributor Author

Choose a reason for hiding this comment

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

/done

Copy link
Contributor

Choose a reason for hiding this comment

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

Double л in полифил word

Copy link
Contributor Author

Choose a reason for hiding this comment

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

/done

Copy link
Contributor

@stalanov stalanov left a comment

Choose a reason for hiding this comment

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

Please fix the issues found

@javascript-translate-bot javascript-translate-bot removed the review needed Review needed, please approve or request changes label Jan 16, 2020
@javascript-translate-bot
Copy link
Contributor

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

@javascript-translate-bot javascript-translate-bot added the changes requested Waiting for changes and /done from PR author label Jan 16, 2020
@javascript-translate-bot javascript-translate-bot added needs +1 One more review needed and removed changes requested Waiting for changes and /done from PR author labels Jan 16, 2020
@javascript-translate-bot javascript-translate-bot added the changes requested Waiting for changes and /done from PR author label Jan 16, 2020
@javascript-translate-bot
Copy link
Contributor

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

@pahuta
Copy link
Contributor Author

pahuta commented Jan 17, 2020

/done

@javascript-translate-bot javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels Jan 17, 2020
@javascript-translate-bot javascript-translate-bot requested a review from a team January 17, 2020 20:20
@pahuta
Copy link
Contributor Author

pahuta commented Jan 19, 2020

/done

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
Member

Choose a reason for hiding this comment

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

точка в конце предложения

Copy link
Member

Choose a reason for hiding this comment

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

Обратите внимание: операция деления 5/2 возвращает округлённый результат, без дробной части. Все операции с числами типа bigint возвращают bigint.

Copy link
Member

Choose a reason for hiding this comment

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

Конвертирование bigint в число всегда происходит неявно, но если значение bigint слишком велико и не подходит под тип number, то дополнительные биты будут отброшены, так что следует быть осторожными с такими преобразованиями.

Copy link
Member

Choose a reason for hiding this comment

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

Прошу следовать англ тексту тут, ни к чему вмешивать asm.js.

Copy link
Member

Choose a reason for hiding this comment

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

| Операция | Встроенный BigInt | JSBI |

Copy link
Member

Choose a reason for hiding this comment

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

из number

Copy link
Member

Choose a reason for hiding this comment

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

...А затем использовать полифил (плагин Babel) для замены вызовов JSBI на встроенные Bigint для браузеров, которые их поддерживают.

Copy link
Member

Choose a reason for hiding this comment

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

Другими словами, данный подход предлагает использовать JSBI вместо встроенных bigint. JSBI внутри себя работает с числами как с bigint, эмулирует их с соблюдением всех требований спецификации. Таким образом, мы можем выполнять JSBI-код в интерпретаторах, которые не поддерживают Bigint, а для тех, которые поддерживают - полифил преобразует вызовы в обычные Bigint.

Copy link
Member

Choose a reason for hiding this comment

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

Ссылки

@javascript-translate-bot javascript-translate-bot added changes requested Waiting for changes and /done from PR author and removed review needed Review needed, please approve or request changes labels Jan 19, 2020
@javascript-translate-bot
Copy link
Contributor

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

@javascript-translate-bot javascript-translate-bot added review needed Review needed, please approve or request changes and removed changes requested Waiting for changes and /done from PR author labels Jan 20, 2020
@javascript-translate-bot javascript-translate-bot requested a review from a team January 20, 2020 08:39
@pahuta
Copy link
Contributor Author

pahuta commented Jan 20, 2020

/done

@iliakan
Copy link
Member

iliakan commented Feb 4, 2020

@pakhuta плиз учти мои изменения, обнови перевод

@pahuta
Copy link
Contributor Author

pahuta commented Feb 5, 2020

@pakhuta плиз учти мои изменения, обнови перевод

@iliakan, я так понял ты про изменения из этого коммита javascript-tutorial/en.javascript.info@7cd8f55#diff-4aebdb789a48074f05b5a4e4391b55eeR45 ?

Если да, то готово.

@pahuta
Copy link
Contributor Author

pahuta commented Feb 5, 2020

/done

@iliakan
Copy link
Member

iliakan commented Feb 5, 2020

Например, вот https://github.com/javascript-tutorial/ru.javascript.info/pull/813/files#r368323582

Я оставил комментарий, который ты не учел. Почему?

Данное решение именно обратное, оно не альтернативное. Если ты внимательно прочитаешь эту секцию статьи и в курсе, что такое полифилы, то разница будет понятна =)

@iliakan
Copy link
Member

iliakan commented Feb 5, 2020

@pakhuta

@pahuta
Copy link
Contributor Author

pahuta commented Feb 5, 2020

@iliakan, не знаю как я пропустил этот коммент. Прошёлся ещё раз по всем твоим комментариям и всё поправил.

@pahuta
Copy link
Contributor Author

pahuta commented Feb 5, 2020

/done

@iliakan
Copy link
Member

iliakan commented Feb 5, 2020

🚀

@iliakan iliakan merged commit df35bee into javascript-tutorial:master Feb 5, 2020
@iliakan
Copy link
Member

iliakan commented Feb 5, 2020

Спасибо!

@pahuta pahuta deleted the add-chapter-of-bigInt branch February 6, 2020 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs +1 One more review needed review needed Review needed, please approve or request changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

Comments