Просьба покритиковать код
 

Добрый день!

Смысл скрипта в следующем: передаем ему массив значений из которых создаются графические объекты с возможностью Drag-n-drop и создания связи между ними.

Просьба покритиковать код и сказать идеи по улучшению.

Заранее благодарен!

1. Кешировать свойство length в локальной переменной для итерации по массивам. Не критично, но как правило хорошего тона (иногда попадаются "живые" коллекции, где length - геттер и обращение к нему вызывает пересчет количества элементов) пригодится.

2. Вы используете индентацию табами, но в некоторые места закрались пробелы.

3. Несколько подряд идущих блоков var можно объединить в один:

var a = 1;
var b = 2;
var c = 3;
// =>
var a = 1,
     b = 2,
     c = 3;

4. В Table.ajax.run при формировании параметров запроса (data) лучше либо использовать encodeURIComponent (На случай, если попадется служебный символ), либо передавать объект вместо строки (jQuery сам конвертирует его в строку, применив необходимые фильтрации).

В целом же все написано достаточно грамотно, и каких-либо серьезных упущений я не заметил.