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

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

−98

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
35
36
37
38
39
40
41
42

<% form_tag(advertiser_creatives_path(@advertiser), :method=>'post' ) do %>
  <% content_tag :table, :class => "form-tabular" do %>
    <% content_tag :thead do %>
      <% content_tag :tr do %>
        <% tag :th, :class=>'hide'%>
        <% content_tag :th do %>
          Concept (optional)
        <%end%>
        <% content_tag :th do %>
           Ad Tag
        <%end%>
        <% content_tag :th do %>
           Click-Through URL
        <%end%>
        <th/>
      <%end%>
    <%end%>
    <% content_tag :tbody, :id=>'creative_sub_forms' do %>
      <% creatives.each_with_index do |creative, index| 
          next unless creative.new_record?
        %>
        <% unless creative.errors.empty? %>
          <tr class='nowrap'>
            <td colspan='3'>
              Error:
              <% creative.errors.full_messages.each do |msg| %>
              <%= msg %><br/>
              <%end%>
            </td>
          </tr>
        <%end%>
        <%=render :partial => 'creative_fields', :locals => {:index => index, :creative => creative}%>
      <%end%>
    <%end%>
    <tr>
      <td colspan='3'><%= link_to_add_creative_fields "Add row"  %></td>
    </tr>
  <%end%>
  <br/>
  <br/>
  <%= submit_tag 'Continue', :class=>'button' %>
<%end%>

Товарищи, заставляйте нубой учить HTML. Ну, или HAML.

cheba, 25 Мая 2010

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

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

−106

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

class User < ActiveRecord::Base

  #Состояния подтвержденности email
  state_machine :email_state, :initial => :unconfirmed, :namespace => 'email' do
    event :confirm do
      transition [:unconfirmed] => :confirmed
    end

    event :unconfirm do
      transition [:confirmed] => :unconfirmed
    end
  end

end

antono, 16 Мая 2010

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

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

−100.2

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

def ordinalize_ru integer
  postfix = case integer
    when 0 then 'ой'
    when 2 then 'ой'
    when 3 then 'ий'
    when 6 then 'ой'
    when 7 then 'ой'
    when 8 then 'ой'
    else 'й'
  end
  [ integer, postfix ].join '-'
end

Когда нет времени вникать в правила родного языка и русской речи. Суровый уральский говнокод.

eveel, 17 Апреля 2010

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

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

−116.2

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

# TODO: выбросить нахер метод with_company_and_state и правильно написать
# этот named scope
named_scope :with_company_and_state_non_uniq, lambda { |company, state|
  { :conditions => [ 'products.company_id = :company_id AND ' +
        'state_mask = :state_mask', { :company_id => company,
        :state_mask => OrderSet::STATES.index(state) || 0 } ],
    :order => 'order_sets.created_at DESC',
    :joins => 'INNER JOIN products' }
}

def self.with_company_and_state(company, state)
  with_company_and_state_non_uniq(company, state).uniq
end

Бывает, что SELECT DISTINCT сделать не всегда возможно. На самом деле, такая выборка сосёт.

eveel, 28 Марта 2010

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

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

−116.6

1
2
3
4
5

class Vector
  def -@
    map(&:-@)
  end
end

Код мой, говнокодом бы не назвал, но без улыбки точно не взглянешь на такое =)
(тут определение унарного минуса через вызов того же унарного минуса у всех элементов вектора, Кэп)

rakoth3d, 11 Марта 2010

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

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

−118.3

1
2
3

from app/controllers/test_controller.rb:13:in `index'
from (irb):4
from ♥:0>>

Сразу предупреждаю: не говнокод (и даже не код), можно минусовать.
День Святого Валентина, Interactive Ruby выдал такое, типа с праздником :)
PS: незнаю где здесь руби.

hardcoder, 15 Февраля 2010

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

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

−121

1
2
3
4
5
6
7
8

def has_currency_rate?
  val = false
  if self.currency.id == self.client.company.currency.id or self.currency_rate.blank?
  else
    val = true
  end
  val
end

тяжело же жилось людям...

rakoth3d, 09 Февраля 2010

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

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

−130.2

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52

class FinancialEventObserver < ActiveRecord::Observer
  observe Payment, Invoice
  def before_save(model)
    event = nil        
    if model.class == Payment
      if model.new_record?        
        event = FinancialEvent.new(:event => FinancialEvent::Event::PAYMENT_INVOICE,
        :arguments => {:client_name => model.invoice.client.short_name, :invoice_number => model.invoice.invoice_number},
          :company_id=>model.invoice.client.company.id)
      end
    elsif model.class == Invoice
      i = Invoice.find_by_id model.id      
      if model.new_record? or i.status != model.status        
        if model.status == Invoice::Status::ESTIMATE
          event = FinancialEvent.new(:event => FinancialEvent::Event::ESTIMATE_SEND,
        :arguments => {:client_name => model.client.short_name, :invoice_number => model.invoice_number},
            :company_id=>model.client.company.id)
        elsif model.status == Invoice::Status::APPROVED
          event = FinancialEvent.new(:event => FinancialEvent::Event::ESTIMATE_APPROVED,
        :arguments => {:client_name => model.client.short_name, :invoice_number => model.invoice_number},
            :company_id=>model.client.company.id)
        elsif model.status == Invoice::Status::REJECTED
          event = FinancialEvent.new(:event => FinancialEvent::Event::ESTIMATE_REJECTED,
        :arguments => {:client_name => model.client.short_name, :invoice_number => model.invoice_number},
            :company_id=>model.client.company.id)
        elsif model.status == Invoice::Status::SEND
          event = FinancialEvent.new(:event => FinancialEvent::Event::INVOICE_SEND,
        :arguments => {:client_name => model.client.short_name, :invoice_number => model.invoice_number},
            :company_id=>model.client.company.id)            
        end
      elsif !model.new_record? and i.state != model.state
        if model.state == Invoice::State::DELETED
          event = FinancialEvent.new(:event => FinancialEvent::Event::INVOICE_DELETED,
        :arguments => {:invoice_number => model.invoice_number},
            :company_id=>model.client.company.id)
        end
      end    
    end
    event.eventable = model.requester unless event.blank? 
    event.save unless event.blank?
    
  end
  def before_destroy(model)
    if model.class == Payment
      event = FinancialEvent.new(:event => FinancialEvent::Event::PAYMENT_DELETED,
        :arguments => {:invoice_number => model.invoice.invoice_number},
      :company_id=>model.invoice.client.company.id)
      event.eventable = model.requester
      event.save
    end    
  end
end

о боже, зачем я открыл этот файл?

rakoth3d, 16 Января 2010

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

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

−131.4

1

some_variable = (some_condition && some_other_condition) ? true : false

Иногда встречаю ГК подобный этому но реализованный в виде case..when. Еще более впечатляет :)

shine, 15 Января 2010

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

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

−126.7

1

<li class="list_item primary_card_<%= index+1 %> <%= primary_card.patient.sex == 'male' ? 'boy' : 'girl' %> critical">

А можно было изменить имя класса в CSS

nimnull, 26 Декабря 2009

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

Говнокод: по колено в коде.

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

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

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

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

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

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

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

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

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

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