Question:

JavaScript code??!! help me!!

by | earlier

whats wrong!!!??

<head>

<script language="JavaScript">



</script>

</head>

Tags:

Report

Answer The Question I've Same Question Too

Follow Question

2 ANSWERS

Sort By: Date | Rating

Mehdi, since the prompt() method evaluates a string, to recast the method as a new String object as you did is not recommended and is not a best practice. It is also not necessary to declare the user1 variable explicitly as a string either. The rest of your reply to the original poster was spot-on.
Report (0) (0) | earlier
Several things:

1. You're using obsolete code. Ditch the  Also, don't use the "language" attribute, use "type".

2. End each command line with a semicolon

3. Use correct, up to date JavaScript.

4. Don't use double quotes in a string that's inside double quotes.

<script type="text/JavaScript" >

var user1 = new String ( 'Spiderpig1' );

var username = new String ( prompt ( 'Please write your

username below:' ) );

if ( user1 == username ) {

alert ( 'Welcome back to Spellingbees,

Simpsonspig1' );

}else{

alert ( 'AHHHHHHHHH!!!!! Well, fine, I will

"obligate" you access to this site. Gosh!!!' );

}

</script>

EDIT:

You make a very valid point, mnmboy.

I've gotten into the habit of always declaring strings *as* strings, because that will allow my editor to use auto-code... which in turn guarantees me that if I call a string function (like .Search, or .ToLowerCase, for example), it'll work.

For what the strings are used for in this question, the declarations may, indeed, be superfluous.
Report (0) (0) | earlier