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

Исправляет код в интерактивном примере для функции reduce #5485

Merged
merged 2 commits into from
Sep 23, 2024

Conversation

yarops
Copy link
Contributor

@yarops yarops commented Aug 7, 2024

Описание

  • Вызов users.reduce(groupNameById) без второго параметра initialValue будет работать не так как проиллюстрировано в примере
  • Без оператора присваивания возникает двусмысленность (функция возвращает значение, а не изменяет текущий массив)

Чек-лист

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

@yarops yarops requested a review from skorobaeus as a code owner August 7, 2024 16:44
@github-actions github-actions bot added the js Контент по JavaScript label Aug 7, 2024
@TatianaFokina TatianaFokina added улучшение Доработка существующего дока Справочный материал демо labels Aug 7, 2024

This comment was marked as outdated.

Copy link

github-actions bot commented Aug 7, 2024

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

@skorobaeus
Copy link
Member

Давайте обсудим, насколько объявлять переменную принципиально? В этом демо многое упрощено и показано схематически.

@yarops
Copy link
Contributor Author

yarops commented Aug 13, 2024

Давайте обсудим, насколько объявлять переменную принципиально? В этом демо многое упрощено и показано схематически.

Привет, вы правы, принципиален второй параметр, а переменная нет. Просто обычно в примерах если функция результирующий массив возвращает - объявляют переменную. Проиллюстрировать отличие от, например, arr.reverse() итп. Т.к. в примере код - его будут Ctrl+C Ctrl+V и пробовать запускать. Мне показалось нет причин чтоб не добавить эту переменную - для наглядности и не объединить код запуска - для удобства 🙂

@skorobaeus
Copy link
Member

Т.к. в примере код - его будут Ctrl+C Ctrl+V и пробовать запускать. Мне показалось нет причин чтоб не добавить эту переменную - для наглядности и не объединить код запуска - для удобства 🙂

Это сильный аргумент, вы меня убедили :) Спасибо большое за вклад!

@TatianaFokina TatianaFokina merged commit e7e6826 into doka-guide:main Sep 23, 2024
6 checks passed
@TatianaFokina
Copy link
Member

Спасибоооо 💖

baileys-li pushed a commit to baileys-li/doka-content that referenced this pull request Dec 19, 2024
…#5485)

* Исправляет код в интерактивном примере

* Добавляет второй параметр
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.

3 participants