Skip to content

Translate 'Operators' page#32

Merged
jakubdrozdek merged 3 commits into
masterfrom
operators
Oct 27, 2019
Merged

Translate 'Operators' page#32
jakubdrozdek merged 3 commits into
masterfrom
operators

Conversation

@jakubdrozdek

Copy link
Copy Markdown
Contributor

No description provided.

@jakubdrozdek

Copy link
Copy Markdown
Contributor Author

@mitroc @sbielenica Poproszę o sprawdzenie :-)


*!*
// both values converted to numbers before the binary plus
// Obydwie wartości są konwertowane do liczb, a następnie przekazywane do dwuargumentowego plusa

@sbielenica sbielenica Oct 26, 2019

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Dlaczego komentarz rozpoczyna się wielką literą? Linia 118 będąca podobnym komentarzem oraz oryginalna wersja linii 114 rozpoczyna się małą literą.

W powyższym kodzie wyrażenie `(a = b + 1)` zwraca wartość, która została przypisana do `a` (czyli `3`), a następnie ta wartość jest używana w kolejnych operacjach.

Funny code, isn't it? We should understand how it works, because sometimes we see it in JavaScript libraries, but shouldn't write anything like that ourselves. Such tricks definitely don't make code clearer or readable.
Śmiesznie to wygląda, prawda? Musimy jednak o tym wiedzieć, zważywszy na fakt, iż w wielu bibliotekach javascriptowych napotkamy taki zapis. Radzimy jednak nie pisać tak we własnym kodzie. Tego typu sztuczki tylko zaciemniają kod.

@sbielenica sbielenica Oct 26, 2019

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Proponuję Tego typu sztuczki sprawiają, że kod jest mniej czytelny czy coś w tym stylu.
Obecne tłumaczenie zaciemniają kod brzmi dziwnie i wydaje mi się, że nie oddaje znaczenia słów readable i clearer w tym kontekście.

Uprzedzam: jestem świadom istnienia definicji https://pl.wikipedia.org/wiki/Zaciemnianie_kodu.

@jakubdrozdek

Copy link
Copy Markdown
Contributor Author

@sbielenica Poprawione.

@jakubdrozdek jakubdrozdek merged commit bc15847 into master Oct 27, 2019
@jakubdrozdek jakubdrozdek deleted the operators branch October 27, 2019 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants