Today I found myself with a common problem: I had a delimited string of an unknown number of parts that that I needed split apart and process. Prior to Java 8, implementing that might looked something like this:
for (String part : string.split("\\n")) {
if (!myList.contains(part)) {
if (!part.isEmpty()) {
myList.add(part);
}
}
}
While that works and seems to be pretty efficient, I felt it could use a stream makeover, as I find the stream operations to be clearer and more concise. What I ended up with was something like this:
Pattern.compile("\\n")
.splitAsStream(string)
.filter(s -> !myList.contains(s))
.forEach( s -> myList.add(s));
I could also have used Arrays.stream()
rather than Pattern
:
Arrays.stream(string.split("\\" + DELIMITER))
.filter(s -> !myList.contains(s))
.filter(s -> !s.isEmpty())
.forEach(s -> myList.add(s));
I haven't done any profiling to see if Pattern.compile()
has any non-negligible performance impact versus String.split()
(and I probably won't, but you can easily "cache" the compiled pattern in an instance or static variable if needed :), but I will point out this difference: when using split()
, streamed or not, we may get a blank value in some situations, so we need to check for that (notice the calls toString.isEmpty()
in both of those implementations). With the Pattern
-based implementation, we don't have that problem.
At any rate, there you have it: you can convert String.split()
to a stream-based implementation fairly easily, and, I think, get more readable code out of it. Any performance implications are left as an exercise for the reader. :)
Recently at work, we found an odd scenario with a REST (-ish ;) endpoint from another team: If the request provided a list of, say, 11 IDs in the query string, the system would only return information on the first 10 of them, silently dropping anything over that seemingly odd limit. The initial reaction was of, course, "Well, let's just increase the limit." To be honest, I had the same reaction, but then I remembered one of my favorite quotes, known as Chesteron's Fence:
In the matter of reforming things, as distinct from deforming them, there is one plain and simple principle; a principle which will probably be called a paradox. There exists in such a case a certain institution or law; let us say, for the sake of simplicity, a fence or gate erected across a road. The more modern type of reformer goes gaily up to it and says, "I don't see the use of this; let us clear it away." To which the more intelligent type of reformer will do well to answer: "If you don't see the use of it, I certainly won't let you clear it away. Go away and think. Then, when you can come back and tell me that you do see the use of it, I may allow you to destroy it."
G.K. Chesterton
Chesterton's context, politics in Great Britain of the 1920s, is, of course, quite different from a software development shop almost 100 years later, but the message is still extremely appropriate: Before you go tearing things down or otherwise changing something you've found, you really need to understand not only what you're changing, but why it was made that way in the first place.
In the case of this REST call, we should ask questions like
-
Are there system load concerns, such as memory or processing time?
-
Are there concerns about the on-the-wire response size?
-
Was there an explicit Product Management decision to set the limit this low for business reasons we don't see reflected in the code?
And so on. Until we can answer those questions (or reasonably rule them out as irrelevant), we need to be very hesitant in making the change. Once we've explained the original developer(s) built that fence, then we can talk about ripping it down.
As the title states, my book has finally been published. You can get it (and you know you want to) at a number of places:
I had fun and learned a lot while working on this. I hope you find it useful.
I recently struggled trying to text in a JavaFX ListView to wrap inside the container like I asked it to, rather than extend (and disappear) past the boundaries of the container. After some discussion on Twitter and a bit of Googling, I found an answer that I thought I'd share here to, perhaps, save someone some time.
In a project I'm working on for my book, I need to share classes between two applications. One, an Android project, requires Java 8. The other, a desktop JavaFX application, needs to run under Java 9, complete with module support. The problem with this is that the Maven tooling isn't quite ready for Java 9, so it's not as simple as I would like. I have, however, found a solution that seems to work.