Can someone have a quick look at my version of the shout code please?

Question

Hi there,

I tried to come up with a solution for the shout code before looking at the solution and here it is:

var shout = prompt("What do you want to shout?"); console.log(shout); alert(shout.toUpperCase()+"!"); console.log(shout.toUpperCase()+"!");

It is different to the teacher's code but does work. Can someone give me a bit of feedback in terms of acceptable this is?

Thanks!

Answer 1 · 2017-10-13T11:59:11Z

October 13, 2017 11:59am

Console.log isn't that important here, so you can remove it. Though early tutorials tend to focus on console.log as a means of determining values and whatnot, it's really just used on the developer side.

Your code does work, so good job! One tweak I'd say is important is to store the new values into variables, just so they can be accessed again. Remember, alert and console.log don't actually store anything. You need a variable to put it in--sort of like storing it in a box. Knowing when you need to store values in variables is going to be very important down the road.

You can either do var message = shout.toUpperCase() + "!"; or do Dragos's way of just changing the existing variable.

Answer 2 · 2017-09-06T21:14:43Z

September 6, 2017 9:14pm

or better ,instead creating a new variable , try this:shout= shout.toUpperCase() + "!"; and then print shout

Answer 3 · 2017-09-06T21:02:41Z

September 6, 2017 9:02pm

for me it looks great.but i think you should create before the alert a variable and asigned to that variable shout.toUpperCase()+"!";.and then put the variable created like argument for alert and last console.log

Welcome to the Treehouse Community

Looking to learn something new?

Phil Sta

Phil Sta

Can someone have a quick look at my version of the shout code please?

3 Answers

Gabriel Rumbaut

Gabriel Rumbaut

dragos busuioc

dragos busuioc

dragos busuioc

dragos busuioc