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

Update languageSelector.ts #237535

Closed
wants to merge 5 commits into from
Closed

Conversation

lighting9999
Copy link

Update:
简化 score 函数的逻辑:
用 reduce 代替 for 循环来计算数组中的最大分数,避免手动管理 maxScore 变量。
将 selector 的类型处理移到不同的分支中,使得逻辑更清晰,并且避免嵌套过多。

类型保护与检查:
在 targetsNotebooks 函数中,增加了对 LanguageFilter 的类型检查,确保其存在 notebookType 属性时才返回 true,并避免了不必要的类型断言。
score 函数中的 pattern 通过 normalizePattern 函数进行预处理,避免了重复的 if 判断。
路径模式的标准化:

使用 normalizePattern 函数对模式进行标准化,确保相对路径的统一性。这个函数会根据传入的 string 或 IRelativePattern 类型调整路径。
可读性增强:

增加了详细的注释,说明了每个功能模块的作用。
score 函数现在更加简洁,逻辑更加直观,减少了重复代码。
主要变化:
使用 reduce 简化了数组处理逻辑。
使用 normalizePattern 统一处理路径模式的标准化。
通过类型保护减少了不必要的类型断言。

@mjbvz mjbvz closed this Jan 9, 2025
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.

3 participants