google / guava

Google core libraries for Java

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Such content will give people a false sense of.

victory-wu opened this issue · comments

Description

question 1

toString in RangeMap is misleading

Although you have used brackets to distinguish,But this situation can still give people an illusion,Especially under high-intensity work.

I think a better way to express it would be a practical, unified unit of quantity.

question 2

The naming of Range.closed and Range.open is also misleading.

The first time I contacted it, I thought it was opening to closing range.

Suggested name change

Range.rangeInner
Range.rangeBound  or   Range.rangeOuter

Example

java code:

RangeMap<Integer, String> rangeMap = TreeRangeMap.create();
rangeMap.put(Range.closed(1, 5), "foo");
rangeMap.put(Range.open(1, 5), "bar"); 
System.out.println(rangeMap.toString());

Expected Behavior

toString out print:

[[1..1]=foo, [2..4]=bar, [5..5]=foo]

Actual Behavior

toString out print:

[[1..1]=foo, (1..5)=bar, [5..5]=foo]

Packages

com.google.common.collect

Platforms

Java 8

Checklist

Using [, ], (, and ) are the standard interval notation as specified by ISO 31-11.

I think what we have currently is working as intended.