помогите разобраться с написанием плагинов

dostel1 · May 18, 2012

1 раз пишу.Подскажите где ошибки


// JavaScript Document
(function($){

	$.fn.plug=function(otions){
		var options=jQuery.extend({color:'red',background:'blue'},options)
		this.each(function(){
		function send(){
		$('this').css('color',options.color)	
		}

			$(this).bind('click',send)

			})
	return this	

	}

})(jQuery)


<script src="jquery.js"></script>
<script src="1.js"></script>
<script>
$('p').plug()

</script>
</head>

<body>
<p>one</p>
<p>two</p>

Great Rash · May 18, 2012

var options=jQuery.extend({color:'red',background:'blue'},options)

Тут неплохо бы проверить, что options - объект, а не строка например.

$(this)

Лучше вынести в отдельную переменную, наверняка в коде будет использоваться не раз. Это существенно ускорит работу плагина.

dostel1 · May 18, 2012

Спасибо за достойные замечания

MiksIr · May 18, 2012

1. $('this') - опечатка.

2. $.fn.plug=function(otions){ - опечатка в options

3. Это тестовый пример или вы реально что-то делать пытаетесь? Ибо в вашем случае this.each не нужен даже, можно просто

$.fn.plug=function(otions){
  var options=jQuery.extend({color:'red',background:'blue'},options)
  $(this).bind('click', function() {
    $(this).css('color',options.color);
  });
  return this;
}

dostel1 · May 18, 2012

MiksIr задача была -понять как создавать плагины.Спасибо

MiksIr · May 18, 2012

А, ну тогда поясню. Внутри плагина this - это коллекция, которую вы выбрали селектором, т.е. в вашем случае все P. Если можете работать с коллекцией - цикл делать необязательно. Т.е. bind вполне можно вызвать на коллекцию, он сам разберется уже. А если вам нужно что-то персонально сделать с каждым элементом коллекции (например, поменять текст внутри каждого P) - тогда делаете цикл по всем элементам. jQuery дает простую возможность делать цикл через коллекция.each(функция).

Еще по коду - функцию send можно вынести за цикл, нет нужды в данном случае ее объявлять столько раз, сколько элементов в коллекции.

Ну и замечание по стилистике - крайне рекомендуется использовать внутри плагина подное обращение к jQuery, т.е не $(this) а jQuery(this). Если хочется краткости, то можно в начале плагина написать var $ = jQuery; Это свзязано с тем, что пользователь вашего плагина может отключить короткий синтаксис (это бывает нужно из-за проблем с другими библиотеками).

dostel1 · May 18, 2012

Еще по коду - функцию send можно вынести за цикл, нет нужды в данном случае ее объявлять столько раз, сколько элементов в коллекции.
Ну и замечание по стилистике - крайне рекомендуется использовать внутри плагина подное обращение к jQuery, т.е не $(this) а jQuery(this). Если хочется краткости, то можно в начале плагина написать var $ = jQuery; Это свзязано с тем, что пользователь вашего плагина может отключить короткий синтаксис (это бывает нужно из-за проблем с другими библиотеками).

Ошибка по циклам полностью понятная .

Я думал данная обертка решает проблемы конфликтов с другими библиотеками.

Edited May 18, 2012 by dostel1

Great Rash · May 18, 2012

Проблему конфликтов решает замыкание.

(function($){
  // я в замыкании и меня не видно снаружи
})(jQuery)

s0rr0w · May 18, 2012

Проблему конфликтов решает замыкание.

Вот и я удивился, нафига что-то куда-то присваивать, если ссылка на объект передается в виде аргумента.

Sign In

помогите разобраться с написанием плагинов

Question

dostel1

Link to comment

Share on other sites

8 answers to this question

Recommended Posts

Great Rash

Link to comment

Share on other sites

dostel1

Link to comment

Share on other sites

MiksIr

Link to comment

Share on other sites

dostel1

Link to comment

Share on other sites

MiksIr

Link to comment

Share on other sites

dostel1

Link to comment

Share on other sites

Great Rash

Link to comment

Share on other sites

s0rr0w

Link to comment

Share on other sites

Join the conversation

Обсуждения

Forums

Important Information