Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

JavaScript The Solution

Nikola Jankovic
Nikola Jankovic
10,793 Points

Just wont read .length ...

var question1= prompt('What is your name'); // 3. Add a second a prompt dialog to capture input from the user and store it in a second variable var question2= prompt('What is your surname'); // 4. Create a third variable and which combines an uppercase version values in the two other variables separated by a space. For example, if the first two variables contain "sally" and "forth", this third variable should contain the string value "SALLY FORTH" var broj= document.write(question1.toUpperCase()+ ' ' +question2.toUpperCase() ); // 5. Create a fourth variable to store a number. The number should be the total number of characters in the third variable. var complete= broj.length; // 6. Add an alert dialog box that says "The string '[insert value of third variable here]' is X number of characters long." For example, if the third variable contained the string "SALLY FORTH" then the alert dialog should says "The string 'SALLY FORTH' is 11 characters long." alert("The string " + complete );

Nikola,

I think you want to concatenate the two variables that you get from the prompt box. Here is how your code should look like:

var question1= prompt('What is your name');
var question2= prompt('What is your surname');
var broj= (question1.toUpperCase()+ ' ' +question2.toUpperCase() );
var complete= broj.length;
alert("The string " + complete );
Gustavo Winter
Gustavo Winter
Courses Plus Student 27,382 Points

you just need to change this

var broj= document.write(question1.toUpperCase()+ ' ' +question2.toUpperCase() );

i will give you a little tip, try to remove "document.write" and test the code again.

1 Answer

Steven Parker
Steven Parker
231,269 Points

You have some stray code in there.

It's a bit hard to read without markdown formatting, but on your step 4 line instead of assigning your value based on combining the strings, you assign it to the result of calling "document.write" with the string combination as the argument. Calling "document.write" is not part of this challenge.

Please see the Markdown Cheatsheet link at the bottom of the "Add an Answer" section for instructions on how to format posted code. :arrow_heading_down: