配列内のオブジェクトの問題Javascript

Question

オブジェクトの配列を検索する際に問題が発生しています。基本的に私のページで必要となるのは、氏名、ユーザー名、電子メール、パスワードなど、私が入力した情報を使って新しい「クライアント」を作成することです。これらのクライアントのそれぞれは、以下に示すように、配列内のオブジェクトです。

var clientlist = [{"username":"John","fullname":"John Doe", 
"email":"john.doe@hotmail.com","type":"client","password":"jdoe2"}, 

このクライアントは既にjsファイル内に作成されています。この同じ構造でこのアレイに追加する新しいオブジェクトを作成する必要があります。例えば、

var clientlist = [{"username":"Peter","fullname":"Peter Jones", 
"email":"peter.jones@hotmail.com","type":"client","password":"pjones1"}, 

私は、コードを書かれているが、それは私がFirebugのを実行したときに、私はすべての要素が「値が」ユーザー名を除き、正しく追加されていることを確認することができ、正常に動作しません。私が追加しているユーザー名が既に存在するかどうかを確認するためにユーザー名を検索することができない場合は、構文ミスかもしれません。私は以下の完全なコードを残し、支援のために事前に感謝します！

var clientlist = [{"username":"John","fullname":"John Doe", 
"email":"john.doe@hotmail.com","type":"client","password":"jdoe2"}, 

var Client = {}; 

function NewClient(){ 
var found; 
var user = $("#username").val(); 

for (var i = 0; i < clientlist.length; i++) { 
    if (clientlist[i].username == user) { 
     found = true; 
    }else{ 
     found = false; 
    } 
} 

if (found == true){ 
    $("#msj").html("User already exists!"); 
} 
else if(found == false){ 
    Client["fullname"] = $("#fullname").val(); 
    Client["username"] = user; 
    Client["email"] = $("#email").val(); 
    Client["type"] = "client"; 
    Client["password"] = $("#password").val(); 

    clientlist[clientlist.length] = Client; 

    $("#msj").html("New client has been created"); 
} 

}

Answer 1

ほとんどミスは：

• は
• が実際に以下のコードは、いくつかの修正動作するはず

新しく追加されたクライアントにプッシュし忘れたclientlist配列を閉じるには、忘れましたあなたが途中で作った間違い。

var clientlist = [{ 
    "username": "John", 
    "fullname": "John Doe", 
    "email": "john.doe@hotmail.com", 
    "type": "client", 
    "password": "jdoe2" 
}]; 

function NewClient() { 
    var found = false; 
    var user = $("#username").val(); 

    for (var i = 0; i < clientlist.length; i++) { 
    if (clientlist[i].username == user) { 
     found = true; 
    } else { 
     found = false; 
    } 
    } 

    if (found) { 
    $("#msj").html("User already exists!"); 
    } else { 
    var newUser = { 
     fullname: $("#fullname").val(), 
     username: user, 
     email: $("#email").val(), 
     type: "client", 
     password: $("#password").val() 
    } 
    clientlist.push(newUser); 
    $("#msj").html("New client has been created"); 
    } 
} 

あなたのためのフィドルメイド：あなたは `clientlist`値の末尾に` `]が欠落している http://codepen.io/gabrielgodoy/pen/xOxoWw?editors=1011

Answer 2

私が推測するには、いくつかの問題を抱えています。

1. ForループclientList
2. のブラケットを終了し、
3. た変数は、クライアントリストに新規ユーザーをプッシュ。

私はそれらを修正して、以下のことが含まれています。あなたが作った

<script> 
var clientlist = [{"username":"John","fullname":"John Doe", 
"email":"john.doe@hotmail.com","type":"client","password":"jdoe2"}] 

function NewClient(){ 
    var found=false; 
    var user = $("#username").val(); 

    for (var i = 0; i < clientlist.length; i++) { 
     if (clientlist[i].username==user) { 
      found = true; 
      break; 
     } 
    } 

    if (found){ 
     $("#msj").html("User already exists!"); 
    } 
    else{ 
     var newUser={ 
      fullname:$("#fullname").val(), 
      username:user, 
      email:$("#email").val(), 
      type:"client", 
      password:$("#password").val() 
     } 

     clientlist.push(newUser); 
     $("#msj").html("New client has been created"); 
    } 
} 
</script>