Skip to content

第十四章 流式编程 #480

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

Closed
wants to merge 6 commits into from
Closed

Conversation

jerryqiang
Copy link
Contributor

修改其中一个演示代码结果

@sjsdfg
Copy link
Collaborator

sjsdfg commented Jun 17, 2020

你这个 PR 有问题吧

@jerryqiang
Copy link
Contributor Author

你这个 PR 有问题吧

是有点问题,应该是没同步至最新版的master
晚点我去拉一下,再pull

@jerryqiang
Copy link
Contributor Author

你这个 PR 有问题吧

似乎合并成功了,之前描述anyMatch 在第一个false停止执行,应改为

anyMatch(Predicate):如果流中的任意一个元素根据提供的 Predicate 返回 true 时,结果返回为 true。在第一个 true时停止执行计算。

@@ -1957,7 +1960,7 @@ Lambda 表达式中的第一个参数 `fr0` 是上一次调用 `reduce()` 的结
### 匹配

- `allMatch(Predicate)` :如果流的每个元素根据提供的 **Predicate** 都返回 true 时,结果返回为 true。在第一个 false 时,则停止执行计算。
- `anyMatch(Predicate)`:如果流中的任意一个元素根据提供的 **Predicate** 返回 true 时,结果返回为 true。在第一个 false 是停止执行计算
- `anyMatch(Predicate)`:如果流中的任意一个元素根据提供的 **Predicate** 返回 true 时,结果返回为 true。在第一个 true时停止执行计算
Copy link
Collaborator

Choose a reason for hiding this comment

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

true 前后加空格

Copy link
Contributor Author

Choose a reason for hiding this comment

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

你直接改下吧,然后关闭我这个request
我这个pr的确有问题

关于空格,文档里的代码格式风格约微有点不统一
有的for {
}
有的for{
}

Copy link
Collaborator

Choose a reason for hiding this comment

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

那是后者有问题
看见可以改

我都懒得改了。。。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

哈哈,影响不大
书的整体完成度更重要
格式问题用阿里规范检测后,统一再修改吧

@jerryqiang jerryqiang closed this Jun 17, 2020
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