Форум

artem55555p · #1 (**permalink**) 15.07.2017, 19:16

Здравствуйте. Хотел спросить Вашего совета и критики.
Вот сайт:
http://www.artem55555p.ru/ru/buy/index

В полях я сделал калькулятор
http://clip2net.com/s/3Ma9kAi
поле BTC не в счет.
вот код:

$(".pole_one").keyup(function() {
		var usd =  $('#usd').text();
		var eur =  $('#eur').text();
		var rub =  $('#rub').text();
		var pole_one = $(".pole_one").val();
		var pole_three = $(".pole_three").val();
		if(pole_three == 'USD'){
			var res = parseFloat(pole_one) * parseFloat(usd);
			if(res > 0) {
				$(".pole_two").val(res);
			}
		}
		if(pole_three == 'EUR'){
			var res2 = parseFloat(pole_one) * parseFloat(eur);
			if(res2 > 0) {
				$(".pole_two").val(res2);
			}
		}
		if(pole_three == 'RUB'){
			var res3 = parseFloat(pole_one) * parseFloat(rub);
			if(res3 > 0) {
				$(".pole_two").val(res3);
			}
		}
	});

	$(".pole_two").keyup(function() {
		var usd =  $('#usd').text();
		var eur =  $('#eur').text();
		var rub =  $('#rub').text();
		var pole_one = $(".pole_one").val();
		var pole_two = $(".pole_two").val();
		var pole_three = $(".pole_three").val();
		if(pole_three == 'USD'){
			var res = parseFloat(pole_two) / parseFloat(usd);
			if(res > 0) {
				$(".pole_one").val(res);
			}
		}
		if(pole_three == 'EUR'){
			var res2 = parseFloat(pole_two) / parseFloat(eur);
			if(res2 > 0) {
				$(".pole_one").val(res2);
			}
		}
		if(pole_three == 'RUB'){
			var res3 = parseFloat(pole_two) / parseFloat(rub);
			if(res3 > 0) {
				$(".pole_one").val(res3);
			}
		}
	});
	
	$(".pole_three").change(function() {
		var usd =  $('#usd').text();
		var eur =  $('#eur').text();
		var rub =  $('#rub').text();
		var pole_one = $(".pole_one").val();
		var pole_three = $(".pole_three").val();
		if(pole_three == 'USD'){
			var res = parseFloat(pole_one) * parseFloat(usd);
			if(res > 0) {
				$(".pole_two").val(res);
			}
		}
		if(pole_three == 'EUR'){
			var res2 = parseFloat(pole_one) * parseFloat(eur);
			if(res2 > 0) {
				$(".pole_two").val(res2);
			}
		}
		if(pole_three == 'RUB'){
			var res3 = parseFloat(pole_one) * parseFloat(rub);
			if(res3 > 0) {
				$(".pole_two").val(res3);
			}
		}
	});

код очень большой. вы можете подсказать, как сделать короче, лучше. и вообще, хороший это вариант или нет?
п.с. js и jquery только учу. Заранее спасибо.

j0hnik · #2 (**permalink**) 15.07.2017, 22:12

вот это ты наворочил

<html>
<head>
</head>
<body>
<input type="number" min="0" id="one">
<input type="number" min="0" id="who">
<select id="three">
	<option value="2190.94">USD 2190.94</option>
	<option value="121712.36">RUB 121712.36</option>
	<option value="1828.38951">EUR 1828.38951</option>
</select>
<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.2.1/jquery.min.js"></script>
	<script>
		var one = $('#one');
		var who = $('#who');
		var three = $('#three');

		one.on('input', function() {
			who.val(one.val()*three.val());
		});
		who.on('input', function() {
			one.val(who.val()/three.val());
		});
		three.on('change', function() {
			who.val(one.val()*three.val());
		});
</script>	
</body>
</html>

рони · #3 (**permalink**) 15.07.2017, 22:32

j0hnik,
может change в строке 20? или нет такого браузера?

j0hnik · #4 (**permalink**) 15.07.2017, 22:40

Сообщение от рони

j0hnik,
может change в строке 20? или нет такого браузера?

пусть будет, в IE не пашет наверное input

artem55555p · #5 (**permalink**) 16.07.2017, 08:18

Спасибо

Форум

Справочник

Discord чат

Курсы javascript