Skip to main content
adicionou 375 caracteres ao conteúdo
Fonte Link
Isac
  • 25,5mil
  • 3
  • 31
  • 51

A lógica que está a utilizar para ir buscar achar o jogador funciona, só o acesso ao campo accountId é que não está correto, este obj[prop].accountId.

Se faz um foreach em participantIdentities significa que cada objeto desse foreach será algo como:

{  
     "player":{  
        "summonerName":"Khal Droggo",
        "accountId":1595535
     },
     "participantId":1
}

O que significa que para aceder ao accountId terá de fazer .player.accountId em vez de .accountId diretamente como tem.

Sendo assim basta trocar o if que tem para:

if (obj[prop].player.accountId == accountId) { //agora .player.accountId
}

Se player pode não existir então convem confirmar se existe utilizando a função hasOwnProperty o que ficaria assim:

if (obj[prop].hasOwnProperty("player") && obj[prop].player.accountId == accountId) {

Exemplo:

function callback(andress, fn){
    $.ajax({
        url: andress,
        type: 'GET',
        dataType: 'json',
        error: function(){},
        success: fn
    });
}

function stats(accountId){
    callback('https://api.myjson.com/bins/1ew27p', function(retorno) {
        var obj = retorno.participantIdentities;
        Object.keys(obj).forEach(function(prop) {
            if (obj[prop].hasOwnProperty("player") && obj[prop].player.accountId == accountId) { //if alterado aqui
                console.log(`Achou jogador ${obj[prop].player.summonerName} com participantId de ${obj[prop].participantId}`);
                return;//se achou não ve mais nenhum
            }
        });
    });
}

stats(211703728);
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>

A lógica que está a utilizar para ir buscar achar o jogador funciona, só o acesso ao campo accountId é que não está correto, este obj[prop].accountId.

Se faz um foreach em participantIdentities significa que cada objeto desse foreach será algo como:

{  
     "player":{  
        "summonerName":"Khal Droggo",
        "accountId":1595535
     },
     "participantId":1
}

O que significa que para aceder ao accountId terá de fazer .player.accountId em vez de .accountId diretamente como tem.

Sendo assim basta trocar o if que tem para:

if (obj[prop].player.accountId == accountId) { //agora .player.accountId
}

Exemplo:

function callback(andress, fn){
    $.ajax({
        url: andress,
        type: 'GET',
        dataType: 'json',
        error: function(){},
        success: fn
    });
}

function stats(accountId){
    callback('https://api.myjson.com/bins/1ew27p', function(retorno) {
        var obj = retorno.participantIdentities;
        Object.keys(obj).forEach(function(prop) {
            if (obj[prop].player.accountId == accountId) { //if alterado aqui
                console.log(`Achou jogador ${obj[prop].player.summonerName} com participantId de ${obj[prop].participantId}`);
                return;//se achou não ve mais nenhum
            }
        });
    });
}

stats(211703728);
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>

A lógica que está a utilizar para ir buscar achar o jogador funciona, só o acesso ao campo accountId é que não está correto, este obj[prop].accountId.

Se faz um foreach em participantIdentities significa que cada objeto desse foreach será algo como:

{  
     "player":{  
        "summonerName":"Khal Droggo",
        "accountId":1595535
     },
     "participantId":1
}

O que significa que para aceder ao accountId terá de fazer .player.accountId em vez de .accountId diretamente como tem.

Sendo assim basta trocar o if que tem para:

if (obj[prop].player.accountId == accountId) { //agora .player.accountId

Se player pode não existir então convem confirmar se existe utilizando a função hasOwnProperty o que ficaria assim:

if (obj[prop].hasOwnProperty("player") && obj[prop].player.accountId == accountId) {

Exemplo:

function callback(andress, fn){
    $.ajax({
        url: andress,
        type: 'GET',
        dataType: 'json',
        error: function(){},
        success: fn
    });
}

function stats(accountId){
    callback('https://api.myjson.com/bins/1ew27p', function(retorno) {
        var obj = retorno.participantIdentities;
        Object.keys(obj).forEach(function(prop) {
            if (obj[prop].hasOwnProperty("player") && obj[prop].player.accountId == accountId) { //if alterado aqui
                console.log(`Achou jogador ${obj[prop].player.summonerName} com participantId de ${obj[prop].participantId}`);
                return;//se achou não ve mais nenhum
            }
        });
    });
}

stats(211703728);
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>

Fonte Link
Isac
  • 25,5mil
  • 3
  • 31
  • 51

A lógica que está a utilizar para ir buscar achar o jogador funciona, só o acesso ao campo accountId é que não está correto, este obj[prop].accountId.

Se faz um foreach em participantIdentities significa que cada objeto desse foreach será algo como:

{  
     "player":{  
        "summonerName":"Khal Droggo",
        "accountId":1595535
     },
     "participantId":1
}

O que significa que para aceder ao accountId terá de fazer .player.accountId em vez de .accountId diretamente como tem.

Sendo assim basta trocar o if que tem para:

if (obj[prop].player.accountId == accountId) { //agora .player.accountId
}

Exemplo:

function callback(andress, fn){
    $.ajax({
        url: andress,
        type: 'GET',
        dataType: 'json',
        error: function(){},
        success: fn
    });
}

function stats(accountId){
    callback('https://api.myjson.com/bins/1ew27p', function(retorno) {
        var obj = retorno.participantIdentities;
        Object.keys(obj).forEach(function(prop) {
            if (obj[prop].player.accountId == accountId) { //if alterado aqui
                console.log(`Achou jogador ${obj[prop].player.summonerName} com participantId de ${obj[prop].participantId}`);
                return;//se achou não ve mais nenhum
            }
        });
    });
}

stats(211703728);
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>