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

    −86

    1. 1
    2. 2
    3. 3
    4. 4
    scope :search_by_title, proc { |title|
  title = sanitize_sql(title).gsub(/'/, '').strip().split().join(':* &') << ":*"
  where("to_tsvector('english', title) @@ to_tsquery('english', '#{title}')").order("case when title ~* '^#{title}' then 1 else 2 end")
}

    в рабочем проекте клиент решил, что он тоже программист.. я менять ничего не стал, просто дописал коммент про facepalm

    Запостил: cheef, 15 Мая 2012

    Комментарии (9) RSS

    • ava niko 15.05.2012 15:31 # 0

      cheef, можешь в двух словах описать здесь где плохо? я по-ruby не понимаю совсем )
      Ответить

      • ava gorsash 16.05.2012 10:43 # 0

        Можно было засунуть все эти гсубы (замена подстрок кажись) и стрипы в sanitize_sql.
        И можно было разделить на несколько независимых строк. Слишком уж длинная цепочка операций
        Ответить

    • ava cheef 15.05.2012 15:49 # +1

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

      • ava guest 16.04.2013 10:17 # −1

        А это одна из причин, почему Ruby такое говно.
        Если язык позволяет запутать код до неузнаваемости - это большая архитектурная проблема языка.
        Попробуйте повторить на том же PHP - не получится. Т.е. даже такое уг как PHP лучше Ruby.
        Ответить

        • ava guest 16.04.2013 10:29 # −1

          Ты еще напиши, если язык позволяет написать произведения Дарьи Донцовой, то он гамно...
          Ответить

    • ava malleus 08.06.2012 12:00 # +1

      если заказчику не указать на его неадекватность, в следующий раз он напишет больше)
      Ответить

    • ava guest 09.11.2012 22:03 # 0

      ох бляяяя... после такого кода... хочется просто не разбиратсья дальше
      Ответить

    • ava akostrikov 22.07.2013 18:40 # 0

      Самое смешное - когда клиент хочет чтобы вы нашли в своём коде ошибку, которую он дописал.
      В первый раз думал - шизофрения началась (я был единственный Ruby-программист и git blame, конечно, не делал).
      Ответить

    Добавить комментарий