Discussion :

[dukoid]

bonjour les amis,

j'ai un soucis, à chaque changement de page, ma requête à l'API REST est effectuée 2 fois même parfois 3 fois.

j'ai 3 pages, à chaque page je fais une requête API REST via $http dans une factory pour récupérer des données.
chaque controller fait donc appelle à cette factory et passe les données à la vue.

la factory Product:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
'use strict';

angular.module('starter')
.factory('Product', function($http) {
    Product.products = [];
    
    Product.getAll = function() {
        return $http.get('http://apirest-xxxxxxxx/api/v1/produits')                      // FAIT 2 fois 
            .success(function(data) {
                Product.products = data;
            });
    };

    ...
    
    return Product;
});

mon controller :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
'use strict';

starter.controller('ProductShowAllController', function($scope, Product) {
    this.products = Product.products;

    Product.getAll()
        .then(function() {
            this.products = Product.products;   
console.log(this.products);                                             //             
        }.bind(this));

    ...

app.js avec les routes :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
var starter = angular.module('starter', ['ngRoute']);
 
starter.config(['$routeProvider',
    function($routeProvider) {
        $routeProvider.
 
        when('/home', {
            templateUrl: 'views/home.html',
            controller: 'HomeController'
        }).
 
        when('/produits', {
            templateUrl: 'views/produits.html',
            controller: 'ProductShowAllController'
        }).
 
       ....

via firebug, je regarde donc le reseau et je vois à chaque changement de page; 2 fois la même requête effectuée.
et console.log(this.products); est exécuté 2 fois, j'ai 2 lignes

une idée ?

[Marco46]

Met un console.log dans ton contrôleur pour voir s'il serait pas exécuté 2 fois par hasard.

A mon avis la solution se trouve dans la structure html de ta page.

[dukoid]

marco46, c'est le cas.

voici l'affichage des produits

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
 
    <div ng-controller="ProductShowAllController as Product">
 
        <div class="row" ng-show="Product.products.length > 1">
            <div class="col-md-7"></div>
            <div class="col-md-5">
                <a ng-click="clickPredicateName()" class="col-md-5">Trier par nom</a>
                <a ng-click="clickPredicatePrice()" class="col-md-5"> Trier par prix</a>
            </div>
            <div class="col-md-12" style="height: 8px;"></div>
        </div>
 
        <a href="#" class="list-group-item active">
            Tous les produits
        </a>
        <div class="list-group">
        <div ng-repeat="product in Product.products | orderBy: orderByPredicate : orderByReverse">
            <a class="list-group-item" href="#/detail/{{product.id}}">
                <div class="row">
                    <div class="offset-md-2 col-md-4">
                        <img src="img/produits/tn_{{ product.image}}.jpg" />
                    </div>
                    <div class="col-md-12 categorie">
                        {{ product.categorie.nom }}
                    </div>
...
                </div>
            </a>
        </div>
 
    </div>
    </div>

je ne vois pas le soucis ?

[Marco46]

Fais voir la description javascript de ta route. Ya des chances que tu déclares le contrôleur dans le html (ng-controller) et dans la route, ce qui fait qu'il serait instancié 2 fois.

[Marco46]

Ah ben sur ton 1er post on le voit :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
when('/produits', {
            templateUrl: 'views/produits.html',
            controller: 'ProductShowAllController'
        }).

Il te faut en virer un des deux.

[dukoid]

aah d'accord, ça marche merci

j'ai donc viré la declaration du controller.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
when('/produits', {
            templateUrl: 'views/produits.html'
        }).

je me crois sur Symfony ou il s'agit que d'une déclaration.
là il exécute le controller donc.



merci marco , tu assures !