Discussion :

[sylvain257]

Bonjour,

J'ai une application front-end dans lequel j'ai une couche avec les entreprises (donnée ponctuelle) qui sont géolocalisées sur une carte (map).

Quand je clique sur un point, j'ai une fiche qui s'ouvre à l'aide d'une modale et dans lequel j'ai un formulaire de mise à jour.

J'ai créé une classe JS "Entreprise" dans laquelle j'ai un constructeur ayant comme paramètres les données natives de l'entreprise, tq le nom, l'adresse, le numéro de tel, etc.

J'ai plusieurs fonctions à l'intérieur de cette classe. Dans l'une d'elles, je récupère une valeur 'intérêt' de 1 à 5 (fonction rate())

J'ai une autre fonction qui permet de mettre à jour le formulaire (function update()) A l'intérieur de cette fonction, on retrouve une requête ajax, dans laquelle on retrouve la liste des paramètres et notamment cette valeur citée précédemment.

Le problème c'est que je n'arrive pas à récupérer la valeur retournée par la fonction rate() dans la fonction update(). La requête AJAX plante systématiquement.

A l'intérieur d'une classe, comment récupérer une valeur d'une fonction pour l'utiliser dans une autre ?

Merci pour vos retours.

Bonne journée,

Sylvain

[SpaceFrog]

Je vois ... Je vois Dans ma boule de crisal ... un bout de code ?
Si si là en bas à droite..


Ha ben non c'est le prix que j'ai oublié d'enlever ...

[Doksuri]

peux-tu nous faire une POC de la structure de ton code ? j'ai du mal a suivre

sinon, pour retourner une valeure, tu fais juste une methode publique (ou alors je n'ai pas compris ton probleme)

Code javascript :

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

1
2
3
4
5
6
7
8
9
10
11
12
class classA {
    #a = 'private A';
    constructor() {
 
    }
    getA() {
        return this.#a;
    }
}
 
const maClassA = new classA();
maClassA.getA(); // retourne 'private A'

[sylvain257]

J'ai ça :


Ma classe telle qu'elle est déclarée :

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
class Carte {
    constructor({reference = "", intitule = "", annee = "", interest = ""}){
        this.reference = reference,
        this.intitule = intitule,
        this.annee = annee,
        this.interest = interest
    }
 
add(){  /// qui permet d'afficher les informations dans une div
 
document.getElementById("info_carte").innerHTML = 
`<form action="" id="cpa_edit">
<div><input type="text" id="fd_reference" value="${this.reference}"></div>
<div><input type="text" id="fd_intitule" value="${this.intitule}"></div>
<div class="my-rating">${this.interest}"</div>
<button type="submit">Soumettre</button>
</form>`
 
}
 
rate(){ /// j'utilise ici la notation par étoiles jQuery avec starRating
$(".my-rating").starRating({
            useFullStars: true,
            emptyColor:'transparent',
            strokeColor: 'black',
            starSize: 22,
            readOnly: false,
            hoverColor:'grey',
            activeColor:'black',
            ratedColor:'gold',
            ratedColors:'yellow',
            useGradient:false,
            initialRating: this.interest,
            disableAfterRate:false,
            onClick: function(currentIndex, currentRating, $el){
                return currentRating;
            }
        });
}
 
update(){
        $.ajax({
            url:"monfichier.php?",
            type:"POST",
            data:{
                "reference":$("fd_reference").val(),
                "intitule":$("fd_intitule").val(),
                "interest":this.interest ?? currentRating //// et c'est là que systématiquement ça renvoie "currentRating" is not defined
            },
            success:function(data){
                M.toast({html: 'Données de la carte mises à jour !', displayLength: 5000, classes: 'toast_success'});
            },
            error:function(err){
                M.toast({html: 'Erreur :' + err, displayLength: 5000, classes: 'toast_error'});
            }
        })
    }
}

et la nouvelle instance de classe donne ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
            var carte = new Carte({
                reference : donnee_reference,
                intitule : donnee_intitule,
                interest : donnee_interest
            })
 
carte.add();
 
            $("#cpa_edit").on('submit', function(e){  /// Mise à jour fiche
                e.preventDefault();
                carte.update();
            })

N'étant pas développeur professionnel, mais étant géomaticien faisant du développement, sûrement que je ne fais pas ça dans les règles de l'art et que certains trucs doivent faire que les pros se tirent les cheveux, que pensez-vous de mon code ?

Merci pour vos retours

Sylvain

[Doksuri]

est-ce que les donnes que tu recois sont correctes ?

donnee_reference, donnee_intitule, donnee_interest ?

[sylvain257]

Oui les données sont correctes, ce sont des données issues d'un click d'une couche de tuiles vectorielles.
(et le return de la fonction rate() renvoie correctement la donnée)

[NoSmoking]

Bonjour,

(et le return de la fonction rate() renvoie correctement la donnée)

certes mais la valeur de la fonction est retournée où ?

currentRating n'est définie que dans le retour de ta fonction sur le onclick, nulle part ailleurs.