関数がgetElementByIdで機能していない

Question

function signup() { 
    var username = document.getElementById("user").value 
    var password = document.getElementById("pass").value 
    if (username != false && password != false) { 
     alert("Whats up " + username); 
    } 
    } 

signup（）に設定するonclickでボタンを作成し、 "username"と "password"というIDを持つ2つのテキストボックスを作成しました。なんらかの理由でこのコードは機能しません。どんな助け？また、私はそれをちょうど警告ユーザー名にしようとしましたが、それでも動作しません。

編集：ここでは、入力のIDをmetioned入力

<form> 
<input type="text" class="cool" id="user" placeholder="Username"><br /> 
<input type="password" class="cool" id="pass" placeholder="Password"><br /> 
<input type="button" class ="button" value="Sign up" name="signup" onclick="signup()"> 
</form> 

Answer 1

を試してみてください。

コードHTMLは次のようになります。

<form> 
    <input type="text" class="cool" id="user" placeholder="Username"><br /> 
    <input type="password" class="cool" id="pass" placeholder="Password"><br /> 
    <input type="button" class ="button" value="Sign up" name="signup-changed" onclick="signup()"> 
</form> 

Answer 2

がusernameとpasswordされているあなたはfalseに等化されて、なぜそれがdocument.getElementById("username")とdocument.getElementById("password")

function signup() { 
    var username = document.getElementById("username").value 
    var password = document.getElementById("password").value 
    if (username != false && password != false) { 
     alert("Whats up " + username); 
    } 
    } 

Answer 3

すべきですか？

要素の値は、ブール値ではなく整数です。 ==や！=のコンパータを書くには、型stringを使用する必要があります。

username == 'string' 

あなたが唯一の関数名と同じを避けるために、ボタンのname属性値を変更する必要が

if(username.length > 0 && password.length > 0) 
    { 
     alert('hi ' + username); 
    }