Skip to content

Recommend using isEmpty() instead of size(). #3139

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 16, 2024
Merged

Recommend using isEmpty() instead of size(). #3139

merged 1 commit into from
Apr 16, 2024

Conversation

dukbong
Copy link
Contributor

@dukbong dukbong commented Apr 15, 2024

In Java, it is recommended to use isEmpty() instead of comparing the size() of a collection object with 0.

  • Added isEmpty() method to the ResultLoaderMap object

@coveralls
Copy link

Coverage Status

coverage: 87.168% (+0.001%) from 87.167%
when pulling b57b86c on dukbong:Replace
into e9e9a29 on mybatis:master.

@hazendaz hazendaz self-assigned this Apr 16, 2024
@hazendaz hazendaz merged commit 7ae38a2 into mybatis:master Apr 16, 2024
@dukbong dukbong deleted the Replace branch April 16, 2024 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@hazendaz hazendaz

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dukbong @coveralls @hazendaz