Card-Forge / Card-Forge/forge

Java Stream + forEachCardInGame

Open
#11,058 6 comments 0 reactions 1 assignee Claimed by @Hanmac View on GitHub
Game Mechanics keep
Dominant language
Java
Stars
2.7k
Forks
1.1k
Avg merge
1d 1h
Merged PRs (30d)
158

Description

The Visitor interface used for `Game::forEachCardInGame` could/should probably be reworked using JavaStream.
Either as Predicate, Consumer or Collector.

For this to clean up, classes like Zone/PlayerZone should return a (filtered) Stream of their cards instead of CardCollection objects.

I need to check out if `Stream.concat` or `Stream.Builder` is better for this.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.