Reducing For loops - Code Optimization

Question

I am new to Java and looking for code optimization techniques. This code is giving the expected output using two for loops and I want to reduce/eliminate the loops (if possible).

public static void main(String[] args) { String dummy = "Hello how are you"; String[] strArr = dummy.split(" "); for(int i=0; i < strArr.length;i++){ String word = strArr[i]; for(int j=word.length(); j > 0; j--){ System.out.print(word.charAt(j - 1)); } System.out.print(" "); } }

Output: olleH woh era uoy

Please advice.

I am not asking for CodeReview, rather the optimization techniques. — user182944
– user182944, Commented Aug 20, 2017 at 15:59
It is impossible to reduce the number of loops. You can change the type of loop to a while or a do-while loop but you need at least 2 looping constructs if you want your code to work — Coder
– Coder, Commented Aug 20, 2017 at 16:02
The problem with this question is that you did not present a problem - you are prematurely optimizing. This doesn't cause a bottleneck, you don't specify any time requirements, etc.. You just wanna "optimize" it, which doesn't make much sense without context. What do you feel is wrong with your code? — Vince
– Vince, Commented Aug 20, 2017 at 16:10
@user182944: Added an example where you "unnest" the for loops. — Gil Vegliach
– Gil Vegliach, Commented Aug 20, 2017 at 16:39

Sergey Kalinichenko · Accepted Answer · 2017-08-20 16:04:44Z

Since the complexity of printing the output would remain the same, all you can do is to "hide" the loops into existing methods that do what you want, not eliminate them, because the amount of work the system needs to perform remains the same in terms of the number of characters that need to be processed.

You can hide the nested loop by using string reversal technique described in this Q&A. The outer loop can be pushed into String.join:

String[] data = {"quick", "brown", "fox"}; System.out.println( String.join( " " , Arrays.stream(data) .map(s -> new StringBuilder(s).reverse().toString()) .toArray(String[]::new) ) );

Demo.

@GilVegliach .collect(Collectors.joining(" ")) would be cooler.

Gil Vegliach · Accepted Answer · 2017-08-20 16:38:01Z

As I stated in my comment the complexity is already O(n) and you cannot get better because you need to read the input string. Anyway, there is a way to "unnest" the two for loops: reverse the whole string, split on spaces, and reverse the resulting array.

Example:

"Hello how are you" "uoy era woh olleH" "uoy" "era" "woh" "olleH" "olleH" "woh" "era" "uoy"

Code:

public static void main(String[] args) { String dummy = "Hello how are you"; int n = dummy.length(); StringBuilder sb = new StringBuilder(n); while (--n >= 0) sb.append(dummy.charAt(n)); String[] tokens = sb.toString().split(" "); n = tokens.length; while (--n >= 0) System.out.print(tokens[n] + " "); }

Instead, if you are after cools java 8 Stream tricks, read dasblinkenlight's answer.

nagendra547 · Accepted Answer · 2017-08-20 16:45:02Z

This is another simple solution using StringTokenizer.

It takes only O(n).

public static void main(String[] args) { System.out.println(reverseStringWordByWord("Hello How are you")); } public static String reverseStringWordByWord(String input) { StringBuilder result = new StringBuilder(); StringTokenizer sToken = new StringTokenizer(input, " "); while (sToken.hasMoreTokens()) { StringBuilder thisToken = new StringBuilder(sToken.nextToken()); result.append(thisToken.reverse() + " "); } return result.toString(); }

Collectives™ on Stack Overflow

Reducing For loops - Code Optimization

3 Answers 3

1 Comment

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Linked

Related