Fabricio Cabral

org.cactoos.text.Replaced uses StringBuffer instead of StringBuilder

10

The class `org.cactoos.text.Replaced` uses `StringBuffer` instead of `StringBuilder` ([here](https://github.com/yegor256/cactoos/blob/3675019c59c64477e8b80a04a25f86d717d9ac4b/src/main/java/org/cactoos/text/Replaced.java#L88)). Let's change it because `StringBuilder` is more efficient.

IterableOf, MapEntry and MapEnvelope should use HashCode instead of own hash code algorithm

7

IterableOf, MapEntry and MapEnvelope (or MapOf if #1384 has been done) should use HashCode instead of own hash code. So let's refactor them to use `HashCode`.

No unit tests for scalar.NoNulls, text.NoNulls, bytes.NoNulls and iterable.NoNulls

4

The decorators `scalar.NoNulls`, `text.NoNulls`, `bytes.NoNulls` and `iterable.NoNulls` haven't unit tests as your "coursins" `collections.NoNulls`, `func.BiFuncNoNulls` and etc.

Some object's attributes without final modifier

10

As you know, Cactoos follows Elegant Objects principles and one of them is immutable objects. To achieve it, you use `final` modifier in object's attributes. But in Cactoos code we...

Need to improve documentation about how uses TeeInput/TeeOutput classes

30

Let's improve the documentation about `TeeInput` and `TeeOuput` classes. For example: 1. How can I copy an `Input` to `Output`? 2. How can I copy an `Output` to `Output`? 3....

Create a Sleep class/object/decorator

6

Let's create a `Sleep` object, because it's util in a lot of situations and we already use `private void sleep()` in some places in Cactoos code.

Randomized has an issue about its constructors parameters order

13

Randomized has an issue about its constructors parameters order. Look: ``` public Randomized() // Ok public Randomized(final Integer len) // Ok public Randomized(final Scalar len) // Ok public Randomized(final Character......

Use of ins't readable value 16 << 10 to describe length

4

`DigestEnvelope`, `GzipInput`, `GzipOutput`, `InputAsBytes`, `ReaderAsBytes`, `WriterAsOutput`, `WriterAsOutputStream` and `LengthOf` use as buffer size value `16

I think we should move some classes to `cactoos-matchers` instead of keep them in `cactoos`. My list: - `BehavesAsCollection` - `BehavesAsList` - `EmptyClosableReader` - `FakeHandler` - `FakeLogger` - `BehavesAsMap` -...

Now, we've a class `Abbreviated` which does something like: `It's a very long phrase so it should be abbrevia...". But we can want abbreviate for left, like: `Reading file .../abc/cdef/fgh/File.docx`....