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

17 #361

Open
wants to merge 1 commit into
base: fix
Choose a base branch
from
Open

17 #361

wants to merge 1 commit into from

Conversation

boerzia
Copy link

@boerzia boerzia commented Nov 24, 2024

2022112452
语法错误修复:
补充缺少的分号(如 put('T', 3) 后添加 ;)。
修正声明错误,将 List ans[] 改为 List ans。
修正循环中 i === 0 为 i = 0。
使用 s.length() 替代 s.length 以正确调用字符串的长度方法。
逻辑错误修复:
确保位操作正确,去掉无效的括号,将 {x << 2} 改为 (x << 2)。
确保滑动窗口的实现逻辑正确,移动计算与哈希映射的结合。
代码规范改进:
初始化变量时使用泛型简化:new ArrayList<>()。
确保测试条件合理,例如长度不足 10 时直接返回结果。
整体验证:
修复后,确保功能符合题意,即返回长度为 10 的所有重复 DNA 子序列。

i Please enter the commit message for your changes. Lines starting
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.

1 participant