2011-12-29 7 views
2

自分のページに一連のラジオボタンがあり、これをチェックすると特定の条件が設定されます。各条件に対して、PRICEから値が減算されます。これは、ユーザーに提供される最終変数です。JavaScript変数の最小値を設定する

if (condition1==0){price-=5;} 
if (condition2==1){price-=4;}  
if (condition3==1){price-=10;} 
var minimum = 1; 
if (price < minimum){price = minimum;) 

条件はすべて私が望むとおりに正確に動作しています。このコードグループでは、基本的に「価格」が1未満にならないようにしています。私の問題は、これがエラーを作り出しているということです。私はJavaに精通しているので、おそらくJavascriptが私に馴染みがないと言うエラーに遭遇しています。

エラーを投げずに、「価格」変数が「最小」変数を下回らないようにするにはどうすればよいですか?

あなたはこのような何か行うことができます
+0

変数名 "minimum"を "minprice"に置き換えてみましたか?ちょうど馬鹿。 –

+0

エラーをお知らせするには – sciritai

+0

のエラーメッセージを表示してください。 – Alfabravo

答えて

4

をではなく、中括弧}これはあなたのコードでエラーが発生しています。

ここに変更点があります。前

if (price < minimum){price = minimum;) 

後:

if (price < minimum){price = minimum;} 
            ^
+0

私はすでに変数として以前に使用された最小値を持っていたことが判明し、それは文字列でした。これは私の間違いでした。ありがとうございました! – MattDavBen

1

if (condition1 === 0) { price = Math.max(price - 5, 1); } 

または関数にそれを一般:price = minimum;後あなたは括弧)を持って

function reducePriceBy(amount) { 
    return Math.max(price - amount, 1); 
} 

そして

if (condition1 === 0) { price = reducePriceBy(5); } 
+0

私はこの機能を多くの場所で使用するので、これは良い考えです。 – MattDavBen

1

あなたは起動する構文エラーを持っている:if (price < minimum){price = minimum;}

最後の括弧は、ブレースされている必要があります。

+0

私はすでに変数として以前に使用された最小値を持っていたことが判明し、それは文字列でした。これは私の間違いでした。ありがとうございました。 – MattDavBen

関連する問題