On the recursive calls to mergeResource(), the call to findMatchingChildResource() to find an existing child Resource need not be made at all, since it will always return the same Resource that is being merged (since its parent was just added to inventory). This will also eliminate the need to execute the (existingResource != null) if-block.