Node.js 計算結果を返す

Question

JavaScript初心者でNode.jsを勉強中です。

http://localhost:3000/tip/200/20
と入力されたら、百分率表記にして"10"と画面に表示させるようにしたいです。
res.sendが書き方が間違っていると思うのですが、アドバイス頂けないでしょうか。

// expressを読み込めるようにしている。
const express = require('express');
// express()をappに格納する。
const app = express();

app.get('/tip/:total/:tipPercentage', (req, res) => {
  const Percentage = (num, per) => {
  return (num/100)*per;
  }
  let total = parseInt(req.params.total)
  let tipPercentage = parseInt(req.params.tipPercentage);
  res.send(`${Percentage}(${total}, ${tipPercentage})`)
})

app.listen(3000, () => console.log('hello i am listening on port 3000'))

Answer 1

2つの引数a,bに対して、bがaの何パーセントかを返したいという認識で間違いないでしょうか？？
それでしたら、return (num/100)*per;→return (per / num) * 100;となるかと思います。
（計算式が間違っていますね）

Comments

[hiro_ike]

ご確認ありがとうございます。似たようなことをやりたいサンプルコードを見つけ、よく考えないで使用しておりました。助かりました。

Answer 2

テンプレートリテラルの{}の使い方で
ちょっとした勘違いがあるようですね。

テンプレートリテラルの{}は
「組み込み式」であって変数ではありません。

「組み込み式」というのは実践的な話をすると「値」を指します。
変数名だけポンと書いても値、これは勿論正しい。
Percentage(a, b)のように書いたら関数実行した返り値という値。
最終的に1個の値になれば良いので、中で三項演算子を使っても構いません。

  const Percentage = (num, per) => {
  return (num/100)*per;
  }

Percentageは関数宣言です。
テンプレートリテラルの中で${Percentage}と指定しても計算結果は出てきません。
文字列に無理やり評価しようとした結果、
(num, per) => { return (num/100)*per; }みたいな文字列になることでしょう。

質問文のコードは「関数実行で計算した結果」が欲しいはずです。。
下記のようにしましょう。

res.send(`${Percentage(total, tipPercentage)}`);

まぁ、そもそもテンプレートリテラル要る？問題もありますね。
Express.js前提ならばこんな感じ

// 数値を文字列にしたいなら.toString()メソッドを利用するのもあり
res.send(Percentage(total, tipPercentage).toString());

// res.sendは勝手にtoStringしてくれそうなので
// そのまま数値突っ込んでも動きそう
res.send(Percentage(total, tipPercentage));

// resはres.jsonにすると良い感じにJSON形式で返してくれる
res.json(Percentage(total, tipPercentage));

ん？Percentageの関数、ちょっと変だな？
100分率にだから100をどっかで掛け算するのが正解でしょうね。

Comments

[hiro_ike]

ご丁寧にありがとうございます。どこが理解していないか分かりました、再度勉強しなおします。助かりました。