1. Ruby / Говнокод #14953

    −131

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    def sum_all_orders
    sum = 0
    Order.where(user_id: self.id).each do |order|
      sum = sum + order.total_sum
    end
    sum
  end

  # вместо
  def sum_all_orders
     orders.sum(&:total_sum)
  end

    В коде магазина, таких ляпов полон проект. Задрался dd жмакать.
    Так сложно API active_support почитать???

    alexes_dev, 25 Февраля 2014

    Комментарии (14)

  2. Ruby / Говнокод #14245

    −87

    1. 1
    2. 2
    # ugh
task = task.task.task

    http://programmingisterrible.com/post/50285403468/a-lifetime-of-terrible-code

    someone, 18 Декабря 2013

    Комментарии (77)

  3. Ruby / Говнокод #13929

    −148

    1. 1
    "class".class.class

    Не говнокод, просто забавно

    __proto__, 12 Октября 2013

    Комментарии (21)

  4. Ruby / Говнокод #13029

    −85

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    def self.process(site, object = nil, &block)
    dont_unblock = false
    block_res = true

    site.lock!
    if site.assets_blocked?
      dont_unblock = true
      raise Asset::BlockedException
    end
    site.assets_blocked = true
    site.save!

    block_res = yield if block_given?

    return false unless block_res
    return object.save if object

    true
  ensure
    site.update_attribute(:assets_blocked, false) unless dont_unblock
  end

  # во всех местах, где с ассетами делалось хоть что-нибудь, копипастился такой кусок:

      result =
      begin
        @asset = @site.assets.new
        @asset.process do
          @asset.attached = params[:file]
          @asset.folder_path = params[:path]
        end
      rescue Asset::BlockedException
        render json: { errors: [I18n.t("layout.assets.errors.assets_blocked")] } and return
      end

    По мотивам http://govnokod.ru/13027: транзакции изобретены не нами, давайте сделаем их сами!

    whitequark, 21 Мая 2013

    Комментарии (2)

  5. Ruby / Говнокод #13027

    −89

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    def set_path
      if name && (!path || path == "/")
        self.path = self.parent.present? ? "#{self.parent.path}/#{name}" : "/#{name}"
      elsif !new_record? && name && path && name_was != name
        parts = path.split("/")
        parts.pop
        self.path = [parts.join("/"), name].join("/")
      elsif !new_record? && name && self.parent_id_changed?
        self.path = self.parent.present? ? "#{self.parent.path}/#{name}" : "/#{name}"
      elsif new_record? && name && path
        self.path = [path, name].join("/")
      end

      if path && self.parent.blank?
        parts = path.split("/")

        self.name = parts.pop

        parent_path = parts.join("/")
        if parent_path.blank? || parent_path == "/"
          self.parent = nil
        else
          possible_parent = site.asset_folders.find_by_path(parent_path)
          self.parent = possible_parent.present? ? possible_parent : self.class.create(path: parent_path, site: site)
        end
      end
      true
     end

    Как не нужно работать с путями в Rails-приложении. Это и еще примерно 500 строк было заменено на 11 строк кода, включая пустые.

    whitequark, 20 Мая 2013

    Комментарии (20)

  6. Ruby / Говнокод #12681

    −92

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    def register_valid_login
		login = params[:login]
		return register_error("Слишком длинный ник!") if (login.length > 16) 
		return register_error("Слишком короткий ник!") if (login.length < 3)
		return register_error("Недопустимые символы в нике!") if !(/^[a-zA-Z\-_]+$/=~login)
		return register_error("Можно использовать либо '-', либо '_' в нике!") if (login.count("-") > 0 && login.count("_") > 0)
		return register_error("Ник должен состоять из трёх частей максимум!") if (login.count("-") > 2 || login.count("_") > 2)
		login = [login] if (login.count("-") == 0 && login.count("_") == 0)
		login = login.split("-") if login.count("-") > 0
		login = login.split("_") if login.count("_") > 0
		return register_error("Начало ника не может быть меньше трёх символов!") if (login[0].length < 1)
		case login.count
			when 2
				return register_error("Если у вас ник вида Xxx_Yyy, то Yyy может состоять минимум из 3х букв") if (login[1].length < 3)
			when 3
				return register_error("Если у вас ник вида AAA_B_CCC, то B должна быть больше одной буквы") if (login[1].length < 1)
		end
		for i in 0..login.count-1
			return register_error("Заглавная буква в середине " + (i+1).to_s + "-ой части ника") if !(/^[a-zA-Z]?[a-z]+$/=~login[i])
		end
		return true
	end

    Процедура проверки ника в ситеме регистрации одного игрового проекта.

    DropWorld, 03 Марта 2013

    Комментарии (31)

  7. Ruby / Говнокод #12521

    −107

    1. 1
    @tag.url = @tag.name.downcase.gsub('А','a').gsub('а','a').gsub('Б','b').gsub('б','b').gsub('В','v').gsub('в','v').gsub('Г','g').gsub('г','g').gsub('Д','d').gsub('д','d').gsub('Е','e').gsub('е','e').gsub('Ё','ye').gsub('ё','e').gsub('Ж','j').gsub('ж','j').gsub('З','z').gsub('з','z').gsub('И','i').gsub('и','i').gsub('Й','i').gsub('й','i').gsub('К','k').gsub('к','k').gsub('Л','l').gsub('л','l').gsub('М','m').gsub('м','m').gsub('Н','n').gsub('н','n').gsub('О','o').gsub('о','o').gsub('П','p').gsub('п','p').gsub('Р','r').gsub('р','r').gsub('С','s').gsub('с','s').gsub('Т','t').gsub('т','t').gsub('У','u').gsub('у','u').gsub('Ф','f').gsub('ф','f').gsub('Х','h').gsub('х','h').gsub('Ц','ts').gsub('ц','ts').gsub('Ч','ch').gsub('ч','ch').gsub('Ш','sh').gsub('ш','sh').gsub('Щ','sh').gsub('щ','sh').gsub('Ъ','').gsub('ъ','').gsub('Ы','y').gsub('ы','y').gsub('Ь','').gsub('ь','').gsub('Э','e').gsub('э','e').gsub('Ю','you').gsub('ю','you').gsub('Я','ya').gsub('я','ya').gsub('`','').gsub(' ','-')

    Оригинальный подход к транслитерации

    heckfy, 31 Января 2013

    Комментарии (2)

  8. Ruby / Говнокод #12087

    −109

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    #Кусок красивого кода контроллера сайта на Ruby On Rails
class QuestionsController < ApplicationController

  #Извлекаем отвеченные вопросы методом говнокода
  def answered
      @subtitle = "Отвеченные"
      #Нам нужны все-все-все вопросы и даже не отвеченные. Скрестим пальцы на случай если их будет много
      @questions = Question.all
      #Обязательно преобразуем все в массив, сделаем N+1 запрос и успешно отфильтруем вопросы без ответа
      @questions.to_a.delete_if { |q| q.answer.nil? }
      #Чуть не забыл, нам же нужно извлечь 10 вопросов для 2-й страницы, а не все что есть
      paginate(@questions)
      #PROFIT
      render "index"
  end

end

    Ну в общем, код хорошо прокоментирован, думаю все и так ясно. =)

    SunnyMagadan, 09 Ноября 2012

    Комментарии (5)

  9. Ruby / Говнокод #11974

    −153

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    class ProblemsController < ApplicationController

  MAX_TAGS_PER_PROBLEM = 5

  def self.MAX_TAGS_PER_PROBLEM
    MAX_TAGS_PER_PROBLEM
  end
end

    antono, 22 Октября 2012

    Комментарии (10)

  10. Ruby / Говнокод #11763

    −166

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    class User < ActiveRecord::Base
    
  def full_name
  	user_info=User.find(self.id)
  	[user_info.name,user_info.surname].join(" ")
  end
end

    Получаем полное имя себя

    1101_debian, 13 Сентября 2012

    Комментарии (8)