Список говнокодов пользователя malleus

Всего: 4

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

−84

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

def self.allow_some_behaviour?
  # better safe than sorry.
  bool = !false ? !false : !!false

  # type checking
  if bool

    # short circuit just in case
    return !true;
  elsif !bool
    return !false
  end
end

найдено в реальном проекте, работало в течении года где то. зачем это сделано - непонятно. почему не выпилили, тоже неясно.

malleus, 08 Июня 2012

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

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

−96

1
2
3
4
5
6

<%products = Array.new
  count = Product.count(:conditions => "novelty = 'true'")
  while products.size < 10 do
    products << Product.find(:first, :conditions => "novelty = 'true'",:offset => rand(count))
    products.uniq!
  end-%>

при количестве новинок меньше 10 получаем бесконечный цикл. счастье в продакшене, там sql запросы не пишутся в лог)

malleus, 13 Сентября 2011

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

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

−98

1

amount = ('-' + batch['settleAmount'].to_s).to_i

работа с платежными системами требует вдумчивости и оригинальности.

malleus, 07 Июля 2011

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

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

−92

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

def get_next_line(max_level, current_level, current_line_items)
        next_line_items = []
        result_string = ''

        if current_level == 1
                next_line_items.push(1)
                result_string = "1\n"
        else
                current_line_number = false
                current_line_number_count = 0
                current_index = 0

                current_line_items.each do |line_number|
                        if current_index == 0
                                current_line_number = line_number
                                current_line_number_count = 1
                        else
                                if line_number == current_line_number
                                        current_line_number_count = current_line_number_count + 1
                                else
                                        next_line_items.push(current_line_number_count)
                                        next_line_items.push(current_line_number)

                                        current_line_number_count = 1
                                        current_line_number = line_number
                                end
                        end

                        if current_index >= current_line_items.length - 1
                                next_line_items.push(current_line_number_count)
                                next_line_items.push(current_line_number)
                        end

                        current_index = current_index + 1
                end

                result_string = next_line_items.join(' ') + "\n"
        end

        if current_level < max_level
                result_string = result_string + get_next_line(max_level, current_level + 1, next_line_items).to_s
        end

        # return result
        result_string
end

puts 'Input max level:'
level = gets

puts ''
puts get_next_line(level.to_i, 1, [])

реальное тестовое задание кандитата.
задача - вывести несколько членов последовательности
1
11
21
1211
....

malleus, 04 Июня 2010

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

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