Сортировка возвращаемых результатов callHierarchy, убраны мусорные переменные из outgoingCalls #3427
Conversation
…ременные из outgoingCalls
WalkthroughВ данном изменении обновлён класс Changes
Sequence Diagram(s)sequenceDiagram
participant Client as Клиент
participant CHP as CallHierarchyProvider
participant Filter as isSymbolSupported
participant Sort as callHierarchyItemComparator
Client ->> CHP: Вызывает outgoingCalls()
CHP ->> CHP: Собирает исходящие вызовы
CHP ->> Filter: Проверка каждого символа
Filter -->> CHP: Результат проверки
CHP ->> Sort: Сортировка вызовов
Sort -->> CHP: Отсортированный список
CHP -->> Client: Отправка списка вызовов
Poem
✨ Finishing Touches
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
src/test/resources/providers/callHierarchy.bsl (1)
8-9: Неиспользуемые переменные в функции.Объявленная переменная
Аи присвоенное значение переменнойБне используются в теле функцииПерваяФункция(). Рекомендуется либо использовать эти переменные, либо удалить их, если они не нужны для тестирования.- Перем А; - Б = 1;src/main/java/com/github/_1c_syntax/bsl/languageserver/providers/CallHierarchyProvider.java (1)
143-145: Четкое определение поддерживаемых символов.Метод
isSymbolSupportedявно определяет, какие типы символов поддерживаются в иерархии вызовов. Рекомендуется добавить документацию к методу.+ /** + * Проверяет, поддерживается ли символ в иерархии вызовов. + * + * @param symbol Проверяемый символ + * @return true, если символ является методом + */ private static boolean isSymbolSupported(Symbol symbol) { return symbol.getSymbolKind() == SymbolKind.Method; }
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
src/main/java/com/github/_1c_syntax/bsl/languageserver/providers/CallHierarchyProvider.java(7 hunks)src/test/java/com/github/_1c_syntax/bsl/languageserver/providers/CallHierarchyProviderTest.java(1 hunks)src/test/resources/providers/callHierarchy.bsl(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (19)
- GitHub Check: build (23, windows-latest)
- GitHub Check: build (23, ubuntu-latest)
- GitHub Check: build (21, macOS-latest)
- GitHub Check: build (21, windows-latest)
- GitHub Check: build (21, ubuntu-latest)
- GitHub Check: build (17, macOS-latest)
- GitHub Check: build (17, windows-latest)
- GitHub Check: build (17, ubuntu-latest)
- GitHub Check: Analyse
- GitHub Check: Benchmark
- GitHub Check: build (23, macOS-latest)
- GitHub Check: build (23, windows-latest)
- GitHub Check: build (23, ubuntu-latest)
- GitHub Check: build (21, macOS-latest)
- GitHub Check: build (21, windows-latest)
- GitHub Check: build (21, ubuntu-latest)
- GitHub Check: build (17, macOS-latest)
- GitHub Check: build (17, windows-latest)
- GitHub Check: build (17, ubuntu-latest)
🔇 Additional comments (5)
src/main/java/com/github/_1c_syntax/bsl/languageserver/providers/CallHierarchyProvider.java (4)
61-63: Хорошее решение для сортировки элементов.Добавление компаратора
callHierarchyItemComparatorобеспечивает стабильную сортировку результатов по двум критериям: detail и selectionRange. ИспользованиеRanges.compareдля сравнения диапазонов является правильным подходом.
99-100: Улучшение читаемости результатов с помощью сортировки.Сортировка входящих вызовов с использованием компаратора улучшает читаемость и предсказуемость результатов. Использование
toList()вместоcollect(Collectors.toList())делает код более лаконичным.
116-116: Правильная фильтрация символов по типу.Добавление фильтрации
isSymbolSupportedпозволяет исключить нерелевантные символы, оставляя только методы в иерархии вызовов.
124-125: Согласованная сортировка исходящих вызовов.Применение того же компаратора к исходящим вызовам обеспечивает единообразие в представлении результатов.
src/test/java/com/github/_1c_syntax/bsl/languageserver/providers/CallHierarchyProviderTest.java (1)
55-55: Корректное обновление позиции в тестах.Позиция
secondFunctionDeclarationPositionобновлена в соответствии с изменениями в тестовом файле. Все тесты проходят успешно с новой позицией.
|
Описание
Fix #3426
Связанные задачи
Closes
Чеклист
Общие
gradlew precommit)Для диагностик
Дополнительно