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

Add ImmutableRangeMap.toImmutableRangeMap(Function keyFunction, Function valueFunction, BinaryOperator mergeFunction) #6822

Open
4 tasks done
perceptron8 opened this issue Nov 6, 2023 · 1 comment

Comments

@perceptron8
Copy link
Contributor

1. What are you trying to do?

I'd like to (be able to) build ImmutableRangeMap from stream using collector given that some of the resulting ranges overlap.

record Job(Instant start, Instant end) {
	Job {
		Objects.requireNonNull(start);
		Objects.requireNonNull(end);
	}
	
	Range<Instant> span() {
		return Range.closedOpen(start, end);
	}
}

2. What's the best code you can write to accomplish that without the new feature?

RangeMap<Instant, Integer> load(List<Job> jobs) {
	RangeMap<Instant, Integer> load = TreeRangeMap.create();
	for (Job job : jobs) {
		load.merge(job.span(), 1, Integer::sum);
	}
	return ImmutableRangeMap.copyOf(load);
}

3. What would that same code look like if we added your feature?

RangeMap<Instant, Integer> load(List<Job> jobs) {
	return jobs.stream().collect(ImmutableRangeMap
		.toImmutableRangeMap(Job::span, always -> 1, Integer::sum));
}

(Optional) What would the method signatures for your feature look like?

public static <T extends @Nullable Object, K extends Comparable<? super K>, V>
Collector<T, ?, ImmutableRangeMap<K, V>> toImmutableRangeMap(
          Function<? super T, Range<K>> keyFunction,
          Function<? super T, ? extends V> valueFunction,
          BinaryOperator<V> mergeFunction
)

Concrete Use Cases

Real use case involves checking if given family is entitled to receive social care. This is based on number of members, their health, income, several legal documents and so on. All conditions are time based and so are the grants.

Packages

com.google.common.collect

Checklist

@Mirrious
Copy link

Hi, I'm interested in this issue, could you please assign it to me?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants