Draft
Conversation
Member
|
No AI! |
Contributor
Author
测试了一下，确实有点毛病，SplitSelect 的页面显示不出包了，至于AI ，本来人写的算法也不咋样... AI的部分不涉及复杂的逻辑，表面上看是没问题，但实际上问题又有，怪 |
Member
|
太坏了 |
Contributor
Author
|
你要不要试试能不能把这个新算法修一下。我觉得这个新算法还行，而且写的比老实现精简。因为使用了一些工具API |
Member
|
你都不知道效果如何知道它是精简还是缺胳膊少腿 |
Contributor
Author
这个实现大体上和旧实现是一样的。只是做了代码精简处理，外加淘汰掉了一些糟糕的字符串匹配。不过在搞清楚为什么split选择会炸之前，只能先挂着了 |
Member
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
使用AI 重构了过去的糟糕代码，请审查一下