Get function in service.js returns null

Asked

Viewed 204 times

0

I have a Json of categories with an array of products within it. Browse all categories and all products according to the category, you are searching. Now the problem is that I want to return the product detail by the product id, but the console is returning null.

Json

.factory('Categorias', function() {

 var categorias = [
 {//Nova Categoria
    "idCategoria": 1,
    "nomeCategoria": "categoria 1",
    "produtos": [
       {//Novo Produto
          "idProduto": 1,
          "nomeProduto": "produto 01A",
       },
       {//Novo Produto
          "idProduto": 2,
          "nomeProduto": "produto 02A",
       }
    ]
 }
];

return {
all: function() {
  return categorias;
},
get: function(categoriaId) {
  for (var i = 0; i < categorias.length; i++) {
    if (categorias[i].idCategoria === parseInt(categoriaId)) {
      return categorias[i];
    }
  }
  return null;
},
 getProduto: function(produtoId) {
 for (var i = 0; i < categorias.length; i++) {
  if (categorias[i].idProduto === parseInt(produtoId)) {
     return categorias[i];
  }
 }
 return null;
}
};
});

Controller

.controller('ProdutoDetalheCtrl', function($scope, $stateParams, Categorias) {
  $scope.produtosCategoria = Categorias.getProduto($stateParams.produtoId)
  console.log($stateParams.produtoId);
  console.log($scope.produtosCategoria)
})

Console

1
null
angularjs json

  • A Function getProduto doesn’t seem right to me...

  • This I know, this is where I have a problem...how do I get inside the products array in getProduct. Ta returning null pq not finding the products

  • Wow, you could have explained that in the question. It’s like you have no idea where the mistake is...

  • I know the error ta ai, but I don’t know how to mount this Function q returns the product by idProduct

  • The variable categorias is to be your Json?

  • that’s right. it’s a json only. I want to search in the getProduct the product within the category.

Show 1 more comment

3 answers

2


As already answered you need two For's one to go through categories and another to go through products.

Alternatively you can use angular.ForEach() to walk theirarrays:

var myApp = angular.module('myApp', []);

myApp.factory('Categorias', function() {
  
    var categorias = [{
        "idCategoria": 1,
        "nomeCategoria": "categoria 1",
        "produtos": [{ 
            "idProduto": 1,
            "nomeProduto": "produto 01A",
        }, { 
            "idProduto": 2,
            "nomeProduto": "produto 02A",
        }]
    }];

    return {
   
        getProduto: function(produtoId) {
            var produto = {};
            angular.forEach(categorias, function(categoriaCorrente) {
                angular.forEach(categoriaCorrente.produtos, function(produtoCorrente) {
                    if (produtoCorrente.idProduto === parseInt(produtoId)) {
                        produto = produtoCorrente;
                    }
                })
            })
            return produto;
        }
    };

});

function MyCtrl($scope, Categorias) {
    $scope.produtosCategoria = Categorias.getProduto(1)
    console.log(1);
    console.log($scope.produtosCategoria)
    console.log("ID: ",$scope.produtosCategoria.idProduto)
    console.log("Nome: ",$scope.produtosCategoria.nomeProduto)
}
<script src="https://ajax.googleapis.com/ajax/libs/angularjs/1.2.23/angular.min.js"></script>
<div ng-app="myApp" ng-controller="MyCtrl">
ID Produto: <b>{{produtosCategoria.idProduto}}</b>
 </br>
Nome Produto: <b>{{produtosCategoria.nomeProduto}}</b>
</div>

If you prefer you can use the foreach of Ecmascript, Example:

 lista.forEach(function(item){
    console.log(item)
  })

  • Well remembered. He can’t apply the forEach right into the object categorias? Type, categorias.forEach(...)?

  • I don’t know if you can do it this way, because in the Angular documentation it’s like I did rs. Only if you can do it using native JS

  • @jbueno just did a test and with ECM da to do :p

  • 1

    I was testing here too, it really works

  • 1

    It worked well tbm your code @Techies. Another knowledge I’ll keep here, to study. Thanks for the force. hugs

1

The "categories[i]. idProduct" you use to compare to the "productId" does not exist. What exists is the "categories[i]. products[y]. idProduct".

So your job had to be something like this:

for (var i = 0; i < categorias.length; i++) {
   for (var y = 0; i < categorias[i].produtos.length; y++) {
      if (categorias[i].produtos[y].idProduto === produtoId) {
          return categorias[i].produtos[y];
      }
   }
}

  • Thanks for the force tbm@AlexandreBusarello. I took your code tbm, but noticed q had 2 errors. a key was missing and parseint. More was worth by force tbm.

  • Fixed the lack of keys and the missing parseint. I wrote here quickly and ended up not answering me. Thanks.

  • quiet. Thanks again for the strength

1

Following its logic, the function getProduto should be like this

getProduto: function(produtoId) {
 for (var i = 0; i < categorias.length; i++){
     for(var c = 0; c < categorias[i].produtos.length; c++){  
         if (categorias[i].produtos[c].idProduto === parseInt(produtoId)) {
             return categorias[i].produtos[c];
         }
     }
 }
 return null;
}

  • It worked here @jbueno. Managed to show the product detail page. What was wrong with the json, it was that the products, even in another category, cannot have the same id. That would be it?

  • now understood, what would be my doubt @jbueno. I could not do 2 for, thanks even for the information. hugs

Browser other questions tagged angularjs json

You are not signed in. Login or sign up in order to post.