Прошу подсказки по Javascript

Question

[Mephistopelus]

Буквально месяц назад начал учить программирование и, в частности, Javascript. Для практики слепил простенький скрипт для перевода скорости соединения в скорость закачки, и страничку, где всё это выводится наружу.

Подскажите, пожалуйста, как заставить второе поле работать так же как первое? Чтобы переводить можно было в оба направления.

Comments

[denver]

Интересно, если поле1=117 а поле2=123, как вы узнаете кого в кого переводить по нажатию на Convert?

[Mephistopelus]

Всем большое спасибо за подсказки! И за более простое решение.

Answer 1

[Денис Инешин]

Вы просто сразу жестко задали направление. У вас прямо даже поля называются «From» и «To».
Попробуйте для начала назвать каждое например «field-1» и «field-2» и сразу думать станет проще.

Далее проверяйте, например на onchange содержимое поля, если начали писать в одном, то второе блокируйте или очищайте.

После останется только сравнивать содержимое полей и типов и делать вычисления.

Answer 2

[VasiliyRylov]

1. Снимите со второго поля readonly
2. if(to.value == '') Прямое направление; else обратное;
3. На событие onfocus на любое из полей, в неактивное записывать '', для того чтобы избежать неоднозначности направления;

Comments

[VasiliyRylov]

Или вот такой маленький хак в начале скрипта, после присвоения from и to:
if(from.value == '' && to.value != '') {
var oldFrom = from;
from = to;
to = oldFrom;
};

[VasiliyRylov]

if(from.value == '' && to.value != '') {
var oldFrom = from;
from = to;
to = oldFrom;
};

[VasiliyRylov]

Только у вас логика вычислений привязана к каждому полю, не делайте так, напишите отдельную функцию для вычисления, тогда код будет не таким нагроможденным и ответ на ваш вопрос станет очевидным.

Answer 3

[Seldon]

Я бы наверное так не делал, много кода мало смыла. Вот так примерно должна выглядеть ваша функция pastebin.com/GdTYTinx пока много ошибок.

Ну а касательно второго поля оно тут не должно работать по логике, ибо зачем? я же могу вверху поставить 1024 килобита внизу попросить мегабиты, или вверху 1 мегабит а внизу 1024 килобита. А по вашему получается что когда оба поля заполнены то не ясно из какого поля брать реальное значение.

Comments

[Seldon]

соотвественно html должен быть таким pastebin.com/9uZ8AEQs вся идея убрать 100 if а поставить вместо них простое выражение с умножением и делением.

[Seldon]

даже вынуждем принести извинения за поспешность публикумых данных, так как в них много недочетов, поэтому выкладываю рабочий пример как должно быть jsfiddle.net/2JWxt/3/

[Mephistopelus]

Укажите, пожалуйста, на мои ошибки (кроме неграмотного использования IF-ов).

[Seldon]

ну вообще все зависит от задачи, но принято
1) не сипользовать навешивание событий в html коде. Не пишите так
получайте еэлемент в DOM по классу или id и навешивайте нужные события в коде

2) я совсем не понял что это значит onclick=«converter(from, fromWhat, toWhat)» почему в функцию вы передаете непонятно что? Если вы объявили что функция принимает 3 параметра — передайте их, в вашем случа функция ничего не принимает значит не надо так писать, объявите переменный как локальные а не как аргументы функции
var converter = function() {
var from = document.units.elements.from,
fromWhat = document.units.elements.fromWhat,
to = document.units.elements.to,
toWhat = document.units.elements.toWhat;

3) также хорошо в коде не писать текст, вообще в любом коде не только js, все собщения об ошибках должны быть вынесены.

4) пишите код в едином стиле а то где условия обернете в скобочки, где нет.

5) Плохо оптимизирован много лишних действий

6) на этом примере сложно указывать ошибки он очень простой и так сказать локальный, в реальных проектах все по другому.

[Mephistopelus]

Спасибо!