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

    Всего: 4

  2. 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)

  3. 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)

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

    −95

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    def space(n)
    str = ''
    n.times do
      str += ' '
    end
    str
  end

    whitequark, 25 Мая 2011

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

  5. Си / Говнокод #5832

    +127

    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
    static void jz_update_dram_prev(unsigned int cur_mclk, unsigned int new_mclk)
{
        /* No risk, no fun: run with interrupts on! */
        if (new_mclk > cur_mclk) {
                /* We're going FASTER, so first update TRAS, RCD, TPC, TRWL
                 * and TRC of DMCR before changing the frequency.
                 */
                jz_update_dram_dmcr(new_mclk);
        } else {
                /* We're going SLOWER: first update RTCOR value
                 * before changing the frequency.
                 */
                jz_update_dram_rtcor(new_mclk);
        }
}

    Кусок кода из официального™ китайского дерева исходников Линукса для одного System-on-Chip.

    Реализует поддержку динамического изменения частоты процессора, а приведенная функция меняет тайминги для памяти (как обычно, весьма альтернативным способом). Что характерно, это вполне себе работает на реальном железе, и вроде бы оно у меня в этом куске ни разу не падало.

    whitequark, 01 Марта 2011

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