Где ошибка в коде?

Question

[Denis]

Всем доброго дня.

Товарищи, в чем здесь ошибка моего понимания логики Js. Объясните, пожалуйста, и если не сочтете лишним - поправьте.

var trouble = prompt('What is wrong?')

var cabel = 1
var router = 2
var computer = 3

if (trouble == 1) {
	console.log('check your cabel')
} else if (trouble == 2){
	console.log('check your router')
} else if (trouble == 3) {
	console.log('reload your PC')
} else {
	console.log('uncorrect trouble')
}

Comments

[Rsa97]

И откуда же нам знать, как вы понимаете логику JS, если ничего кроме фрагмента кода вы не привели?

[Roman]

switch case здесь уместнее использовать по логике проверки одного значения на несколько условий.
switch(trouble)
{
case cabel:
...
break;
}

точка с запятой в конце выражения желательно.

[Дмитрий]

console.log(typeof trouble);

Answer 1

[Антон Спирин]

var trouble = +prompt('What is wrong?');

var CABEL = 1;
var ROUTER = 2;
var COMPUTER = 3;

switch (trouble) {
  case CABEL:
    console.log('check your cabel');
    break;
  
  case ROUTER:
    console.log('check your router');
    break;
  
  case COMPUTER:
    console.log('reload your PC');
    break;
  
  default:
    console.log('uncorrect trouble');
}

Answer 2

[Stimulate]

var trouble = prompt('What is wrong?');

switch (trouble) {
	case 1:
		console.log('check your cabel');
    break;
	
	case 2:
		console.log('check your router');
    break;
	
	case 3:
		console.log('reload your PC');
    break;
	
	default:
		console.log('uncorrect trouble');
	break;
}

Comments

[gleendo]

Stimulate +prompt('What is wrong?');

Answer 3

[Codebaker]

1) prompt возвращает строки, сравнение с числами - некорректно
2) в блоках с if ()... вы постоянно сравниваете с константами, хотя чуть ранее объявили их "смысловую нагрузку"

Соответственно, лучше будет сделать так:

1) var trouble = parseInt(prompt('What is wrong?'))
2) if (trouble == cabel) {...
...
else if (trouble == router) { ...
...
else if (trouble == computer) { ....

Comments

[eRKa]

Сравнение корректно. При сравнении со строкой число автоматом приведется к строке. Проверьте в консоли "1" == 1; // true

[Codebaker]

@eRKa, это контракт, понимаете? parseInt() говорит нам (в том числе нам через год-два, или другим, кто будет поддерживать этот код), что именно вот тут автор кода ждал число, а не объект или строку, например.
Неявные преобразования - они работают, но код - штука самодокументирующая. Я об этом.

Answer 4

[eRKa]

У меня Ваш код работает
тут
А вообще не стоит пренебрегать ";" в конце строки.

Comments

[Denis]

Попробовал в разных песочницах js, нигде не сработал :(

[eRKa]

Denis,
"Странно, у меня такая же нога и не болит" (с)
Причина не в коде, однозначно. Вы хоть скажите что именно не так, какая то ошибка выскакивает или логика не так отрабатывает?