Q&A
タイトルはなるべく質問要件に寄せてください。このタイトルだと作成自体になりますが、実際起きているのはもっと細かい問題です。
質問失礼します。
JavaScriptにてこちらのサイトを参考に電卓の作成を行っているのですが
一か所上手く理解できないところがあります。
JavaScript
1function myCalculate(myData){ // 演算ボタンを押した 2if (myFlg==0){ // １回前に入力したものは数値か？ 3myFlg = 1; // １回前に入力したものは演算子 4myWork = myTotal + myCalc + myInput; // 一連の計算式を作る 5myTotal = eval(myWork); // 計算式を計算させる 6myInput = ""; // 現在入力している値をクリア 7document.myForm.myLine.value = myTotal; // 合計を表示 8} 9if (myData == "="){ // 演算ボタンは[＝]か？ 10myTotal = 0; // 合計をクリア 11myCalc = "+"; // 演算子を[+]とする 12}else{ // 演算ボタンは[＝]以外である 13myCalc = myData; // 演算子を退避させておく 14} 15}
myWork = myTotal + myCalc + myInput
myTotal = eval(myWork)
この部分ですが、私の理解では演算子の判断ができないのでは？
宣言時にmyCalc = "+"となっている為、このコードではすべて+が入ってしまうのでは？
と思い疑問です。
実際にコードが正しく動いていることは確認できたのですが理解が出来ていないためご教示願いたいです。
以上です。よろしくお願いいたします。
回答4件
0
演算子の問題は起きません。
+は文字列の連結にもなるものです。
「入力された値を取得」でしたら見た目数値でも型は文字列です。
対象の情報の型が文字列なので、+は文字列連結の働きをします。
ですからevalには1+2という文字列が渡されます。
投稿2020/03/10 05:26
編集2020/03/10 06:06総合スコア80850
とはいえ、ユーザー入力値そのまま突っ込んでevalは脅威にしかならないのでやめたほうが良いかと。
`+`以外が入ったときのパターンはどうなるのだろう、と思ったのですが私の勘違いでした。
evalは基本的に使ってはいけない関数なのでしょうか。
みんな数字を素直に打ってくれるなんて保証ないですからね。htmlでtype=numberを指定したとしてもブラウザのデベロッパーツールから幾らでも改竄できます。
ユーザー入力を受け付けるということは「誰が何を打ち込んでくるかわからない」ということです。
どうしてもeval使いたいのでしたらホワイトリスト方式で「許可する値のみ受け付ける」ようにすべきです。
まあ、使わない実装は幾らでもできるので、そちらを検討すべきですね。
0
ベストアンサー
function myCalculate(myData){ // 演算ボタンを押した内で、
myCalc = myData;とクリックした演算子を代入しているので問題ないですね。
myCalc = "+";は初期化の意味で代入しているようです。
演算ボタンをクリックせずに"＝"ボタンを押したときに、
myWork = myTotal + myCalc + myInput; // 一連の計算式を作る
で生成される演算式が 0+入力値 となるようにしています。
ただ、他の方も言われてますが、eval はセキュリティ上NGですので、このプログラムは参考にすべきではないですね。検索すれば他にも参考になるコード例はいくらでも見つかります。
投稿2020/03/10 06:00
編集2020/03/10 06:06
総合スコア33699
ありがとうございます。
` if (myData == "=")`
`myCalc = "+";`
の記載があったのですべて＋になってしまうのではないか、と疑問でしたが納得できました。
evalが問題ある関数なことの方に少々驚きました・・・。迂闊に使えませんね。。
0
追記
myCalc = myData; // 演算子を退避させておく
失礼しました、この部分を見落としていました。
+以外の演算子が押された場合にどうなるのか、という事が疑問でしたが
きちんと保持されていたのですね。
それよりもeval()自体がよくないのですね・・・。
便利な関数のように見えたのですが問題があるのでしょうか。
投稿2020/03/10 06:00
総合スコア44
下記を参照してください。
必要以上に eval を使用しないで！
https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/eval#Dont_use_eval_needlessly!
質問は編集できますので、回答として投稿するのは使い方まちがってます。
使い慣れていなくてすみません、勉強になります。
この回答は削除依頼をだしておいてください。
あなたの回答
tips
プレビュー
