Почему не работает код?

Question

[mindx]

Только учусь. Простой код, и не работает... Пишет f1 is not defined. Хотя все есть....
Что не так?
https://codepen.io/Mindx/pen/wjGOWQ

Answer 1

[Александр]

Вы определяете переменную r1,а затем используете вместо неё p. Условия else if написаны задом наперед. При приведении к числу одна и та же переменная написана по разному.

function f1(){
  var nu1, nu2, r1;
  nu1 = document.getElementById('n1').value;
  nu1 = parseInt(nu1);

  nu2 = document.getElementById('n2').value;
  nu2 = parseInt(nu2);

  r1 = document.getElementById('r');

  if (nu1>nu2){
    r1.innerHTML = 'Первое число больше второго!';
  }
  else if(nu1==nu2){
    r1.innerHTML = 'Числа равны!!';
  }
    else{
    r1.innerHTML = 'Второе число больше первого!';
  }
}

Comments

[mindx]

Храни господь Вас! Честно минут 40 втыкал, пытался решить.... Только потом увидел , что переменные разные...
Наверно если бы переписал, все бы получилось...
Спасибо!

[Александр]

mindx,Чтобы в будущем не повторять данную ошибку советую давать переменным более осмысленные именования.

[mindx]

Александр, да тренировался же) Спасибо еще раз!

Answer 2

[Артём Кан]

Как вариант:

function f1() {
	var nu1, nu2, r1;

	nu1 = +document.getElementById('n1').value; // Унарный плюс превращает тип в число
	nu2 = +document.getElementById('n2').value;

  if (isNaN(nu1) || isNaN(nu2) || nu1 == '' || nu2 == '') {
    r.innerHTML = 'Введите числа!' // Число проверка; Так как 'r' это id, можно обойтись без getElById
  } else if (nu1 > nu2) {
     r.innerHTML = 'Первое число больше второго!';
  } else if (nu1 < nu2) {
    r.innerHTML = 'Второе число больше первого!';
  } else {
     r.innerHTML = 'Числа равны!!';
  }
}

Будьте внимательнее! Удачи!

Comments

[mindx]

nu1 = +document.getElementById('n1').value;
Те при такой записи ненужно будет дописывать nu1 = parseInt(num); ?

[Артём Кан]

mindx, если в DOM есть элемент с id box, то переменная id, которая равна getElelementById('box'), уже создана. Можно как не писать, как и написать.