Как оптимизировать этот код?

Question

[Дуболом Одноклеточный]

У меня есть что-то вроде "кастомного textarea", т.е. при нажатии на кнопку (я сделал пока только при нажатии на цифры) клавиатуры, кнопку на которую я нажал должна отображаться в span'е:

$(document).on("keydown", e => {
  switch(e.keyCode) {
    case 48:
      $("span").append("0");
      break;
    case 49:
      $("span").append("1");
      break;
    case 50:
      $("span").append("2");
      break;
    case 51:
      $("span").append("3");
      break;
    case 52:
      $("span").append("4");
      break;
    case 53:
      $("span").append("5");
      break;
    case 54:
      $("span").append("6");
      break;
    case 55:
      $("span").append("7");
      break;
    case 56:
      $("span").append("8");
      break;
    case 57:
      $("span").append("9");
      break;
  }
});

Как эту кашу оптимизировать?

Comments

[Ярослав Иванов]

Циклом

[Дуболом Одноклеточный]

Я пробовал, не получается, есть какой нибудь наглядный пример?

[Ярослав Иванов]

Дуболом Одноклеточный, а ну раз пробовал и не получается то тогда нет смысла

[Михаил]

if >= 48 <= 57
$("span").append(e.keyCode - 48);

[Игорь]

Но это не оптимизация, а решение задачи разными способами.
Интерпретатор сам оптимизирует.

Другое дело, что эта портянка и было бы неплохо её укоротить.

[Дуболом Одноклеточный]

Михаил, Нельзя поставить меньше или равно перед числом

Answer 1

[twobomb]

$(document).on("keydown", e => {
	if(e.key.search(/\d/) != -1)
      $("span").append(e.key);
});

Comments

[Дуболом Одноклеточный]

Спасибо большое! Здоровья счастья вам!

Answer 2

[Валерий Павлов]

Особо JS не знаю, но по сути так.

$(document).on("keydown", e => {
	for(i = 48; i < 58; i++)
	{
		if(e.keyCode == i)
		{
			$("span").append(i-48);
		}
	}
});