Skip to content

Feature : Opitimize WordLookupLayout thru searchWithTiers#78

Draft
JYinherit wants to merge 7 commits into
OctagonalStar:mainfrom
JYinherit:main
Draft

Feature : Opitimize WordLookupLayout thru searchWithTiers#78
JYinherit wants to merge 7 commits into
OctagonalStar:mainfrom
JYinherit:main

Conversation

@JYinherit

Copy link
Copy Markdown
Contributor

lib/funcs/utili.dart 中原本用于题目混淆项筛选的 searchWithTiers(词根、词距检索分级)功能内嵌进了 WordLookupLayout 对应的词汇页面检索里

为了更好地展示检索到的不同相似级别的词汇,我调整了内部的布局:将原本简单的 GridView 重构为了 CustomScrollView + SliverGrid

现在当你在词汇总览里输入阿拉伯语单词进行搜索时,结果将会如下进行分类并分块显示:

完全匹配/包含:本身单词或包含该内容的延伸单词
极高相似 (同根同词性):词根相同且词性相同(派生联系紧密)
较高相似 (近根同词性):形似词
中等相似 (同根异性):同词根的其他派生词
近似词汇 (近根干扰项):可能的视觉、拼写易混淆词

如果你搜索的是中文,页面仍会像以前一样使用改进过的中文 Levenshtein 匹配返回排序结果,但加上了一个统一的「中文检索结果」标题来保持界面一致性

JYinherit and others added 5 commits April 18, 2026 15:44
Signed-off-by: JYinherit <jyinherit@qq.com>
Signed-off-by: JYinherit <jyinherit@qq.com>
Signed-off-by: JYinherit <jyinherit@qq.com>
Signed-off-by: JYinherit <jyinherit@qq.com>
@OctagonalStar OctagonalStar self-requested a review April 29, 2026 07:45

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

何意味?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

你删了原本的测试代码是过不了pr合并检查的。而且代码规范检查应该放另一个测试单元里,不适合和构建放一起

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

你原本那边的action不通过是因为你没有设置secret环境变量。我原本action构建apk中使用了自己的签名,而在你的仓库下没有相应的签名环境变量所以才会fail。

正确的解决方案是
你先rebase消除对构建文件的修改或者回滚那次修改
你自己生成自己的一份apk签名
参考我原本构建apk的命令中的环境变量名,设置你github仓库的secret环境变量

你这样直接build的发布,每次签名都不同,会导致软件不能正常更新。

@JYinherit JYinherit marked this pull request as draft April 29, 2026 08:09
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