Как рационально организовать обработку keyboards events?

Тогда мне придется объявить еще с дюжину переменных для каждой кнопки, которую я хочу обрабатывать) Это как-то не очень.

А что насчет технических недочетов?

Кстати, в вашем варианте я вижу, что при вызове фукнции вы дважды обращаетесь к массиву actions. Это зачем?

Я говорю об этом:

$(document).on("keydown", function(event) {
  return actions[event.which] && actions[event.which](event);
});

У меня вроде бы одно обращение:

var t

keyAction = function(e) {
  return (t = actions[e.keyCode]) && t(e)
}

Никита Полевой: Занесение кодов в переменные несёт описательный характер (пояснять их в комментариях менее эффективно) и позволяет держать в голове меньше информации, не говоря о наглядности для человека, которому может придётся разбираться в вашем коде.

return actions[event.which] && actions[event.which](event);

if (actions[event.which]) {
return actions[event.which](event);
}

Эти два выражения равноценны. Проверка нужна, чтобы в случае отсутствия функции соответствующей коду не применять вызов на undefined (это создаёт ошибку). В вашем коде происходит тоже самое. Если вынесете присваивание отдельно, то станет нагляднее:

t = actions[e.keyCode];
return t && t(e);

Из технических проблем: хранение функций в разреженном массиве вместо объекта. На работе скрипта это не сказывается, но очень грязно. Ещё объявление переменных без ключевого слова var. В таких случаях переменные создаются в глобальной области видимости, а не в той, в которой были объявлены. Это очень быстро приводит к проблемам.

Ну, можно и в объекте, так и было пару версий назад, но потом я из каких-то слухов решил, что обращение к массиву будет обрабатываться быстрее, чем обращение к объекту. Я не прав?

Вот, переделал так, чтобы все было в одном объекте, который собран конструктором. По поводу переменных тоже все исправил.

Зато код теперь выглядит отвратительно( Можно было как-то это сделать без постоянного упоминания this и base?

Никита Полевой: Массивы предназначены для работы с упорядоченными данными, желательно одного типа. Если массив становится разреженным и/или ему добавляется любое свойство не являющееся целым числом, тогда интерпретатор работает с ним как с объектом со всеми вытекающими, включая понижение скорости обращения и размер занимаемой памяти.

Если вы пишете var base = this, то используйте эту переменную везде, где она эквивалентна this в текущей области. Иметь одну сущность под разными именами не лучшая идея. Сам this при работе с объектами неизбежен, и я не понимаю зачем было оборачивать всё в конструктор.

В конструктор обернул, потому что не знал, как в одно действие создать объект со свойствами, ссылающимися на свойства этого же объекта (например, я пытался объявить свойства container : $() и items : container.children(), получается ошибка). В итоге получилось вот это

Ошибки наконец-то исчезли ?)

Никита Полевой: Да. Почитайте про форматирование кода (например learn.javascript.ru/coding-style ) и про паттерны программирования. Будет ещё лучше.

Спасибо большое)

Красота) Массивная какая штуковина