Traiter variable via une fonction

Version imprimable

21/03/2019, 15h44
Benji12

Traiter variable via une fonction
Bonjour,

Disclaimer : Je débute en js, suis plutôt back ;)

J'essaie de faire une petite fonction, mais le résultat ne correspond pas à ce que j'attends :(

Dans mon html, je récupère plusieurs variables via data-set :

Code:

let {lat, lng, price, id} = $item.dataset

Je voudrais faire une function pour traiter l'affichage du prix, qui peut être soit renseigné, soit vide. S'il est renseigné, je voudrais afficher price + € sinon N.C.
Voilà ce que j'ai fait :
Code:

1 2 3 4 5 6 7 8 9 10 11 function testNum(price) { if (!price || price !== 0){ let price = (price + ' €') return price } else { let price = 'N.C.' return price } }
A ce niveau, console.log retourne undefined pour toutes les données. :(

Je ne vois pas ce qui cloche ;)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
function testNum(price) {
  if (!isNaN(price) && price.length>0 && price !== 0){
    price = (price + ' €')
 
  }
  else
  {
    price = 'N.C.'
 
  }
  return price
  }
 
  console.log(testNum(""));

Exemple :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
function testNum(price) {
  if (price && price !== 0){     
    return price + ' €'
  }
  else
  {     
    return 'N.C.'
  }
}
 
console.log(testNum(0));// affiche "N.C."
console.log(testNum(3));// affiche "3 €"
console.log(testNum());// affiche "N.C."

Merci à tous les deux pour votre retour. ;)
C'est tjrs intéressant de voir comment on peut traiter le même problème avec différentes approches.
Du coup, j'ai fait deux fonctions que j'ai encapsulé, mais je ne sais pas si c'est une bonne pratique en js.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 function havePrice(price) { if (price && price !== 0) { return true } else { return false } } function displayPrice(havePrice) { return (havePrice ? (price + ' €') : "N.C."); }
Il y peut être moyen de factoriser encore... je ne sais pas... ;)

Par contre j'ai une autre question, un peu différente mais dans la suite logique.

Je souhaite afficher des popups (avec le prix ou pas) sur une map avec leaflet. Jusque là, pas de soucis...

Sur cette map, je récupère également 2 types de points.

J'ai donc fait, une function pour récupérer le type de point. En fonction du type, je voudrais afficher différemment les popups.

J'ai donc une const popup :
Code:

1 2 3 4 5 6 7 const popup = L.popup({ className: 'marker', maxWidth: 100 }) .setLatLng([lat, lng]) .setContent(displayPrice(havePrice(price))) .openOn(map)
Quelle serait la meilleure approche, sachant qu'en fonction du type de point, la popup est la même, sauf pour className: "tow_marker" et le setContent(name). J'ai juste ces deux données qui changent.

Est ce que je fais des functions par type et j'injecte dans la const ?
Je traite directement le if dans la const ?

Quelle est la meilleure pratique ?

J'ai tenté un truc :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 function poiType(poitype, price){ if (poitype == "Town") { let m = "town-marker" let c = name return [m, c] } else { let m = "marker" let c = displayPrice(havePrice(price)) return [m, c] } }
mais dans tous les cas je passe en false :(

Bonjour,

Citation:

Il y peut être moyen de factoriser encore... je ne sais pas...

je ne vois pas bien, sur ce coup, l’intérêt de passer par deux fonctions !

Citation:

Par contre j'ai une autre question, un peu différente mais dans la suite logique.

https://www.developpez.com/template/kit/kitinfo.png Poser plusieurs questions dans un seul et même fil nuit grandement à la qualité du forum : notamment, lors d'une recherche, tout le contenu du fil n'est pas indiqué dans le titre, elle devient donc plus ardue.

Notre but est de maintenir une base de connaissances facile à exploiter par tous les visiteurs, utilisable par tous. Vous avez pu bénéficier de ce forum en posant vos questions, laissez la possibilité aux suivants de bénéficier des réponses en leur offrant un sujet moins dispersé.

une discussion = une question

21/03/2019, 20h24
Benji12

Citation:

Envoyé par NoSmoking

je ne vois pas bien, sur ce coup, l’intérêt de passer par deux fonctions !

Je risque d'avoir besoin de la fonction havePrice ailleurs, puis c'est une habitude pour moi de découper autant se faire se peut les functions. ;)

Citation:

Envoyé par NoSmoking

Poser plusieurs questions dans un seul et même fil nuit grandement à la qualité du forum : notamment, lors d'une recherche, tout le contenu du fil n'est pas indiqué dans le titre, elle devient donc plus ardue.

J'entends bien, mais comme c'est une suite logique avec le début de ma question, mais si ça pose souci, je peux la recopier dans un nouveau fil ?
21/03/2019, 20h58
Beginner.

Citation:

Envoyé par Benji12

Je risque d'avoir besoin de la fonction havePrice ailleurs, puis c'est une habitude pour moi de découper autant se faire se peut les functions.

Je pense aussi qu'il n'y a pas vraiment d’intérêt à faire cela dans ce cas précis... Cela n'est pas plus lisible (au contraire), c'est moins rapide, cela oblige à faire deux appels de fonctions...

De plus la variable "price" dans la fonction displayPrice n'est pas définie...

EDIT : J'ai supprimé une partie du message...
21/03/2019, 21h29
NoSmoking

Citation:

Envoyé par Benji12

J'entends bien, mais comme c'est une suite logique avec le début de ma question, ...

En n’exposant pas le contexte d’application clairement ainsi que le but final on se retrouve souvent avec des réponses, bien que satisfaisantes, qui ne répondent pas réellement au problème.

De plus en connaissant la finalité il est également possible de réorienter l’approche donc la solution.

Une solution valable dans un contexte donné ne l’ait plus forcément dans un autre.
On verra ce qu’il en ressort dans la suite de la discussion.
21/03/2019, 21h31
Benji12

Citation:

Envoyé par Beginner.

De plus la variable "price" dans la fonction displayPrice n'est pas définie...

Effectivement, j'ai oublié le price quand j'ai recopié.

Alors, pour le js et le front, je ne sais pas, mais je bosse sous RoR et j'ai cette habitude de découper les functions dès que possible.
C'est peut ^tre pas une bonne pratique en js... ;) J'en prend note
21/03/2019, 21h59
Benji12

Citation:

Envoyé par NoSmoking

En n’exposant pas le contexte d’application clairement ainsi que le but final on se retrouve souvent avec des réponses, bien que satisfaisantes, qui ne répondent pas réellement au problème.

De plus en connaissant la finalité il est également possible de réorienter l’approche donc la solution.

Une solution valable dans un contexte donné ne l’ait plus forcément dans un autre.
On verra ce qu’il en ressort dans la suite de la discussion.

Tu as très certainement raison, j'en prends note pour mes prochaines interrogations ;)