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

Добавляет доку про метод множеств isDisjointFrom() #5672

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

punkmachine
Copy link
Contributor

Описание

Вкратце описал что делает метод.

Я вот не помню кого надо просить, чтобы иллюстрацию пересечения нарисовать как в статьях от Вити (пример: https://doka.guide/js/set-intersection/)

Closes #5658

Чек-лист

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

@github-actions github-actions bot added js Контент по JavaScript дока Справочный материал labels Jan 26, 2025
@punkmachine
Copy link
Contributor Author

Очень давно не писал Доку, ощущается, как первый мёрж реквест сюда😅.

@vitya-ne
Copy link
Contributor

@skorobaeus Света, нам нужна твоя незаменимая помощь с иллюстрацией. Вот пример:
image

@vitya-ne vitya-ne closed this Jan 26, 2025
@vitya-ne vitya-ne reopened this Jan 26, 2025
@skorobaeus
Copy link
Member

Загрузила схему, можно подставлять в нужное место в тексте :)

@Inventoris Inventoris changed the title Добавил доку метода множеств isDisjointFrom() Добавляет доку про метод множеств isDisjointFrom() Jan 27, 2025
Copy link
Member

@ra1nbow1 ra1nbow1 left a comment

Choose a reason for hiding this comment

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

Привет. Добавил пару предложений

js/set-is-disjoint-from/index.md Outdated Show resolved Hide resolved
@punkmachine
Copy link
Contributor Author

@vitya-ne @ra1nbow1 спасибо за ревью! Правки внёс.

@skorobaeus большое спасибо за иллюстрацию!

Comment on lines 56 to 57
Аргументом для метода `isDisjointFrom()` может быть множество, созданное через [конструктор `new Set()`](/js/set-constructor/), а также [`Map`](/js/map/).

Copy link
Contributor

@vitya-ne vitya-ne Jan 29, 2025

Choose a reason for hiding this comment

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

множество, созданное через конструктор new Set()

Стоит также не забывать про set-like объекты.

Я бы предложил использовать тут блок используемый в других методах, например: https://doka.guide/js/set-is-superset-of/#kak-pishetsya

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Я вот тут могу ошибаться, но кажется кроме Map и Set как раз таки и нет таковых объектов в JS.
Просто если я прав, тогда не особо вижу смысла указывать про set-like объекты.

Copy link
Contributor

@vitya-ne vitya-ne Jan 30, 2025

Choose a reason for hiding this comment

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

кроме Map и Set как раз таки и нет таковых объектов в JS.

Есть и даже перечислены:
image

То чего нет, можно же создать. А это и не так сложно:
image

Просто если я прав, тогда не особо вижу смысла указывать про set-like объекты.

Если бы ты был тут прав, то и понятия "set-like объекты" не существовало.

Текущая редакция:
"множество, созданное через конструктор new Set()"
звучит примерно как "коллекция созданная через конструктор new Array()". Кажется это назвается ни как иначе как массив ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Хм, был не прав, спасибо <3. Вечером закину в статью.

Copy link

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

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.

Не хватает материала о методе set - isDisjointFrom
4 participants