Note that there are some explanatory texts on larger screens.

plurals
  1. PO
    primarykey
    data
    text
    <p>There are quite a few things wrong with your markup and javascript. You have unclosed paragraph tags, you set the scope of currency to be global but then define a local variable with the same name when you want to set it (which is why currency is never being set globally), you're using an if/else statement where a switch/case is more appropriate... there's probably a lot more, and the more you learn the more you'll discover. Having said all that, because it bothered me, here's a modified version of your code which seems to do what you were after :</p> <pre><code>&lt;html&gt; &lt;head&gt; &lt;title&gt;Rounded Commodity Pricing&lt;/title&gt; &lt;script type="text/javascript"&gt; var currency; function setCurrency(val) { currency = val; } function finddivid(marketprice, tradevalue) { var KWDex = 0.281955 var GBPex = 0.625907 var USDex = 1 var CADex = 0.998727 var EURex = 0.784594 var currencyMarketprice; switch (currency) { case "KWD": currencyMarketprice = Math.ceil(marketprice * KWDex); break; case "GBP": currencyMarketprice = Math.ceil(marketprice * GBPex); break; case "USD": currencyMarketprice = Math.ceil(marketprice * USDex); break; case "CAD": currencyMarketprice = Math.ceil(marketprice * CADex); break; case "EUR": currencyMarketprice = Math.ceil(marketprice * EURex); break; } if (tradevalue % currencyMarketprice == 0) return ("tonnage = " + tradevalue / currencyMarketprice + " mt, and price = " + currencyMarketprice + " " + currency + " per mt"); else { for (var counter = currencyMarketprice + 1; counter &lt; (currencyMarketprice * 2); counter++) { if (tradevalue % counter == 0) return ("tonnage = " + tradevalue / counter + " mt, and price = " + counter + " " + currency + " per mt"); } } } function calculate() { var mktprc = document.getElementById('mktprc'); var trdval = document.getElementById('trdval'); document.getElementById('showMeArea').value = finddivid(mktprc.value, trdval.value); } &lt;/script&gt; &lt;/head&gt; &lt;p&gt; Select currency: &lt;input type="button" value="KWD" onclick="setCurrency('KWD')"&gt; &lt;input type="button" value="USD" onclick="setCurrency('USD')"&gt; &lt;input type="button" value="GBP" onclick="setCurrency('GBP')"&gt; &lt;input type="button" value="EUR" onclick="setCurrency('EUR')"&gt; &lt;input type="button" value="CAD" onclick="setCurrency('CAD')"&gt; &lt;/p&gt; &lt;p&gt;Enter today's price of commodity in USD: &lt;input id="mktprc" input type="number"&gt;&lt;p&gt; &lt;p&gt;Enter value of trade: &lt;input id="trdval" input type="number"&gt;&lt;/p&gt; &lt;p&gt;&lt;input type="button" value="Calculate" OnClick="calculate();"&gt;&lt;/p&gt; &lt;p&gt; &lt;input id="showMeArea" readonly="true" size="100"&gt; &lt;/p&gt; &lt;/html&gt; </code></pre>
    singulars
    1. This table or related slice is empty.
    1. This table or related slice is empty.
    plurals
    1. This table or related slice is empty.
    1. This table or related slice is empty.
    1. This table or related slice is empty.
    1. VO
      singulars
      1. This table or related slice is empty.
    2. VO
      singulars
      1. This table or related slice is empty.
 

Querying!

 
Guidance

SQuiL has stopped working due to an internal error.

If you are curious you may find further information in the browser console, which is accessible through the devtools (F12).

Reload