添加自定义字典支持，并修复不能正确插入F#特殊标识符的 bug

stratosblue / ChinesePinyinIntelliSenseExtender

VisualStudio中文代码拼音补全拓展

MIT License

32 stars 6 forks source link

Closed ijklam closed 2 years ago

ijklam commented 2 years ago

添加自定义字典支持，并修复不能正确插入F#特殊标识符的 bug。

通过使用新的字库顺便支持了多音字和繁体字的输入。

wherewhere commented 2 years ago

围观大佬

stratosblue commented 2 years ago

@Tangent-90 非常感谢大佬的贡献

代码行数有点多，没法在线Review了。。。我需要抽个时间看完代码再进行合并。。。

大概看了一点点代码，有一点小疑问:

ChineseCheckUtil.cs 中缓存由 ConditionalWeakTable 变更为了 ConcurrentDictionary 是出于什么考虑呢；
CharacterTable.CreateTableAsync 中 SemaphoreSlim 似乎没有锁住任何东西；

ijklam commented 2 years ago

ChineseCheckUtil.cs 中缓存由 ConditionalWeakTable 变更为了 ConcurrentDictionary 是出于什么考虑呢；

是为了防止添加重复的 key，在我这边运行偶而会报 key 重复的错，所以我就全部用了 ConcurrentDictionary，用 ConditionalWeakTable 的话还要锁住整个函数

CharacterTable.CreateTableAsync 中 SemaphoreSlim 似乎没有锁住任何东西

这个确实弄错了😂😂😂，锁应该放在外面的

stratosblue commented 2 years ago

@Tangent-90 哈哈哈，好的，感谢做了这么多修改。我先合并了，接下来稍做修改后再发布

stratosblue commented 2 years ago

@Tangent-90 一不注意改的有点多，新版本已发，大佬看下按预期工作吗

ijklam commented 2 years ago

@stratosblue

大佬太强了，居然能想到把字典都放在一起用，我当时就只想一次用一个的，这个好

这样也可以方便的在自带的字典上加自定义拼写，赞

stratosblue commented 2 years ago

@Tangent-90 哈哈哈，大佬过誉了，还是大佬引入字典文件的思路好