Skip to content

Добавлен метод contains() для проверки наличия элемента - #5

Open
leemuar wants to merge 2 commits into
masterfrom
claude/onescript-stack-implementation-9vt9lu
Open

Добавлен метод contains() для проверки наличия элемента#5
leemuar wants to merge 2 commits into
masterfrom
claude/onescript-stack-implementation-9vt9lu

Conversation

@leemuar

@leemuar leemuar commented Aug 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • Реализован contains(value) — проверка наличия значения среди элементов стека (через Найти()/Find() базового массива, строгое равенство: примитивы по значению, объекты по ссылке)
  • Добавлены тесты: наличие метода, поиск существующего значения, поиск отсутствующего значения, поведение на пустом стеке, отсутствие побочных эффектов на count()
  • Обновлена документация: doc-комментарий к методу в формате стандарта 1С, contains() добавлен в таблицу методов и пример использования в README

Test plan

  • Полный набор тестов (29/29) прогнан на реальном движке OneScript — все зелёные

Generated by Claude Code

claude added 2 commits August 26, 2026 01:26
- Реализован contains(value) на основе Найти()/Find() базового массива
- Добавлены тесты: наличие метода, поиск существующего и
  отсутствующего значения, поведение на пустом стеке,
  отсутствие побочных эффектов на count()
- Обновлена документация (doc-комментарий по стандарту 1С в коде,
  README): добавлен contains() в таблицу методов и пример
  использования, отмечена семантика сравнения (строгое равенство)
- Приватная переменная mStackContainer переименована в _StackContainer
- Внутри непустых скобок добавлены пробелы после ( и перед )
- Между функциями/процедурами — двойной перенос строки
- Внутри каждой функции добавлены пустые строки сразу после сигнатуры
  и перед End(Function|Procedure)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants