Javascript-форум (https://javascript.ru/forum/)
-   Общие вопросы Javascript (https://javascript.ru/forum/misc/)
-   -   Проблемка с toggle (https://javascript.ru/forum/misc/53212-problemka-s-toggle.html)

рони 26.01.2015 15:07

danik.js,
у тебя мания банить за одно и тоже десять раз ?

рони 26.01.2015 15:36

Цитата:

Сообщение от Valentin1918
в этом куске кода я вообще ничего не понимаю:

этот код возврашает удалённый метод toggle под новым названием mytoggle

danik.js 26.01.2015 15:58

Цитата:

Сообщение от рони
у тебя мания банить за одно и тоже десять раз ?

Я не модер, банить не могу. Или ты букву «р» пропустил? Так я ведь заранее предупредил.

рони 26.01.2015 16:20

danik.js,
считаю что ты не прав, и я снова опубликую этот код если у кого-то будет в нём потребность в старом toggle -
сократил его до твоих любимых a,b,c
$.fn.mytoggle = function() {
     var b = arguments;
     return this.each(function(d, c) {
         var a = function() {
             var a = 0;
             return function() {
                 b[a++ % b.length].apply(c, arguments)
             }
         }();
         $(c).click(a)
     })
 };

считаю что данный код должен занимать как можно меньше места, потому что он достаточно элементарный для того кто понимает, а кто пока нет - у того он займёт меньше места. кому нужна подробность пусть идёт по твоей ссылке
http://javascript.ru/forum/jquery/43...tml#post289163

danik.js 26.01.2015 17:17

Цитата:

Сообщение от рони
считаю что данный код должен занимать как можно меньше места

Хули тогда пробелы не удалишь, а? Мистер обфускатор.

рони 26.01.2015 17:36

danik.js,
ок без пробелов
$.fn.mytoggle=function(){var b=arguments;return this.each(function(d,c){var a=function(){var a=0;return function(){b[a++%b.length].apply(c,arguments)}}();$(c).click(a)})};

danik.js 26.01.2015 17:43

Цитата:

Сообщение от рони
el

Ты что! Эту переменную нужно срочно сократить до одной буквы! И не до «e», а, например, «d», чтоб никто не догадался!
Цитата:

Сообщение от рони
$.fn.mytoggle

Это же сколько лишних байт! Код должен занимать как можно меньше места, поэтому должно быть $.fn.a

Цитата:

Сообщение от рони
function(i,el){var a=

Это ж целый лишний var! i у нас объявлена, так что ее можно использовать для ссылки на function! Сэкономим 4 байта - это не хухры-мухры!

рони 26.01.2015 17:49

danik.js,
input[type=checkbox]:not(old)
что такое old и где об этом прочесть?

рони 26.01.2015 18:02

Цитата:

Сообщение от danik.js
Эту переменную нужно срочно сократить до одной буквы! И не до «e», а, например, «d»,

исправил смотри код выше -- а по css не подскажешь?

ruslan_mart 26.01.2015 18:33

danik.js, завязывай с флудом и матом. ;)
Я никого тут не поддерживаю, но пусть рони пишет так, как ему удобнее. То что он тут помогает новичкам бесплатно - это уже благое дело. Если автора что-то не устраивает, то пусть учит JS и называет переменные как ему нравится, или же в раздел "Работа" за идеальным кодом.


Часовой пояс GMT +3, время: 15:29.