Javascript.RU

Создать новую тему Ответ
 
Опции темы Искать в теме
  #1 (permalink)  
Старый 10.02.2011, 14:31
Новичок на форуме
Отправить личное сообщение для stnw Посмотреть профиль Найти все сообщения от stnw
 
Регистрация: 13.11.2010
Сообщений: 7

Просьба покритиковать код
Добрый день!

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

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

Заранее благодарен!
Вложения:
Тип файла: txt demo.js.txt (8.1 Кб, 17 просмотров)
Ответить с цитированием
  #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 сам конвертирует его в строку, применив необходимые фильтрации).

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



Опции темы Искать в теме
Искать в теме:

Расширенный поиск


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
Как создать bodyclick код royksopp Общие вопросы Javascript 8 14.10.2011 16:39
код странно отрабатывает комбинацию клавиш ffx Общие вопросы Javascript 4 20.01.2011 10:58
Не работает код из внешнего файла в IE6 Petka jQuery 2 03.03.2010 16:09
Opera отдаёт неправильный код функциональных клавиш. NT Man Opera, Safari и др. 1 19.01.2010 02:45
Как узнать динамический код страницы? gosha13 Общие вопросы Javascript 5 27.07.2009 13:31