VPF::Как правильно компоновать библиотеку?

solenko

Дата 23.1.2008, 02:38 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1473
Регистрация: 15.1.2006
Где: Украина

Репутация: 9
Всего: 67

Вот решил собрать небольшую библиотеку для работы с xml. Как выяснилось практически весь код разный для IE и остальных браузеров. Вот и возник вопрос как лучше писать:

Код


DOMXML.prototype.xpath = function() {
    if (ie) {
//код для IE
    }else{
//код для остальных
    }
}

или

Код


if (ie) {
    DOMXML.prototype.xpath = function() {
//код для IE
    }
}else{
    DOMXML.prototype.xpath = function() {
//код для остальных
    }
}

В первом случа получаем проверки при каждом вызове. Минусов сторого пока не вижу (действительно очень мало повторяющегося кода).
Так все таки как лучше? Или в данном случае вообще лучше подгружать файл в зависимости от браузера?

--------------------

Ла-ла-ла-ла
Заметьте, нет официального подтверждения, что это не просто четыре слога.

Sardar

Дата 23.1.2008, 04:46 (ссылка)

(нет голосов)

Загрузка ...

Бегун

Профиль
Группа: Модератор
Сообщений: 6986
Регистрация: 19.4.2002
Где: Нидерланды, Groni ngen

Репутация: 78
Всего: 317

Второй способ лучше, т.к. исключает if, а функция штука легковесная.

Цитата(solenko @ 23.1.2008, 01:38

)

Или в данном случае вообще лучше подгружать файл в зависимости от браузера?

Определять браузер по userAgent'у или ещё как - резкое не уважение к пользователю smile

Нужно проверять по функционалу, т.е. тестировать необходимые функции, следовательно на сервере выделить "правильный" JS не получиться. На клиенте генерить <script> не очень красиво, не уверен что оно того стоит (скрипт ведь не сотни кб?)

--------------------

Опыт - сын ошибок трудных © А. С. Пушкин
Процесс написания своего велосипеда повышает профессиональный уровень программиста. © Opik
Оценить мои качества можно тут.

egao

Дата 23.1.2008, 10:02 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 109
Регистрация: 6.8.2007

Репутация: -1
Всего: нет

Код


DOMXML.prototype.xpath = select( browserType, {
    'ie': function(){ ... }
,
    'default': function(){ ... }
})

solenko

Дата 23.1.2008, 11:51 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1473
Регистрация: 15.1.2006
Где: Украина

Репутация: 9
Всего: 67

Sardar, как и подсказывала логика. Просто напугало, что нигде не видел такого описания.

egao, а что это за заверь такой, select? Или эта ваша обертка для switch?

--------------------

Ла-ла-ла-ла
Заметьте, нет официального подтверждения, что это не просто четыре слога.

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| JavaScript: Общие вопросы \| Следующая тема »