rubish #1601 30 мая 2013 ну посты же я конечно знал, что рабисты ебаные наркоманы, но чтобы настолько http://habrahabr.ru/post/181434/ Методы не могут быть длиннее чем 5 строк кода. на самом деле в этом что-то есть Колы я выросту - то хочу буты такым як я годные смайлы Поделиться сообщением Ссылка на сообщение
TheDeadSkin #1602 30 мая 2013 да-да, в методах на 5 строк кода определённо что-то есть и тут как никогда в тему будет сказать, что это весьма ТОНКО Поделиться сообщением Ссылка на сообщение
rubish #1603 30 мая 2013 да-да, в методах на 5 строк кода определённо что-то есть и тут как никогда в тему будет сказать, что это весьма ТОНКОну как бы 5 строк - это черезчур, но методы должны быть максимально короткими. ну и мерять надо не в количестве строк, а в вызовах.хотя ребята по ссылке эпичные долбоебы, ведь их подходы увеличивают циколматическую сложность, а это беда похуже, чем метод на страницу Колы я выросту - то хочу буты такым як я годные смайлы Поделиться сообщением Ссылка на сообщение
TheDeadSkin #1604 30 мая 2013 я думаю так и имеется ввиду строка=выховда и не спорю я, что методы должны быть короткими, но короткими пропорционально логике метода, а не короткими со статическим органичением кол-ва строк там в каментах выразили мысль которая у меня сходу возникла - тоесть выходит если логика метода не влезает в 5 строк мне нужно хуярить 5 подметодов на один раз чтобы по факту строк на метод было 25? бред же ебаный, код будет фрагментирован Поделиться сообщением Ссылка на сообщение
DIMAN123456789 #1605 30 мая 2013 Или писать очень длинные строки И в качестве аргументов передавать мапы параметров Поделиться сообщением Ссылка на сообщение
Tinplz #1606 30 мая 2013 Изначально мерять код строками это полный бред. Уже именно с этого момента их гайд провальный.А много методов это не проблема, по-крайней мере в плюсах, просто маркируешь их инлайнами, и компилятор сам всё разворачивает. Что там шарпы и жавы с их джитом делают со всей этой поебней - не в курсе, но что-то мне подсказывает, что ни к чему хорошему это не приводит. Поделиться сообщением Ссылка на сообщение
Nekon #1607 30 мая 2013 гуру js, нужна помощь.Короче есть фиксированное меню .И есть блоки с соответственно содержимым.Мне надо, чтобы при прокрутке до определенного блока присваивался active класс соответственному пункту меню(т.е пользователь докрутил до 6 блока, в меню должен загореться 6 пункт), как вообще поймать, что была произведена прокрутка до какого-то блока.Желательно с использованием jquery, еще лучше, если кто плагин готовый знает. Поделиться сообщением Ссылка на сообщение
KotZhilkina #1608 30 мая 2013 гуру js, нужна помощь.Короче есть фиксированное меню .И есть блоки с соответственно содержимым.Мне надо, чтобы при прокрутке до определенного блока присваивался active класс соответственному пункту меню(т.е пользователь докрутил до 6 блока, в меню должен загореться 6 пункт), как вообще поймать, что была произведена прокрутка до какого-то блока.Желательно с использованием jquery, еще лучше, если кто плагин готовый знает.не балгодари http://twitter.github.io/bootstrap/javascript.html#scrollspy Публикация отключена Поделиться сообщением Ссылка на сообщение
rubish #1609 30 мая 2013 я думаю так и имеется ввиду строка=выховда и не спорю я, что методы должны быть короткими, но короткими пропорционально логике метода, а не короткими со статическим органичением кол-ва строк там в каментах выразили мысль которая у меня сходу возникла - тоесть выходит если логика метода не влезает в 5 строк мне нужно хуярить 5 подметодов на один раз чтобы по факту строк на метод было 25? бред же ебаный, код будет фрагментированнихуя. иф и энд тоже за строку считают Колы я выросту - то хочу буты такым як я годные смайлы Поделиться сообщением Ссылка на сообщение
TheDeadSkin #1610 30 мая 2013 тоесть метод не может быть больше чем if hui = true then call deleteelse call nedeleteend ахахах, прекратите Поделиться сообщением Ссылка на сообщение
KotZhilkina #1611 30 мая 2013 тоесть метод не может быть больше чем if hui = true thencall deleteelsecall nedeleteend ахахах, прекратите ну это суть руби погружайся с нами Публикация отключена Поделиться сообщением Ссылка на сообщение
TheDeadSkin #1612 30 мая 2013 да может это не так уж и плоховсё-таки можно одним методом в пять строк с условием развернуть фейсбук, ну и на крайняк если условие не выполнено, то хотя бы просто какой-то твиттер Поделиться сообщением Ссылка на сообщение
KotZhilkina #1613 30 мая 2013 if hui = true thencall huielsecall nothuiend if nothui = true thencall nothuielsecall notnothuiend if notnothui = true thencall notnothuielsecall notnotnothuiend if notnotnothui = true thencall notnothuielsecall notnotnotnothuiend Публикация отключена Поделиться сообщением Ссылка на сообщение
TheDeadSkin #1614 30 мая 2013 а если выполнится каждое условие, то всё будет очень плохо? Поделиться сообщением Ссылка на сообщение
Nekon #1615 30 мая 2013 впадлу мне было качать бутстрап и разбираться в нём, короче сделал свою прокрутку с блекджеком и шлюхами Поделиться сообщением Ссылка на сообщение
KotZhilkina #1616 30 мая 2013 (изменено) ветвление через множественные пятистрочные методы... пойду-ка запатентовывать алгоритм branching through multiple five-row classes впадлу мне было качать бутстрап и разбираться в нём, короче сделал свою прокрутку с блекджеком и шлюхамиа причем тут бустрап? он тут вообще не причем - все на jquery же... код /* =============================================================* bootstrap-scrollspy.js v2.3.2* http://twitter.githu....html#scrollspy* =============================================================* Copyright 2012 Twitter, Inc.** Licensed under the Apache License, Version 2.0 (the "License");* you may not use this file except in compliance with the License.* You may obtain a copy of the License at** http://www.apache.or...ses/LICENSE-2.0** Unless required by applicable law or agreed to in writing, software* distributed under the License is distributed on an "AS IS" BASIS,* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.* See the License for the specific language governing permissions and* limitations under the License.* ============================================================== */ !function ($) { "use strict"; // jshint ;_; /* SCROLLSPY CLASS DEFINITION * ========================== */ function ScrollSpy(element, options) { var process = $.proxy(this.process, this) , $element = $(element).is('body') ? $(window) : $(element) , href this.options = $.extend({}, $.fn.scrollspy.defaults, options) this.$scrollElement = $element.on('scroll.scroll-spy.data-api', process) this.selector = (this.options.target || ((href = $(element).attr('href')) && href.replace(/.*(?=#[^\s]+$)/, '')) //strip for ie7 || '') + ' .nav li > a' this.$body = $('body') this.refresh() this.process() } ScrollSpy.prototype = { constructor: ScrollSpy , refresh: function () { var self = this , $targets this.offsets = $([]) this.targets = $([]) $targets = this.$body .find(this.selector) .map(function () { var $el = $(this) , href = $el.data('target') || $el.attr('href') , $href = /^#\w/.test(href) && $(href) return ( $href && $href.length && [[ $href.position().top + (!$.isWindow(self.$scrollElement.get(0)) && self.$scrollElement.scrollTop()), href ]] ) || null }) .sort(function (a, b) { return a[0] - b[0] }) .each(function () { self.offsets.push(this[0]) self.targets.push(this[1]) }) } , process: function () { var scrollTop = this.$scrollElement.scrollTop() + this.options.offset , scrollHeight = this.$scrollElement[0].scrollHeight || this.$body[0].scrollHeight , maxScroll = scrollHeight - this.$scrollElement.height() , offsets = this.offsets , targets = this.targets , activeTarget = this.activeTarget , i if (scrollTop >= maxScroll) { return activeTarget != (i = targets.last()[0]) && this.activate ( i ) } for (i = offsets.length; i--;) { activeTarget != targets && scrollTop >= offsets && (!offsets[i + 1] || scrollTop <= offsets[i + 1]) && this.activate( targets ) } } , activate: function (target) { var active , selector this.activeTarget = target $(this.selector) .parent('.active') .removeClass('active') selector = this.selector + '[data-target=' + target + '],' + this.selector + '[href=' + target + ']' active = $(selector) .parent('li') .addClass('active') if (active.parent('.dropdown-menu').length) { active = active.closest('li.dropdown').addClass('active') } active.trigger('activate') } } /* SCROLLSPY PLUGIN DEFINITION * =========================== */ var old = $.fn.scrollspy $.fn.scrollspy = function (option) { return this.each(function () { var $this = $(this) , data = $this.data('scrollspy') , options = typeof option == 'object' && option if (!data) $this.data('scrollspy', (data = new ScrollSpy(this, options))) if (typeof option == 'string') data[option]() }) } $.fn.scrollspy.Constructor = ScrollSpy $.fn.scrollspy.defaults = { offset: 10 } /* SCROLLSPY NO CONFLICT * ===================== */ $.fn.scrollspy.noConflict = function () { $.fn.scrollspy = old return this } /* SCROLLSPY DATA-API * ================== */ $(window).on('load', function () { $('[data-spy=scroll]').each(function () { var $spy = $(this) $spy.scrollspy($spy.data()) }) }) }(window.jQuery); Изменено 30 мая 2013 пользователем Hed-kun Публикация отключена Поделиться сообщением Ссылка на сообщение
rubish #1617 30 мая 2013 тоесть метод не может быть больше чем if hui = true thencall deleteelsecall nedeleteend ахахах, прекратите ну там прямым текстом именно это написано Колы я выросту - то хочу буты такым як я годные смайлы Поделиться сообщением Ссылка на сообщение
Обязательное_поле #1618 30 мая 2013 module Sra4_v_teme_programmirovanie < Application.Base.BaseImp.BaseofBasesAbsrtact.IBaseApp.AppFamily.Rabi_dla_uebkov class SporResolver define razrulit_situaciu |sabj| gem install -rake --traced -~~-v1.0(1) stringy_clazz_autoloader_run do_what_i_said render resolve if??<%{$#.username.#$}%prav%> + {{ko-ko_pok_pok_rails_rulit}} bender exec rake run task razrulit_situaciu sabj end Lilaend тоесть метод не может быть больше чем if hui = true thencall deleteelsecall nedeleteend ахахах, прекратите ну это суть руби погружайся с намиmodule Sra4_v_teme_programmirovanie < Application.Base.BaseImp.BaseofBasesAbsrtact.IBaseApp.AppFamily.Rabi_dla_uebkov class SporResolver define razrulit_situaciu |sabj| gem install -rake --traced -~~-v1.0(1) stringy_clazz_autoloader_run do_what_i_said render resolve if??<%{$#.username.#$}%prav%> + {{ko-ko_pok_pok_rails_rulit}} bender exec rake run task razrulit_situaciu sabj end Lilaend Поделиться сообщением Ссылка на сообщение