API cleanup

[zoltish]

Good evening,

I think it would be productive to clean up the API around the library a bit.

There are for example numerous methods/variables that are rather confusing (e.g. box.panicModeRemoveAll() and box.all2), and quite a few calls include "internal" in their names even though they are public. Is there a reason for this?

Its not the end of the world, but I do think it would make a difference - and one of the things I appreciated with Realm earlier on was how everything had its place. You could never see any of its implementation details, and you could fully focus on creating awesome stuff.

Thanks!
Zoltan

[greenrobot]

Not our focus right now, but we'll look into it at some point. Public internal methods are usually methods which are used from sub packages (e.g. query) and thus cannot be package local.

[greenrobot-team]

I suppose the official API could be extracted to an interface (here Box) and internally an implementation with additional methods could be used (here BoxImpl).

Example:

public interface Box<T> {
}
public class BoxImpl<T> implements Box<T> {
}
Box<T> box = boxStore.boxFor(T.class);

-ut