Different functions in Java I wonder what is better code practice or just what looks better in Java:

Version 1:

protected Boolean doSomething(int amount) {
    if (amount &lt; 1) return false;

    return insertToDb(fillImages(fetchInfo(REQUEST_URL, amount)));
}


Version 2:

protected Boolean doSomething(int amount) {
    if (amount &lt; 1) return false;

    Data[] data = fetchInfo(REQUEST_URL, amount);

    fillImages(data);
    // or even
    // data = fillImages(data);

    return insertToDb(data);
}


This could be a problem if we had more functions chained like that. I would like to make my code nicer, for anyone else who might have to edit it, and just to know that I am not writing anything that looks stupid.
I definitely prefer the second version. The steps taken just "leap out" of the code and the method is easier to read. It will also be a lot easier to debug (or to pinpoint the problem in a stack trace).

Always go for readability, it makes for easier understanding when going back to your old code and it's a lot easier for someone else to understand what is going on.

I also prefer the "data = fillImages(data)" if possible. It makes clear that the data is modified.