for loop print - as many as the word length?

Question

How can I print out dashes "-" in the same length as the word length? I used for-loop but only got 1 dash.

 for(int i=0; i<secretWordLen; i++) theOutput = "-";

Main:

public String processInput(String theInput) { String theOutput = null; String str1 = new String(words[currentJoke]); int secretWordLen = str1.length(); if (state == WAITING) { theOutput = "Connection established.. Want to play a game? 1. (yes/no)"; state = SENTKNOCKKNOCK; } else if (state == SENTKNOCKKNOCK) { if (theInput.equalsIgnoreCase("yes")) { //theOutput = clues[currentJoke]; //theOutput = words[currentJoke]; for(int i=0; i<secretWordLen; i++) theOutput = "-"; state = SENTCLUE;

possible duplicate of stackoverflow.com/questions/7107297/… — holap
– holap, Commented Nov 18, 2013 at 15:05

Cyrille Ka · Accepted Answer · 2013-11-18 15:10:10Z

Use StringBuilder:

StringBuilder builder = new StringBuilder(); for(int i=0; i<secretWordLen; i++) { builder.append('-'); } theOutput = builder.toString();

This will do if all you want in theOutput is the series of dashes. If you want to have something before, just use builder.append() before appending the dashes.

The solution with += would work too (but needs theOutput to be initialized to something before, of course, so you don't append to null). Behind the scenes, Java will transform any += instruction into a code that uses StringBuilder. Using it directly makes it more clear what is happening, is more efficient in this case, and is in general a good thing to learn about how String are manipulated in Java.

Thanks, yup the += solution will append null as you mentioned if not initialized.

user1567896 · Accepted Answer · 2013-11-18 14:57:01Z

1

You are overwriting your output-variable in each iteration.

Change it to:

theOutput += "-";

answered Nov 18, 2013 at 14:57

user1567896

2,4082 gold badges27 silver badges43 bronze badges

Comments

sternze · Accepted Answer · 2013-11-18 14:55:54Z

0

instead of theOutput = "-"; use theOutput += "-";

answered Nov 18, 2013 at 14:55

sternze

1,6442 gold badges10 silver badges15 bronze badges

Comments

Suresh Atta · Accepted Answer · 2013-11-18 14:55:58Z

You have to append the result each time.

for(int i=0; i<secretWordLen; i++) theOutput += "-";

When you write theOutput += "-"; that's the shorthand of

 theOutput = theOutput +"-";

Collectives™ on Stack Overflow

for loop print - as many as the word length?

4 Answers 4

1 Comment

Comments

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

Comments

Comments

Comments

Linked

Related