2016-11-02 4 views
0

過去10では、最小値を10とし、6を最大値 "1"とは別に指定します。私はなぜ、誰かが私に知らせてくれるか、答えが偉大になるように私をプッシュできるかどうかはわかりません。最小値を決定する

function btnMinMax_onclick() 
{ 
    // assign textbox elements to variables for easier access 
    var firstTextbox = document.getElementById("txtNum1"); 
    var secondTextbox = document.getElementById("txtNum2"); 
    var minTextbox = document.getElementById("txtMin"); 
    var maxTextbox = document.getElementById("txtMax"); 

    var num1 = firstTextbox.value; 
    var num2 = secondTextbox.value; 

    var isMin = determineMinimum(num1, num2); 
    var isMax = determineMaximum(num1, num2); 

    minTextbox.value = isMin; 
    maxTextbox.value = isMax; 
} 

function determineMinimum(number1, number2) 
{ 
    if (number1 < number2){ 
     return number1; 
    } else { 
     return number2; 
    } 
} 

function determineMaximum(number1, number2) 
{ 
    if (number2 > number1){ 
     return number2; 
    } else { 
     return number1; 
    } 
} 
+1

'Math.min'と' Math.max'は、このために最適です...また、あなたからそれを変換するために '.value'をparseFloatはしたいですあなたは字句ではない数字を比較するように数値に文字列を入力してください –

+0

@JaromandaX私はdecideMaximumとdecideMinimum関数を使用する必要があります – Bowie

+0

right ...だから、数値を比較しているので最初に値を解析します –

答えて

3

テキストボックスから引っ張っている値はString、ないNumberです。明示的に変換する必要があります。数値ではなく、辞書順に並べ替えます。あなたは本当の整数値ではなく、整数のように見える文字列を持っている

var num1 = parseInt(firstTextbox.value, 10); 
var num2 = parseInt(secondTextbox.value, 10); 

var num1 = firstTextbox.value; 
var num2 = secondTextbox.value; 

をする:

最も簡単な解決策は、変更することです。整数以外の値(例:3.14)を受け入れる必要がある場合は、代わりにparseFloatを使用してください。

+0

ありがとう、愚かな間違い部。 – Bowie

0

さて、あなたはここに問題を抱えている:

var num1 = firstTextbox.value; 
var num2 = secondTextbox.value; 

var isMin = determineMinimum(num1, num2); 
var isMax = determineMaximum(num1, num2); 

.valueはあなたにstringを取得します。しかし、数値を比較したいので、比較する前に文字列を数値に変換する必要があります。

あなたはこのようにそれを行うことができます:

function determineMinimum(number1, number2) { 
    number1 = Number(number1); 
    number2 = Number(number2); 

    // your code here 
} 
+0

名前が 'is'で始まる変数は、常にブール値を返します。 –

+0

@TânNguyễn:それは必要ですが、それを強制する言語機能はありません。ブール値を常に保存しているとは言えません。 OPは名前を使用した、私はそれを再利用するための答えをノックするつもりはない。 – ShadowRanger

関連する問題