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

Добавляет доку для метода массива with() #5013

Merged
merged 22 commits into from
Jan 12, 2024

Conversation

vitya-ne
Copy link
Contributor

@vitya-ne vitya-ne commented Dec 6, 2023

Описание

Добавляет доку для метода массива with()

Closes #4835

Чек-лист

  • Текст оформлен согласно руководству по стилю
  • Ссылки на внутренние материалы начинаются со слеша и заканчиваются слэшем либо якорем на заголовок (/css/color/, /tools/json/, /tools/gulp/#kak-ponyat)
  • Ссылки на картинки, видео и демки относительные (images/example.png, demos/example/, ../demos/example/)

@github-actions github-actions bot added js Контент по JavaScript дока Справочный материал labels Dec 6, 2023
@TatianaFokina TatianaFokina changed the title Doka/array with Добавляет доку для метода массива with() Dec 6, 2023
@vitya-ne
Copy link
Contributor Author

vitya-ne commented Dec 6, 2023

привет,
я не добавил данные в раздел /people. Это можно сделать только как часть работы над контентом ?

@TatianaFokina
Copy link
Member

привет, я не добавил данные в раздел /people. Это можно сделать только как часть работы над контентом ?

Привет! Да, папки авторов хранятся в репозитории с контентом. Можешь добавить свою папку прямо в этом пулреквесте.

@vitya-ne
Copy link
Contributor Author

vitya-ne commented Dec 6, 2023

немножко странно... я использовал букву ё, но получил:
image

@TatianaFokina
Copy link
Member

немножко странно...

Спеллер вообще в последние месяцы в странном настроении 😄 Сейчас всё должно быть ок.

@vitya-ne
Copy link
Contributor Author

vitya-ne commented Dec 6, 2023

привет, я не добавил данные в раздел /people. Это можно сделать только как часть работы над контентом ?

Привет! Да, папки авторов хранятся в репозитории с контентом. Можешь добавить свою папку прямо в этом пулреквесте.

добавил

@vitya-ne
Copy link
Contributor Author

vitya-ne commented Dec 6, 2023

может лучше оставить:
Благодаря поддержке отрицательных индексов, `with()` значительно упрощает способ изменения...
image

@TatianaFokina
Copy link
Member

может лучше оставить: `` Благодаря поддержке отрицательных индексов, with() значительно упрощает способ изменения...

Надо подумать. Прочитала не так сначала.

Может так?

💡 Способ изменения элементов в конце массива значительно упрощается благодаря поддержке отрицательных индексов в with(). Например, для изменения последнего элемента массива без использования with() дополнительно потребуется:

@vitya-ne
Copy link
Contributor Author

vitya-ne commented Dec 6, 2023

может лучше оставить: `` Благодаря поддержке отрицательных индексов, with() значительно упрощает способ изменения...

Надо подумать. Прочитала не так сначала.

Может так?

💡 Способ изменения элементов в конце массива значительно упрощается благодаря поддержке отрицательных индексов в with(). Например, для изменения последнего элемента массива без использования with() дополнительно потребуется:

Можно. Но, теперь меня смущает 'Например,'. Это ведь как раз обратный пример, как плохо без with()

@TatianaFokina
Copy link
Member

Продолжаем экспериментировать!

💡 Способ изменения элементов в конце массива значительно упрощается благодаря поддержке отрицательных индексов в with(). В противном случае, для изменения последнего элемента массива без использования with() дополнительно потребуется:

@vitya-ne
Copy link
Contributor Author

vitya-ne commented Dec 6, 2023

Продолжаем экспериментировать!

💡 Способ изменения элементов в конце массива значительно упрощается благодаря поддержке отрицательных индексов в with(). В противном случае, для изменения последнего элемента массива без использования with() дополнительно потребуется:

а если так?
with() — упрощает изменения элемента, когда известен его индекс от конца массива. Обычно для изменения последнего элемента массива требуется:

@TatianaFokina
Copy link
Member

а если так? with() — упрощает изменения элемента, когда известен его индекс от конца массива. Обычно для изменения последнего элемента массива требуется:

Отлично!

@vitya-ne
Copy link
Contributor Author

vitya-ne commented Dec 8, 2023

@TatianaFokina, я немного изменил формулировки

Copy link
Member

@HellSquirrel HellSquirrel left a comment

Choose a reason for hiding this comment

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

@vitya-ne приветик и спасибо за отличную доку. Я даже не знала про такой метод :) Добавила немного комментариев :)

js/array-with/index.md Outdated Show resolved Hide resolved
js/array-with/index.md Outdated Show resolved Hide resolved
js/array-with/index.md Outdated Show resolved Hide resolved
js/array-with/index.md Outdated Show resolved Hide resolved
js/array-with/index.md Outdated Show resolved Hide resolved
// [ 'ВС', 'ПН', 'ВТ', 'СР' ]
```

💡 При создании нового массива `with()` выполнит преобразование всех незаполненных ячеек к `undefined`:
Copy link
Member

Choose a reason for hiding this comment

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

Кажется в примере ниже не создается новый массив, а расширяется копия исходного

Suggested change
💡 При создании нового массива `with()` выполнит преобразование всех незаполненных ячеек к `undefined`:
💡 При создании нового массива `with()` выполнит преобразование всех незаполненных ячеек к `undefined`:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Я не очень понял твой комент.
with() создаёт новый массив.
Ты имела ввиду, что в примере нужно добавить строку с присвоением ?
вот так:

const numbers = [0, , 11, 20, , 30]
const newNumbers = numbers.with(2, 10)
console.log(newNumbers)
// [ 0, undefined, 10, 20, undefined, 30 ]

Copy link
Member

Choose a reason for hiding this comment

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

В твоем примере все хорошо, это я просто не туда посмотрела. Извини пжлст :)

js/array-with/index.md Outdated Show resolved Hide resolved
js/array-with/index.md Outdated Show resolved Hide resolved
vitya-ne and others added 3 commits December 29, 2023 21:18
Co-authored-by: Polina Gurtovaia <zloymult@gmail.com>
Co-authored-by: Polina Gurtovaia <zloymult@gmail.com>
Co-authored-by: Polina Gurtovaia <zloymult@gmail.com>
Copy link

github-actions bot commented Jan 2, 2024

Превью контента из 4156f24 опубликовано.

@vitya-ne
Copy link
Contributor Author

vitya-ne commented Jan 2, 2024

@TatianaFokina, пока не знаю что это, но спасибо
изображение

@TatianaFokina
Copy link
Member

TatianaFokina commented Jan 2, 2024

@TatianaFokina, пока не знаю что это, но спасибо

Это ачивка за контрибьютоголизм 😄

// [ 'ВС', 'ПН', 'ВТ', 'СР' ]
```

💡 При создании нового массива `with()` выполнит преобразование всех незаполненных ячеек к `undefined`:
Copy link
Member

Choose a reason for hiding this comment

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

В твоем примере все хорошо, это я просто не туда посмотрела. Извини пжлст :)

@HellSquirrel HellSquirrel merged commit 7fc3fbc into doka-guide:main Jan 12, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
js Контент по JavaScript дока Справочный материал
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Методы массива: with
3 participants