Ruby — Говнокод.ру

Ruby / Говнокод #16813

−93

1

deals = Deal.joins(:deal_locations).select("deals.id, (SELECT array(SELECT category_id FROM deal_categories WHERE deal_id = deals.id ORDER BY category_id ASC)) AS category, deals.start_date, deals.end_date, deals.address, deals.theme, deals.alias, deals.metro, deals.district, deals.discount, deals.coupon_count, deals.issued_coupons, deals.main_image, count(*) OVER() AS total_count, trunc(abs(extract(epoch FROM age(deals.end_date, clock_timestamp()))/86400)) AS days_left").where("deal_locations.deal_id = deals.id AND deal_locations.location_id = ? AND deals.category_id IN (#{categories[0...-2]}) AND deals.id NOT IN (#{ex_deals.map{|e| e}.join(", ")})", params[:location].to_i).active.in_stock.limit(limit).offset(offset).order("deals.start_date desc")

Rails. Да, да, активрекорд, я помню о тебе.

adastreamer, 07 Октября 2014

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

Ruby / Говнокод #16324

−151

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15

def get_team_choices_with_index
    school_ids = params[:school_id].split(",")
    @index = params[:index]

    @teams_array = Array.new
    school_ids.each do |s|
      next if s.eql? "0"
      School.find(s).teams.each do |t|
        @teams_array << t
      end
    end

    @teams_array.flatten!
    @teams_array
  end

А должно быть что-то вроде
@teams_array = Team.where(school_id: school_ids)

dj_Andreus, 14 Июля 2014

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

Ruby / Говнокод #16072

−136

1
2
3
4

it 'traders email should be nil' do
   trader = FactoryGirl.create(:trader, :email => nil)
   trader.email.should be nil
end

captain obvious test

andrejlr, 28 Мая 2014

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

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

−131

01
02
03
04
05
06
07
08
09
10
11
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)

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

−87

1
2

# ugh
task = task.task.task

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

someone, 18 Декабря 2013

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

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

−148

1

"class".class.class

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

__proto__, 12 Октября 2013

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

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

−85

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
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)

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

−89

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
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)

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

−92

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
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)

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

−107

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