2016-04-01 10 views
0

このエラーが発生したときに私のウェブサイトを改装していました。
シンプルなユーザー名 - パスワードのJavascript呼び出し関数です。 (私は任意のユーザー名、パスワード、リンク、または関数名を与えたくない機密保持のため)ここでJavascript OnClick機能が動作しません(部分コード付き)

はコードです:

<head> 
<script> 
function ----() { 
var username = prompt("Please enter your username."); 
if (username.toLowerCase() == "-----"){ 
    var password = prompt("Please enter your password."); 
    if (password.toLowerCase() == "-----"){ 
    window.open('http://example.com',"_self") 
} 
else{ 
alert('Incorrect password. Please try again.') 
} 
} 
else{ 
alert('Incorrect username. Please try again.') 
} 
} 
</script> 
</head> 

<body> 
<p><a onclick="----()" href="javascript:void(0);">------</a></p> 
</body> 

私のコードを批判しないでください。私は1〜2年間しかコーディングしていません。私はこの改装の経験からもっと学びたいです!

ありがとうございます!
ジェファーソン・ユー

+1

誰もがあなたのコード内でこれらの関数名、ユーザ名、およびパスワードを観察することができました。これはまったく安全ではありません。 – Comptonburger

+0

これはあまり安全ではないことは分かっていますが、このコードは深刻なウェブサイトではなくカジュアルな目的で使用されるため、大丈夫です。 – JeffersonYu1

答えて

0

冗長なelse文があります。説明するためにfooというサンプル関数を追加しました。あなたのコードに字下げを適用してみましょう。これは、構文上のエラーを作成する手間を省きます。あなたのサイトを閲覧し

<html> 
<head> 
<script> 
function foo() { 
    var username = prompt("Please enter your username."); 
    if (username.toLowerCase() == "user"){ 
     var password = prompt("Please enter your password."); 
     if (password.toLowerCase() == "pass"){ 
      window.open('http://example.com',"_self"); 
     } 
    } 
    else{ 
     alert('Incorrect password. Please try again.'); 
    } 
} 
</script> 
</head> 
<body> 
<p><a onclick="foo()" href="javascript:void(0);">click me</a></p> 
</body>enter code here 
</html> 
+0

それでも動作しません... www.dsandb.weebly.com/login.htmlのリンクを試すことができます – JeffersonYu1

+0

この問題を迅速かつ効果的に解決するために手伝ってください。 – JeffersonYu1

関連する問題