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

Solution5 #367

Open
wants to merge 2 commits into
base: fix
Choose a base branch
from
Open

Solution5 #367

wants to merge 2 commits into from

Conversation

2023120243
Copy link

@2023120243 2023120243 commented Nov 30, 2024

2023120243
1、在numSubseq函数中,首先i的边界不对,没有考虑数组最后一个,另外取余数的符号不是“/”,而是“%”;
2、在pretreatment函数中,初值设置的不对,应该从1开始,从0开始会导致所有数组数值被处理成0;
3、在binarySearch函数中,循环的判断条件不对,另外再循环中的逻辑处理有错,需要更改;
4、另外,在本次改动中增加了测试文件,在整个文件模块中添加了Solution5_Test的子模块,将Solution5和测试文件均放在了子模块的src/zjs中。

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