Показать сообщение отдельно
  #2 (permalink)  
Старый 10.02.2011, 16:59
Аватар для B@rmaley.e><e
⊞ Развернуть
Отправить личное сообщение для B@rmaley.e><e Посмотреть профиль Найти все сообщения от B@rmaley.e><e
 
Регистрация: 11.01.2010
Сообщений: 1,810

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 сам конвертирует его в строку, применив необходимые фильтрации).

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